Re: RFR: 8350903: Remove explicit libjvm.so dependency for libVThreadEventTest

2025-03-07 Thread Jiangli Zhou
On Mon, 3 Mar 2025 17:36:00 GMT, Jiangli Zhou wrote: >> Please review the test fix that removes `libVThreadEventTest` explicit >> dependency to `libjvm`, by removing the call to `JNI_GetCreatedJavaVMs` in >> `Agent_OnAttach`. There is a `vm` argument passed via `Agent_OnAttach`. With >> the ch

Re: RFR: 8351322: Parameterize link option for pthreads

2025-03-07 Thread snake66
On Thu, 6 Mar 2025 13:53:31 GMT, Antonio Vieiro wrote: >> Replace hardcoded instances of `-lpthread` with `$(LIBPTHREAD)`, so that >> it's possible to parameterize this for platforms that use different flags >> for enabling posix threads. >> >> This work is a continuation of the work done by G

Re: RFR: 8350903: Remove explicit libjvm.so dependency for libVThreadEventTest

2025-03-07 Thread Magnus Ihse Bursie
On Mon, 3 Mar 2025 17:36:00 GMT, Jiangli Zhou wrote: >> Please review the test fix that removes `libVThreadEventTest` explicit >> dependency to `libjvm`, by removing the call to `JNI_GetCreatedJavaVMs` in >> `Agent_OnAttach`. There is a `vm` argument passed via `Agent_OnAttach`. With >> the ch

Integrated: 8347433: Deprecate XML interchange in java.management/javax/management/modelmbean/DescriptorSupport for removal

2025-03-07 Thread Kevin Walls
On Fri, 10 Jan 2025 14:43:00 GMT, Kevin Walls wrote: > DescriptorSupport has a constructor and a method providing creation from, and > export to, XML. > > These are unused in the JDK and have no practical known examples of usage. > XML parsing is best done by an independent implementation, no

Re: RFR: 8351322: Parameterize link option for pthreads

2025-03-07 Thread Magnus Ihse Bursie
On Fri, 7 Mar 2025 00:18:14 GMT, David Holmes wrote: >>> What is the intended way of using this? Do you run make with >>> LIBPTHREAD=-pthread or do you apply a patch on libraries.m4 for the >>> specific way of linking to pthread? >> >> This is in preparation of the upcoming BSD port, which use

Re: RFR: 8343191: Cgroup v1 subsystem fails to set subsystem path [v17]

2025-03-07 Thread Sergey Chernyshev
On Fri, 7 Mar 2025 04:34:24 GMT, David Holmes wrote: >>> OK for me now. `test_cgroupSubsystem_linux.cpp` needs a copyright update as >>> well. >> >> Thanks for your review @jerboaa ! I cheched the >> test_cgroupSubsystem_linux.cpp, it's already updated to 2025 in the master >> branch. > > @se