From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52054) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bzvQJ-0000r4-CI for qemu-devel@nongnu.org; Thu, 27 Oct 2016 20:53:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bzvQG-0003GI-6l for qemu-devel@nongnu.org; Thu, 27 Oct 2016 20:53:31 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57798) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1bzvQG-0003Fx-1M for qemu-devel@nongnu.org; Thu, 27 Oct 2016 20:53:28 -0400 Date: Thu, 27 Oct 2016 22:53:22 -0200 From: Eduardo Habkost Message-ID: <20161028005322.GP5057@thinpad.lan.raisama.net> References: <1476878743-144953-12-git-send-email-imammedo@redhat.com> <1476975522-154506-1-git-send-email-imammedo@redhat.com> <20161020185148.GJ5057@thinpad.lan.raisama.net> <20161021105327.2bce44f7@nial.brq.redhat.com> <20161028010751-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161028010751-mutt-send-email-mst@kernel.org> Subject: Re: [Qemu-devel] [PATCH] fixup! pc: add 'etc/boot-cpus' fw_cfg file for machine with more than 255 CPUs List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: Igor Mammedov , liuxiaojian6@huawei.com, rkrcmar@redhat.com, qemu-devel@nongnu.org, peterx@redhat.com, kevin@koconnor.net, kraxel@redhat.com, pbonzini@redhat.com, lersek@redhat.com, chao.gao@intel.com On Fri, Oct 28, 2016 at 01:08:32AM +0300, Michael S. Tsirkin wrote: > On Fri, Oct 21, 2016 at 10:53:27AM +0200, Igor Mammedov wrote: > > On Thu, 20 Oct 2016 16:51:48 -0200 > > Eduardo Habkost wrote: > > > > > On Thu, Oct 20, 2016 at 04:58:42PM +0200, Igor Mammedov wrote: > > > > Currently firmware uses 1 byte at 0x5F offset in RTC CMOS > > > > to get number of CPUs present at boot. However 1 byte is > > > > not enough to handle more than 255 CPUs. So add a new > > > > fw_cfg file that would allow QEMU to tell it. > > > > For compat reasons add file only for machine types that > > > > support more than 255 CPUs. > > > > > > > > Signed-off-by: Igor Mammedov > > > > > > I suggest squashing this into the patch, to clarify why we are > > > setting it to 0. > > > > > > Signed-off-by: Eduardo Habkost > > > > Could you squash it in while merging series? > > I'm not sure who's merging it (Eduardo do you want to?) > if me, I'd rather see a clean repost, with a squash and acks > included. It's already merged. See commit 080ac219cc7d9c55adf925c3545b7450055ad625. -- Eduardo