From:   "Tian, Kevin" <kevin.tian@intel.com>
To:     "Liu, Yi L" <yi.l.liu@intel.com>,
        "alex.williamson@redhat.com" <alex.williamson@redhat.com>,
        "jgg@nvidia.com" <jgg@nvidia.com>
CC:     "cohuck@redhat.com" <cohuck@redhat.com>,
        "eric.auger@redhat.com" <eric.auger@redhat.com>,
        "nicolinc@nvidia.com" <nicolinc@nvidia.com>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
        "mjrosato@linux.ibm.com" <mjrosato@linux.ibm.com>,
        "chao.p.peng@linux.intel.com" <chao.p.peng@linux.intel.com>,
        "yi.y.sun@linux.intel.com" <yi.y.sun@linux.intel.com>,
        "peterx@redhat.com" <peterx@redhat.com>,
        "jasowang@redhat.com" <jasowang@redhat.com>,
        "shameerali.kolothum.thodi@huawei.com" 
        <shameerali.kolothum.thodi@huawei.com>,
        "lulu@redhat.com" <lulu@redhat.com>,
        "suravee.suthikulpanit@amd.com" <suravee.suthikulpanit@amd.com>,
        "intel-gvt-dev@lists.freedesktop.org" 
        <intel-gvt-dev@lists.freedesktop.org>,
        "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-s390@vger.kernel.org" <linux-s390@vger.kernel.org>
Subject: RE: [PATCH v2 03/14] vfio: Accept vfio device file in the driver
 facing kAPI
Thread-Topic: [PATCH v2 03/14] vfio: Accept vfio device file in the driver
 facing kAPI
Thread-Index: AQHZOgo008xv7p4OlUyVdUrCFkzTu67C02xg
Date:   Tue, 7 Feb 2023 03:25:17 +0000
Message-ID: <BN9PR11MB52766989CAD2B3DDA3B3021D8CDB9@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20230206090532.95598-1-yi.l.liu@intel.com>
 <20230206090532.95598-4-yi.l.liu@intel.com>
In-Reply-To: <20230206090532.95598-4-yi.l.liu@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?iUCP1B/nZoZ4CNDaFtOEFBLtcZspezRreHMt4TsydXu9hIN+smORgr/76+RZ?=
 =?us-ascii?Q?+dQafQpV8Rs67WwC6Ooqz9gwRY9bnu0gjffE9tCyeu5TvPmLRg5CnYq8rjjM?=
 =?us-ascii?Q?gLHhbcFjvsVPcal5tTGjrWfPiFmhYDh65nzXhxIJYKL0sSeAuc1GL5LrmLZy?=
 =?us-ascii?Q?xp5ynsyrZThtZ1Ui6xPXfeFbxO+Ij9i7sJkI+RnFuBE2al0HouXxcaqg0tfJ?=
 =?us-ascii?Q?6iF1R/7LOqaRIppMT5va9JY2zxrcGzzQJbaV0Dz5UzO30xLaSIzi7AqZSvfW?=
 =?us-ascii?Q?r0JRwvFM5jxpSt5K/kRMejxDA8I6rSyhBBrhorbHgVY5J1XE0aEAZqE/MYll?=
 =?us-ascii?Q?0eWksFMaW1intxH3c2Oq9yLlSB3v1RbT0eXf6jmBQq3qnFkSeblhBJHP+g41?=
 =?us-ascii?Q?7Usv8x52nXoob3bVjBtWvwzDUv//g3st0wqVCZZ7p6fJsuMX22DRBVxXE92a?=
 =?us-ascii?Q?a3cPILRR/ioKatmvCcgTbwytU/A3Jp2CQY7bwRL8kPcHC03TOKQwyV/qtGMT?=
 =?us-ascii?Q?YqtV6iOMe1QXCnIzcf720pVgOxd5NnKRvnljBDa+tcY1ESvdPcC/7aBofGCK?=
 =?us-ascii?Q?SJjvCB1yoACUkkIpHhKDWb2OI4t9qxjjpOzk5USbCm+/31/KGAU+Kgmwe9rg?=
 =?us-ascii?Q?tKqeWiu/pOvUj/WyuwsRB5v3bLtRM37Mxb7+VTjlwfgSDs0CkMGbibPh/CLS?=
 =?us-ascii?Q?/i0/+klaUvs61CCPskdDThViP8YqdJ0cyKR24bQ8dAad5EJlUy1BxOgUbcZk?=
 =?us-ascii?Q?wAiBWsbgDTizfLDHEr7vcjORKFCzjolBhZdQTQrZnQy/jLhEt5IrXjk4VJk9?=
 =?us-ascii?Q?FuF1rCgDP4amlTk0kxQU6mhmg4+0yA3YX7hZukAfJgSlffy5/l8uJ7pkZASZ?=
 =?us-ascii?Q?xkzfUO1pkKIw4eaV8UR+a0OQSLXAnTWarzSxMqQFBLqSLlBMqNKreCl6z3Gg?=
 =?us-ascii?Q?LwVVmtSUxXg1lKKZYfGhy5TNWhmE1ZbXrIeGKiY2xK8i5ULBtXExYyenep3P?=
 =?us-ascii?Q?IBAGYFSzDt0HID1vdYHGG/AJpgwUMYh58H7g84xTYSWC+kouJUX+B/2ESeyR?=
 =?us-ascii?Q?tsG0c3URgOfcKlpHt8LDqeWbRT5Wirv0XsXgMe3k0ExTozTSVMo97Th3vVHA?=
 =?us-ascii?Q?sj9K6Mc+UUhua3iPwUSxltMBm5liU1UbuU3+n0TwdLzoFXopYXxGpn9z8BKY?=
 =?us-ascii?Q?9uxaNDLvpIq4njp+IRBYIJBl58Vhdxb9tDewwJpfPPhh6C7pzecQ5kxvskF6?=
 =?us-ascii?Q?LAd59kiirEivVUgBG26g6RRHAjf7unKXqdInai+QTvKy37m5OqpMdKdL2NgF?=
 =?us-ascii?Q?ZeSyi1zADE2EksVtCizF32AiN6rLaWgluCpNozbSvkwZqusB9cAzJ2DUS0U2?=
 =?us-ascii?Q?9t2nqfvoQwzMkoE2rdYSgZVV8huWFvuwj/Az9rb3BnxzV2tdUupC+sP835E3?=
 =?us-ascii?Q?c9RLkIBqGy3oL7iqKvsWKz8FM0icR09hrvmw8Gh+R0EUGB0JEOxGUCx4nNsp?=
 =?us-ascii?Q?PYsNJ2hotDZY9ZZoTzhd4nanbp8HRwylVHCNeMOt/vlE7j6F8DXc77pvTKih?=
 =?us-ascii?Q?+CWpqD8QFKGd4T7N5CWQ73UyipCkTMt6aRDt34C5?=
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: 60597574-06a9-4296-f85f-08db08baef34
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Feb 2023 03:25:17.3066
 (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: qncIhUh1KbWWil1oUj+NhUk9zVrAIYLHb1LjxlPAWMEWlIa2kOuNzndCFRQl5VS4N1gaF+q7351xEvN51ZokFQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB8041
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> From: Liu, Yi L <yi.l.liu@intel.com>
> Sent: Monday, February 6, 2023 5:05 PM
>=20
> This makes the vfio file kAPIs to accepte vfio device files, also a
> preparation for vfio device cdev support.
>=20
> For the kvm set with vfio device file, kvm pointer is stored in struct
> vfio_device_file, and use kvm_ref_lock to protect kvm set and kvm
> pointer usage within VFIO. This kvm pointer will be set to vfio_device
> after device file is bound to iommufd in the cdev path.
>=20
> Signed-off-by: Yi Liu <yi.l.liu@intel.com>

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