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@lists.xen.org https://lists.xen.org/xen-devel