>>> On 21.11.18 at 14:21, <andrew.coop...@citrix.com> wrote: > --- a/xen/arch/x86/mm/mem_sharing.c > +++ b/xen/arch/x86/mm/mem_sharing.c > @@ -901,8 +901,8 @@ static int share_pages(struct domain *sd, gfn_t sgfn, > shr_handle_t sh, > struct two_gfns tg; > struct rmap_iterator ri; > > - get_two_gfns(sd, gfn_x(sgfn), &smfn_type, NULL, &smfn, > - cd, gfn_x(cgfn), &cmfn_type, NULL, &cmfn, > + get_two_gfns(sd, sgfn, &smfn_type, NULL, &smfn, > + cd, cgfn, &cmfn_type, NULL, &cmfn, > 0, &tg);
I guess the function invocation fits on two lines now. In any event Reviewed-by: Jan Beulich <jbeul...@suse.com> Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel