SeaBIOS May 2014

seabios@seabios.org

16 participants
46 discussions

Re: [SeaBIOS] [PATCH] acpi: Fix acpi dsdt compile warning.

by Kevin O'Connor

On Fri, May 09, 2014 at 09:20:21AM -0400, Kevin O'Connor wrote: > The _EJ0 method should not return a value. FYI, I have pushed this patch along with the three other patches I sent on Thursday. -Kevin

6 years, 5 months

[PATCH] acpi: Fix acpi dsdt compile warning.

by Kevin O'Connor

The _EJ0 method should not return a value. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/fw/acpi-dsdt.dsl | 1 - src/fw/ssdt-pcihp.dsl | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/fw/acpi-dsdt.dsl b/src/fw/acpi-dsdt.dsl index 56243c3..3556dca 100644 --- a/src/fw/acpi-dsdt.dsl +++ b/src/fw/acpi-dsdt.dsl @@ -139,7 +139,6 @@ DefinitionBlock ( Method(PCEJ, 1, NotSerialized) { // _EJ0 method - eject callback Store(ShiftLeft(1, Arg0), B0EJ) - Return (0x0) } /* Hotplug notification method supplied by SSDT */ diff --git a/src/fw/ssdt-pcihp.dsl b/src/fw/ssdt-pcihp.dsl index 67e485f..cb24c11 100644 --- a/src/fw/ssdt-pcihp.dsl +++ b/src/fw/ssdt-pcihp.dsl @@ -29,7 +29,7 @@ DefinitionBlock ("ssdt-pcihp.aml", "SSDT", 0x01, "BXPC", "BXSSDTPCIHP", 0x1) Name(_ADR, 0xAA0000) ACPI_EXTRACT_METHOD_STRING ssdt_pcihp_ej0 Method(_EJ0, 1) { - Return (PCEJ(_SUN)) + PCEJ(_SUN) } } } -- 1.9.0

6 years, 5 months

Re: [SeaBIOS] [PATCH] If an int 1587 call is made from an option rom, stay in bigreal mode.

by Kevin O'Connor

On Sat, Apr 19, 2014 at 12:30:01PM -0400, Kevin O'Connor wrote: > Modify the int 1587 handler to check if the POST phase is still > running. If it is, use bigreal mode segment limits so that the caller > remains in bigreal mode when the 1587 handler completes. This helps > with SeaVGABIOS' use of "direct" framebuffer accesses (an option rom > may attempt to display text during its option rom execution which can > cause SeaVGABIOS to make the int 1587 calls). FYI, I have committed this patch. -Kevin

6 years, 5 months

[PATCH v2] ld: don't hardcode "-melf_i386"

by Roger Pau Monne

On FreeBSD, the name of the emulation is "-melf_i386_fbsd", so allow SeaBIOS to fetch the emulation to use from the environment variables. This allows SeaBIOS to compile on FreeBSD with gcc. Signed-off-by: Roger Pau Monné <roger.pau(a)citrix.com> Cc: Ian Campbell <ian.campbell(a)citrix.com> --- Changes in v2: - Rebase on top of master --- Makefile | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index e567ba1..4338f94 100644 --- a/Makefile +++ b/Makefile @@ -24,6 +24,7 @@ STRIP=$(CROSS_PREFIX)strip PYTHON=python CPP=cpp IASL:=iasl +LD32BIT_FLAG:=-melf_i386 # Source files SRCBOTH=misc.c stacks.c output.c string.c x86.c block.c cdrom.c mouse.c kbd.c \ @@ -154,8 +155,8 @@ $(OUT)romlayout16.lds: $(OUT)ccode32flat.o $(OUT)code32seg.o $(OUT)ccode16.o $(O @echo " Building ld scripts" $(Q)./scripts/buildversion.sh $(OUT)version.c $(Q)$(CC) $(CFLAGS32FLAT) -c $(OUT)version.c -o $(OUT)version.o - $(Q)$(LD) -melf_i386 -r $(OUT)ccode32flat.o $(OUT)version.o -o $(OUT)code32flat.o - $(Q)$(LD) -melf_i386 -r $(OUT)ccode16.o $(OUT)romlayout.o -o $(OUT)code16.o + $(Q)$(LD) $(LD32BIT_FLAG) -r $(OUT)ccode32flat.o $(OUT)version.o -o $(OUT)code32flat.o + $(Q)$(LD) $(LD32BIT_FLAG) -r $(OUT)ccode16.o $(OUT)romlayout.o -o $(OUT)code16.o $(Q)$(OBJDUMP) -thr $(OUT)code32flat.o > $(OUT)code32flat.o.objdump $(Q)$(OBJDUMP) -thr $(OUT)code32seg.o > $(OUT)code32seg.o.objdump $(Q)$(OBJDUMP) -thr $(OUT)code16.o > $(OUT)code16.o.objdump -- 1.7.7.5 (Apple Git-26)

6 years, 5 months

[PATCH] Fix int 1589 calls when CONFIG_ENTRY_EXTRASTACK is enabled.

by Kevin O'Connor

The int 1589 call is entered in real mode and returns in protected mode. However, the code to use the "extra stack" does not support that. Fix this by never using the "extra stack" on int 1589 calls. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/post.c | 2 +- src/romlayout.S | 10 ++++++++++ src/system.c | 4 ++-- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/src/post.c b/src/post.c index 0d9e66f..0fdd28e 100644 --- a/src/post.c +++ b/src/post.c @@ -61,7 +61,7 @@ ivt_init(void) SET_IVT(0x12, FUNC16(entry_12)); SET_IVT(0x13, FUNC16(entry_13_official)); SET_IVT(0x14, FUNC16(entry_14)); - SET_IVT(0x15, FUNC16(entry_15)); + SET_IVT(0x15, FUNC16(entry_15_official)); SET_IVT(0x16, FUNC16(entry_16)); SET_IVT(0x17, FUNC16(entry_17)); SET_IVT(0x18, FUNC16(entry_18)); diff --git a/src/romlayout.S b/src/romlayout.S index f6b7979..57e8bcc 100644 --- a/src/romlayout.S +++ b/src/romlayout.S @@ -378,6 +378,12 @@ entry_pcibios16: ENTRY_ARG handle_pcibios iretw +// int 1589 entry point + DECLFUNC entry_1589 +entry_1589: + ENTRY_ARG handle_1589 + iretw + // BIOS32 support .code32 DECLFUNC entry_bios32 @@ -682,6 +688,10 @@ entry_10_0x0f: IRQ_ENTRY_ARG 11 ORG 0xf859 + .global entry_15_official +entry_15_official: + cmpb $0x89, %ah + je entry_1589 // 1589 calls return in protected mode IRQ_ENTRY_ARG 15 // 0xfa6e - vgafont8 in font.c diff --git a/src/system.c b/src/system.c index 3cb2228..756dc31 100644 --- a/src/system.c +++ b/src/system.c @@ -191,9 +191,10 @@ handle_1588(struct bregs *regs) } // Switch to protected mode -static void +void VISIBLE16 handle_1589(struct bregs *regs) { + debug_enter(regs, DEBUG_HDL_15); set_a20(1); pic_reset(regs->bl, regs->bh); @@ -355,7 +356,6 @@ handle_15(struct bregs *regs) case 0x86: handle_1586(regs); break; case 0x87: handle_1587(regs); break; case 0x88: handle_1588(regs); break; - case 0x89: handle_1589(regs); break; case 0x90: handle_1590(regs); break; case 0x91: handle_1591(regs); break; case 0xc0: handle_15c0(regs); break; -- 1.9.0

6 years, 5 months

[PATCH] build: Make sure romlayout.o is recompiled on Kconfig change.

by Kevin O'Connor

Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index e567ba1..608d033 100644 --- a/Makefile +++ b/Makefile @@ -146,7 +146,7 @@ $(OUT)code32seg.o: $(OUT)autoconf.h $(patsubst %.c, $(OUT)src/%.o,$(SRC32SEG)) ; $(OUT)ccode32flat.o: $(OUT)autoconf.h $(patsubst %.c, $(OUT)src/%.o,$(SRC32FLAT)) ; $(call whole-compile, $(CFLAGS32FLAT), $(addprefix src/, $(SRC32FLAT)),$@) -$(OUT)romlayout.o: src/romlayout.S $(OUT)asm-offsets.h +$(OUT)romlayout.o: src/romlayout.S $(OUT)autoconf.h $(OUT)asm-offsets.h @echo " Compiling (16bit) $@" $(Q)$(CC) $(CFLAGS16) -c -D__ASSEMBLY__ $< -o $@ -- 1.9.0

6 years, 5 months

[PATCH] xhci: Use msleep() instead of mdelay() for port status delay.

by Kevin O'Connor

Use msleep() so that interrupts and other threads can occur during the delay. Signed-off-by: Kevin O'Connor <kevin(a)koconnor.net> --- src/hw/usb-xhci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/hw/usb-xhci.c b/src/hw/usb-xhci.c index f64f562..ca1fe25 100644 --- a/src/hw/usb-xhci.c +++ b/src/hw/usb-xhci.c @@ -705,7 +705,7 @@ configure_xhci(void *data) writel(&xhci->op->usbcmd, reg); // FIXME: try find a more elegant way than a fixed delay - mdelay(100); + msleep(100); usb_enumerate(&xhci->hub); // XXX - should walk list of pipes and free unused pipes. -- 1.9.0

6 years, 5 months

Re: [SeaBIOS] [Xen-devel] [PATCH v3] Hvmloader: Modify ACPI to only supply _EJ0 methods for PCIslots that support hotplug by runtime patching

by Kevin O'Connor

On Thu, May 08, 2014 at 07:29:15AM +0000, Gonglei (Arei) wrote: > > > > --- /dev/null > > > > +++ b/tools/firmware/hvmloader/tools/acpi_extract.py > > > > @@ -0,0 +1,308 @@ > > > > +#!/usr/bin/python > > > > +# Copyright (C) 2011 Red Hat, Inc., Michael S. Tsirkin <mst(a)redhat.com> > > > > +# > > > > +# This file may be distributed under the terms of the GNU GPLv3 license. > > > > > > Well, Xen code is under GPLv2 so I don't think you can do this? > > > > > > Unless Michael is willing to release it under a different licence? > > > > I'd be fine with "GPLv2 or later" so code can flow back to seabios > > as well - but you will have to track down everyone else who > > changed this file since I wrote it and get their permission. > > > Thanks, Michael. I search the git log of above two python files, and > find two person has changed those two files, Johannes and Kevin. > > CC'ing Johannes and Kevin for their permission, which using those > two files in Xen code. Thanks in advance! I'm okay with those two python scripts being used as "GPLv2 or later". -Kevin

6 years, 5 months

Re: [SeaBIOS] [Xen-devel] [PATCH v3] Hvmloader: Modify ACPI to only supply _EJ0 methods for PCIslots that support hotplug by runtime patching

by Ian Campbell

On Thu, 2014-05-08 at 07:29 +0000, Gonglei (Arei) wrote: > > + rmvc_pcrm = inl(PCI_RMV_BASE); > > > > > > > > > So .. what is at this special 0xae0c address? > > > > > > Is there some code in QEMU that needs this treatment as well? > > > > Yep, the 0xae0c is defined in upstream QEMU, which show the hotplugging ability > of PCI devices. Is this considered to be a ABI by qemu? Or is it subject to change? How is this going to interact with http://lists.xen.org/archives/html/xen-devel/2014-05/msg00026.html ? (At leas then it is a Xen interface I suppose) Ian.

6 years, 5 months

Re: [SeaBIOS] [Xen-devel] [PATCH v3] Hvmloader: Modify ACPI to only supply _EJ0 methods for PCIslots that support hotplug by runtime patching

by Michael S. Tsirkin

On Thu, May 08, 2014 at 07:29:15AM +0000, Gonglei (Arei) wrote: > Hi, all > > > -----Original Message----- > > From: Michael S. Tsirkin [mailto:mst@redhat.com] > > Sent: Thursday, May 08, 2014 4:15 AM > > To: Konrad Rzeszutek Wilk > > Cc: Gonglei (Arei); qemu-devel(a)nongnu.org; xen-devel(a)lists.xen.org; > > Huangweidong (C); Ian.Campbell(a)citrix.com; stefano.stabellini(a)eu.citrix.com; > > Hanweidong (Randy); fabio.fantoni(a)m2r.biz; JBeulich(a)suse.com; > > anthony.perard(a)citrix.com; Gaowei (UVP) > > Subject: Re: [Xen-devel] [PATCH v3] Hvmloader: Modify ACPI to only supply _EJ0 > > methods for PCIslots that support hotplug by runtime patching > > > > On Wed, May 07, 2014 at 02:48:48PM -0400, Konrad Rzeszutek Wilk wrote: > > > On Sun, May 04, 2014 at 05:25:15PM +0800, arei.gonglei(a)huawei.com > > wrote: > > > > From: Gaowei <gao.gaowei(a)huawei.com> > > > > > > > > In Xen platform, after using upstream qemu, the all of pci devices will show > > > > hotplug in the windows guest. In this situation, the windows guest may > > occur > > > > blue screen when VM' user click the icon of VGA card for trying unplug VGA > > card. > > > > However, we don't hope VM's user can do such dangerous operation, and > > showing > > > > all pci devices inside the guest OS is unfriendly. > > > > > > > > This is done by runtime patching: > > > > - Rename _EJ0 methods for PCI slots in DSDT to EJ0_:note that this has > > the > > > > same checksum, but is ignored by OSPM. > > > > - At compile time, look for these methods in ASL source,find the > > matching AML, > > > > and store the offsets of these methods in a table named > > aml_ej0_data. > > > > - At run time, go over aml_ej0_data, check which slots not support > > hotplug and > > > > patch the ACPI table, replacing _EJ0 with EJ0_. > > > > > > How does this work with traditional QEMU ? Is this only when running with > > SeaBIOS and upstream QEMU? > > > > The traditional QEMU is unaffected and works fine as normal. Yes, it is only useful > With seabios and upstream QEMU. > > > > > > > > > Signed-off-by: Gaowei <gao.gaowei(a)huawei.com> > > > > Signed-off-by: Gonglei <arei.gonglei(a)huawei.com> > > > > --- > > > > v3->v2: > > > > reformat on the latest master > > > > v2->v1: > > > > rework by Jan Beulich's suggestion: > > > > - adjust the code style > > > > > > > > tools/firmware/hvmloader/acpi/Makefile | 44 ++- > > > > tools/firmware/hvmloader/acpi/acpi2_0.h | 4 + > > > > tools/firmware/hvmloader/acpi/build.c | 22 ++ > > > > tools/firmware/hvmloader/acpi/dsdt.asl | 1 + > > > > tools/firmware/hvmloader/acpi/mk_dsdt.c | 2 + > > > > tools/firmware/hvmloader/ovmf.c | 6 +- > > > > tools/firmware/hvmloader/rombios.c | 4 + > > > > tools/firmware/hvmloader/seabios.c | 8 + > > > > tools/firmware/hvmloader/tools/acpi_extract.py | 308 > > +++++++++++++++++++++ > > > > .../hvmloader/tools/acpi_extract_preprocess.py | 41 +++ > > > > 10 files changed, 428 insertions(+), 12 deletions(-) > > > > create mode 100644 tools/firmware/hvmloader/tools/acpi_extract.py > > > > create mode 100644 > > tools/firmware/hvmloader/tools/acpi_extract_preprocess.py > > > > > > > > diff --git a/tools/firmware/hvmloader/acpi/Makefile > > b/tools/firmware/hvmloader/acpi/Makefile > > > > index 2c50851..f79ecc3 100644 > > > > --- a/tools/firmware/hvmloader/acpi/Makefile > > > > +++ b/tools/firmware/hvmloader/acpi/Makefile > > > > @@ -24,30 +24,46 @@ OBJS = $(patsubst %.c,%.o,$(C_SRC)) > > > > CFLAGS += $(CFLAGS_xeninclude) > > > > > > > > vpath iasl $(PATH) > > > > +vpath python $(PATH) > > > > + > > > > +.DELETE_ON_ERROR:$(filter dsdt_%.c,$(C_SRC)) > > > > + > > > > all: acpi.a > > > > > > > > ssdt_s3.h ssdt_s4.h ssdt_pm.h ssdt_tpm.h: %.h: %.asl iasl > > > > iasl -vs -p $* -tc $< > > > > - sed -e 's/AmlCode/$*/g' $*.hex >$@ > > > > + sed -e 's/AmlCode/$*/g' $*.hex >$@.tmp > > > > + $(call move-if-changed,$@.tmp $@) > > > > rm -f $*.hex $*.aml > > > > > > > > mk_dsdt: mk_dsdt.c > > > > $(HOSTCC) $(HOSTCFLAGS) $(CFLAGS_xeninclude) -o $@ mk_dsdt.c > > > > > > > > dsdt_anycpu_qemu_xen.asl: dsdt.asl mk_dsdt > > > > - awk 'NR > 1 {print s} {s=$$0}' $< > $@ > > > > - ./mk_dsdt --dm-version qemu-xen >> $@ > > > > + awk 'NR > 1 {print s} {s=$$0}' $< > $@.tmp > > > > + sed -i 's/AmlCode/dsdt_anycpu_qemu_xen/g' $@.tmp > > > > + ./mk_dsdt --dm-version qemu-xen >> $@.tmp > > > > + sed -i 's/aml_ej0_name/dsdt_anycpu_qemu_xen_aml_ej0_name/g' > > $@.tmp > > > > + sed -i 's/aml_adr_dword/dsdt_anycpu_qemu_xen_aml_adr_dword/g' > > $@.tmp > > > > + $(call move-if-changed,$@.tmp $@) > > > > > > > > # NB. awk invocation is a portable alternative to 'head -n -1' > > > > dsdt_%cpu.asl: dsdt.asl mk_dsdt > > > > - awk 'NR > 1 {print s} {s=$$0}' $< > $@ > > > > - ./mk_dsdt --maxcpu $* >> $@ > > > > + awk 'NR > 1 {print s} {s=$$0}' $< > $@.tmp > > > > + sed -i 's/AmlCode/dsdt_$*cpu/g' $@.tmp > > > > + ./mk_dsdt --maxcpu $* >> $@.tmp > > > > + $(call move-if-changed,$@.tmp $@) > > > > > > > > -$(filter dsdt_%.c,$(C_SRC)): %.c: iasl %.asl > > > > - iasl -vs -p $* -tc $*.asl > > > > - sed -e 's/AmlCode/$*/g' $*.hex >$@ > > > > - echo "int $*_len=sizeof($*);" >>$@ > > > > - rm -f $*.aml $*.hex > > > > +$(filter dsdt_%.c,$(C_SRC)): %.c: iasl python %.asl > > > > + cpp -P $*.asl > $*.asl.i.orig > > > > + python ../tools/acpi_extract_preprocess.py $*.asl.i.orig > $*.asl.i > > > > + iasl -vs -l -tc -p $* $*.asl.i > > > > + python ../tools/acpi_extract.py $*.lst > $@.tmp > > > > + echo "int $*_len=sizeof($*);" >>$@.tmp > > > > + if grep -q "$*_aml_ej0_name" $@.tmp; then echo "int > > $*_aml_ej0_name_len=sizeof($*_aml_ej0_name);" >>$@.tmp; fi > > > > + if grep -q "$*_aml_adr_dword" $@.tmp; then echo "int > > $*_aml_adr_dword_len=sizeof($*_aml_adr_dword);" >>$@.tmp;fi > > > > + $(call move-if-changed,$@.tmp $@) > > > > + rm -f $*.aml $*.hex $*.asl.i.orig $*.asl.i $*.lst > > > > > > > > iasl: > > > > @echo > > > > @@ -57,6 +73,12 @@ iasl: > > > > @echo > > > > @exit 1 > > > > > > > > +python: > > > > + @echo > > > > + @echo "python is needed" > > > > + @echo > > > > + @exit 1 > > > > + > > > > build.o: ssdt_s3.h ssdt_s4.h ssdt_pm.h ssdt_tpm.h > > > > > > > > acpi.a: $(OBJS) > > > > @@ -64,7 +86,7 @@ acpi.a: $(OBJS) > > > > > > > > clean: > > > > rm -rf *.a *.o $(IASL_VER) $(IASL_VER).tar.gz $(DEPS) > > > > - rm -rf ssdt_*.h dsdt*.c *~ *.aml *.hex mk_dsdt dsdt_*.asl > > > > + rm -rf ssdt_*.h dsdt*.c *~ *.aml *.hex mk_dsdt dsdt_*.asl *.asl.i > > *.asl.i.orig *.lst *.tmp > > > > > > > > install: all > > > > > > > > diff --git a/tools/firmware/hvmloader/acpi/acpi2_0.h > > b/tools/firmware/hvmloader/acpi/acpi2_0.h > > > > index 7b22d80..4ba3957 100644 > > > > --- a/tools/firmware/hvmloader/acpi/acpi2_0.h > > > > +++ b/tools/firmware/hvmloader/acpi/acpi2_0.h > > > > @@ -396,6 +396,10 @@ struct acpi_config { > > > > int dsdt_anycpu_len; > > > > unsigned char *dsdt_15cpu; > > > > int dsdt_15cpu_len; > > > > + unsigned short *aml_ej0_name; > > > > + unsigned short *aml_adr_dword; > > > > + int aml_ej0_name_len; > > > > + int aml_adr_dword_len; > > > > }; > > > > > > > > void acpi_build_tables(struct acpi_config *config, unsigned int physical); > > > > diff --git a/tools/firmware/hvmloader/acpi/build.c > > b/tools/firmware/hvmloader/acpi/build.c > > > > index f1dd3f0..ccce420 100644 > > > > --- a/tools/firmware/hvmloader/acpi/build.c > > > > +++ b/tools/firmware/hvmloader/acpi/build.c > > > > @@ -30,6 +30,8 @@ > > > > #define align16(sz) (((sz) + 15) & ~15) > > > > #define fixed_strcpy(d, s) strncpy((d), (s), sizeof(d)) > > > > > > > > +#define PCI_RMV_BASE 0xae0c > > > > + > > > > extern struct acpi_20_rsdp Rsdp; > > > > extern struct acpi_20_rsdt Rsdt; > > > > extern struct acpi_20_xsdt Xsdt; > > > > @@ -404,6 +406,8 @@ void acpi_build_tables(struct acpi_config *config, > > unsigned int physical) > > > > unsigned char *dsdt; > > > > unsigned long > > secondary_tables[ACPI_MAX_SECONDARY_TABLES]; > > > > int nr_secondaries, i; > > > > + unsigned int rmvc_pcrm = 0; > > > > + unsigned int len_aml_addr = 0, len_aml_ej0 = 0; > > > > > > > > /* Allocate and initialise the acpi info area. */ > > > > mem_hole_populate_ram(ACPI_INFO_PHYSICAL_ADDRESS >> > > PAGE_SHIFT, 1); > > > > @@ -440,6 +444,24 @@ void acpi_build_tables(struct acpi_config *config, > > unsigned int physical) > > > > memcpy(dsdt, config->dsdt_anycpu, config->dsdt_anycpu_len); > > > > nr_processor_objects = HVM_MAX_VCPUS; > > > > } > > > > + len_aml_addr = > > config->aml_adr_dword_len/sizeof(config->aml_adr_dword[0]); > > > > + len_aml_ej0 = > > config->aml_ej0_name_len/sizeof(config->aml_ej0_name[0]); > > > > + if (config->aml_adr_dword_len && config->aml_ej0_name_len && > > (len_aml_addr == len_aml_ej0)) > > > > + { > > > > + rmvc_pcrm = inl(PCI_RMV_BASE); > > > > > > > > > So .. what is at this special 0xae0c address? > > > > > > Is there some code in QEMU that needs this treatment as well? > > > > Yep, the 0xae0c is defined in upstream QEMU, which show the hotplugging ability > of PCI devices. > > Including other special address about acpi hotplug defined in hw/acpi/pcihp.c: > > #define ACPI_PCI_HOTPLUG_STATUS 2 > #define ACPI_PCIHP_ADDR 0xae00 > #define ACPI_PCIHP_SIZE 0x0014 > #define ACPI_PCIHP_LEGACY_SIZE 0x000f > #define PCI_UP_BASE 0x0000 > #define PCI_DOWN_BASE 0x0004 > #define PCI_EJ_BASE 0x0008 > #define PCI_RMV_BASE 0x000c > #define PCI_SEL_BASE 0x0010 > > > > > + printf("rmvc_pcrm is %x\n", rmvc_pcrm); > > > > + for (i = 0; i < len_aml_addr; i++ ) { > > > > + /* Slot is in byte 2 in _ADR */ > > > > + unsigned char slot = dsdt[config->aml_adr_dword[i] + 2] & > > 0x1F; > > > > + /* Sanity check */ > > > > + if (memcmp(dsdt + config->aml_ej0_name[i], "_EJ0", 4)) { > > > > + goto oom; > > > > + } > > > > + if (!(rmvc_pcrm & (0x1 << slot))) { > > > > + memcpy(dsdt + config->aml_ej0_name[i], "EJ0_", 4); > > > > + } > > > > + } > > > > + } > > > > > > > > /* > > > > * N.B. ACPI 1.0 operating systems may not handle FADT with > > revision 2 > > > > diff --git a/tools/firmware/hvmloader/acpi/dsdt.asl > > b/tools/firmware/hvmloader/acpi/dsdt.asl > > > > index 247a8ad..1e7695b 100644 > > > > --- a/tools/firmware/hvmloader/acpi/dsdt.asl > > > > +++ b/tools/firmware/hvmloader/acpi/dsdt.asl > > > > @@ -16,6 +16,7 @@ > > > > * this program; if not, write to the Free Software Foundation, Inc., 59 > > Temple > > > > * Place - Suite 330, Boston, MA 02111-1307 USA. > > > > */ > > > > +ACPI_EXTRACT_ALL_CODE AmlCode > > > > > > > > DefinitionBlock ("DSDT.aml", "DSDT", 2, "Xen", "HVM", 0) > > > > { > > > > diff --git a/tools/firmware/hvmloader/acpi/mk_dsdt.c > > b/tools/firmware/hvmloader/acpi/mk_dsdt.c > > > > index a4b693b..555e062 100644 > > > > --- a/tools/firmware/hvmloader/acpi/mk_dsdt.c > > > > +++ b/tools/firmware/hvmloader/acpi/mk_dsdt.c > > > > @@ -372,7 +372,9 @@ int main(int argc, char **argv) > > > > /* hotplug_slot */ > > > > for (slot = 1; slot <= 31; slot++) { > > > > push_block("Device", "S%i", slot); { > > > > + printf("ACPI_EXTRACT_NAME_DWORD_CONST > > aml_adr_dword\n"); > > > > stmt("Name", "_ADR, %#06x0000", slot); > > > > + printf("ACPI_EXTRACT_METHOD_STRING > > aml_ej0_name\n"); > > > > push_block("Method", "_EJ0,1"); { > > > > stmt("Store", "ShiftLeft(1, %#06x), B0EJ", slot); > > > > stmt("Return", "0x0"); > > > > diff --git a/tools/firmware/hvmloader/ovmf.c > > b/tools/firmware/hvmloader/ovmf.c > > > > index 28dd7bc..ea11564 100644 > > > > --- a/tools/firmware/hvmloader/ovmf.c > > > > +++ b/tools/firmware/hvmloader/ovmf.c > > > > @@ -123,7 +123,11 @@ static void ovmf_acpi_build_tables(void) > > > > .dsdt_anycpu = dsdt_anycpu, > > > > .dsdt_anycpu_len = dsdt_anycpu_len, > > > > .dsdt_15cpu = NULL, > > > > - .dsdt_15cpu_len = 0 > > > > + .dsdt_15cpu_len = 0, > > > > + .aml_ej0_name = NULL, > > > > + .aml_adr_dword = NULL, > > > > + .aml_ej0_name_len = 0, > > > > + .aml_adr_dword_len = 0, > > > > }; > > > > > > > > acpi_build_tables(&config, ACPI_PHYSICAL_ADDRESS); > > > > diff --git a/tools/firmware/hvmloader/rombios.c > > b/tools/firmware/hvmloader/rombios.c > > > > index 810bd24..803c9fa 100644 > > > > --- a/tools/firmware/hvmloader/rombios.c > > > > +++ b/tools/firmware/hvmloader/rombios.c > > > > @@ -179,6 +179,10 @@ static void rombios_acpi_build_tables(void) > > > > .dsdt_anycpu_len = dsdt_anycpu_len, > > > > .dsdt_15cpu = dsdt_15cpu, > > > > .dsdt_15cpu_len = dsdt_15cpu_len, > > > > + .aml_ej0_name = NULL, > > > > + .aml_adr_dword = NULL, > > > > + .aml_ej0_name_len = 0, > > > > + .aml_adr_dword_len = 0, > > > > }; > > > > > > > > acpi_build_tables(&config, ACPI_PHYSICAL_ADDRESS); > > > > diff --git a/tools/firmware/hvmloader/seabios.c > > b/tools/firmware/hvmloader/seabios.c > > > > index dd7dfbe..ca01d27 100644 > > > > --- a/tools/firmware/hvmloader/seabios.c > > > > +++ b/tools/firmware/hvmloader/seabios.c > > > > @@ -33,6 +33,10 @@ > > > > > > > > extern unsigned char dsdt_anycpu_qemu_xen[]; > > > > extern int dsdt_anycpu_qemu_xen_len; > > > > +extern unsigned short dsdt_anycpu_qemu_xen_aml_ej0_name[]; > > > > +extern unsigned short dsdt_anycpu_qemu_xen_aml_adr_dword[]; > > > > +extern int dsdt_anycpu_qemu_xen_aml_ej0_name_len; > > > > +extern int dsdt_anycpu_qemu_xen_aml_adr_dword_len; > > > > > > > > struct seabios_info { > > > > char signature[14]; /* XenHVMSeaBIOS\0 */ > > > > @@ -99,6 +103,10 @@ static void seabios_acpi_build_tables(void) > > > > .dsdt_anycpu_len = dsdt_anycpu_qemu_xen_len, > > > > .dsdt_15cpu = NULL, > > > > .dsdt_15cpu_len = 0, > > > > + .aml_ej0_name = dsdt_anycpu_qemu_xen_aml_ej0_name, > > > > + .aml_adr_dword = dsdt_anycpu_qemu_xen_aml_adr_dword, > > > > + .aml_ej0_name_len = > > dsdt_anycpu_qemu_xen_aml_ej0_name_len, > > > > + .aml_adr_dword_len = > > dsdt_anycpu_qemu_xen_aml_adr_dword_len, > > > > }; > > > > > > > > acpi_build_tables(&config, rsdp); > > > > diff --git a/tools/firmware/hvmloader/tools/acpi_extract.py > > b/tools/firmware/hvmloader/tools/acpi_extract.py > > > > new file mode 100644 > > > > index 0000000..ccec089 > > > > --- /dev/null > > > > +++ b/tools/firmware/hvmloader/tools/acpi_extract.py > > > > @@ -0,0 +1,308 @@ > > > > +#!/usr/bin/python > > > > +# Copyright (C) 2011 Red Hat, Inc., Michael S. Tsirkin <mst(a)redhat.com> > > > > +# > > > > +# This file may be distributed under the terms of the GNU GPLv3 license. > > > > > > Well, Xen code is under GPLv2 so I don't think you can do this? > > > > > > Unless Michael is willing to release it under a different licence? > > > > I'd be fine with "GPLv2 or later" so code can flow back to seabios > > as well - but you will have to track down everyone else who > > changed this file since I wrote it and get their permission. > > > Thanks, Michael. I search the git log of above two python files, and > find two person has changed those two files, Johannes and Kevin. > > CC'ing Johannes and Kevin for their permission, which using those > two files in Xen code. Thanks in advance! > > CC'ing seabios maillist too. > > > Best regards, > -Gonglei As an alternative, you can get a copy from qemu source which has already been relicensed to GPLv2+. Also I wonder: would it be possible for you to load the acpi tables from qemu instead of generating your own? This is what kvm with seabios do now, and that would help make sure you get all the features automatically. -- MST

6 years, 5 months

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS May 2014