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 V4 04/11] vfio/pci: Move to single error path
Thread-Topic: [PATCH V4 04/11] vfio/pci: Move to single error path
Thread-Index: AQHZeS7Myrfqz9PnuEuDQL/XmYyy8q9ARFqw
Date:   Fri, 28 Apr 2023 06:34:08 +0000
Message-ID: <BN9PR11MB52768E041B430C7F98D9FB278C6B9@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <cover.1682615447.git.reinette.chatre@intel.com>
 <521fd5184ef052ff768a90bbe670cfc4e375eff9.1682615447.git.reinette.chatre@intel.com>
In-Reply-To: <521fd5184ef052ff768a90bbe670cfc4e375eff9.1682615447.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?I/PplqdlOPRV11Nyf6EfvdBx+xc+1iXWVocTXAb+TS1ACaAu5hAmCW9C7umC?=
 =?us-ascii?Q?0vAFxeZHOgpZ2Bw0miUuC6BAuD8PJkFZp7j3Jzbj6ZNtU/Lnp59RSvi7yJYd?=
 =?us-ascii?Q?qNj0a1prcj6YkhuqkiDv+Guw2Wp2hVEf05Th8H4A1WbmhIRRi9+NsI4lYwdl?=
 =?us-ascii?Q?6S9m4bWnTn3K6ARRMmNpdXJv52F2iBb+QfIfHCgz/4hHeeBDj7y2pdrO5uM6?=
 =?us-ascii?Q?KGTDGfRJxGiTVdeX8Pe962ma90ziX41IOxXEPBfnnThDuBFx5m96dT3rE+19?=
 =?us-ascii?Q?FDNYJLBlnGhXmDQw5b7oXYROcI2EqSzpIRjNSqKKQBlGchCKjeadj2eealuH?=
 =?us-ascii?Q?LpBH79KDwGVTZaMBZ/OdNRsurgWvd31jEJZOncLVSQaAUHsd5tBP5RToNur4?=
 =?us-ascii?Q?S5Q4HLahCaXq7Ehy7EJsqYYTS/GAUcOSyeL4+uoEYGFY57DboE4YiJbuIYPX?=
 =?us-ascii?Q?Qufk4gUHl1klKrkDlpXlx8O1xJebbgMpIzo7q0+2OoALSSH7y1Oc0uKdNMbf?=
 =?us-ascii?Q?6hHCtm7Abi+G3FEjQLojDfJ2zr5IzaNAzIHY3/gz+1ZG8B0IxaVjhDP7LtxT?=
 =?us-ascii?Q?0y03F2iQ30eHiM/odnoaQCRDa1oFvZx4AnadJXr8XiCM4s7i24R6L0ByUPts?=
 =?us-ascii?Q?mXd7Si0WZGYm/wSpRawmfZlMRaDiA30nitaIrRaaA1FwhLymBnAzRkD+yUEs?=
 =?us-ascii?Q?3Myc9kKKb52NowdXdvmamyyLLt8HqEf6e7GLW4e0OCWMEkKVLUiGm3PuMZ+t?=
 =?us-ascii?Q?YwLXqWzW3iYyzOtmQkiW+nHCY+GiAe62OT+Kt+HqesZQiC+l3nyM8fHlfXXK?=
 =?us-ascii?Q?7FIr6aXSX9lMDibJ90DCAwgjo3EWs7+SZTsxpYvhhutu/V12XXPwTvYSGQsS?=
 =?us-ascii?Q?YulDlZFfsDQCmGo5s9cDdU9gRAMzU9BrAGpZRWxxUfXFGX3tmgg71MTBI7zk?=
 =?us-ascii?Q?kOcAN+xnXMWTxOz3Q1KNE3u4g2DuyRXBAtVGrjBi0l79SzeR4LzC7LN3npKj?=
 =?us-ascii?Q?W0XZrfa69897aWeBn51kU5hol/gyGXnpRpMd6Ep1WimnObnZedQI9tcYMuO+?=
 =?us-ascii?Q?PMAXALVZeINbUMqOGzeubyFi0RkNowfP+mkJP8H9yTRyULwS+qxmgYWxPvar?=
 =?us-ascii?Q?dyExDo/c3mzRFP8pOU0bAMxMhzHNpKOsDgV2k2j3NXGLeFQOdGCyGt7Z7aDb?=
 =?us-ascii?Q?eHHktEvlLhqeLNMvwK5z+vCTgvG95HqClWVgfJv37mgOTlz8snQv6g3fCEfX?=
 =?us-ascii?Q?e4mdr4I86aJ+06zyl23UviqBrOY5YLU2aOJHnFhqUPprvqYu1z14eWF4vc1n?=
 =?us-ascii?Q?sWG+s4+FGQKsXOqmhurMeGKDoOlBO1aR9DkDCe/69ghc831NncOuhZpPET3C?=
 =?us-ascii?Q?jlS25KrZwG8UE6OWm2yHobx1SqcHD+M2xMoCHq24+1lL5yQDExB7Xj+N6hah?=
 =?us-ascii?Q?470sGAlZfq9NPePwCu5DF6x7/8MfrXYW5lwqJW/oJ7fPwpexE5daGEvt5pR8?=
 =?us-ascii?Q?Gt6n3IEd0Nqxtv+ue6TVPAGTt8xEyzpbD8yl/R9QUSafojjAoElb92qY80zH?=
 =?us-ascii?Q?Wupcs31hZ6UxMvZMPBSWc5965FLp1LjeXvSnLo3U?=
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: 146fc405-2b28-469f-5f39-08db47b29257
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Apr 2023 06:34:08.7455
 (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: c6BlzzNgnPMKMh6JCGrTa0pyUsIKRIcXWTynTTPruNum5fyTEoBaPMKf1/dPXU7lK6kwUXtKbrf96qM6bD6ijw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5581
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_PASS,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: Friday, April 28, 2023 1:36 AM
>=20
> Enabling and disabling of an interrupt involves several steps
> that can fail. Cleanup after failure is done when the error
> is encountered, resulting in some repetitive code.
>=20
> Support for dynamic contexts will introduce more steps during
> interrupt enabling and disabling.
>=20
> Transition to centralized exit path in preparation for dynamic
> contexts to eliminate duplicate error handling code.
>=20
> Signed-off-by: Reinette Chatre <reinette.chatre@intel.com>

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