[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login
From: |
ronnie sahlberg |
Subject: |
Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login |
Date: |
Thu, 15 Nov 2012 08:13:29 -0800 |
On Thu, Nov 15, 2012 at 7:54 AM, Paolo Bonzini <address@hidden> wrote:
> Il 15/11/2012 15:57, ronnie sahlberg ha scritto:
>> I dont know if we should switch to use synchronous code here.
>> It is much nicer if all code is async.
>
> bdrv_open is generally synchronous, so I think Peter's patch is ok.
I was thinking about the case where you disconnect/reconnect a device
at runtime. Like swapping the medium in a CDROM.
If bdrv_open() is synchronous and blocks for a long time, would that
not impact the rest of QEMU?
Otherwise:
Acked-by: address@hidden
>
> Paolo
>
>> Is it possible to add a timeout instead that would break out if the
>> connect/login has not completed within a certain amount of time?
>
- [Qemu-devel] [PATCH] iscsi: fix deadlock during login, Peter Lieven, 2012/11/15
- Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login, Peter Lieven, 2012/11/15
- Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login, Paolo Bonzini, 2012/11/16
- Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login, Peter Lieven, 2012/11/16
- Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login, Peter Lieven, 2012/11/19