On Tue, Nov 22, 2022 at 01:32:20PM -0500, Kevin O'Connor wrote:
On Mon, Nov 21, 2022 at 11:12:55AM +0100, Gerd Hoffmann wrote:
On Fri, Nov 18, 2022 at 03:27:55PM +0100, Igor Mammedov wrote:
Since QEMU commit dbce582a119 (acpi: pc/q35: drop ad-hoc PCI-ISA bridge AML routines and let bus ennumeration generate AML) SeaBIOS fails to parse ISA bridge AML with:
parse_termlist: parse error, skip from 92/517 ... ACPI: no PS/2 keyboard present
due to Alias term in DSDT which isn't handled by SeaBIOS properly. Add dumb Alias parsing which just skips over term, so the rest of AML could be parsed successfully.
Signed-off-by: Igor Mammedov imammedo@redhat.com Reported-by: Volker Rümelin vr_qemu@t-online.de
Reviewed-by: Gerd Hoffmann kraxel@redhat.com
Thanks. Looks fine to me as well.
Gerd - feel free to commit this (and the virtio changes as well) when you feel they are ready.
Done.
FWIW, if we want a release in the short term, I'd suggest making a v1.16.1 "stable" release.
Well. Currently we have ...
# git log --oneline rel-1.16.0.. 3208b098f51a virtio: finalize features before using device 5ea5c64c20e9 virtio-mmio: read/write the hi 32 features for mmio 61e901bbaadf acpi: parse Alias object 85d56f812f4d virtio-blk: Fix incorrect type conversion in virtio_blk_op() 46de2eec93bf virtio-blk: use larger default request size dc88f9b72df5 malloc: use large ZoneHigh when there is enough memory 3b91e8e9fe93 malloc: use variable for ZoneHigh size 01774004c7f7 reset: force standard PCI configuration access d24f42b0d819 pci: refactor the pci_config_*() functions
... where most changes (all except the two 'malloc' patches) are bugfixes. Cherry-picking 7/9 patches into 1.16.1 feels somewhat silly. Just grabbing a master branch snapshot looks like the simpler solution here.
Independent from that I think it makes sense to plan a new 1.17.0 release.
take care, Gerd