[win-pv-devel] [PATCH] Disallow RSS configuration if there is only a single vCPU

[Paul Durrant <paul.durrant@xxxxxxxxxx>]

It's not going to be useful and having RSS enabled in a single vCPU
VM seems to confuse a Windows Domain Controller installed in that VM.

Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
---
 src/xenvif/frontend.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/xenvif/frontend.c b/src/xenvif/frontend.c
index b510cc1..463d163 100644
--- a/src/xenvif/frontend.c
+++ b/src/xenvif/frontend.c
@@ -1937,7 +1937,11 @@ FrontendSetHashAlgorithm(
         break;
 
     case XENVIF_PACKET_HASH_ALGORITHM_TOEPLITZ:
-        status = (Frontend->DisableToeplitz != 0) ?
+        // Don't allow toeplitz hashing to be configured for a single
+        // queue, or if it has been explicitly disabled
+        ASSERT(__FrontendGetNumQueues(Frontend) != 0);
+        status = (__FrontendGetNumQueues(Frontend) == 1 ||
+                  Frontend->DisableToeplitz != 0) ?
                  STATUS_NOT_SUPPORTED :
                  STATUS_SUCCESS;
         break;
-- 
2.5.3


_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/win-pv-devel