From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34115) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X1UFv-0000z7-L6 for qemu-devel@nongnu.org; Mon, 30 Jun 2014 01:36:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X1UFq-0006el-Mn for qemu-devel@nongnu.org; Mon, 30 Jun 2014 01:35:55 -0400 Received: from [59.151.112.132] (port=29191 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X1UFq-0006dy-7R for qemu-devel@nongnu.org; Mon, 30 Jun 2014 01:35:50 -0400 Date: Mon, 30 Jun 2014 13:33:42 +0800 From: Hu Tao Message-ID: <20140630053342.GA8535@G08FNSTD100614.fnst.cn.fujitsu.com> References: <1403687054-13682-1-git-send-email-hutao@cn.fujitsu.com> <20140629152022.GC29448@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: <20140629152022.GC29448@redhat.com> Subject: Re: [Qemu-devel] [PATCH] numa: check for busy memory backend List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: Igor Mammedov , qemu-devel@nongnu.org, Yasunori Goto On Sun, Jun 29, 2014 at 06:20:22PM +0300, Michael S. Tsirkin wrote: > On Wed, Jun 25, 2014 at 05:04:14PM +0800, Hu Tao wrote: > > ..to prevent one memory backend from being used by more than one numa > > node. > > Thanks, but please always make the msg content self-contained > so it can be understood without the subject. > E.g. here, just drop "..to". > > Are you sure we want this? Is there a chance sharing a backend > can be useful? This patch is actually a bug fix. Even if we will want backend sharing, we can do it after. > > Igor, what's your take? > > > > > Signed-off-by: Hu Tao > > --- > > numa.c | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/numa.c b/numa.c > > index e471afe..6c1c554 100644 > > --- a/numa.c > > +++ b/numa.c > > @@ -279,6 +279,13 @@ void memory_region_allocate_system_memory(MemoryRegion *mr, Object *owner, > > exit(1); > > } > > > > + if (memory_region_is_mapped(seg)) { > > + char *path = object_get_canonical_path_component(OBJECT(backend)); > > + error_report("memory backend %s is busy", path); > > + g_free(path); > > + exit(1); > > + } > > + > > memory_region_add_subregion(mr, addr, seg); > > vmstate_register_ram_global(seg); > > addr += size; > > -- > > 1.9.3