From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60806) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wwr2i-0007Cj-76 for qemu-devel@nongnu.org; Tue, 17 Jun 2014 06:55:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Wwr2a-0005L4-0I for qemu-devel@nongnu.org; Tue, 17 Jun 2014 06:55:08 -0400 Received: from mx1.redhat.com ([209.132.183.28]:9858) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wwr2Z-0005L0-NV for qemu-devel@nongnu.org; Tue, 17 Jun 2014 06:54:59 -0400 Date: Tue, 17 Jun 2014 13:55:20 +0300 From: "Michael S. Tsirkin" Message-ID: <20140617105520.GB6991@redhat.com> References: <20140616110933.GA10791@redhat.com> <20140617084753.GA30206@G08FNSTD100614.fnst.cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140617084753.GA30206@G08FNSTD100614.fnst.cn.fujitsu.com> Subject: Re: [Qemu-devel] [PATCH v2 0/4] fixes for pci tree, v2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Hu Tao Cc: Paolo Bonzini , Igor Mammedov , qemu-devel@nongnu.org, Yasunori Goto On Tue, Jun 17, 2014 at 04:47:53PM +0800, Hu Tao wrote: > On Mon, Jun 16, 2014 at 02:09:33PM +0300, Michael S. Tsirkin wrote: > > On Mon, Jun 16, 2014 at 06:05:40PM +0800, Hu Tao wrote: > > > Michael, > > > > > > This is fixes for your pci tree v2. please review. thanks! > > > > Patches 1,4 applied. Thanks! > > What about patches 2,3? If you need I can resend patch 2 based on Igor's > comments. About patch 3, I assume you leave it as it is not a > regression. Patch 2 isn't a regression either, right? Please go ahead and repost patches 2,3 addressing comments. They are unrelated so don't have to be in a series, even. > > > > > changes to v1: > > > > > > 1. adds the missing query-memdev and change into querying qom for memdevs(patch 1) > > > 2. tweak error message (patch 2) > > > 3. improve error handling for memory-backend-file (patch 3) > > > 4. fixup the problem that nodes not start from 0 (patch 4) > > > > > > Hu Tao (4): > > > qmp: add query-memdev > > > check if we have space left for hotplugged memory > > > memory-backend-file: improve error handling > > > fixup! numa: add -numa node,memdev= option > > > > > > exec.c | 26 ++++++++++++----- > > > hw/mem/pc-dimm.c | 7 +++++ > > > numa.c | 89 +++++++++++++++++++++++++++++++++++++++++++++++++++++++- > > > qapi-schema.json | 40 +++++++++++++++++++++++++ > > > qmp-commands.hx | 38 ++++++++++++++++++++++++ > > > 5 files changed, 191 insertions(+), 9 deletions(-) > > > > > > -- > > > 1.9.3