Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2025-01-09 Thread Severin Gehwolf
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2025-01-09 Thread Severin Gehwolf
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2025-01-08 Thread Archie Cobbs
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2025-01-08 Thread Archie Cobbs
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2025-01-08 Thread Henry Jen
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2025-01-08 Thread Alan Bateman
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2025-01-07 Thread Severin Gehwolf
On Fri, 20 Dec 2024 07:53:07 GMT, Matthias Baesken wrote: >> Please review this fairly simple change to improve how the >> `JimageDiffGenerator` works. The original implementation is pretty naive and >> read all bytes into memory and then compared them. This improved version >> only reads byte

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-29 Thread Matthias Baesken
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-20 Thread Severin Gehwolf
On Fri, 20 Dec 2024 07:53:07 GMT, Matthias Baesken wrote: > Hi Severin, I added it to our build/test queue. Thanks, Matthias! - PR Comment: https://git.openjdk.org/jdk/pull/22835#issuecomment-2556610466

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-20 Thread Severin Gehwolf
On Fri, 20 Dec 2024 02:12:27 GMT, Fei Yang wrote: > Yes, I can confirm that it works on my linux-aarch64 platform. Great! Thanks! Thank you! - PR Comment: https://git.openjdk.org/jdk/pull/22835#issuecomment-2556609682

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-19 Thread Matthias Baesken
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-19 Thread Fei Yang
On Thu, 19 Dec 2024 18:16:47 GMT, Severin Gehwolf wrote: > @MBaesken @RealFYang Could you please test this PR since you originally ran > into JDK-8346239 (this bug) and > [JDK-8344036](https://bugs.openjdk.org/browse/JDK-8344036). Thanks! Yes, I can confirm that it works on my linux-aarch64 pl

Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-19 Thread Severin Gehwolf
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-19 Thread Severin Gehwolf
Please review this fairly simple change to improve how the `JimageDiffGenerator` works. The original implementation is pretty naive and read all bytes into memory and then compared them. This improved version only reads bytes on a bound buffer into memory compares those bytes and if equal conti