From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47297) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WkpvL-0007aw-C0 for qemu-devel@nongnu.org; Thu, 15 May 2014 03:17:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WkpvA-00026l-A4 for qemu-devel@nongnu.org; Thu, 15 May 2014 03:17:51 -0400 Message-ID: <537469DD.7010503@hp.com> Date: Thu, 15 May 2014 15:16:45 +0800 From: "Li, ZhenHua" MIME-Version: 1.0 References: <1399964954-10842-1-git-send-email-zhen-hual@hp.com> <1399964954-10842-5-git-send-email-zhen-hual@hp.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 4/4] Support more than 255 cpus: runtime check List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Filippov Cc: qemu-trivial@nongnu.org, qemu-devel , qemu-stable Maybe it should be 4 bytes for 4096 (0x1000). On 05/13/2014 04:19 PM, Max Filippov wrote: > On Tue, May 13, 2014 at 11:09 AM, Li, Zhen-Hua wrote: >> From: "Li, ZhenHua" >> >> There is some runtime check for max cpu count. Make them support 4096 cpus. >> >> Signed-off-by: Li, ZhenHua >> --- >> hw/i386/acpi-build.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c >> index c98df88..5c3bf10 100644 >> --- a/hw/i386/acpi-build.c >> +++ b/hw/i386/acpi-build.c > [...] > >> @@ -680,7 +680,7 @@ build_append_notify_method(GArray *device, const char *name, >> for (i = 0; i < count; i++) { >> GArray *target = build_alloc_array(); >> build_append_nameseg(target, format, i); >> - assert(i < 256); /* Fits in 1 byte */ >> + assert(i < 4096); /* Fits in 1 byte */ > The comment is no longer true. > Also the function build_append_notify_method is called with format argument > set to "CP%0.02X", looks like this should be changed to "CP%0.03X". >