Re: [PATCH 2/18 V2]: PVH xen: add XENMEM_add_to_physmap_range

[Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>]

On Mon, Mar 18, 2013 at 03:25:51PM +0000, Jan Beulich wrote:
> >>> On 18.03.13 at 15:28, Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> wrote:
> > On Fri, Mar 15, 2013 at 05:20:18PM -0700, Mukesh Rathor wrote:
> >> In this patch we add a new function xenmem_add_to_physmap_range(), and
> >> change xenmem_add_to_physmap_once parameters so it can be called from
> >> xenmem_add_to_physmap_range. There is no PVH specific change here.
> >> 
> >> Changes in V2:
> >>    - Do not break parameter so xenmem_add_to_physmap_once() but pass in
> >>      struct xen_add_to_physmap.
> > 
> > Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> 
> I'm confused: A little earlier you sent a response raising several
> issues, and now you send a Reviewed-by?

Sorry, I looked at it a first and it looked OK, so I did the Reviewed-by.

Then looked at some other patches, some cog in my brain did a late page-fault
on this patch and I went back in here. Then realized that there certain
things I was not sure off and asked Mukesh.

So, Reviewed-by rescinded for this patch right now. Thanks for catching that
process error.

> 
> Jan
>