netdev - Re: [PATCH] ifcvf/vDPA: fix misuse virtio-net device config size for blk dev

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <9957a8be-f93a-5d23-d697-22968f31766f@intel.com>
Date:   Wed, 1 Dec 2021 10:36:42 +0800
From:   "Zhu, Lingshan" <lingshan.zhu@...el.com>
To:     Stefano Garzarella <sgarzare@...hat.com>
Cc:     jasowang@...hat.com, mst@...hat.com,
        virtualization@...ts.linux-foundation.org, kvm@...r.kernel.org,
        netdev@...r.kernel.org, stable@...r.kernel.org
Subject: Re: [PATCH] ifcvf/vDPA: fix misuse virtio-net device config size for
 blk dev



On 11/30/2021 5:32 PM, Stefano Garzarella wrote:
> 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@...el.com>
>> Reported-and-suggested-by: Stefano Garzarella <sgarzare@...hat.com>
>> Fixes: 6ad31d162a4e ("vDPA/ifcvf: enable Intel C5000X-PL virtio-block 
>> for vDPA")
>> Cc: <stable@...r.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.
Sure, Thanks!

Thanks,
Zhu Lingshan
>
> In both cases:
>
> Reviewed-by: Stefano Garzarella <sgarzare@...hat.com>
>
> Thanks,
> Stefano
>