On Thu, Aug 24, 2017 at 06:55:53PM +0100, Andrew Cooper wrote: > It is redundant with the *page parameter. Rename the grant_frame parameter to > indicate that it is local, and highlight the correctness of the change. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
Reviewed-by: Wei Liu <wei.l...@citrix.com> > > - rc = acquire_grant_for_copy(td, trans_gref, rd->domain_id, > - readonly, &grant_frame, page, > - &trans_page_off, &trans_length, > - false); > + rc = acquire_grant_for_copy( > + td, trans_gref, rd->domain_id, readonly, page, &trans_page_off, > + &trans_length, false); The change of style seems unnecessary, but I don't think I care that much. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
