On Wed, Nov 12, 2025 at 04:45:05PM +0100, Jan Beulich wrote: > It's really FIRST_IRQ_VECTOR vector which is meant here, i.e. effectively > there is a form of open-coding in use right now. > > No change in practice, due to the present aliasing. > > Signed-off-by: Jan Beulich <[email protected]> > --- > While doing the HPET work I had to fiddle with this, even if in the end > no vector below FIRST_DYNAMIC_VECTOR is being used there (for now at > least). > > --- a/xen/arch/x86/msi.c > +++ b/xen/arch/x86/msi.c > @@ -158,7 +158,7 @@ void msi_compose_msg(unsigned vector, co > { > memset(msg, 0, sizeof(*msg)); > > - if ( vector < FIRST_DYNAMIC_VECTOR ) > + if ( vector < FIRST_IRQ_VECTOR )
Should vector also be rejected if it's > LAST_IRQ_VECTOR? Thanks, Roger.
