From 775c50d3352ab35b47fe44c01e04af0d9f18274b Mon Sep 17 00:00:00 2001 From: Paul Durrant Date: Mon, 7 Nov 2016 16:15:24 +0000 Subject: [PATCH] Disallow RSS configuration if there is only a single vCPU 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 --- 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.39.5