Re: [PATCH v3 15/19] xen: Update header guards - RISC-V

[Jan Beulich <jbeulich@xxxxxxxx>]

On 04.10.2024 10:17, Frediano Ziglio wrote:
> Updated headers related to RISC-V.
> 
> Signed-off-by: Frediano Ziglio <frediano.ziglio@xxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

> ---
>  xen/arch/riscv/include/asm/acpi.h           | 6 +++---
>  xen/arch/riscv/include/asm/asm.h            | 6 +++---
>  xen/arch/riscv/include/asm/atomic.h         | 6 +++---
>  xen/arch/riscv/include/asm/bitops.h         | 6 +++---
>  xen/arch/riscv/include/asm/bug.h            | 6 +++---
>  xen/arch/riscv/include/asm/byteorder.h      | 6 +++---
>  xen/arch/riscv/include/asm/cache.h          | 6 +++---
>  xen/arch/riscv/include/asm/cmpxchg.h        | 6 +++---
>  xen/arch/riscv/include/asm/config.h         | 6 +++---
>  xen/arch/riscv/include/asm/cpufeature.h     | 6 +++---
>  xen/arch/riscv/include/asm/csr.h            | 6 +++---
>  xen/arch/riscv/include/asm/current.h        | 6 +++---
>  xen/arch/riscv/include/asm/domain.h         | 6 +++---
>  xen/arch/riscv/include/asm/early_printk.h   | 6 +++---
>  xen/arch/riscv/include/asm/event.h          | 6 +++---
>  xen/arch/riscv/include/asm/fence.h          | 6 +++---
>  xen/arch/riscv/include/asm/fixmap.h         | 6 +++---
>  xen/arch/riscv/include/asm/flushtlb.h       | 6 +++---
>  xen/arch/riscv/include/asm/guest_access.h   | 6 +++---
>  xen/arch/riscv/include/asm/guest_atomics.h  | 6 +++---
>  xen/arch/riscv/include/asm/io.h             | 6 +++---
>  xen/arch/riscv/include/asm/irq.h            | 6 +++---
>  xen/arch/riscv/include/asm/mm.h             | 6 +++---
>  xen/arch/riscv/include/asm/monitor.h        | 6 +++---
>  xen/arch/riscv/include/asm/nospec.h         | 6 +++---
>  xen/arch/riscv/include/asm/p2m.h            | 6 +++---
>  xen/arch/riscv/include/asm/page-bits.h      | 6 +++---
>  xen/arch/riscv/include/asm/page.h           | 6 +++---
>  xen/arch/riscv/include/asm/pmap.h           | 6 +++---
>  xen/arch/riscv/include/asm/processor.h      | 6 +++---
>  xen/arch/riscv/include/asm/regs.h           | 6 +++---
>  xen/arch/riscv/include/asm/riscv_encoding.h | 4 ++--
>  xen/arch/riscv/include/asm/sbi.h            | 6 +++---
>  xen/arch/riscv/include/asm/setup.h          | 6 +++---
>  xen/arch/riscv/include/asm/smp.h            | 4 ++--
>  xen/arch/riscv/include/asm/spinlock.h       | 6 +++---
>  xen/arch/riscv/include/asm/string.h         | 6 +++---
>  xen/arch/riscv/include/asm/system.h         | 6 +++---
>  xen/arch/riscv/include/asm/time.h           | 6 +++---
>  xen/arch/riscv/include/asm/traps.h          | 6 +++---
>  xen/arch/riscv/include/asm/types.h          | 6 +++---
>  41 files changed, 121 insertions(+), 121 deletions(-)

For the bulk of the changes you didn't Cc anyone who can ack them. Note that
the RISCV section of ./MAINTAINERS presently has only R: entries. You need
to Cc THE REST in such a case.

It is probably also a good idea to Cc the person(s) doing the bulk of the
work in a given area at a given point in time. Then again, Oleksii, maybe
you want to consider adding yourself as R: in that section, too? That way
you will be Cc-ed on patches (as long as people respect ./MAINTAINERS).

Jan