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

[PATCH v9 14/16] xen/arm: account IO handlers for emulated PCI MSI-X


  • To: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Date: Tue, 29 Aug 2023 23:19:46 +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=arcselector9901; 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=FTPldD6+q4SM1yUJw2n6MueChQx31PeEONHRMx/GHEQ=; b=ns9wTHOEpWCTuKo7mx/SJ1d8NZS7aF2vHjxBZVQsOnN+Sll7V8kL7hxM2aJXpobdWlLvhpy+MZOdgc/3Ge7xWiqmTJtV5uXuKKBGzyKf+ZPW+CU/m8iY04+M4cgewOuWfGKqXhw/h7CZNxd5n5/V7BT5N0R9O1xsHZ1ubjDQAsY3Q76D8gLaM48GCqionVMEE9IvC/cUBzk9qzvqUBKoM9adl2E+fzPZe/dikwBTyCfi5MtEfaUgTepztBJow4ND3V++S/zQCLldUltdNtX2VvrPMke+Ul77SSKX5UyOKtI4xGN/cDzcqs3F932Q9YED8IsauTmvn9MGMGUNGwH/BA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NlAOaI2TKockDGVytM7PMphHd0wtYNZgqK38rsmUqoyh0IJtwb5WEPwML9CfBwjGbW3XimHjHpJCeOf1ECdcpbP28P7kKqvYYfc8YrAq6uqypi6rvMQrKAMFqzkoVZ8fDHw2scx5QrFIn88nAl2esMUhvdBfHPPjqcstMNHCOnDIL1qzU4FWzc7+S2M/b2G5v/m2vJWLfIp77gQ+JrQ0av/mS9VWNY5CNlZEmzP1769SwNaXAwFhyyLNNR7gFPCWhB3AcBMCEn8UkVO6AP6WLzhEKpJfqxA9pV2F7gqJNtRsXg6eCJ+7sOk62eaujIcEfExr08BT0Nd44Sxi8VD+Kw==
  • Cc: Stewart Hildebrand <stewart.hildebrand@xxxxxxx>, Oleksandr Andrushchenko <Oleksandr_Andrushchenko@xxxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Julien Grall <jgrall@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • Delivery-date: Tue, 29 Aug 2023 23:20:06 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHZ2s9M8h1s1RnmZka4sXTrGcJzww==
  • Thread-topic: [PATCH v9 14/16] xen/arm: account IO handlers for emulated PCI MSI-X

From: Oleksandr Andrushchenko <oleksandr_andrushchenko@xxxxxxxx>

At the moment, we always allocate an extra 16 slots for IO handlers
(see MAX_IO_HANDLER). So while adding IO trap handlers for the emulated
MSI-X registers we need to explicitly tell that we have additional IO
handlers, so those are accounted.

Signed-off-by: Oleksandr Andrushchenko <oleksandr_andrushchenko@xxxxxxxx>
Acked-by: Julien Grall <jgrall@xxxxxxxxxx>
---
Cc: Julien Grall <julien@xxxxxxx>
Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>
---
This actually moved here from the part 2 of the prep work for PCI
passthrough on Arm as it seems to be the proper place for it.

Since v5:
- optimize with IS_ENABLED(CONFIG_HAS_PCI_MSI) since VPCI_MAX_VIRT_DEV is
  defined unconditionally
New in v5
---
 xen/arch/arm/vpci.c | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/xen/arch/arm/vpci.c b/xen/arch/arm/vpci.c
index 58e2a20135..01b50d435e 100644
--- a/xen/arch/arm/vpci.c
+++ b/xen/arch/arm/vpci.c
@@ -134,6 +134,8 @@ static int vpci_get_num_handlers_cb(struct domain *d,
 
 unsigned int domain_vpci_get_num_mmio_handlers(struct domain *d)
 {
+    unsigned int count;
+
     if ( !has_vpci(d) )
         return 0;
 
@@ -154,7 +156,17 @@ unsigned int domain_vpci_get_num_mmio_handlers(struct 
domain *d)
      * For guests each host bridge requires one region to cover the
      * configuration space. At the moment, we only expose a single host bridge.
      */
-    return 1;
+    count = 1;
+
+    /*
+     * There's a single MSI-X MMIO handler that deals with both PBA
+     * and MSI-X tables per each PCI device being passed through.
+     * Maximum number of emulated virtual devices is VPCI_MAX_VIRT_DEV.
+     */
+    if ( IS_ENABLED(CONFIG_HAS_PCI_MSI) )
+        count += VPCI_MAX_VIRT_DEV;
+
+    return count;
 }
 
 /*
-- 
2.41.0



 


Rackspace

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