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

Re: [PATCH] Set FrontendMaxQueues by default


  • To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
  • From: "Durrant, Paul" <xadimgnik@xxxxxxxxx>
  • Date: Wed, 7 Jun 2023 10:52:01 +0100
  • Delivery-date: Wed, 07 Jun 2023 09:52:06 +0000
  • List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>

On 26/05/2023 09:17, Owen Smith wrote:
Limiting the number of queues to the same value defined in xennet's INF
file prevents xenvif from allocating many more resources than required.

Currently, xenvif will allocate a queue for each active CPU, but xennet
only reports a maximum of 8 RSS queues to Windows. When vCPU count is
greather than 8, additional queues are not used and just consume resources.

Signed-off-by: Owen Smith <owen.smith@xxxxxxxxx>

Acked-by: Paul Durrant <paul@xxxxxxx>

---
  src/xenvif.inf | 1 +
  1 file changed, 1 insertion(+)

diff --git a/src/xenvif.inf b/src/xenvif.inf
index d81fe52..4b5c9f7 100644
--- a/src/xenvif.inf
+++ b/src/xenvif.inf
@@ -88,6 +88,7 @@ HKR,,"BootFlags",0x00010003,0x81
  [XenVif_Parameters]
  HKR,"Parameters",,0x00000010
  HKR,"Parameters","RequestKey",0x00000000,%RequestKey%
+HKR,"Parameters","FrontendMaxQueues",0x00010001,0x00000008
[XenVif_Inst.CoInstallers]
  CopyFiles=CoInst_CopyFiles




 


Rackspace

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