Re: RFR: 8189685: need PerfMemory class update and a volatile_static_field support in VMStructs [v4]

2023-08-29 Thread Chris Plummer
On Wed, 23 Aug 2023 18:26:03 GMT, Chris Plummer wrote: >> During [JDK-8151815](https://bugs.openjdk.org/browse/JDK-8151815) it was >> noted that the PerfMemory _initialized and _destroyed fields should be >> volatile, but VMStructs didn't have the needed support for doing that, so it >> was le

Re: RFR: 8189685: need PerfMemory class update and a volatile_static_field support in VMStructs [v4]

2023-08-26 Thread Yasumasa Suenaga
On Wed, 23 Aug 2023 18:26:03 GMT, Chris Plummer wrote: >> During [JDK-8151815](https://bugs.openjdk.org/browse/JDK-8151815) it was >> noted that the PerfMemory _initialized and _destroyed fields should be >> volatile, but VMStructs didn't have the needed support for doing that, so it >> was le

Re: RFR: 8189685: need PerfMemory class update and a volatile_static_field support in VMStructs [v4]

2023-08-25 Thread Serguei Spitsyn
On Wed, 23 Aug 2023 18:26:03 GMT, Chris Plummer wrote: >> During [JDK-8151815](https://bugs.openjdk.org/browse/JDK-8151815) it was >> noted that the PerfMemory _initialized and _destroyed fields should be >> volatile, but VMStructs didn't have the needed support for doing that, so it >> was le

Re: RFR: 8189685: need PerfMemory class update and a volatile_static_field support in VMStructs [v4]

2023-08-25 Thread David Holmes
On Wed, 23 Aug 2023 18:26:03 GMT, Chris Plummer wrote: >> During [JDK-8151815](https://bugs.openjdk.org/browse/JDK-8151815) it was >> noted that the PerfMemory _initialized and _destroyed fields should be >> volatile, but VMStructs didn't have the needed support for doing that, so it >> was le

Re: RFR: 8189685: need PerfMemory class update and a volatile_static_field support in VMStructs [v4]

2023-08-23 Thread Chris Plummer
> During [JDK-8151815](https://bugs.openjdk.org/browse/JDK-8151815) it was > noted that the PerfMemory _initialized and _destroyed fields should be > volatile, but VMStructs didn't have the needed support for doing that, so it > was left as a future task. @YaSuenag provided a patch at the time t