|
| From: | Alberto Garcia |
| Subject: | Re: [Qemu-devel] [PATCH 3/3] virtio-scsi: Forbid devices with different iothreads sharing a blockdev |
| Date: | Wed, 23 Jan 2019 16:09:10 +0100 |
| User-agent: | Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 23 Jan 2019 10:46:54 AM CET, Paolo Bonzini wrote:
> On 22/01/19 16:53, Alberto Garcia wrote:
>> This patch forbids attaching a disk to a SCSI device if its using a
>> different AioContext. Test case included.
>>
>> Signed-off-by: Alberto Garcia <address@hidden>
>
> Should this be handled in common code? However, everything else looks
> good. Thanks!
You mean a common function with the code below?
>> + ctx = blk_get_aio_context(sd->conf.blk);
>> + if (ctx != s->ctx && ctx != qemu_get_aio_context()) {
>> + error_setg(errp, "Cannot attach a blockdev that is using "
>> + "a different iothread");
>> + return;
>> + }
Who else would be the user?
Berto
| [Prev in Thread] | Current Thread | [Next in Thread] |