On 06/18/2017 01:40 AM, John Paul Adrian Glaubitz wrote:
> Hi Erik!
> 
> On 06/14/2017 02:04 PM, John Paul Adrian Glaubitz wrote:
>> On Wed, Jun 14, 2017 at 01:50:06PM +0200, Erik Helin wrote:
>>> thanks for contributing and signing the OCA!
>>
>> Thanks for reviewing my patches ;-).
> 
> My OCA has been completed now and I'm now showing up on the list
> of signees:
> 
>> http://www.oracle.com/technetwork/community/oca-486395.html
> 
> So, at least from the bureaucracy side, I should be all set now.

Great, thanks!

>>> I think the first three patches (hotspot-add-missing-log-header.diff,
>>> hotspot-fix-checkbytebuffer.diff, rename-sparc-linux-atomic-header.diff) all
>>> look good, thanks for fixing broken code. Consider them Reviewed by me.
>>> Every patch needs a corresponding issue in the bug tracker, so I went ahead
>>> and created:
>>> - https://bugs.openjdk.java.net/browse/JDK-8182163
>>> - https://bugs.openjdk.java.net/browse/JDK-8182164
>>> - https://bugs.openjdk.java.net/browse/JDK-8182165
> 
> So, for these three patches. What else needs to be done to get them
> in a state so they can be merged? I understand they need to be reviewed
> by a second reviewer as they concern Hotspot code.

Yep, you need a second reviewer for these patches.

> In another message, you also mentioned:
> 
>> Also, have you run the tier 1 testing for hotspot (the tests that need to
>> pass for every commit)? You can run those tests by running (from the
>> top-level "root" repo):
>>
>> $ make test TEST=hotspot_tier1
>>
>> or, if you want to try the new run-test functionality
>>
>> $ make run-test-hotspot_tier1
> 
> Should those be run on the Linux sparc64 machine or just on Linux x86_64?

The tests should be run in order to verify your changes. Since your
changes make Linux/sparc64 work again, they should be run on Linux/sparc64.

> I'm asking because running the testsuite on Linux sparc64 will only be
> possible with all four patches applied as they are build fixes. Running
> the testsuite on Linux x86_64 will be possible, of course.

Ok, this is the part I don't really understand yet. Why run with Zero on
Linux/sparc64? Shouldn't it be possible to run HotSpot "natively" (with
the template interpreter, C1, C2) on Linux/sparc64?

Thanks,
Erik

> After fixing the build on Linux sparc, I will have more patches ready
> to fix the Zero builds on even more targets ;-).
> 
> Cheers,
> Adrian
> 

Reply via email to