RFR: 8237842: Separate definitions for default cache line and padding sizes

Aleksey Shipilev shade at openjdk.org
Wed Dec 6 10:28:00 UTC 2023


On Wed, 6 Dec 2023 10:13:38 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> Well I bring this up because you are adding a comment that further sediments the understanding of the need to use *2. As I said I think the patch looks good and I'm just taking the opportunity to talk about this now that someone is yet again juggling around this *2 value. Changing to "some" is probably a good idea.

Yes, true. I rewrote the comment to hopefully much weaker wording that now implies we are not actually that sure `*2` adjustment is needed.

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

PR Comment: https://git.openjdk.org/jdk/pull/16973#issuecomment-1842594038


More information about the hotspot-dev mailing list