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

Re: [XenPPC][Patch] memory alignment check for allocMem

To: geyi <kudva@xxxxxxxxxxxxxx>
From: Hollis Blanchard <hollisb@xxxxxxxxxx>
Date: Mon, 20 Nov 2006 14:27:47 -0600
Cc: xen-ppc-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 20 Nov 2006 12:28:06 -0800
List-id: Xen PPC development <xen-ppc-devel.lists.xensource.com>

On Mon, 2006-11-20 at 14:13 -0500, geyi wrote:
> 
> The other one is for _allocMem2, the memory-allocate function for
> domain restore. You can find it in my previous patch.

I've been meaning to say something, but haven't had an opportunity to
really review your patches...

"allocMem2" is a terrible name at least, and at worst is duplicating
code. I am not at all comfortable with that. If we need to refactor
something we can work with the Cambridge guys to do it.

-- 
Hollis Blanchard
IBM Linux Technology Center

_______________________________________________
Xen-ppc-devel mailing list
Xen-ppc-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ppc-devel

References:
- [XenPPC][Patch] memory alignment check for allocMem
  - From: geyi
- Re: [XenPPC][Patch] memory alignment check for allocMem
  - From: Jimi Xenidis
- Re: [XenPPC][Patch] memory alignment check for allocMem
  - From: geyi

Prev by Date: Re: [XenPPC][Patch] memory alignment check for allocMem
Next by Date: Re: [XenPPC] [linux-ppc-2.6] [XEN][POWERPC] Forgot to free foregin page by Xen VM area
Previous by thread: Re: [XenPPC][Patch] memory alignment check for allocMem
Next by thread: [XenPPC] [xenppc-unstable] [XEND][POWERPC] Fix typo where '&' was used instead of '%'
Index(es):
- Date
- Thread

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