qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev,


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH] block/iscsi: introduce bdrv_co_{readv, writev, flush_to_disk}
Date: Tue, 08 Oct 2013 14:35:29 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130923 Thunderbird/17.0.9

Il 08/10/2013 14:33, Kevin Wolf ha scritto:
>>> > > this converts read, write and flush functions from aio to coroutines.
>> > 
>> > I'm not sure it's already the time for this...  Cancellation sucks in
>> > QEMU, and this is going to make things even worse.
> Not sure what you're referring to. If you mean iscsi_aio_cancel(), isn't
> it dead code anyway since we changed block.c to use coroutines for
> everything? bdrv_co_io_em() even throws the acb away, so even if you
> wanted, there's no way to cancel the request even today.

SCSI tries to use cancellation, and this results in VCPU threads
starving all other threads.  So I would like to introduce cancellation
points for coroutines.

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]