Thanks for the quick response! Just for my own knowledge, what Junfeng described is the process to fix the bug if a bug is present in the main dpdk repo?
On Thu, Jun 1, 2023 at 3:24 AM Ferruh Yigit <ferruh.yi...@amd.com> wrote: > > On 6/1/2023 9:26 AM, Ferruh Yigit wrote: > > On 6/1/2023 5:49 AM, Rushil Gupta wrote: > >> gVNIC requires physical address to be passed in the adminq command. > >> This was initially rightly pointed by ferruh.yigit@. > >> Fixed by passing 'driver_info_mem->iova'. > >> > >> Signed-off-by: Rushil Gupta <rush...@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); > >> > > > > Yep, this was my point, let me squashed onto original patch in next-net. > > > > > > Squashed into relevant commit in next-net, thanks. > Can you please verify latest code in next-net?
