private E dequeue() {
// assert takeLock.isHeldByCurrentThread();
Node h = head;
Node first = h.next;
h.next = h; // help GC
head = first;
E x = first.item;
first.item = null;
return x;
}

From what I understand, G1 marks the whole stack each collection, so shouldn’t it see that everything is dead right off the bat? I thought the problem was that the dead nodes in the tenured generation were keeping everything alive even in the new gen, at least until you do an expensive full (compacting?) gc?

Maybe the bug is over my head and I’m missing it…


private boolean casHead(Node<E> cmp, Node<E> val) {
return headUpdater.compareAndSet(this, cmp, val);
}


would become


private boolean casHead(Node<E> cmp, Node<E> val) {
final result = headUpdater.compareAndSet(this, cmp, val);
if (result) h.setNext(null);
return result;
}