RFR: 8251570: 8215624 causes assert(worker_id <' _n_workers) failed: Invalid worker_id(Internet mail)
linzang(臧琳)
linzang at tencent.com
Fri Aug 14 23:24:03 UTC 2020
Hi Stefan,
Thanks for help fix the issue. Sorry for not realize this bug in original patch.
> Note that even if the user tries to specify the Number of threads to use for the parallel heap inspection, there's no guarantee at all that we'll use that many threads. I didn't change that with this patch, but I'm not sure if that was apparent for the authors and reviewers of 8215624<tel:8215624>.
I considered the parallel heap inspection thread number has same behavior with ‘parallelGCThreads‘. And I think it is reasonable that user may not be guaranteed to get as many threads as they want.
Thanks,
Lin
On Aug 15, 2020, at 4:05 AM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
Thanks, Kim!
StefanK
On 2020-08-14 21:26, Kim Barrett wrote:
On Aug 14, 2020, at 2:25 PM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
https://cr.openjdk.java.net/~stefank/8251570/webrev.02.delta/
https://cr.openjdk.java.net/~stefank/8251570/webrev.02/
Still looks good. That is, I don’t see anything wrong with this one either.
More information about the hotspot-gc-dev
mailing list