Re: [Qemu-devel] [PATCH 1/9] acpi: remove unused acpi-dsdt.aml

2018-02-27 Thread Igor Mammedov 
On Tue, 27 Feb 2018 13:42:38 +0100
Auger Eric  wrote:

> Hi,
> 
> On 22/02/18 13:42, Igor Mammedov wrote:
> > SeaBIOS blob which is currently shipped with QEMU
> > doesn't need acpi-dsdt.aml nor is able to use it
> > and code that loaded it QEMU was removed by  
> as code that loaded it in QEMU was removed by?
Thanks, I'll fix it on respin.

> > (commit 9fb7aaaf4c "pc: drop external DSDT loading")
> > in 2013.
> > 
> > Signed-off-by: Igor Mammedov 
> > CC: Gerd Hoffmann   
> Otherwise
> Reviewed-by: Eric Auger 
> 
> Thanks
> 
> Eric
> > ---
> >  Makefile  |   1 -
> >  pc-bios/acpi-dsdt.aml | Bin 4405 -> 0 bytes
> >  2 files changed, 1 deletion(-)
> >  delete mode 100644 pc-bios/acpi-dsdt.aml
> > 
> > diff --git a/Makefile b/Makefile
> > index 90e05ac..f3a4cf9 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -648,7 +648,6 @@ bepocz
> >  ifdef INSTALL_BLOBS
> >  BLOBS=bios.bin bios-256k.bin sgabios.bin vgabios.bin vgabios-cirrus.bin \
> >  vgabios-stdvga.bin vgabios-vmware.bin vgabios-qxl.bin vgabios-virtio.bin \
> > -acpi-dsdt.aml \
> >  ppc_rom.bin openbios-sparc32 openbios-sparc64 openbios-ppc QEMU,tcx.bin 
> > QEMU,cgthree.bin \
> >  pxe-e1000.rom pxe-eepro100.rom pxe-ne2k_pci.rom \
> >  pxe-pcnet.rom pxe-rtl8139.rom pxe-virtio.rom \
> > diff --git a/pc-bios/acpi-dsdt.aml b/pc-bios/acpi-dsdt.aml
> > deleted file mode 100644
> > index 
> > 558c10f51ccbbf9ec2f47a4a998a7055059a8963..
> > GIT binary patch
> > literal 0
> > HcmV?d1
> > 
> > literal 4405  
> > zcmb7{O>f)C8OI;KNTx=#Ov$uk$9XY?b_=xIjb0q@_EJP5WkrsxFrpHqP*76NE}  
> > zqzSOzpn&5RO*WTe*G|Kam8znLL9
> > zdJ#bTx_LkF0GjuGY(WhGo!+3koGWcQ9rFPU5B+94((<~g4WH$C9dAv`{m^gT  
> > zZEJrW$A5|A$IoMJl)(Ns@V^7|L@K9JFq{e&^9IOQm8M#H0x!0S}3=w`>a8*i9l  
> > zMGe0XR&=-HYff+FBQoL^UcVI z#Tgm?=lKNy@f?7`Y?dceyma7Ch?1^<&1QdpC#wIrasZas-`*@%==
> > z2|2%0`vJHO7J2;;UQ)-|gCMNeL^TdtX>}ZQ>$N1Pgb_W)N-Ls=$)m@-itRY~WHYgk
> > zgX+BqrWEW?)FoC3!tE_lT@6}k^@E_hy_E!2ipVPzkypAAJ^BL$Apdw8JA0Oxf|hkN
> > zXbsXi&~OfL^l_GN27^7ov3`59aWhLwfmMtLJY9eLvcCx1(^-fP`A#LS5&
> > z_E*O-9LM?DYzmXYSH~mx^T>vO|2H#*DO<8=Sc*kf_+yTS?9DqcX}p{p+4*D-kG#yi
> > zb|d180Xv{$XK)dCIxy@PNAErQ+8n3KJVco~H$7En{Z-6#s#%p5=+|  
> > z>%skMU4%Dqj4^z*PT^@  
> > zyarVauZe}V > zk3PQD^ylV=;pX_@&#t?*5T1l_HN#AsceGfh3=SFYnmfKX
> > ze-#(NT@&+50P!S?b2^3B<~-vDTWf3I8Q$TO7yo4fv_alm4 z?+N9w#kTgj@ws7H > zg5DcPKhZIcg>jsYb+#sOA%+7j58pBiUkMT(uE)EZc=I=hhhb|Mzl_$me4&!?nw8e>  
> > zrn?k)tz9iS(8fRw?snkyc5t5KZ$5k#v#U?yN%1MgInZJNd^U)+Nr_tgvleDJyAxVO
> > zPWv%F!Kn)RgVL?v9+vVZzHHF#-SR=yHLN$FWK%oSQ8ZIwpzxryXxg(Ge)CX;b46Zg
> > zSP;*+ADX6;JTX4^)VU|xo+|Q8P4P9NjA+U|QIaS2hTGy7TG*Z{@=Q$);fbc)6D4`3
> > zS@1I > zJyXsFl@q1RX(^nR!fC0TC}mFkpyCLnoH>Ovr*fi{Ihn%A6i%jcqLexDH;OrNO!%zi  
> > z70$fMiBjgo54$v z%863ubQDfU;dE3^lrkr7#-#ZyDx5`?6Q#^qQaDQrXG!HmDRV9>oXZO5vdW24=5!TK  
> > zSK)M3PLwj|io&^~aIUDFC}qx7g>zNmTva(y%AB|xl-BJ9h4X^SiBjfVQ#jWY=2
> > zrOdgmaIPzy>nbNone(E;c~RlKsB)r|IX4te+zyF%j(;^bR8EvK=Ou;nlEQgO > zURF3SE1Z{APLwj|6@~MP!g)pIL@9G#RXDFIoL5y!9~k>_^uO`jgU*EWn+e7WD5_  
> > zEWB0eR-;?pa+f=I@RvWyJ!OYu+`;CiEbnf2?s)wi8uTm00?U7ygQ`6
> > zCiz!mc9@K*KBea2QFnpf=yXS7<8GMt+Vm$6i>qw;%L3#K{9WM*1%OT{c#v2UJ3Z > zb > z`$y-TG<((k{_NG^(H^mT=dv^X|43hx(${$U+PU<#_oT0#ruWaM$J5Rarmsus>pXq^  
> > zT>AQZ($|Maw@suE&!;y<`g94=kqD@_qz  
> > zV0*#s-WcMfm}eH?9)hk>GJY{)I`G1PBt~VzbmU(20$kE(UXx6W8+$q?xy%b%yZW%q
> > z{)wleKHuy9jcpE}*<9c)y5YFHS*&<~ODl{%!&5$OWOp*J;^)+h)37m<7T}A0
> > zZU426&7a``5jTMQ$ > z+hW5DI+aRg8*mW1l?u2kQL9pg2WMw1+E%*`w$|W**tBCmxpiGQZHeN#C{81NEYv5W
> > U_=PAMqG*c36NN8|mMC`Mf01wzJpcdz
> >

Re: [Qemu-devel] [PATCH 1/9] acpi: remove unused acpi-dsdt.aml

2018-02-27 Thread Auger Eric 
Hi,

On 22/02/18 13:42, Igor Mammedov wrote:
> SeaBIOS blob which is currently shipped with QEMU
> doesn't need acpi-dsdt.aml nor is able to use it
> and code that loaded it QEMU was removed by
as code that loaded it in QEMU was removed by?
> (commit 9fb7aaaf4c "pc: drop external DSDT loading")
> in 2013.
> 
> Signed-off-by: Igor Mammedov 
> CC: Gerd Hoffmann 
Otherwise
Reviewed-by: Eric Auger 

Thanks

Eric
> ---
>  Makefile  |   1 -
>  pc-bios/acpi-dsdt.aml | Bin 4405 -> 0 bytes
>  2 files changed, 1 deletion(-)
>  delete mode 100644 pc-bios/acpi-dsdt.aml
> 
> diff --git a/Makefile b/Makefile
> index 90e05ac..f3a4cf9 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -648,7 +648,6 @@ bepocz
>  ifdef INSTALL_BLOBS
>  BLOBS=bios.bin bios-256k.bin sgabios.bin vgabios.bin vgabios-cirrus.bin \
>  vgabios-stdvga.bin vgabios-vmware.bin vgabios-qxl.bin vgabios-virtio.bin \
> -acpi-dsdt.aml \
>  ppc_rom.bin openbios-sparc32 openbios-sparc64 openbios-ppc QEMU,tcx.bin 
> QEMU,cgthree.bin \
>  pxe-e1000.rom pxe-eepro100.rom pxe-ne2k_pci.rom \
>  pxe-pcnet.rom pxe-rtl8139.rom pxe-virtio.rom \
> diff --git a/pc-bios/acpi-dsdt.aml b/pc-bios/acpi-dsdt.aml
> deleted file mode 100644
> index 
> 558c10f51ccbbf9ec2f47a4a998a7055059a8963..
> GIT binary patch
> literal 0
> HcmV?d1
> 
> literal 4405
> zcmb7{O>f)C8OI;KNTx=#Ov$uk$9XY?b_=xIjb0q@_EJP5WkrsxFrpHqP*76NE}
> zqzSOzpn&5RO*WTe*G|Kam8znLL9
> zdJ#bTx_LkF0GjuGY(WhGo!+3koGWcQ9rFPU5B+94((<~g4WH$C9dAv`{m^gT
> zZEJrW$A5|A$IoMJl)(Ns@V^7|L@K9JFq{e&^9IOQm8M#H0x!0S}3=w`>a8*i9l
> zMGe0XR&=-HYff+FBQoL^UcVI z2|2%0`vJHO7J2;;UQ)-|gCMNeL^TdtX>}ZQ>$N1Pgb_W)N-Ls=$)m@-itRY~WHYgk
> zgX+BqrWEW?)FoC3!tE_lT@6}k^@E_hy_E!2ipVPzkypAAJ^BL$Apdw8JA0Oxf|hkN
> zXbsXi&~OfL^l_GN27^7ov3`59aWhLwfmMtLJY9eLvcCx1(^-fP`A#LS5&
> z_E*O-9LM?DYzmXYSH~mx^T>vO|2H#*DO<8=Sc*kf_+yTS?9DqcX}p{p+4*D-kG#yi
> zb|d180Xv{$XK)dCIxy@PNAErQ+8n3KJVco~H$7En{Z-6#s#%p5=+|
> z>%skMU4%Dqj4^z*PT^@
> zyarVauZe}V zk3PQD^ylV=;pX_@&#t?*5T1l_HN#AsceGfh3=SFYnmfKX
> ze-#(NT@&+50P!S?b2^3B<~-vDTWf3I8Q$TO7yo4fv_alm4 zg5DcPKhZIcg>jsYb+#sOA%+7j58pBiUkMT(uE)EZc=I=hhhb|Mzl_$me4&!?nw8e>
> zrn?k)tz9iS(8fRw?snkyc5t5KZ$5k#v#U?yN%1MgInZJNd^U)+Nr_tgvleDJyAxVO
> zPWv%F!Kn)RgVL?v9+vVZzHHF#-SR=yHLN$FWK%oSQ8ZIwpzxryXxg(Ge)CX;b46Zg
> zSP;*+ADX6;JTX4^)VU|xo+|Q8P4P9NjA+U|QIaS2hTGy7TG*Z{@=Q$);fbc)6D4`3
> zS@1I zJyXsFl@q1RX(^nR!fC0TC}mFkpyCLnoH>Ovr*fi{Ihn%A6i%jcqLexDH;OrNO!%zi
> z70$fMiBjgo54$voXZO5vdW24=5!TK
> zSK)M3PLwj|io&^~aIUDFC}qx7g>zNmTva(y%AB|xl-BJ9h4X^SiBjfVQ#jWY=2
> zrOdgmaIPzy>nbNone(E;c~RlKsB)r|IX4te+zyF%j(;^bR8EvK=Ou;nlEQgO zURF3SE1Z{APLwj|6@~MP!g)pIL@9G#RXDFIoL5y!9~k>_^uO`jgU*EWn+e7WD5_
> zEWB0eR-;?pa+f=I@RvWyJ!OYu+`;CiEbnf2?s)wi8uTm00?U7ygQ`6
> zCiz!mc9@K*KBea2QFnpf=yXS7<8GMt+Vm$6i>qw;%L3#K{9WM*1%OT{c#v2UJ3Z zb z`$y-TG<((k{_NG^(H^mT=dv^X|43hx(${$U+PU<#_oT0#ruWaM$J5Rarmsus>pXq^
> zT>AQZ($|Maw@suE&!;y<`g94=kqD@_qz
> zV0*#s-WcMfm}eH?9)hk>GJY{)I`G1PBt~VzbmU(20$kE(UXx6W8+$q?xy%b%yZW%q
> z{)wleKHuy9jcpE}*<9c)y5YFHS*&<~ODl{%!&5$OWOp*J;^)+h)37m<7T}A0
> zZU426&7a``5jTMQ$ z+hW5DI+aRg8*mW1l?u2kQL9pg2WMw1+E%*`w$|W**tBCmxpiGQZHeN#C{81NEYv5W
> U_=PAMqG*c36NN8|mMC`Mf01wzJpcdz
>

Re: [Qemu-devel] [PATCH 1/9] acpi: remove unused acpi-dsdt.aml

2018-02-22 Thread Gerd Hoffmann 
On Thu, Feb 22, 2018 at 01:42:48PM +0100, Igor Mammedov wrote:
> SeaBIOS blob which is currently shipped with QEMU
> doesn't need acpi-dsdt.aml nor is able to use it
> and code that loaded it QEMU was removed by
> (commit 9fb7aaaf4c "pc: drop external DSDT loading")
> in 2013.
> 
> Signed-off-by: Igor Mammedov 
> CC: Gerd Hoffmann 

Reviewed-by: Gerd Hoffmann

[Qemu-devel] [PATCH 1/9] acpi: remove unused acpi-dsdt.aml

2018-02-22 Thread Igor Mammedov 
SeaBIOS blob which is currently shipped with QEMU
doesn't need acpi-dsdt.aml nor is able to use it
and code that loaded it QEMU was removed by
(commit 9fb7aaaf4c "pc: drop external DSDT loading")
in 2013.

Signed-off-by: Igor Mammedov 
CC: Gerd Hoffmann 
---
 Makefile  |   1 -
 pc-bios/acpi-dsdt.aml | Bin 4405 -> 0 bytes
 2 files changed, 1 deletion(-)
 delete mode 100644 pc-bios/acpi-dsdt.aml

diff --git a/Makefile b/Makefile
index 90e05ac..f3a4cf9 100644
--- a/Makefile
+++ b/Makefile
@@ -648,7 +648,6 @@ bepocz
 ifdef INSTALL_BLOBS
 BLOBS=bios.bin bios-256k.bin sgabios.bin vgabios.bin vgabios-cirrus.bin \
 vgabios-stdvga.bin vgabios-vmware.bin vgabios-qxl.bin vgabios-virtio.bin \
-acpi-dsdt.aml \
 ppc_rom.bin openbios-sparc32 openbios-sparc64 openbios-ppc QEMU,tcx.bin 
QEMU,cgthree.bin \
 pxe-e1000.rom pxe-eepro100.rom pxe-ne2k_pci.rom \
 pxe-pcnet.rom pxe-rtl8139.rom pxe-virtio.rom \
diff --git a/pc-bios/acpi-dsdt.aml b/pc-bios/acpi-dsdt.aml
deleted file mode 100644
index 
558c10f51ccbbf9ec2f47a4a998a7055059a8963..
GIT binary patch
literal 0
HcmV?d1

literal 4405
zcmb7{O>f)C8OI;KNTx=#Ov$uk$9XY?b_=xIjb0q@_EJP5WkrsxFrpHqP*76NE}
zqzSOzpn&5RO*WTe*G|Kam8znLL9
zdJ#bTx_LkF0GjuGY(WhGo!+3koGWcQ9rFPU5B+94((<~g4WH$C9dAv`{m^gT
zZEJrW$A5|A$IoMJl)(Ns@V^7|L@K9JFq{e&^9IOQm8M#H0x!0S}3=w`>a8*i9l
zMGe0XR&=-HYff+FBQoL^UcVI}ZQ>$N1Pgb_W)N-Ls=$)m@-itRY~WHYgk
zgX+BqrWEW?)FoC3!tE_lT@6}k^@E_hy_E!2ipVPzkypAAJ^BL$Apdw8JA0Oxf|hkN
zXbsXi&~OfL^l_GN27^7ov3`59aWhLwfmMtLJY9eLvcCx1(^-fP`A#LS5&
z_E*O-9LM?DYzmXYSH~mx^T>vO|2H#*DO<8=Sc*kf_+yTS?9DqcX}p{p+4*D-kG#yi
zb|d180Xv{$XK)dCIxy@PNAErQ+8n3KJVco~H$7En{Z-6#s#%p5=+|
z>%skMU4%Dqj4^z*PT^@
zyarVauZe}V
zrn?k)tz9iS(8fRw?snkyc5t5KZ$5k#v#U?yN%1MgInZJNd^U)+Nr_tgvleDJyAxVO
zPWv%F!Kn)RgVL?v9+vVZzHHF#-SR=yHLN$FWK%oSQ8ZIwpzxryXxg(Ge)CX;b46Zg
zSP;*+ADX6;JTX4^)VU|xo+|Q8P4P9NjA+U|QIaS2hTGy7TG*Z{@=Q$);fbc)6D4`3
zS@1IOvr*fi{Ihn%A6i%jcqLexDH;OrNO!%zi
z70$fMiBjgo54$vzNmTva(y%AB|xl-BJ9h4X^SiBjfVQ#jWY=2
zrOdgmaIPzy>nbNone(E;c~RlKsB)r|IX4te+zyF%j(;^bR8EvK=Ou;nlEQgO_^uO`jgU*EWn+e7WD5_
zEWB0eR-;?pa+f=I@RvWyJ!OYu+`;CiEbnf2?s)wi8uTm00?U7ygQ`6
zCiz!mc9@K*KBea2QFnpf=yXS7<8GMt+Vm$6i>qw;%L3#K{9WM*1%OT{c#v2UJ3ZpXq^
zT>AQZ($|Maw@suE&!;y<`g94=kqD@_qz
zV0*#s-WcMfm}eH?9)hk>GJY{)I`G1PBt~VzbmU(20$kE(UXx6W8+$q?xy%b%yZW%q
z{)wleKHuy9jcpE}*<9c)y5YFHS*&<~ODl{%!&5$OWOp*J;^)+h)37m<7T}A0
zZU426&7a``5jTMQ$