RFR: 8376287: Crashes when using -XX:ObjArrayMarkingStride=0 [v2]

[Thomas Schatzl]

On Thu, 29 Jan 2026 08:28:01 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Guanqiang Han has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove unused assert
>
> Looks good, but the corresponding sanity assert (ObjArrayMarkingStride > 0) could be removed in Shenandoah now.
> 
> After some investigation we found that not only G1 is affected, but also Serial and Parallel GC to a varying degree, so the summary of the CR changed.

> Hi @tschatzl , I've removed the unused assert and updated the CR summary. Could you please take a look? Hi @shipilev , thank you for the review.

Looking in more detail on the uses of `ObjArrayMarkingStride`, I see that it is cast to `int`, and at least one calculation multiplies it with 2 before casting to int.

So it must be <= `MAX_INT / 2`.

Apologies for not looking in more detail earlier.

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

PR Comment: https://git.openjdk.org/jdk/pull/29479#issuecomment-3817194993