Vmxnet3 uses capability registers to advertise the supported capabilities of UPT device. It uses DCR0_REG command register for this purpose. However, the register command enum misses one command which technically is not used by dpdk vmxnet3 driver yet. This can cause issue for commands added later. For example, DCR0_REG and MAX_CAPABILITIES register commands when issued will lead to incorrect command being executed in the hypervisor. The return values can be unexpected and can result in some capabilities being disabled and affect vnics requesting Uniform Passthroguh (UPT) mode.
This patch fixes this issue by adding a reserved command in its place. Fixes: 5241d61bd865 ("net/vmxnet3: support capability registers") Cc: sta...@dpdk.org Signed-off-by: Ronak Doshi <ronak.do...@broadcom.com> -- Change log v2: fixed typo v3: updated commit message with more information --- drivers/net/vmxnet3/base/vmxnet3_defs.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/vmxnet3/base/vmxnet3_defs.h b/drivers/net/vmxnet3/base/vmxnet3_defs.h index 24c235876e..a6bb281d8d 100644 --- a/drivers/net/vmxnet3/base/vmxnet3_defs.h +++ b/drivers/net/vmxnet3/base/vmxnet3_defs.h @@ -126,6 +126,7 @@ typedef enum { VMXNET3_CMD_RESERVED7, VMXNET3_CMD_RESERVED8, VMXNET3_CMD_GET_MAX_QUEUES_CONF, + VMXNET3_CMD_RESERVED11, VMXNET3_CMD_GET_MAX_CAPABILITIES, VMXNET3_CMD_GET_DCR0_REG, } Vmxnet3_Cmd; -- 2.11.0