[jdk21u-dev] RFR: 8319439: Move BufferNode from PtrQueue files to new files

Goetz Lindenmaier goetz at openjdk.org
Fri Mar 7 19:48:24 UTC 2025


I would like to bacvkport this change to make future 
backports easier.

It is low risk, so this is feasible although it does not fix
a bug. It is a pure move of code to a new file. The code is unchanged.
At most, it might slightly affect the size of libjvm.so.

I had to do some resolkves. 
Especially the moved code differes. 
So I moved the classes by hand. 
They differ because 
8314276: Improve PtrQueue API around size/capacity
8309899: Rename PtrQueueSet::buffer_size()
are not in 21.
The latter would be a candidate for backport as well, 
it should have low risk. But resolving against 
function names is not as annoying as applying
the changes in other files. 
What do you think?

Resolved files in detail:

src/hotspot/share/gc/g1/g1YoungGCPostEvacuateTasks.cpp
src/hotspot/share/gc/g1/jvmFlagConstraintsG1.cpp
Resolved includes.

src/hotspot/share/gc/shared/ptrQueue.cpp
src/hotspot/share/gc/shared/ptrQueue.hpp
>From these files, I copied the deleted content to
the two new files.
I verified that the original change copies these
without changing them as well.

test/hotspot/gtest/gc/shared/test_bufferNodeAllocator.cpp
Trivial resolve.

-------------

Commit messages:
 - Resolved files.
 - Backport 1e76ba0cd0de38da6fc2b8147627496ee2f2837b

Changes: https://git.openjdk.org/jdk21u-dev/pull/1458/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1458&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8319439
  Stats: 311 lines in 17 files changed: 185 ins; 118 del; 8 mod
  Patch: https://git.openjdk.org/jdk21u-dev/pull/1458.diff
  Fetch: git fetch https://git.openjdk.org/jdk21u-dev.git pull/1458/head:pull/1458

PR: https://git.openjdk.org/jdk21u-dev/pull/1458


More information about the jdk-updates-dev mailing list