Re: [XEN PATCH] xen/arch/arm/traps: remove inlining of handle_ro_raz()

2023-06-15 Thread Julien Grall
Hi, On 15/06/2023 15:49, Michal Orzel wrote: Hi Federico, NIT: for Arm, we use "xen/arm: " as a component name in a commit title or if you want to also specify the sub component, then "xen/arm: : " e.g. "xen/arm: traps: " On 15/06/2023 15:50, Federico Serafini wrote: To comply with MISRA C

Re: [XEN PATCH] xen/arch/arm/traps: remove inlining of handle_ro_raz()

2023-06-15 Thread Michal Orzel
Hi Federico, NIT: for Arm, we use "xen/arm: " as a component name in a commit title or if you want to also specify the sub component, then "xen/arm: : " e.g. "xen/arm: traps: " On 15/06/2023 15:50, Federico Serafini wrote: > > > To comply with MISRA C:2012 Rule 8.10 ("An inline function shall

[XEN PATCH] xen/arch/arm/traps: remove inlining of handle_ro_raz()

2023-06-15 Thread Federico Serafini
To comply with MISRA C:2012 Rule 8.10 ("An inline function shall be declared with the static storage class"), remove inline function specifier from handle_ro_raz() since asking the compiler to inline such function does not seem to add any kind of value. Signed-off-by: Federico Serafini --- xen/a