Re: [PATCH v1 03/17] xen/riscv: add missing APLIC register offsets, masks to asm/aplic.h

[Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>]

On 7/28/26 2:02 PM, Jan Beulich wrote:
> On 20.07.2026 18:02, Oleksii Kurochko wrote:
> > @@ -25,6 +27,7 @@
> >   #define APLIC_DOMAINCFG_DM      BIT(2, U)
> >   #define APLIC_DOMAINCFG_BE      BIT(0, U)
> >   
> > +/* sourcecfg register fields */
> >   #define APLIC_SOURCECFG_SM_INACTIVE     0x0
> >   #define APLIC_SOURCECFG_SM_DETACH       0x1
> >   #define APLIC_SOURCECFG_SM_EDGE_RISE    0x4
> > @@ -32,7 +35,39 @@
> >   #define APLIC_SOURCECFG_SM_LEVEL_HIGH   0x6
> >   #define APLIC_SOURCECFG_SM_LEVEL_LOW    0x7
> >   
> > +/* target register fields */
> >   #define APLIC_TARGET_HART_IDX_SHIFT 18
> > +#define APLIC_TARGET_EIID_MASK      0x7ff
> So why would these live here, far ahead of ...
>
> > +#define APLIC_DOMAINCFG         0x0000
> > +#define APLIC_SOURCECFG_BASE    0x0004
> > +#define APLIC_SOURCECFG_LAST    0x0ffc
> > +
> > +#define APLIC_SMSICFGADDR       0x1bc8
> > +#define APLIC_SMSICFGADDRH      0x1bcc
> > +
> > +#define APLIC_SETIP_BASE        0x1c00
> > +#define APLIC_SETIP_LAST        0x1c7c
> > +#define APLIC_SETIPNUM          0x1cdc
> > +
> > +#define APLIC_CLRIP_BASE        0x1d00
> > +#define APLIC_CLRIP_LAST        0x1d7c
> > +#define APLIC_CLRIPNUM          0x1ddc
> > +
> > +#define APLIC_SETIE_BASE        0x1e00
> > +#define APLIC_SETIE_LAST        0x1e7c
> > +#define APLIC_SETIENUM          0x1edc
> > +
> > +#define APLIC_CLRIE_BASE        0x1f00
> > +#define APLIC_CLRIE_LAST        0x1f7c
> > +#define APLIC_CLRIENUM          0x1fdc
> > +
> > +#define APLIC_SETIPNUM_LE       0x2000
> > +
> > +#define APLIC_GENMSI            0x3000
> > +
> > +#define APLIC_TARGET_BASE       0x3004
> > +#define APLIC_TARGET_LAST       0x3ffc
> ... the register definition itself. Would you mind taking a look at the top
> third (or so) of arch/x86/include/asm/msr-index.h? There you'll find MSR
> index values of MSRs we use, immediately followed by field definitions (of
> course only where applicable). With such an arrangement, the extra comments
> you add can easily be omitted.
I'll move the definitions mentioned above to the suggested location and 
follow this pattern in future changes. I just thought it would be better 
to keep the APLIC register definitions together in memory layout order.
Thanks.

~ Oleksii