regarding backporting 8305056 to 17u

[Astigeevich, Evgeny]

Looked into the PR and found: https://github.com/openjdk/jdk/pull/13227#issuecomment-1519963429
The patch relies on inlining of ‘memcpy’ by compilers which might be a concern.


From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.org> on behalf of "Langer, Christoph" <christoph.langer at sap.com>
Date: Wednesday, 2 August 2023 at 13:33
To: Vladimir Kempik <vladimir.kempik at gmail.com>, jdk-updates-dev <jdk-updates-dev at openjdk.java.net>
Subject: RE: regarding backporting 8305056 to 17u

Hi Vladimir,

I think you should try the backport, regardless of it being in non risc-v only code.

Cheers
Christoph

From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.org> On Behalf Of Vladimir Kempik
Sent: Mittwoch, 2. August 2023 13:20
To: jdk-updates-dev <jdk-updates-dev at openjdk.java.net>
Subject: regarding backporting 8305056 to 17u

Hello maintainers of openjdk 17 udpates
I would like to know your opinion on backporting https://bugs.openjdk.org/browse/JDK-8305056 to 17u-dev before I start backport process
it’s important for risc-v to prevent misaligned load/stores when generating/patching executable code it opcodes aren’t naturally aligned.
Sadly this backport isn’t risc-v only, but I can try to use ifdefs to protect other ports.

Regards, Vladimir



Amazon Development Centre (London) Ltd. Registered in England and Wales with registration number 04543232 with its registered office at 1 Principal Place, Worship Street, London EC2A 2FA, United Kingdom.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/jdk-updates-dev/attachments/20230802/58f2feca/attachment-0001.htm>