RFR: 8354668: Missing REX2 prefix accounting in ZGC barriers leads to incorrect encoding [v3]

[Dean Long]

On Thu, 17 Apr 2025 03:21:08 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> ZGC bookkeeps multiple place holders in barrier code snippets through relocations, these are later used to patch appropriate contents (mostly immediate values) in instruction encoding. While most of the relocation records the patching offsets from the end of the instruction, SHL instruction, which is used for pointer coloring, computes the patching offset from the starting address of the instruction.
>> 
>> Thus, in case the destination register operand of SHL instruction is an extended GPR register, we miss accounting additional REX2 prefix byte in patch offset, thereby corrupting the encoding since runtime patches the primary opcode byte resulting into ILLEGAL instruction exception.
>> 
>> This patch fixes reported failures by computing the relocation offset of SHL instruction from end of instruction, thereby making the patch offset agnostic to REX/REX2 prefix.
>> 
>> Please review and share your feedback.
>> 
>> Best Regards,
>> Jatin
>> 
>> PS: Validation were performed using latest Intel Software Development Emulator after modifying static register allocation order in x86_64.ad file giving preference to EGPRs.
>
> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comments resolutions

When I made my suggestions, I didn't realize it would also require changes on the Graal side.  So I would suggest a separate PR only if the Graal team agrees.

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

PR Comment: https://git.openjdk.org/jdk/pull/24664#issuecomment-2813856674