Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Tian, Kevin" <kevin.tian@intel.com>
To:     "Chatre, Reinette" <reinette.chatre@intel.com>,
        "jgg@nvidia.com" <jgg@nvidia.com>,
        "yishaih@nvidia.com" <yishaih@nvidia.com>,
        "shameerali.kolothum.thodi@huawei.com" 
        <shameerali.kolothum.thodi@huawei.com>,
        "alex.williamson@redhat.com" <alex.williamson@redhat.com>
CC:     "tglx@linutronix.de" <tglx@linutronix.de>,
        "darwi@linutronix.de" <darwi@linutronix.de>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
        "Jiang, Dave" <dave.jiang@intel.com>,
        "Liu, Jing2" <jing2.liu@intel.com>,
        "Raj, Ashok" <ashok.raj@intel.com>,
        "Yu, Fenghua" <fenghua.yu@intel.com>,
        "tom.zanussi@linux.intel.com" <tom.zanussi@linux.intel.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH V5 03/11] vfio/pci: Prepare for dynamic interrupt context
 storage
Thread-Topic: [PATCH V5 03/11] vfio/pci: Prepare for dynamic interrupt context
 storage
Thread-Index: AQHZhB+NML0SOweBJ02fBu9B9TGuza9dwX7w
Date:   Wed, 17 May 2023 02:12:01 +0000
Message-ID: <BN9PR11MB5276B08334B4AD974D6302508C7E9@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <cover.1683740667.git.reinette.chatre@intel.com>
 <eab289693c8325ede9aba99380f8b8d5143980a4.1683740667.git.reinette.chatre@intel.com>
In-Reply-To: <eab289693c8325ede9aba99380f8b8d5143980a4.1683740667.git.reinette.chatre@intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?I2+We/WSn/aUOcjmF9YrtXF4jkgnFMdh2vYICtTUm6VkF9Ze/P8w8VIJdH9z?=
 =?us-ascii?Q?2mi/rijvWQyNtgN5fOkmTI4TTsXEiLGU6dngSin8jSUkHU0wyelbRqfbPoBO?=
 =?us-ascii?Q?vJT78IxoXuPibSYOGLpgSv9aD+wbRJkvGkmqlDe4jG7zFCgCs0nagjVSlvm1?=
 =?us-ascii?Q?VbSQr4Zme7I0vT4JJFmGPOADqpJeQr0SH9khFFAvmGzTo34LoKLhY0IjAIat?=
 =?us-ascii?Q?O0jYv1pZ0EYgigt++ueZCjVQR3/h7t0p/iv/nDxJGn83LcenKZ1xTWblfXhe?=
 =?us-ascii?Q?CPg1PPbUFemK4ZK8V+HEb6DOWzH45Tnr9ZHOqi/Ex03i5Hl5xmh7o7M4+z1i?=
 =?us-ascii?Q?LrEG9tlVxNEBxEqKU9LCsGaprY0EhFweFqnEDaB4Be8Btgyhhs/sQTwlV6lw?=
 =?us-ascii?Q?JJgehlpl2WmIdz/cf/KnIKc7K/QMZEMKsyjk7jadXa7/qQmBmlnA7fM+mY97?=
 =?us-ascii?Q?M1PFcMFkxuj8Eg5AXsvw5ndSMd7nR3XDS3HM99MfBw+8UHxLszAXF/VxP/qe?=
 =?us-ascii?Q?e+y2O/E7u3DPjN3+b0ao3YOWxlrGYOl0FtQmyJpFej6jpILzub6KWYDZzT+T?=
 =?us-ascii?Q?U7vp4z4WB1EMMqSme17akj1S4uFfcXhADLcaYp6E/LlCLoXxa/L26n/2Mg2U?=
 =?us-ascii?Q?LPyRVrvEhEuvtX3FgNyLYeB8or+XVwxIuDEwYu04jGx748FwbJChM4MMNv5V?=
 =?us-ascii?Q?qdRpsp8DfWP3H0FG/UOmeIE1kyLY1mZa1PD2vSTLpLqK9NyHr/FYEXNouNVd?=
 =?us-ascii?Q?43cnGopGFcOQQthMy2Mrr/dAXkrix42a9UBqRiDp0d9980LhFvucXzg+yDbV?=
 =?us-ascii?Q?eatsI80h8hStPfDWUNS1+cMqDcoXFBToDSAOYszDiDM9XPfidbvXuWvLIZDD?=
 =?us-ascii?Q?BipaFDr9f60Dv1DJ/TQRBxyDkcq6AxeGn3dDzJylhmrU9Z7r+PF7dc/s3S8I?=
 =?us-ascii?Q?T67dWe8FZuGPl4etNd9rcKaqNtGJEzm/XHHmneye0gxQiXnCvCIo08wcm7Iq?=
 =?us-ascii?Q?SolSCzLCo+lhqbfJfWhmG3pUZjs+ZNBfNJ18jY72EyMFA4rZA9QwmmjW+Lbb?=
 =?us-ascii?Q?gyTDUe4p8igMIev2wtNf+Lw5edpn74IumIsyOg3JZeIMs/WwCuxesZx9fTm8?=
 =?us-ascii?Q?TXByDXjnOTakytNakxFDl/TRNhu8HGLgIIhKETirMAZUWWo+SJQnd49ALwvy?=
 =?us-ascii?Q?J5c88SdyhS8Wm7PdlrC3V6PivF+rdPoINm4XZyzPPF4ekmwqeOUl9nsrPV7M?=
 =?us-ascii?Q?D3/QpZlRxy6p3ID91uFYQ39PfntFsFC2PJxDkmzXD6ILdZt7QMuxLYVkVYd+?=
 =?us-ascii?Q?iUrYhpG9zAdCH1F5Qt29Igcrb6rT/48JT9ifNcSajJM9ClQ9VuSbgJ9vflVR?=
 =?us-ascii?Q?Nk7RHk/ZAkjCUKY4nXzzTC4RfiZ8BtNPOdQYppK5XC1oNiuYxR9GK8sAcDbg?=
 =?us-ascii?Q?nrCAepGDAiiq42Tx3B7v6WEH4ZFQKAmdE/UEKZk7W0b4CQiibzqccmfv9yqd?=
 =?us-ascii?Q?S6phWijBEuOl3mqWQpHIfEFN9CHjS0GhnRpFhQvZvpy6QvCV7x9vTgiFEvQ1?=
 =?us-ascii?Q?A7QrMK9n0tZGbNPrmedE/hKlWO01w/PWRxk1Rgcm?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BN9PR11MB5276.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b18abab6-0416-4efa-d5c3-08db567c19bf
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 May 2023 02:12:01.0396
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: tZtPfbsjKrhAcbDtmRAf/Vpfs25VN7M+bHp0iR70mWASUrCggW8MGhBRs7ywhQvTHvlbYAC11qliWTEbcnszzg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB7568
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> From: Chatre, Reinette <reinette.chatre@intel.com>
> Sent: Thursday, May 11, 2023 11:45 PM
>=20
> Interrupt context storage is statically allocated at the time
> interrupts are allocated. Following allocation, the interrupt
> context is managed by directly accessing the elements of the
> array using the vector as index.
>=20
> It is possible to allocate additional MSI-X vectors after
> MSI-X has been enabled. Dynamic storage of interrupt context
> is needed to support adding new MSI-X vectors after initial
> allocation.
>=20
> Replace direct access of array elements with pointers to the
> array elements. Doing so reduces impact of moving to a new data
> structure. Move interactions with the array to helpers to
> mostly contain changes needed to transition to a dynamic
> data structure.
>=20
> No functional change intended.
>=20
> Signed-off-by: Reinette Chatre <reinette.chatre@intel.com>

Reviewed-by: Kevin Tian <kevin.tian@intel.com>