On Mon, 5 Oct 2020 14:29:02 GMT, Lin Zang <lz...@openjdk.org> wrote: >> Hi @linzang, >> >> I started reviewing your patch in more detail. However, I ended up with a >> large number of major comments and I started >> to feel that it would probably be more productive to create an alternative >> patch for this, so I did. >> https://github.com/openjdk/jdk/compare/master...pliden:8252105_parallel_object_iterator >> >> Some of the main differences compared to your proposed patch: >> * Root scanning is also now done in parallel >> * Same code paths regardless of number of worker threads used >> * Arrays are chunked for better work stealing/balance >> * A terminator is used for better work stealing/balance >> * All of the functionality is isolated (as much as possible) to ZHeapIterator >> >> I think the easiest way forward is probably if you close this PR, and I'll >> open a new one. > > Hi @pliden, > Sorry for late reply. > I will close this PR and let's discuss in your new one. > > -Lin
@linzang Ok, thanks. I created https://github.com/openjdk/jdk/pull/507. ------------- PR: https://git.openjdk.java.net/jdk/pull/103
