On 11/30/2021 5:32 PM, Stefano Garzarella wrote:

On Mon, Nov 29, 2021 at 05:31:44PM +0800, Zhu Lingshan wrote:

quoted

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 <redacted>
Reported-and-suggested-by: Stefano Garzarella [off-list ref]
Fixes: 6ad31d162a4e ("vDPA/ifcvf: enable Intel C5000X-PL virtio-block 
for vDPA")
Cc: <redacted>
---
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.

Sure, Thanks!

Thanks,
Zhu Lingshan

In both cases:

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>

Thanks,
Stefano