From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Date: Sat, 27 Sep 2008 10:51:31 +0000 Subject: Re: [patch] stop passing in global variable as argument to cmos_init() Message-Id: <48DE1033.5020704@redhat.com> List-Id: References: <48DCDD8E.4070409@sgi.com> In-Reply-To: <48DCDD8E.4070409@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: > Hi, > > Looking through the ia64 code I came across this little gem. > > At some point someone added a new argument to hw/pc.c:cmos_init() named > 'smp_cpus', and then passed in the global variable 'smp_cpus' as the > argument. This propagated through to the ia64 code as well. > > I checked, this isn't present in the upstream QEMU code, so lets kill > it in the KVM branch. One small step to get closer to upstream :-) > Applied, thanks (though personally I prefer not depending on global variables and their initialization order, etc.) -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.