[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.