[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] x86/MSI: adjust permitted vector range
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Date: Mon, 2 Feb 2026 16:54:34 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=pxKF+nXhb81kRca0jah7A9axpTFmCqfPSnAmvOVeh6g=; b=Wj0zAwMUPLgf/4iUspRm6i5vw9zytbCl+6O74Tf4r1yU7Upa7+uUMM+gEebgMm3dtnX9c6hOzS5+jqwmPwm2T7/J/nGN4nq/323z4HuyNEgftflbOt4TKysj+loh6Sh4BKtayJVeflbTI5F71x5Db3NnaE+sCw8JwsOTyrMSIWTia7MngJSWWCI4xSe3fjaE+K7bYfJysPwm6rH44DSUcCxRPdbMJ+Ow9WWik7RGVbu4VDOi7GhGKe3VZKZQYH3iylMJpgQHEMG/yMm7QAbAv3ZInm0ya2JKzwjp1XYf9yMMcBUbMJhufJlEcmhSC/nnRCpQT5YjJMOHLsUalssITg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=nCV4cyEQe/jq0e0jyAMvGOwNbs9OFJnsoK+YErtmxR5yJQ7y5Y9DN5/2IhXZRviI5oNNwzpfQQJ0zpMw69IYxiWR3fhH+6wL+HzDzPp9CCtvPdFUiY9alEP9mROfclJhi8+mP1eo1mphDLbNmCJM6R+0azNcWyP+3K0KSPpmL/4g6oRJrdVsfCTdjqjVu1iQqMTmPzEY90KUt5ICojetr7B/11vplSYrKZuThfm9zboJEfWFL2/Iobz9Y/4MPDUEIXbPab9mjPhCy7q9TwsP0Sd9awti3xrO7bJlGJ49OCc9d9AyYAK3pUk0aW2B1XkuN2ZpJ0M9njlOM4Qrd3VNQA==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- Delivery-date: Mon, 02 Feb 2026 15:54:47 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
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 <jbeulich@xxxxxxxx>
> ---
> 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.
|
