RFR (XL): 8019972: PPC64 (part 9): platform files for interpreter only VM.

David Holmes david.holmes at oracle.com
Thu Jul 18 22:28:40 PDT 2013

Hi Goetz,

Only a brief glance through ...

I think orderAccess_linux_ppc.inline.hpp should have:

   34 #ifndef _LP64
   35 #error "Atomic currently only impleneted for PPC64"
   36 #endif

the same as in atomic_linux_ppc.inline.hpp (the jlong variants will only 
be atomic on ppc64).

BTW typo: 35 #error "Atomic currently only impleneted for PPC64"

I also find the ppc_ prefix used in the assembly code somewhat redundant.


On 18/07/2013 1:34 AM, Lindenmaier, Goetz wrote:
> Hi,
> This time with webrev. Sorry for the double mails.
> This change contains all the files in cpu/ppc and os_cpu/linux_ppc needed for
> the PPC64 interpreter port on linux.
> With this change you can do a core build on ppc64 and run the VM interpreter only.
> It executes simple programs as jvm98.
> The change requires
> *         8016697: Use stubs to implement safefetch
> *         8020059: The flag introduced by 8014972 is not defined ...
> which will arrive soon in the staging repository.
> I marked the change as XL as it contains a lot of code.  Nevertheless the
> code has no impact on the existing Oracle platforms.
> The change touches a single shared file, globals.hpp, removing a
> special case introduced for the port.  This is because we
> integrated some changes earlier than originally intended.
> Please review the change.  Does it need testing on Oracle side?
> http://cr.openjdk.java.net/~goetz/webrevs/8019972-ppc_files/
> Best regards,
>    Goetz.

More information about the hotspot-dev mailing list