[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 5/5] vPCI: re-init extended-capabilities when MMCFG availability changed
- To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Stewart Hildebrand <stewart.hildebrand@xxxxxxx>
- Date: Thu, 19 Feb 2026 17:21:57 -0500
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=suse.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ojjnlfHIOkVjavQa9jwEY++80EHK6ug+8MZT/dYIeMs=; b=zGfrjq74uS4AF1ubKUW5UrmADFg6sQuIte/QGIki1qyQaqSzlt+zhBQm3S2rBLAI74Biz1yVArfmkdIiqQdjKvcq0h5dYp+eqb2sDyBMtbdTcHEPYF/Nwd5h8jxU+3ZSAXIdNODWn9fNK9+fBAS+CII5+2I/ZiDfVvfsZmPHJG7B7kWfde3o7EROe0HYfmcN2SHC93o660mPAUmYsJJXnYMTJGPHHj/wDiRVpO0678NhBdjRY0NbTiWVWCWQQLTLTbNDcNAlAH+9uXV7tRIPbz7Zv4C1R6/aMLEHfSkUYGSOzv1O8yJ0EqQBcqb8lSEkw1ekN30FWe3iztokSQbPnA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=gowNUnQaxDmpeM4Zos+mrzJMW3yNc18BtwMY/1xBISpxgXlIQ6QnI8gGT/wNl5wZhojkNgS/imCC9b0iZMBfbW7jXt1ssMrmYM25gOaHAO0smX5LP5/fdFt8TfP4Kt3Wr18zKbv4eec4rKagt/YCz+tBQZbaYNvPV6BLLNqQAfG4kMAHzq0fuxpiuzaNCMxIeUQf3mwY7P0Um9cLoOKxMof0dYUXglt4n0wGzAWO2pEE2hLb4bYIxwAZXCKJSZZ1oIVuJMd1j9zcLFmcBwtD8NvXoRFRhAYpdDbJ2vkxv/+maPfVDmKDnhx6KtY1Ciu074tqKU4ykSr/3ZWp0tLn3A==
- Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
- Delivery-date: Thu, 19 Feb 2026 22:22:45 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 2/10/26 05:55, Jan Beulich wrote:
> --- a/xen/drivers/vpci/cap.c
> +++ b/xen/drivers/vpci/cap.c
> @@ -376,6 +379,20 @@ void vpci_cleanup_capabilities(struct pc
> }
> }
>
> +int vpci_reinit_ext_capabilities(struct pci_dev *pdev)
> +{
> + if ( !pdev->vpci )
> + return 0;
> +
> + vpci_cleanup_capabilities(pdev, true);
In the case where pdev->ext_cfg transitions from true to false, it doesn't look
like this would actually result in the respective capability->cleanup() hook
being called, due to reliance on pci_find_ext_capability().
|
