Opened 12 years ago

Closed 12 years ago

#191 closed enhancement (fixed)

New OpenJdk release

Reported by: mrt Owned by:
Priority: blocker Milestone: GA4
Component: general Version: 1.6.0 Build 25 GA3
Severity: high Keywords:
Cc:

Description

Change History (12)

comment:2 by Silvan Scherrer, 12 years ago

Milestone: EnhancedGA4

comment:4 by Silvan Scherrer, 12 years ago

Priority: minormajor
Severity: mediumhigh

comment:5 by Silvan Scherrer, 12 years ago

Priority: majorblocker

comment:6 by dmik, 12 years ago

b27 changes have been merged in in r406. Will try building it now.

comment:7 by dmik, 12 years ago

hotspot is successfully built, the Java itself has failed at some strange place (build.tools.hasher.Hasher.main). Investigating.

comment:8 by dmik, 12 years ago

Found the reason of the build break. This was a really tricky regression of r 22024 in Odin. This is a real mess I'd say but in fact reading from the standard input when it is a redirected file was broken in Java since then.

The matter is that when the standard input is redirected, the call to GetNumberOfConsoleInputEvents ends up in HMDeviceOpen32Class::_DeviceRequest() (instead of HMDeviceConsoleInClass::_DeviceRequest() as in case of the not redirected standard input). Before r 22024 HMDeviceOpen32Class::_DeviceRequest() would always erroneously return TRUE which satisfied the java code in io_util_md.c (handleStdinAvailable) just by accident. After I fixed the wrong return values in r 22024, it started to return FALSE as it should. However, in this case handleStdinAvailable() delegates its work to handleNonSeekAvailable which in turn calls PeekNamedPipe which seems to fail somehow on redirected stdin (with "invalid function") and this in turn aborts FileInputStream::available with an I/O exception.

I have to check on how to properly solve that. (I reverted the change to make Java build further but we need to fix it in Odin anyway).

comment:9 by dmik, 12 years ago

I created http://svn.netlabs.org/odin32/ticket/90 for tracking the issue in Odin.

comment:10 by dmik, 12 years ago

Odin ticket 90 is done. The build now goes further (finally).

comment:11 by dmik, 12 years ago

Builds and seems to work. Doing further testing.

comment:12 by dmik, 12 years ago

Resolution: fixed
Status: newclosed

Can be closed.

Note: See TracTickets for help on using tickets.