| Message ID | 20190926151733.25349-1-philmd@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | hw: Convert various reset() handler to DeviceReset | expand |
Patchew URL: https://patchew.org/QEMU/20190926151733.25349-1-philmd@redhat.com/ Hi, This series failed the docker-mingw@fedora build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #! /bin/bash export ARCH=x86_64 make docker-image-fedora V=1 NETWORK=1 time make docker-test-mingw@fedora J=14 NETWORK=1 === TEST SCRIPT END === The full log is available at http://patchew.org/logs/20190926151733.25349-1-philmd@redhat.com/testing.docker-mingw@fedora/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
On Thu, Sep 26, 2019 at 05:17:22PM +0200, Philippe Mathieu-Daudé wrote: > Hi. > > Following the thread discussion between Peter/Markus/Damien about > reset handlers: > https://www.mail-archive.com/qemu-devel@nongnu.org/msg617103.html > I started to remove qemu_register_reset() calls from few qdevified > devices (the trivial ones). > > Regards, > > Phil. How do you want these patches merged? Trivial tree? > Philippe Mathieu-Daudé (11): > hw/acpi/piix4: Convert reset handler to DeviceReset > hw/ide/piix: Convert reset handler to DeviceReset > hw/isa/piix4: Convert reset handler to DeviceReset > hw/pci-host/piix: Convert reset handler to DeviceReset > hw/ide/sii3112: Convert reset handler to DeviceReset > hw/ide/via82c: Convert reset handler to DeviceReset > hw/isa/vt82c686: Convert reset handler to DeviceReset > hw/input/lm832x: Convert reset handler to DeviceReset > hw/pci-host/bonito: Convert reset handler to DeviceReset > hw/timer/etraxfs: Convert reset handler to DeviceReset > hw/misc/vmcoreinfo: Convert reset handler to DeviceReset > > hw/acpi/piix4.c | 7 +++---- > hw/ide/piix.c | 8 +++----- > hw/ide/sii3112.c | 7 +++---- > hw/ide/via.c | 10 ++++------ > hw/input/lm832x.c | 12 +++++------- > hw/isa/piix4.c | 7 +++---- > hw/isa/vt82c686.c | 11 ++++------- > hw/misc/vmcoreinfo.c | 5 ++--- > hw/pci-host/bonito.c | 8 +++----- > hw/pci-host/piix.c | 8 +++----- > hw/timer/etraxfs_timer.c | 7 +++---- > 11 files changed, 36 insertions(+), 54 deletions(-) > > -- > 2.20.1
Hi Michael, On 10/5/19 11:46 PM, Michael S. Tsirkin wrote: > On Thu, Sep 26, 2019 at 05:17:22PM +0200, Philippe Mathieu-Daudé wrote: >> Hi. >> >> Following the thread discussion between Peter/Markus/Damien about >> reset handlers: >> https://www.mail-archive.com/qemu-devel@nongnu.org/msg617103.html >> I started to remove qemu_register_reset() calls from few qdevified >> devices (the trivial ones). >> >> Regards, >> >> Phil. > > How do you want these patches merged? Trivial tree? I was hoping Eduardo would take them but he is busy and I even forgot to Cc him. They might go via Trivial or else via Paolo's Misc... >> Philippe Mathieu-Daudé (11): >> hw/acpi/piix4: Convert reset handler to DeviceReset >> hw/ide/piix: Convert reset handler to DeviceReset >> hw/isa/piix4: Convert reset handler to DeviceReset >> hw/pci-host/piix: Convert reset handler to DeviceReset >> hw/ide/sii3112: Convert reset handler to DeviceReset >> hw/ide/via82c: Convert reset handler to DeviceReset >> hw/isa/vt82c686: Convert reset handler to DeviceReset >> hw/input/lm832x: Convert reset handler to DeviceReset >> hw/pci-host/bonito: Convert reset handler to DeviceReset >> hw/timer/etraxfs: Convert reset handler to DeviceReset >> hw/misc/vmcoreinfo: Convert reset handler to DeviceReset
On Mon, Oct 07, 2019 at 12:01:54PM +0200, Philippe Mathieu-Daudé wrote: > Hi Michael, > > On 10/5/19 11:46 PM, Michael S. Tsirkin wrote: > > On Thu, Sep 26, 2019 at 05:17:22PM +0200, Philippe Mathieu-Daudé wrote: > > > Hi. > > > > > > Following the thread discussion between Peter/Markus/Damien about > > > reset handlers: > > > https://www.mail-archive.com/qemu-devel@nongnu.org/msg617103.html > > > I started to remove qemu_register_reset() calls from few qdevified > > > devices (the trivial ones). > > > > > > Regards, > > > > > > Phil. > > > > How do you want these patches merged? Trivial tree? > > I was hoping Eduardo would take them but he is busy and I even forgot to Cc > him. They might go via Trivial or else via Paolo's Misc... I can merge them if somebody reviews the series. How exactly are we making sure device_reset() is really being called for all devices touched by this series? Are all buses guaranteed to be children of main-system-bus? > > > > Philippe Mathieu-Daudé (11): > > > hw/acpi/piix4: Convert reset handler to DeviceReset > > > hw/ide/piix: Convert reset handler to DeviceReset > > > hw/isa/piix4: Convert reset handler to DeviceReset > > > hw/pci-host/piix: Convert reset handler to DeviceReset > > > hw/ide/sii3112: Convert reset handler to DeviceReset > > > hw/ide/via82c: Convert reset handler to DeviceReset > > > hw/isa/vt82c686: Convert reset handler to DeviceReset > > > hw/input/lm832x: Convert reset handler to DeviceReset > > > hw/pci-host/bonito: Convert reset handler to DeviceReset > > > hw/timer/etraxfs: Convert reset handler to DeviceReset > > > hw/misc/vmcoreinfo: Convert reset handler to DeviceReset
On Mon, 7 Oct 2019 at 22:24, Eduardo Habkost <ehabkost@redhat.com> wrote: > > On Mon, Oct 07, 2019 at 12:01:54PM +0200, Philippe Mathieu-Daudé wrote: > > Hi Michael, > > > > On 10/5/19 11:46 PM, Michael S. Tsirkin wrote: > > > On Thu, Sep 26, 2019 at 05:17:22PM +0200, Philippe Mathieu-Daudé wrote: > > > > Hi. > > > > > > > > Following the thread discussion between Peter/Markus/Damien about > > > > reset handlers: > > > > https://www.mail-archive.com/qemu-devel@nongnu.org/msg617103.html > > > > I started to remove qemu_register_reset() calls from few qdevified > > > > devices (the trivial ones). > > > > > > > > Regards, > > > > > > > > Phil. > > > > > > How do you want these patches merged? Trivial tree? > > > > I was hoping Eduardo would take them but he is busy and I even forgot to Cc > > him. They might go via Trivial or else via Paolo's Misc... > > I can merge them if somebody reviews the series. How exactly are > we making sure device_reset() is really being called for all > devices touched by this series? Are all buses guaranteed to be > children of main-system-bus? At least one of them is not (vmcoreinfo), as noted in review comments on that patch. So while some of these patches might be ok, some are not, and so we need to review them first before they get merged via any route I think. thanks -- PMM