Re: [Xen-devel] [PATCH RFC 04/13] x86/mm: carve out create_grant_pv_mapping

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Wed, Mar 29, 2017 at 11:27:49AM +0100, Andrew Cooper wrote:
> On 27/03/17 10:10, Wei Liu wrote:
> > We will later split out PV specific code to pv/mm.c.
> >
> > No functional change.
> >
> > Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> 
> Please could you take the time to correctly mfn_t/gfn_t the calls. 
> Having both addr and frame, and them being otherwise unqualified, is
> confusing to read.
> 

I'm not sure what you want here.

The create_grant_host_mapping function is hooked into grant table code,
which is using "unsigned long" everywhere.

Modifying it here would require fixing all call sites as well. It'd be
better be done in a separate patch.

Wei.

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