Skip to content
Permalink
Browse files
8273381: Assert in PtrQueueBufferAllocatorTest.stress_free_list_alloc…
…ator_vm

Reviewed-by: sjohanss, tschatzl
  • Loading branch information
Kim Barrett committed Oct 6, 2021
1 parent 9759fcb commit c80a612709f1e483575e2843c1f0ea84e1b9a257
Showing 1 changed file with 7 additions and 1 deletion.
@@ -180,12 +180,18 @@ class BufferNode::TestSupport::ProcessorThread : public JavaTestThread {
{}

virtual void main_run() {
bool shutdown_requested = false;
while (true) {
BufferNode* node = _cbl->pop();
if (node != NULL) {
_allocator->release(node);
} else if (!Atomic::load_acquire(_continue_running)) {
} else if (shutdown_requested) {
return;
} else if (!Atomic::load_acquire(_continue_running)) {
// To avoid a race that could leave buffers in the list after this
// thread has shut down, continue processing until the list is empty
// *after* the shut down request has been received.
shutdown_requested = true;
}
ThreadBlockInVM tbiv(this); // Safepoint check.
}

3 comments on commit c80a612

@openjdk-notifier
Copy link

@openjdk-notifier openjdk-notifier bot commented on c80a612 Oct 6, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@GoeLin
Copy link
Member

@GoeLin GoeLin commented on c80a612 Jan 28, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/backport jdk17u-dev

@openjdk
Copy link

@openjdk openjdk bot commented on c80a612 Jan 28, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@GoeLin the backport was successfully created on the branch GoeLin-backport-c80a6127 in my personal fork of openjdk/jdk17u-dev. To create a pull request with this backport targeting openjdk/jdk17u-dev:master, just click the following link:

➡️ Create pull request

The title of the pull request is automatically filled in correctly and below you find a suggestion for the pull request body:

Hi all,

This pull request contains a backport of commit c80a6127 from the openjdk/jdk repository.

The commit being backported was authored by Kim Barrett on 6 Oct 2021 and was reviewed by Stefan Johansson and Thomas Schatzl.

Thanks!

If you need to update the source branch of the pull then run the following commands in a local clone of your personal fork of openjdk/jdk17u-dev:

$ git fetch https://github.com/openjdk-bots/jdk17u-dev GoeLin-backport-c80a6127:GoeLin-backport-c80a6127
$ git checkout GoeLin-backport-c80a6127
# make changes
$ git add paths/to/changed/files
$ git commit --message 'Describe additional changes made'
$ git push https://github.com/openjdk-bots/jdk17u-dev GoeLin-backport-c80a6127

Please sign in to comment.