From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Date: Wed, 12 Nov 2008 13:12:29 +0000 Subject: Re: [patch] remove vcpu_info array v5 Message-Id: <491AD63D.8000001@redhat.com> List-Id: References: <4909C00F.8050704@sgi.com> In-Reply-To: <4909C00F.8050704@sgi.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: kvm-ia64@vger.kernel.org Jes Sorensen wrote: > Avi Kivity wrote: >> Jes Sorensen wrote: >>> What I meant was that I was able to compile the code, and there was >>> only >>> one piece left that needed that function, which is why I moved it and >>> made it static in the acpi code. >>> >> >> That's because there is another static array in acpi.c... > > Which array are you talking about? ACPI has it's own limitations, like > we need ACPI 3.0 to go beyond 256 processors :-( > Sorry. apic.c: static APICState *local_apics[MAX_APICS + 1]; A lucky typo, though, since it brought the acpi issue into the discussion. >>> Well we have to start somewhere. Just because there's limits in other >>> areas too, doesn't mean we should discard valid improvements elsewhere. >>> I'll get to some of that at some point, but there's soo many cards used >>> to build this house.... >> >> Your patch builds the 16383rd floor while ignoring the 15th. Qemu >> scaling issues are much, much more painful than what you were >> addressing. > > Well I am not ignoring the 15th floor, I am just putting in expandable > rails that can go all the way to the 16383rd. Next I'll get to look at > putting in the button to make the elevator stop at the 15th+ floors :) Well I want 16384 buttons in the elevator, not one button you need to press 16383 times to get to your floor :) -- error compiling committee.c: too many arguments to function