From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andre Przywara Subject: Re: [PATCH 1/4] hvm: NUMA guest: extend memops hypercall Date: Fri, 4 Jul 2008 14:48:39 +0200 Message-ID: <486E1C27.300@amd.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Keir Fraser Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org Keir Fraser wrote: > On 4/7/08 12:14, "Andre Przywara" wrote: > >>> Should unprivileged domUs be allowed to specify the NUMA node they allocate >>> from, regardless of whether they even run there? Seems like a breakage of >>> guest isolation to me. >> Good catch (I code in HVM land most of the time), I hope this small >> (attached) patch fixes this. > > Looking some more, I still don't see that this patch can work. Don't the > subfunctions in memory.c go and OR in MEMF_node() values on top of what the > caller may have specified?? Maybe I don't get your question right, but the only part where the caller specified node number is used is the line I handled in the last patch. Later they only use the member memflags of struct memop_args, not struct xen_memory_reservation. If the node number is not specified (or blocked), it will be later determined by looking at the current scheduled pCPU (and thus node), but this is the current behavior anyway. Regards, Andre. -- Andre Przywara AMD-OSRC (Dresden) Tel: x84917