Received-SPF: pass (google.com: domain of linux-kernel+bounces-193362-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: "Tian, Kevin" <kevin.tian@intel.com>
To: Arnd Bergmann <arnd@kernel.org>, "Zeng, Xin" <xin.zeng@intel.com>,
	"Cabiddu, Giovanni" <giovanni.cabiddu@intel.com>, Alex Williamson
	<alex.williamson@redhat.com>, "Cao, Yahui" <yahui.cao@intel.com>
CC: Arnd Bergmann <arnd@arndb.de>, Jason Gunthorpe <jgg@ziepe.ca>, "Yishai
 Hadas" <yishaih@nvidia.com>, Shameer Kolothum
	<shameerali.kolothum.thodi@huawei.com>, "kvm@vger.kernel.org"
	<kvm@vger.kernel.org>, qat-linux <qat-linux@intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] vfio/qat: add PCI_IOV dependency
Thread-Topic: [PATCH] vfio/qat: add PCI_IOV dependency
Thread-Index: AQHasPdozFJ7uzjsf0GWpzSyP9bM07GteM2g
Date: Wed, 29 May 2024 02:25:29 +0000
Message-ID: <BN9PR11MB5276C0C078CF069F2BBE01018CF22@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20240528120501.3382554-1-arnd@kernel.org>
In-Reply-To: <20240528120501.3382554-1-arnd@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?6smgDZupVub64Ps81keQ/P525EMvRf96RV6oGzRz8lO1xcxDdQFrY3+SE2B/?=
 =?us-ascii?Q?wT2BVWe89fu4pFj6glAfxpHWO16r4V7nBHHWl797eC1c0JVPfM1Zcek+Q6Hp?=
 =?us-ascii?Q?HKyqaZeBmbR0JeKPOBWWFmsEi/5LZ1rORteWI5FlHI4RdbnKTEab4g0UExiU?=
 =?us-ascii?Q?Qe0TZlz9SX0Y0CIxjfRHSA64k1+u8ZkfbL9pIUFGUp3S+Y5nI3DbF/LHtACL?=
 =?us-ascii?Q?qVQa5MyZMmw9AyHyZ5tWYQzUSMjlaTk7s2Z+WDrFLpAKGZpIt7NVSUD/zvjF?=
 =?us-ascii?Q?qlMwcyDGY/C5yMom7eBwm/4oVPAKtn+etqm2R/kEVy4Y/WmwzgXRRz/11FK/?=
 =?us-ascii?Q?+vBkJlDljjPSqw1UQ1sb84g1xaVZfSgofd/fXtj1hf8GLY3UoYO5vTIcKu5G?=
 =?us-ascii?Q?WH40lT8u1VHFHvfF4XaUyBF5+lUlDi0bwUr7hvsPKncdGMg8R14JfrQv+1fF?=
 =?us-ascii?Q?IJ7ajceLHLHF34/zXNCiWoVbqFFqoU1FvZnlqsHSgZMgO0J31QqCJI1q2e50?=
 =?us-ascii?Q?KZLnPNfYaV5iP/5fwJyu13kdOopWTOMAOng7CJy7RmiuDqeDQRp370pNRJma?=
 =?us-ascii?Q?aMp2W3RJ/5f4PMAgYRdVEZ0bKy8CR2v1wcMuoL+3jlFs/1l++pzufp8h8naP?=
 =?us-ascii?Q?Pw691V5xmGxJnkhlnmdJcYN6vGkMq0dKmuUjvT+MS8DaQgnP+YvGVuHUl8Iw?=
 =?us-ascii?Q?fBUS+vDgPEjkSf+oMK8In8ENS13Jm2k6iAuQZSpBFNDFK7DkeH3AdHaUGkUj?=
 =?us-ascii?Q?mpSh8ypzBWIgIGDLxp03udy0HJ7ovD2P2nMAktfnzWgATcHtT1V0R8TbO+e8?=
 =?us-ascii?Q?A4JvnezBPP9v7eUq9rSj0lxc60eZwR2+LgcJ/jreiXfGfwuZRRd6941sop1d?=
 =?us-ascii?Q?8TgZEo1q3fj50cJ22p487bAo87IKoyc+To1TrTvAIwcH/3O2kABV8GXsRSmO?=
 =?us-ascii?Q?ZOrby7ypUhTr+KvGZ0W8cFQsnhWE9Xcx/SLssBYHsISd6Piis7EeRfJNEGst?=
 =?us-ascii?Q?FbS/pnXR1ORCawI3RGDpJAcSF5btezT0vfVyAcWP7/o85IwRV6YgVlHnY75L?=
 =?us-ascii?Q?ZPgfPS5xaP3nWZm2lVFhzWnBlbc/9l3K1Bsz+Iub28vgaijmZGKAb4AWwBKe?=
 =?us-ascii?Q?Judvz94oyosqhxTYdfyBfcmVph/LgY+YomzkZgTctXuD/rgLnxkq17qc7+KB?=
 =?us-ascii?Q?0TPAVV8mkd5XXv/8BlniTq4TRQVDFyeOQY3LkYgUs16MqhhkYfrv6fHVChNu?=
 =?us-ascii?Q?QP/P5l7fwhfiuEYntOcf/0AEqWh54L1gT5WGrVjMWgCBTBkRjIae5KJQ7pWv?=
 =?us-ascii?Q?lhwlitsWEei4zDzzzdXwToiDPKGF9Ezcr8vQMfgcNGWp91DlFO7QZcktLXNF?=
 =?us-ascii?Q?BdB14WbJZjCKx4R1pw7d6uVObcT8+U54j+9S/lgF4zTB6Pymm4mMqYDCDIrc?=
 =?us-ascii?Q?cT7Hdr1UZAAGOXzI1MiSA9fBvMaTnSJZKblgHgfqm5+SeqH8FrjoosOQ71ze?=
 =?us-ascii?Q?O2Xtt4lUWtDJPmTaZUK5QaP9mLU2HP0h53DIsYNU0cawiDeOsmmdF00RaSUc?=
 =?us-ascii?Q?XW/xBBwsYQe+AGvYBQJ6hL8iskGyd/U5dFF7+KI5?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
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: dda5e284-19f1-4030-56ca-08dc7f869b84
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 May 2024 02:25:29.1122
 (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: iJozdHQL11zo1KzwbX8Vi0mRsq01uKR2HlEVv/ns7RcMOuERYwPVZKxEZNP46QtAXNtPla59nqNrTrbsjKm+qA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB7092
X-OriginatorOrg: intel.com

> From: Arnd Bergmann <arnd@kernel.org>
> Sent: Tuesday, May 28, 2024 8:05 PM
>=20
> From: Arnd Bergmann <arnd@arndb.de>
>=20
> The newly added driver depends on the crypto driver, but it uses exported
> symbols that are only available when IOV is also turned on:
>=20
> x86_64-linux-ld: drivers/vfio/pci/qat/main.o: in function
> `qat_vf_pci_open_device':
> main.c:(.text+0xd7): undefined reference to `qat_vfmig_open'
> x86_64-linux-ld: drivers/vfio/pci/qat/main.o: in function
> `qat_vf_pci_release_dev':
> main.c:(.text+0x122): undefined reference to `qat_vfmig_cleanup'
> x86_64-linux-ld: main.c:(.text+0x12d): undefined reference to
> `qat_vfmig_destroy'
> x86_64-linux-ld: drivers/vfio/pci/qat/main.o: in function
> `qat_vf_resume_write':
> main.c:(.text+0x308): undefined reference to `qat_vfmig_load_setup'
> x86_64-linux-ld: drivers/vfio/pci/qat/main.o: in function
> `qat_vf_save_device_data':
> main.c:(.text+0x64c): undefined reference to `qat_vfmig_save_state'
> x86_64-linux-ld: main.c:(.text+0x677): undefined reference to
> `qat_vfmig_save_setup'
> x86_64-linux-ld: drivers/vfio/pci/qat/main.o: in function
> `qat_vf_pci_aer_reset_done':
> main.c:(.text+0x82d): undefined reference to `qat_vfmig_reset'
> x86_64-linux-ld: drivers/vfio/pci/qat/main.o: in function
> `qat_vf_pci_close_device':
> main.c:(.text+0x862): undefined reference to `qat_vfmig_close'
> x86_64-linux-ld: drivers/vfio/pci/qat/main.o: in function
> `qat_vf_pci_set_device_state':
> main.c:(.text+0x9af): undefined reference to `qat_vfmig_suspend'
> x86_64-linux-ld: main.c:(.text+0xa14): undefined reference to
> `qat_vfmig_save_state'
> x86_64-linux-ld: main.c:(.text+0xb37): undefined reference to
> `qat_vfmig_resume'
> x86_64-linux-ld: main.c:(.text+0xbc7): undefined reference to
> `qat_vfmig_load_state'

at a glance those undefined symbols don't use any symbol under
IOV. They are just wrappers to certain callbacks registered by
by respective qat drivers which support migration.

Probably they'd better be moved out of CONFIG_PCI_IOV in
"drivers/crypto/intel/qat/qat_common/Makefile" to remove
this dependency in vfio variant driver.

>=20
> Add this as a second dependency.
>=20
> Fixes: bb208810b1ab ("vfio/qat: Add vfio_pci driver for Intel QAT SR-IOV =
VF
> devices")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
>  drivers/vfio/pci/qat/Kconfig | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>=20
> diff --git a/drivers/vfio/pci/qat/Kconfig b/drivers/vfio/pci/qat/Kconfig
> index bf52cfa4b595..fae9d6cb8ccb 100644
> --- a/drivers/vfio/pci/qat/Kconfig
> +++ b/drivers/vfio/pci/qat/Kconfig
> @@ -1,8 +1,9 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  config QAT_VFIO_PCI
>  	tristate "VFIO support for QAT VF PCI devices"
> -	select VFIO_PCI_CORE
>  	depends on CRYPTO_DEV_QAT_4XXX
> +	depends on PCI_IOV
> +	select VFIO_PCI_CORE
>  	help
>  	  This provides migration support for Intel(R) QAT Virtual Function
>  	  using the VFIO framework.
> --
> 2.39.2