From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:51565)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <bsd@redhat.com>) id 1Up2NH-0005kZ-5E
	for qemu-devel@nongnu.org; Tue, 18 Jun 2013 16:19:33 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <bsd@redhat.com>) id 1Up2NE-0008By-4B
	for qemu-devel@nongnu.org; Tue, 18 Jun 2013 16:19:30 -0400
Received: from mx1.redhat.com ([209.132.183.28]:55821)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <bsd@redhat.com>) id 1Up2ND-0008Bs-S3
	for qemu-devel@nongnu.org; Tue, 18 Jun 2013 16:19:28 -0400
From: Bandan Das <bsd@redhat.com>
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>
Date: Tue, 18 Jun 2013 16:19:18 -0400
Message-ID: <jpg1u7zdveh.fsf@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain
Subject: Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory
	policy
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Eduardo Habkost <ehabkost@redhat.com>
Cc: andre.przywara@amd.com, aliguori@us.ibm.com, qemu-devel@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>, y-goto@jp.fujitsu.com, afaerber@suse.de, Wanlong Gao <gaowanlong@cn.fujitsu.com>

Eduardo Habkost <ehabkost@redhat.com> 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)

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