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:     Eric Farman <farman@linux.ibm.com>,
        Matthew Rosato <mjrosato@linux.ibm.com>,
        Alex Williamson <alex.williamson@redhat.com>,
        Cornelia Huck <cohuck@redhat.com>,
        Jason Gunthorpe <jgg@nvidia.com>,
        "Liu, Yi L" <yi.l.liu@intel.com>
CC:     Zhenyu Wang <zhenyuw@linux.intel.com>,
        "Wang, Zhi A" <zhi.a.wang@intel.com>,
        Jani Nikula <jani.nikula@linux.intel.com>,
        "Joonas Lahtinen" <joonas.lahtinen@linux.intel.com>,
        "Vivi, Rodrigo" <rodrigo.vivi@intel.com>,
        Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>,
        "Halil Pasic" <pasic@linux.ibm.com>,
        Vineeth Vijayan <vneethv@linux.ibm.com>,
        "Peter Oberparleiter" <oberpar@linux.ibm.com>,
        Heiko Carstens <hca@linux.ibm.com>,
        Vasily Gorbik <gor@linux.ibm.com>,
        Alexander Gordeev <agordeev@linux.ibm.com>,
        Christian Borntraeger <borntraeger@linux.ibm.com>,
        Sven Schnelle <svens@linux.ibm.com>,
        Tony Krowiak <akrowiak@linux.ibm.com>,
        Jason Herne <jjherne@linux.ibm.com>,
        Harald Freudenberger <freude@linux.ibm.com>,
        Diana Craciun <diana.craciun@oss.nxp.com>,
        Eric Auger <eric.auger@redhat.com>,
        Kirti Wankhede <kwankhede@nvidia.com>,
        Abhishek Sahu <abhsahu@nvidia.com>,
        Yishai Hadas <yishaih@nvidia.com>,
        "intel-gvt-dev@lists.freedesktop.org" 
        <intel-gvt-dev@lists.freedesktop.org>,
        "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>,
        "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-s390@vger.kernel.org" <linux-s390@vger.kernel.org>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>
Subject: RE: [PATCH v1 7/7] vfio: Remove vfio_free_device
Thread-Topic: [PATCH v1 7/7] vfio: Remove vfio_free_device
Thread-Index: AQHY49cc6IlfQ6/8HUyTbs6M71+22a4p3CKQ
Date:   Tue, 1 Nov 2022 09:11:40 +0000
Message-ID: <BN9PR11MB5276DC6065E67B0AF6C4127E8C369@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20221019162135.798901-1-farman@linux.ibm.com>
 <20221019162135.798901-8-farman@linux.ibm.com>
In-Reply-To: <20221019162135.798901-8-farman@linux.ibm.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?SkyRIPOYxnZgMTm0248637MnkDRjWa6bn2/uL0MvqNOwweeRbMqMIKFLmZVa?=
 =?us-ascii?Q?1ir/YL261iQSVhVi/dJwKlUKQCaccFMctwIP48G78Ff1cLJzRRS5fVCTqpaZ?=
 =?us-ascii?Q?GSXT+3kqEH9ApIO8EQRDoWh0Bi7Zs9ZO+w0mumKIiewffhia42Q05nOtLvuQ?=
 =?us-ascii?Q?RopMGJtHcGYR1qjLWT8Ezzfrxp0ItlePco1b2MGKXluiTKWNBuVj4q5swNnV?=
 =?us-ascii?Q?INAlrB9TG9hT7MvP3zFw47BAbmNRL+2AYbP1y3h5dj2EzyoZJngYT74DRta7?=
 =?us-ascii?Q?1pJhs+SsDSflD9JrCWFYTkTHDAI4ez3tdtr3mKO2OEgbfx0d/3Dka77BoxLY?=
 =?us-ascii?Q?uy6fc/zt6raP9t/26i7DbA3gkPTd08dijgfWtydG9fDSmQZ1yOayGl/DSjpx?=
 =?us-ascii?Q?SpD5p08PQ+cjTCGN7S0b+0kXBQlIZ4rKqae3uwrUhlggZ/bGg/Pdn8uWOtAu?=
 =?us-ascii?Q?CzkoXG5/CDxJLx2WeoSzzvKrSXiZn8VRmZscbWPBgxZYURe1KPPNVJ2aK1Ea?=
 =?us-ascii?Q?mFqraTWSnQNSZsPn8jdDqp0iljmfN/gyYoWy6QVexLst2rjDIKyNmpcCh389?=
 =?us-ascii?Q?dOfm7d3wJMVGFvppJKpcQe+hwItg3po/4gLL3xlI9pLxb67D1zt4dqiXbTAl?=
 =?us-ascii?Q?H7FWMzLgwHc+wT6j/ardx8+D2HDRM9qjMedGi1sxj8S7fLS1O6hPDuXc1umI?=
 =?us-ascii?Q?dsQhTvXEfnzuZvvPTKGpga9zMKQbdqqhEsCAytlOpXLp0QFE/WgOMaP1k15+?=
 =?us-ascii?Q?TJ7VZOFs7FAJ75pLN8GMqrgqn1M0pB8gbS6U8mozwMHTlK4XlkyEofil7899?=
 =?us-ascii?Q?LtmbYegYnxCIQXItlvpJOYnZJ0YpTVuDr8/d62E2vX1uIiPGvHoiwcH8WduL?=
 =?us-ascii?Q?mVhty6DQMuEWvdsnr+1lYz3GtQxppnLWUUoIsW4uvsJD+KI0WGbmI8cE7zNU?=
 =?us-ascii?Q?SVsSnIlZEVI9c1AjqtsE0Eltn+22nz1GtrbaZk3xfDl/HhfIMu1U9TNJMkik?=
 =?us-ascii?Q?kqqFYLb3Z3SLLy4ZC4RkIU83CX6l9JfU9X9sb0wtGY3P9zCTy+wK9b7wt++e?=
 =?us-ascii?Q?5qQMS0bUhYa9UANgh0u2CwzZFm43VR41/V7cNXw3MQV7PyMFEQauR2ilirGO?=
 =?us-ascii?Q?Ky3ersqPnt8RvaOS/soGSgPYHPr3F5fteEoEGGI/0JMXEVKcZxocBeBVLM5q?=
 =?us-ascii?Q?mENKIvJ7qjxWYP49zM6sAn3ks8FBLS5OzlOUsm1GiE3X2mHo+goQq6gvb1Wa?=
 =?us-ascii?Q?MJjU7WncqObAEJoiXR7ITsBhAjO+aetIS/B0+h2VeEwxCkaFqEmTplOZzRi6?=
 =?us-ascii?Q?1kCNfEhoI231CIQbWiu+z/1KLrWQrD4D7i8t2bctodlOveddfvQkhjI3HXY9?=
 =?us-ascii?Q?OlLLln3bJ0P8nAidpyxosPo5JxbtMEBC/lyZuvSzFoUmgLWx+KsAi4sTdWRF?=
 =?us-ascii?Q?O5k3z6vH8OHD3nbCStteliWWE8/s1ajw16/1GvV8nv6Az6yZq/HNPsLPv4Jw?=
 =?us-ascii?Q?ptn9H/Dt63DPH6yelwg0ow5327dO+OSon4tPnkHOtKFI/zXqbKnCs922N/Lj?=
 =?us-ascii?Q?qYypRJDTKu64BlzOY2aYMkODP3dVnIhD6GDEijz4?=
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: f6d7830e-af78-4757-890a-08dabbe916a9
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Nov 2022 09:11:40.8173
 (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: UqGy64TBx1bi/4oAKWjo8RHwCKyqjnTOiDp8JVhO6xyoid7AzKgbZ8m8JKBxsupqiC9irUsHa0/C4QS0gZR9kg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ1PR11MB6252
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.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
        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: Eric Farman <farman@linux.ibm.com>
> Sent: Thursday, October 20, 2022 12:22 AM
>=20
> With the "mess" sorted out, we should be able to inline the
> vfio_free_device call introduced by commit cb9ff3f3b84c
> ("vfio: Add helpers for unifying vfio_device life cycle")
> and remove them from driver release callbacks.
>=20
> Signed-off-by: Eric Farman <farman@linux.ibm.com>

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