On Mon, 5 Jun 2023, Bernhard Beschow wrote:
> Am 22. Mai 2023 15:42:03 UTC schrieb Bernhard Beschow <shen...@gmail.com>:
> >
> >
> >Am 15. Mai 2023 20:52:40 UTC schrieb Stefano Stabellini
> ><sstabell...@kernel.org>:
> >>On Sat, 13 May 2023, Bernhard Beschow wrote:
> >>> Am 21. April 2023 07:38:10 UTC schrieb "Michael S. Tsirkin"
> >>> <m...@redhat.com>:
> >>> >On Mon, Apr 03, 2023 at 09:41:17AM +0200, Bernhard Beschow wrote:
> >>> >> There is currently a dedicated PIIX3 device model for use under Xen.
> >>> >> By reusing
> >>> >> existing PCI API during initialization this device model can be
> >>> >> eliminated and
> >>> >> the plain PIIX3 device model can be used instead.
> >>> >>
> >>> >> Resolving TYPE_PIIX3_XEN_DEVICE results in less code while also making
> >>> >> Xen
> >>> >> agnostic towards the precise south bridge being used in the PC
> >>> >> machine. The
> >>> >> latter might become particularily interesting once PIIX4 becomes
> >>> >> usable in the
> >>> >> PC machine, avoiding the "Frankenstein" use of PIIX4_ACPI in PIIX3.
> >>> >
> >>> >xen stuff so I assume that tree?
> >>>
> >>> Ping
> >>
> >>I am OK either way. Michael, what do you prefer?
> >>
> >>Normally I would suggest for you to pick up the patches. But as it
> >>happens I'll have to likely send another pull request in a week or two
> >>and I can add these patches to it.
> >>
> >>Let me know your preference and I am happy to follow it.
> >
> >Hi Stefano,
> >
> >Michael's PR was merged last week. How about including this series into your
> >PR then?
>
> Ping
Sorry for the late reply, it looks like patch #3 breaks the build:
[1888/4025] Compiling C object libcommon.fa.p/hw_isa_piix3.c.o
FAILED: libcommon.fa.p/hw_isa_piix3.c.o
cc -m64 -mcx16 -Ilibcommon.fa.p -Iui -I../ui -I/usr/include/capstone
-I/usr/include/pixman-1 -I/usr/include/libpng16 -I/usr/include/spice-server
-I/usr/include/spice-1 -I/usr/include/p11-kit-1 -I/usr/include/libusb-1.0
-I/usr/include/SDL2 -I/usr/include/glib-2.0
-I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/libmount
-I/usr/include/blkid -I/usr/include/gio-unix-2.0 -I/usr/include/slirp
-I/usr/include/gtk-3.0 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz
-I/usr/include/freetype2 -I/usr/include/fribidi -I/usr/include/uuid
-I/usr/include/cairo -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/atk-1.0
-I/usr/include/at-spi2-atk/2.0 -I/usr/include/dbus-1.0
-I/usr/lib/x86_64-linux-gnu/dbus-1.0/include -I/usr/include/at-spi-2.0
-I/usr/include/vte-2.91 -I/usr/include/virgl -I/usr/include/cacard
-I/usr/include/nss -I/usr/include/nspr -I/usr/include/PCSC
-fdiagnostics-color=auto -Wall -Winvalid-pch -Werror -std=gnu11 -O2 -g
-fstack-protector-strong -U_FORTIFY_SOURCE -D_FORTIFY_SOURC
E=2 -Wundef -Wwrite-strings -Wmissing-prototypes -Wstrict-prototypes
-Wredundant-decls -Wold-style-declaration -Wold-style-definition -Wtype-limits
-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wempty-body
-Wnested-externs -Wendif-labels -Wexpansion-to-defined -Wimplicit-fallthrough=2
-Wmissing-format-attribute -Wno-missing-include-dirs -Wno-shift-negative-value
-Wno-psabi -isystem /builds/Vikram.garhwal/qemu-ioreq/linux-headers -isystem
linux-headers -iquote . -iquote /builds/Vikram.garhwal/qemu-ioreq -iquote
/builds/Vikram.garhwal/qemu-ioreq/include -iquote
/builds/Vikram.garhwal/qemu-ioreq/host/include/x86_64 -iquote
/builds/Vikram.garhwal/qemu-ioreq/host/include/generic -iquote
/builds/Vikram.garhwal/qemu-ioreq/tcg/i386 -pthread -D_GNU_SOURCE
-D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -fno-strict-aliasing -fno-common
-fwrapv -fPIE -D_DEFAULT_SOURCE -D_XOPEN_SOURCE=600 -DNCURSES_WIDECHAR=1
-D_REENTRANT -Wno-undef -DSTRUCT_IOVEC_DEFINED -MD -MQ libcommon.fa.p/hw
_isa_piix3.c.o -MF libcommon.fa.p/hw_isa_piix3.c.o.d -o
libcommon.fa.p/hw_isa_piix3.c.o -c ../hw/isa/piix3.c
../hw/isa/piix3.c:265:13: error: ‘pci_piix3_realize’ defined but not used
[-Werror=unused-function]
265 | static void pci_piix3_realize(PCIDevice *dev, Error **errp)
| ^~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
