[PATCH] net/gve: fix bug in verify driver compatibility

Rushil Gupta rushilg at google.com
Thu Jun 1 06:49:50 CEST 2023


gVNIC requires physical address to be passed in the adminq command.
This was initially rightly pointed by ferruh.yigit at .
Fixed by passing 'driver_info_mem->iova'.

Signed-off-by: Rushil Gupta <rushilg at google.com>
---
 drivers/net/gve/gve_ethdev.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/gve/gve_ethdev.c b/drivers/net/gve/gve_ethdev.c
index 2c1e73d07a..aa75abe102 100644
--- a/drivers/net/gve/gve_ethdev.c
+++ b/drivers/net/gve/gve_ethdev.c
@@ -276,7 +276,8 @@ gve_verify_driver_compatibility(struct gve_priv *priv)
 			(char *)driver_info->os_version_str2);
 
 	err = gve_adminq_verify_driver_compatibility(priv,
-		sizeof(struct gve_driver_info), (dma_addr_t)driver_info);
+		sizeof(struct gve_driver_info),
+		(dma_addr_t)driver_info_mem->iova);
 	/* It's ok if the device doesn't support this */
 	if (err == -EOPNOTSUPP)
 		err = 0;
-- 
2.41.0.rc2.161.g9c6817b8e7-goog



More information about the dev mailing list