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

[Goetz Lindenmaier]

> 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.

Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:

  Fix whitespace

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

Changes:
  - all: https://git.openjdk.org/jdk21u-dev/pull/1458/files
  - new: https://git.openjdk.org/jdk21u-dev/pull/1458/files/5f4fe4fa..3460e80a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1458&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1458&range=00-01

  Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 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