Re: [PATCH 1/5] OvmfPkg/VirtioNetDxe: map VRING using VirtioRingMap()

[Brijesh Singh <brijesh.singh@amd.com>]

Hi Laszlo,

Thanks for quick reviews. I will go through each feedback address them in v2.



On 09/05/2017 06:47 AM, Laszlo Ersek wrote:

[...]

> 
> Please also modify the commit message similarly: "map the VRING system
> physical address[es] to device address[es]".
> 
> 
> Would it be OK with you to submit only these two patches next?


I saw that you looked at other patches, just wondering, do you still want me
to submit two patches and advance in small steps ?


> 
> I wouldn't like to look at the rest of the patches just now. I expect
> quite a few tweaks -- especially because I would *really* like to keep
> "TechNotes.txt" up to date! --, and I'd like to keep my focus, and
> advance in small steps. I must re-read TechNotes.txt myself, in parallel
> to the progress that we make with this series.
> 
> Once we consider these patches complete, we can test them and commit
> them in isolation. Further versions of the series won't have to repeat
> these patches.
> 
> I'll strive to be very responsive, so that you don't have to wait long
> after every small step.
>