[PATCH v2 0/2] vhost: fix async address mapping
Kevin Traynor
ktraynor at redhat.com
Tue Feb 1 12:29:28 CET 2022
On 01/02/2022 08:28, Maxime Coquelin wrote:
>
>
> On 1/19/22 16:10, xuan.ding at intel.com wrote:
>> From: Xuan Ding <xuan.ding at intel.com>
>>
>> This patchset fixes the issue of incorrect DMA mapping in PA mode.
>> Due to the ambiguity of host_phys_addr naming in the guest page
>> struct, rename it to host_iova.
>>
>> v2:
>> * Change the order of patch.
>
> I'm not sure why you changed the order of the patches.
> Now, the second one is the fix, so it will make the backport more
> difficult. Either both are considered to be fixes. I think it can make
> sense as the renaming does not introduce risk of regression and will
> make backporting patches easier in the future.
>
> Other solution is to reverse the order again, but I think tagging the
> renaming as a fix is OK for me here.
>
> What do you think?
>
Either way sounds ok to me, but can you also add stable tag(s). There
isn't a stable tag on either patch at present. Thanks.
> Regards,
> Maxime
>
>>
>> Xuan Ding (2):
>> vhost: rename field in guest page struct
>> vhost: fix physical address mapping
>>
>> lib/vhost/vhost.h | 11 ++--
>> lib/vhost/vhost_user.c | 130 ++++++++++++++++++++---------------------
>> lib/vhost/virtio_net.c | 11 ++--
>> 3 files changed, 75 insertions(+), 77 deletions(-)
>>
>
More information about the dev
mailing list