RFR: 8296401: ConcurrentHashTable::bulk_delete might miss to delete some objects [v8]

[Robbin Ehn]

On Mon, 16 Jan 2023 13:11:18 GMT, Leo Korinth <lkorinth at openjdk.org> wrote:

>> ConcurrentHashTable::bulk_delete might miss to delete some objects if a bucket has more than 256 entries. Current uses of ConcurrentHashTable are not harmed by this behaviour. 
>> 
>> I modified gtest:ConcurrentHashTable to detect the problem (first commit), and fixed the problem in the code (second commit).
>> 
>> Tests passes tier1-3.
>
> Leo Korinth has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision:
> 
>  - Merge branch '_master_jdk' into _8296401_ConcurrentHashTable
>  - rerun test
>  - growable array is limited by (signed) int, this should not be a problem
>  - GrowableArray is indexed by int :-(
>  - renam stack limit, and add comments
>  - Make ndel a reference to a pointer so that we can poison it. Also fix
>    POISON_PTR to be of correct type (the pointer is const, not the memory
>    pointed to).
>  - fix counting bug
>  - growable stacked
>  - Revert "working!"
>    
>    This reverts commit 5366f22c7202eaa2182976c084d02e9af4f56de0.
>  - working!
>  - ... and 1 more: https://git.openjdk.org/jdk/compare/e2024092...1b91238a

Marked as reviewed by rehn (Reviewer).

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

PR: https://git.openjdk.org/jdk/pull/10983