[riscv-port-jdk11u:riscv-port] RFR: 8291893: riscv: remove fence.i used in user space [v2]

Fei Yang fyang at openjdk.org
Thu Mar 28 01:56:45 UTC 2024


On Wed, 27 Mar 2024 08:23:15 GMT, Gui Cao <gcao at openjdk.org> wrote:

>> Please help review this backport to riscv-port-jdk11u.
>> Backport of [JDK-8291893](https://bugs.openjdk.org/browse/JDK-8291893)
>> 
>> In addition to the backport [8291947](https://bugs.openjdk.org/browse/JDK-8291947), [8310656](https://bugs.openjdk.org/browse/JDK-8310656) to align and minimize the difference with 17u upstream.
>> 
>> Testing:
>> 
>> - [x] Run tier1-3 tests on SOPHON SG2042 (release)
>
> Gui Cao has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   8310656: RISC-V: __builtin___clear_cache can fail silently.

src/hotspot/os_cpu/linux_riscv/riscv_flush_icache.hpp line 28:

> 26: #ifndef OS_LINUX_RISCV_FLUSH_ICACHE_LINUX_HPP
> 27: #define OS_LINUX_RISCV_FLUSH_ICACHE_LINUX_HPP
> 28: 

I think we should add one extra include here: `#include "memory/allocation.hpp"`.

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

PR Review Comment: https://git.openjdk.org/riscv-port-jdk11u/pull/16#discussion_r1542218546


More information about the riscv-port-dev mailing list