RFR (trivial): 8219888: aarch64 : add CPU detection code for HiSilicon TSV110

[Patrick Zhang OS]

The top comment “says” this if-clause was for ThunderX2, which looks a little strange.

I would suggest either we place all vendor specific descriptions together at the top, or have a new if-clause for each (much cleaner).


  // ThunderX2
  if ((_cpu == CPU_CAVIUM && (_model == 0xAF)) ||
      (_cpu == CPU_BROADCOM && (_model == 0x516)) ||
  // HiSilicon TSV110
      (_cpu == CPU_HISILICON && (_model == 0xd01))) {
    if (FLAG_IS_DEFAULT(AvoidUnalignedAccesses)) {
      FLAG_SET_DEFAULT(AvoidUnalignedAccesses, true);
    }
    if (FLAG_IS_DEFAULT(UseSIMDForMemoryOps)) {
      FLAG_SET_DEFAULT(UseSIMDForMemoryOps, true);
    }
  }



Regards

Patrick



-----Original Message-----

From: aarch64-port-dev <aarch64-port-dev-bounces at openjdk.java.net> On Behalf Of Yangfei (Felix)

Sent: Friday, March 1, 2019 9:23 AM

To: hotspot-runtime-dev at openjdk.java.net

Cc: aarch64-port-dev <aarch64-port-dev at openjdk.java.net>

Subject: [aarch64-port-dev ] RFR (trivial): 8219888: aarch64 : add CPU detection code for HiSilicon TSV110



Hi,



    Please review this trivial patch adding support for HiSilicon TSV110.

    Bug: https://bugs.openjdk.java.net/browse/JDK-8219888

    Webrev: http://cr.openjdk.java.net/~fyang/8219888/webrev.00/



    Tested on Huawei Kunpeng 920 server platform.  OK to push?



Thanks,

Felix