RFR: 8303022: "assert(allocates2(pc)) failed: not in CodeBuffer memory" When linking downcall handle [v2]

[Jorn Vernee]

On Wed, 8 Mar 2023 18:40:05 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update copyright years
>
> I think you need to add these changes to other ports too and ask for testing on them.

@vnkozlov Yes, this is true. The only other existing port of this code is RISCV. However, to fix that port properly, someone needs to repeat the experiment on RISCV in order to figure out what the base size and the size per argument should be.

I don't have access to a RISCV machine, so I figured I would file a followup issue for the RISCV maintainers to fix separately.

@feilongjiang Could you comment on this? If you could figure out the needed sizes for RISCV I could add the needed changes to this patch. Otherwise I could file a followup issue if that seems more convenient.

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

PR: https://git.openjdk.org/jdk/pull/12908