[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v13 1/2] arm/pci: add pci_passhtrough_enabled check to pci_device_reset


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Mykyta Poturai <Mykyta_Poturai@xxxxxxxx>
  • Date: Fri, 18 Jul 2025 11:52:38 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none
  • 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=NMP0k57Hu3y4ZkCDfW9AgHCn3p1fAo8CheFa23TymMo=; b=WMW4zsaBWIMW0lW6CDlujXWYzjfXnsdKM5qMaEV8w5eDsWArLAZKGFORKZSTk2Y5oyhqDCejqLFPtR7YHlmliWT0yB33xBdGxH5EWV65U+63tNOMICg5oAIXtxu2DUPsSqn2TJUmIQaLuGiZTX6zLqNCA1OlB2R3iwQJdBzC8QqW69SHy0OPoiEjqH63hNvcMzG0ndz6IL6EWhPGDc2VvG40one9T6+FjlgNUly0703lwovjEFxGIv4mg6jNw99KUTo12ZQndc1yi5oGGR4xCQla/BQ7ZRS112t+xS0+ihDzoVcwJmbW5Zt415W+0BW8nkiGMEZwfdLmnWNF1uhcvw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Fnwq8wEx1fvz8i/K2D81LYiD78Ka4CnMegFkBNonoRojK2y6LJXMIwvZ1whg/QOWiZWSZLhFiKrwolbXG1xLmzDiXzVGgszvJoVb7ktQ+P8+35XxRzsu3da5ZCvVH3y/a8Vw9D2CS7jQAHqIy61k5T2EO5g9M1UHr8qcUOlHDGHsF3AhHVY/8Ub7gFurHqhAEIgF7nczgTvYgaygN5j2Q4UlyIz4X98/NX3stT6jm+I/uxygSJ3mByoDHDMYbdWRLGJmQMJhnGLGXjptJhqDFPzQ0W3THxhHpDmf6fB6vHs/IBOXeA71oW/zNOu6pBdouMTxyKY1M5u/ZQyYwbAgsA==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Julien Grall <julien@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Fri, 18 Jul 2025 11:52:53 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHb9iVc9+KBe39cBkCCdxD6apzzJrQ0bRQAgANbioA=
  • Thread-topic: [PATCH v13 1/2] arm/pci: add pci_passhtrough_enabled check to pci_device_reset

On 16.07.25 11:36, Jan Beulich wrote:
> On 16.07.2025 09:43, Mykyta Poturai wrote:
>> Without pci-passthrough=on Xen does not know anything about present PCI
>> devices due to PHYSDEVOP_pci_device_add not executing.
> 
> While the latter half of the sentence is true, Xen may know of PCI devices
> be other means. It scans the bus itself after all on x86, under certain
> conditions. Furthermore "pci-passthrough" is an Arm-only option, when the
> change here affects x86 as much. The description therefore will need some
> improvement; the code change itself is okay (on the assumption that it can
> be justified properly).
> 
> Jan

Will adding an additional note that all of this only relates to Arm and 
will not change x86 functionality be sufficient? As far as I understand, 
this sentence is completely true for Arm, even the yet-to-be-upstreamed 
bus scanning routines in Xen are guarded by is_pci_passthroug_enabled in 
the latest patches I've seen them in.

-- 
Mykyta

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.