[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/3] gnttab: Drop the frame parameter from get_paged_frame()



On Fri, Aug 25, 2017 at 10:38:19AM +0100, Andrew Cooper wrote:
> >> +    *page = mfn_valid(mfn) ? mfn_to_page(mfn_x(mfn)) : NULL;
> >>      if ( (!(*page)) || (!get_page(*page, rd)) )
> > Mind dropping those unneeded parentheses?
> 
> I'm planning separate cleanup to this function (including renaming it). 
> I'd prefer to defer changes like this to there.
> 

Sure.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.