RFR: 8295282: Use Zicboz/cbo.zero to zero-out memory on RISC-V [v11]
Ludovic Henry
luhenry at openjdk.org
Tue Oct 25 09:23:51 UTC 2022
On Tue, 25 Oct 2022 08:07:21 GMT, Fei Yang <fyang at openjdk.org> wrote:
>>> I overall agree with the 64 bytes is an industry standard. What I want to ensure is that we can eagerly enable `UseBlockZeroing` whenever we can.
>>
>> I strongly disagree.
>>
>> Until processor reports Zic64b supported we should assume the cache line size could be anything (2^N). I'm running some tests on risc-v fpga core and it has 16-bytes cache block size.
>> having to patch ( change default cache line size) and rebuild openjdk to just try new Zicbo{z,p,m} opcodes ( once they are implemented) - isn't a nice way
>
>> > I overall agree with the 64 bytes is an industry standard. What I want to ensure is that we can eagerly enable `UseBlockZeroing` whenever we can.
>>
>> I strongly disagree.
>>
>> Until processor reports Zic64b supported we should assume the cache line size could be anything (2^N). I'm running some tests on risc-v fpga core and it has 16-bytes cache block size. having to patch ( change default cache line size) and rebuild openjdk to just try new Zicbo{z,p,m} opcodes ( once they are implemented) - isn't a nice way
>
> Now I see the concern here. I agree to have the an option for cache-line size in that respect.
I'll revert to what I had to set CacheLineSize then.
-------------
PR: https://git.openjdk.org/jdk/pull/10718
More information about the hotspot-dev
mailing list