Re: [PATCH v5 03/23] xen/riscv: introduce nospec.h

[Julien Grall <julien@xxxxxxx>]

Hi Oleksii,

On 26/02/2024 17:38, Oleksii Kurochko wrote:
>  From the unpriviliged doc:
>    No standard hints are presently defined.
>    We anticipate standard hints to eventually include memory-system spatial
>    and temporal locality hints, branch prediction hints, thread-scheduling
>    hints, security tags, and instrumentation flags for simulation/emulation.
>
> Also, there are no speculation execution barriers.
>
> Therefore, functions evaluate_nospec() and block_speculation() should
> remain empty until a specific platform has an extension to deal with
> speculation execution.
>
> Signed-off-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
> ---
>   Changes in V5:
>     - new patch
> ---
>   xen/arch/riscv/include/asm/nospec.h | 25 +++++++++++++++++++++++++
>   1 file changed, 25 insertions(+)
>   create mode 100644 xen/arch/riscv/include/asm/nospec.h
>
> diff --git a/xen/arch/riscv/include/asm/nospec.h 
> b/xen/arch/riscv/include/asm/nospec.h
> new file mode 100644
> index 0000000000..4fb404a0a2
> --- /dev/null
> +++ b/xen/arch/riscv/include/asm/nospec.h
> @@ -0,0 +1,25 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
New file should use the SPDX tag GPL-2.0-only. I guess this could be 
fixed on commit?
Cheers,

--
Julien Grall