> -----Original Message----- > From: Julien Grall [mailto:[email protected]] > Sent: 01 October 2018 11:26 > To: Paul Durrant <[email protected]>; [email protected] > Cc: Andrew Cooper <[email protected]>; George Dunlap > <[email protected]>; Ian Jackson <[email protected]>; Konrad > Rzeszutek Wilk <[email protected]>; Stefano Stabellini > <[email protected]>; Tim (Xen.org) <[email protected]>; Wei Liu > <[email protected]> > Subject: Re: [PATCH v12 5/9] memory: add check_get_page_from_gfn() as a > wrapper... > > Hi Paul, > > On 09/27/2018 03:33 PM, Paul Durrant wrote: > > ...for some uses of get_page_from_gfn(). > > > > There are many occurrences of the following pattern in the code: > > > > q = <readonly look-up> ? P2M_ALLOC : P2M_UNSHARE; > > page = get_page_from_gfn(d, gfn, &p2mt, q); > > > > if ( p2m_is_paging(p2mt) ) > > { > > if ( page ) > > put_page(page); > > > > p2m_mem_paging_populate(d, gfn); > > return <-EAGAIN or equivalent>; > > } > > > > if ( (q & P2M_UNSHARE) && p2m_is_shared(p2mt) ) > > { > > if ( page ) > > put_page(page); > > > > return <-EAGAIN or equivalent>; > > } > > > > if ( !page ) > > return <-EINVAL or equivalent>; > > > > There are some small differences between the exact way the occurrences > > are coded but the desired semantic is the same. > > > > This patch introduces a new common implementation of this code in > > check_get_page_from_gfn() and then converts the various open-coded > patterns > > into calls to this new function. > > > > Signed-off-by: Paul Durrant <[email protected]> > > Reviewed-by: Roger Pau Monne <[email protected]> > > Reviewed-by: Jan Beulich <[email protected]> > > --- > > Cc: Andrew Cooper <[email protected]> > > Cc: George Dunlap <[email protected]> > > Cc: Ian Jackson <[email protected]> > > Cc: Julien Grall <[email protected]> > > Cc: Konrad Rzeszutek Wilk <[email protected]> > > Cc: Stefano Stabellini <[email protected]> > > Cc: Tim Deegan <[email protected]> > > Cc: Wei Liu <[email protected]> > > > > v11: > > - Forward declare p2m_type_t in p2m-common.h to allow the duplicate > > declarations of check_get_page_from_gfn() to be removed, and hence > add > > Jan's R-b. > > I think this would be worth adding a word in the commit message about > the forward declaration.
Ok. I'll add something to v12. > > For the Arm bits: > > Acked-by: Julien Grall <[email protected]> > Thanks, Paul > Cheers, > > -- > Julien Grall _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
