From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39312) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UpDVv-0003xS-OE for qemu-devel@nongnu.org; Wed, 19 Jun 2013 04:13:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UpDVu-0000Zb-5e for qemu-devel@nongnu.org; Wed, 19 Jun 2013 04:13:11 -0400 Received: from [222.73.24.84] (port=3121 helo=song.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UpDVt-0000Tc-R5 for qemu-devel@nongnu.org; Wed, 19 Jun 2013 04:13:10 -0400 Message-ID: <51C1653F.4000902@cn.fujitsu.com> Date: Wed, 19 Jun 2013 16:01:03 +0800 From: Wanlong Gao MIME-Version: 1.0 References: <1371542991-15911-1-git-send-email-gaowanlong@cn.fujitsu.com> <1371542991-15911-4-git-send-email-gaowanlong@cn.fujitsu.com> <51C02665.1060205@redhat.com> <20130618190049.GA2825@otherpad.lan.raisama.net> In-Reply-To: Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy Reply-To: gaowanlong@cn.fujitsu.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Bandan Das Cc: andre.przywara@amd.com, aliguori@us.ibm.com, Eduardo Habkost , qemu-devel@nongnu.org, y-goto@jp.fujitsu.com, Paolo Bonzini , afaerber@suse.de, Wanlong Gao On 06/19/2013 04:19 AM, Bandan Das wrote: > Eduardo Habkost writes: > >> On Tue, Jun 18, 2013 at 11:20:37AM +0200, Paolo Bonzini wrote: >> [...] >>> Also, please use QemuOpts instead of yet another homegrown parser. >>> Eduardo, I think you had the most recent attempt to convert -numa to >>> QemuOpts? >> >> I had one, but I believe it is more complex than it should have been. I >> was creating a "numa-node" config section while keeping "-numa" just for >> compatbility, but I don't think we really need to do that. > > Ah, I was working on an update to Eduardo's earlier proposals for multiple CPU ranges > and part of the change was to convert to QemuOpts. > > Probably needs more testing but posted it anyway since we are already discussing this : > [PATCH v3] vl.c: Support multiple CPU ranges on -numa option > (hasn't shown up in the archives yet) Here is the archive: http://thread.gmane.org/gmane.comp.emulators.qemu/217491 So, are you all ACK with this? And we are not considering compatible thing by using "cpu" instead of "cpus" here? Thanks, Wanlong Gao > > Bandan > >> If you want to take a look, the old attemp is at: >> git://github.com/ehabkost/qemu-hacks.git work/old-numa-node-config-section-experiment >