Re: [Xen-devel] [PATCH v4 12/16] xen/mm: Switch common/memory.c to use typesafe MFN

[Julien Grall <julien.grall@xxxxxxx>]

Hi,

On 23/02/18 18:05, Wei Liu wrote:
> On Fri, Feb 23, 2018 at 05:46:39PM +0000, Julien Grall wrote:
> > On 23/02/18 17:26, Wei Liu wrote:
> > > On Wed, Feb 21, 2018 at 02:02:55PM +0000, Julien Grall wrote:
> > > > A new helper copy_mfn_to_guest is introduced to easily to copy a MFN to
> > > > the guest memory.
> > > >
> > > > Not functional change intended
> > > Is there a reason to not make all guest accessors tyep-safe instead?
> > Could you clarify what you mean? MFN and xen_pfn_t have different type on
> > Arm. So I am not sure how you would be able to make them typesafe.
> What I meant was to make copy_{to,from}_guest* type-safe. I just feel it
> a bit strange you only created a wrapper for this file. I wonder why.
Oh, I can have a look at that.

> Note I'm just asking question. That's not necessarily a good idea to
> turn them all in the end.
Cheers,


--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel