* Re: [PATCH] ifcvf/vDPA: fix misuse virtio-net device config size for blk dev
[not found] <20211129093144.8033-1-lingshan.zhu@intel.com>
@ 2021-11-30 9:32 ` Stefano Garzarella
0 siblings, 0 replies; only message in thread
From: Stefano Garzarella @ 2021-11-30 9:32 UTC (permalink / raw)
To: Zhu Lingshan; +Cc: kvm, mst, netdev, stable, virtualization
On Mon, Nov 29, 2021 at 05:31:44PM +0800, Zhu Lingshan wrote:
>This commit fixes a misuse of virtio-net device config size issue
>for virtio-block devices.
>
>A new member config_size in struct ifcvf_hw is introduced and would
>be initialized through vdpa_dev_add() to record correct device
>config size.
>
>Signed-off-by: Zhu Lingshan <lingshan.zhu@intel.com>
>Reported-and-suggested-by: Stefano Garzarella <sgarzare@redhat.com>
>Fixes: 6ad31d162a4e ("vDPA/ifcvf: enable Intel C5000X-PL virtio-block for vDPA")
>Cc: <stable@vger.kernel.org>
>---
> drivers/vdpa/ifcvf/ifcvf_base.c | 41 +++++++++++++++++++++++++--------
> drivers/vdpa/ifcvf/ifcvf_base.h | 9 +++++---
> drivers/vdpa/ifcvf/ifcvf_main.c | 24 ++++---------------
> 3 files changed, 41 insertions(+), 33 deletions(-)
The patch LGTM. Maybe we could add in the description that we rename
some fields and functions in a more generic way.
In both cases:
Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
Thanks,
Stefano
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
^ permalink raw reply [flat|nested] only message in thread