[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 03/22] hw/ide: restore replay support of IDE
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v1 03/22] hw/ide: restore replay support of IDE |
Date: |
Wed, 7 Oct 2020 19:53:37 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 10/7/20 6:00 PM, Alex Bennée wrote:
> A recent change to weak reset handling broke replay due to the use of
> aio_bh_schedule_oneshot instead of the replay aware
> replay_bh_schedule_oneshot_event.
>
> Fixes: 55adb3c456 ("ide: cancel pending callbacks on SRST")
> Suggested-by: Pavel Dovgalyuk <pavel.dovgalyuk@ispras.ru>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> hw/ide/core.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/ide/core.c b/hw/ide/core.c
> index 0e32abd779..693b352d5e 100644
> --- a/hw/ide/core.c
> +++ b/hw/ide/core.c
> @@ -2289,8 +2289,8 @@ void ide_ctrl_write(void *opaque, uint32_t addr,
> uint32_t val)
> s = &bus->ifs[i];
> s->status |= BUSY_STAT;
> }
> - aio_bh_schedule_oneshot(qemu_get_aio_context(),
> - ide_bus_perform_srst, bus);
> + replay_bh_schedule_oneshot_event(qemu_get_aio_context(),
> + ide_bus_perform_srst, bus);
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> }
>
> bus->cmd = val;
>
- [PATCH v1 00/22] testing and plugin fixes with gitdm updates (pre PR), Alex Bennée, 2020/10/07
- [PATCH v1 02/22] hw/misc/mips_cpc: Start vCPU when powered on, Alex Bennée, 2020/10/07
- [PATCH v1 03/22] hw/ide: restore replay support of IDE, Alex Bennée, 2020/10/07
- [PATCH v1 06/22] tests/docker: Add genisoimage to the docker file, Alex Bennée, 2020/10/07
- [PATCH v1 10/22] contrib/gitdm: Add ByteDance to the domain map, Alex Bennée, 2020/10/07
- [PATCH v1 01/22] configure: fix performance regression due to PIC objects, Alex Bennée, 2020/10/07
- [PATCH v1 04/22] cirrus: Fixing and speedup the msys2/mingw CI, Alex Bennée, 2020/10/07
- [PATCH v1 05/22] cirrus: msys2/mingw speed is up, add excluded target back, Alex Bennée, 2020/10/07
- [PATCH v1 07/22] contrib/gitdm: Add more academic domains, Alex Bennée, 2020/10/07
- [PATCH v1 11/22] contrib/gitdm: Add Google to the domain map, Alex Bennée, 2020/10/07
- [PATCH v1 09/22] contrib/gitdm: Add Baidu to the domain map, Alex Bennée, 2020/10/07
- [PATCH v1 08/22] contrib/gitdm: Add more individual contributors, Alex Bennée, 2020/10/07