Re: [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler

2019-07-02 Thread Laszlo Ersek
On 07/02/19 15:41, Laszlo Ersek wrote: > On 07/02/19 13:52, Laszlo Ersek wrote: >> Hi Phil, >> >> On 07/02/19 02:12, Philippe Mathieu-Daudé wrote: >>> The pflash device lacks a reset() function. >>> When a machine is resetted, the flash might be in an >>> inconsistent state, leading to unexpected

Re: [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler

2019-07-02 Thread Philippe Mathieu-Daudé
On 7/2/19 1:52 PM, Laszlo Ersek wrote: > Hi Phil, > > On 07/02/19 02:12, Philippe Mathieu-Daudé wrote: >> The pflash device lacks a reset() function. >> When a machine is resetted, the flash might be in an >> inconsistent state, leading to unexpected behavior: >>

Re: [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler

2019-07-02 Thread Laszlo Ersek
On 07/02/19 13:52, Laszlo Ersek wrote: > Hi Phil, > > On 07/02/19 02:12, Philippe Mathieu-Daudé wrote: >> The pflash device lacks a reset() function. >> When a machine is resetted, the flash might be in an >> inconsistent state, leading to unexpected behavior: >>

Re: [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler

2019-07-02 Thread Laszlo Ersek
Hi Phil, On 07/02/19 02:12, Philippe Mathieu-Daudé wrote: > The pflash device lacks a reset() function. > When a machine is resetted, the flash might be in an > inconsistent state, leading to unexpected behavior: > https://bugzilla.redhat.com/show_bug.cgi?id=1678713 > Resolve this issue by adding

Re: [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler

2019-07-02 Thread Philippe Mathieu-Daudé
Cc'ing Marc-André, On 7/2/19 8:15 AM, no-re...@patchew.org wrote: > Patchew URL: https://patchew.org/QEMU/20190702001301.4768-1-phi...@redhat.com/ > > === TEST SCRIPT BEGIN === > #!/bin/bash > make docker-image-fedora V=1 NETWORK=1 > time make docker-test-debug@fedora TARGET_LIST=x86_64-softmmu

Re: [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler

2019-07-02 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190702001301.4768-1-phi...@redhat.com/ Hi, This series failed the asan 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 make

[Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler

2019-07-01 Thread Philippe Mathieu-Daudé
The pflash device lacks a reset() function. When a machine is resetted, the flash might be in an inconsistent state, leading to unexpected behavior: https://bugzilla.redhat.com/show_bug.cgi?id=1678713 Resolve this issue by adding a DeviceReset() handler. Fix also two minor issues, and clean a bit