On 11.12.2024 18:27, Oleksii Kurochko wrote: > Make all upper bounds (end addresses) for areas inclusive to align > with the corresponding definitions. > > Signed-off-by: Oleksii Kurochko <oleksii.kuroc...@gmail.com>
I expected this would be straightforward to ack, but ... > --- a/xen/arch/riscv/include/asm/config.h > +++ b/xen/arch/riscv/include/asm/config.h > @@ -41,17 +41,17 @@ > * Start addr | End addr | Slot | area description > * > ============================================================================ > * ..... L2 511 Unused > - * 0xffffffffc0a00000 0xffffffffc0c00000 L2 511 Fixmap > + * 0xffffffffc0a00000 0xffffffffc0bfffff L2 511 Fixmap > * ..... ( 2 MB gap ) > - * 0xffffffffc0400000 0xffffffffc0800000 L2 511 FDT > + * 0xffffffffc0400000 0xffffffffc07fffff L2 511 FDT > * ..... ( 2 MB gap ) > - * 0xffffffffc0000000 0xffffffffc0200000 L2 511 Xen > + * 0xffffffffc0000000 0xffffffffc01fffff L2 511 Xen > * ..... L2 510 Unused > - * 0x3200000000 0x7f40000000 L2 200-509 Direct map > + * 0x3200000000 0x7f7fffffff L2 200-509 Direct map ... this isn't just an adjustment by -1. If the old table entry was wrong, the description wants to say so. Else an adjustment to the number is needed here. Jan
