Re: [SeaBIOS] [PATCH] KVM vcpu hotplug seabios infrastructure

[Isaku Yamahata]

On Mon, Jun 21, 2010 at 02:52:51PM +0800, Liu, Jinsong wrote:
> Isaku,
>
> Thanks for comments!
> I didn't fine changeset 12cbb43b9dc8a16f712188cb308a0eb0321f3e20, I pull from git://git.kernel.org/pub/scm/virt/kvm/seabios.git, any misunderstanding?

I'm referring to git://git.linuxtogo.org/home/kevin/seabios.git
According to http://www.coreboot.org/SeaBIOS,
it seems that Keven maintains that repo as the seabios upstream.
Kevin when pushing patches, which repository should we refer?
> As for makefile rule for acpi-ssdt-15.hex and acpi-ssdt-anycpu.hex, we in fact combine them into 1 generic rule in our old patch, as attached.
> However, the old makefile require iasl for user compiling seabios.
> Kevin prefer not using iasl, so we update it as new version.

Something like the below didn't work?
(I just wrote it just to show my idea. I didn't any test.)

+src/%.hex: src/acpi-ssdt.dsl src/mk_ssdt.c
+       @echo "Compiling SSDT for general os which support more cpus"
+       $(Q)$(CC) -o mk_$* src/mk_ssdt.c
+       head -n -2 $< > src/$*.dsl.i
+       ./mk_$* >> src/$*.dsl.i
+       $(Q)cpp -P src/$*.dsl.i > src/$*.dsl
+       $(Q)iasl -tc -p $@ src/acpi-ssdt-anycpu.dsl
+       $(Q)sed -i -e 's/AmlCode/$*/g' $@
+       $(Q)rm mk_$* src/$*.dsl.i src/$*.aml src/$*.dsl

thanks,
> Thanks,
> Jinsong
>
> Isaku Yamahata wrote:
> > Hi Jison.
> >
> > Can you check the changeset of
> > 12cbb43b9dc8a16f712188cb308a0eb0321f3e20 and update Makefile changes?
> >
> > The rule for acpi-ssdt-15.hex and acpi-ssdt-anycpu.hex seems
> > same, so it can be combined into a single generic rule.
> > The rules removes the files at the last, but it would be more
> > reliable to use Makefile intermediate and let make to remove
> > them automatically.
> >
> > On Sun, Jun 20, 2010 at 11:40:02PM +0800, Liu, Jinsong wrote:
> > > Kevin O'Connor wrote:
> > > > On Wed, Jun 16, 2010 at 12:19:24PM +0800, Liu, Jinsong wrote:
> > > > > Gleb Natapov wrote:
> > > > > > FWIW looks good to me. This patch removes dynamic creation of
> > > > > > Processor scope and supports only 15/255 cpus configuration, but
> > > > > > given that new code is much more complex that old one I don't see
> > > > > > other possibilities (unless we'll add aml compiler to seabios
> > > > > > runtime).
> > > > I'm okay with the approach (two SSDTs selected by cpu count).  The
> > > > build changes don't look right (15.c is too cryptic).  However, more
> > > > importantly, I want to see where we end up going with the
> > > > qemu/seabios acpi interface discussion before committing this.
> > > >
> > > > -Kevin
> > >
> > > Kevin,
> > >
> > > Thanks for comments!
> > >
> > > In your last comments, you prefer not to use iasl to compile seabios.
> > > Now I update the patch as attached (only need update Makefile):
> > > cancel cryptic name like 15.c from Makefile, and we give detailed
> > >
> > > description for cpu number in new version; 
> > > 2. user don't need iasl again to compile seabios, except if they
> > > want to change dsdt/ssdt itself;
> > >
> > > The process to apply patch and build in my desktop is:
> > > $ git apply kvm-vcpu-hotplug-seabios.patch
> > > $ git add src/acpi-ssdt.dsl
> > > $ git add src/mk_ssdt.c
> > > $ git add src/acpi-ssdt-15.hex
> > > $ git add src/acpi-ssdt-anycpu.hex
> > > $ git commit -a
> > > $ make
> > >
> > > We test the updated patch, it works OK with linux2.6.32 hot-add and
> > > hot-remove.
> > >
> > > Thanks,
> > > Jinsong
> > > SeaBIOS mailing list
> > > [e-mail]
> > > http://www.seabios.org/mailman/listinfo/seabios