Re: [PATCH] qemu-kvm/rbd: add queueing delay based on queuesize (using use qemu_mutex_* and qemu_cond_*)

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

 



2010/7/20 Yehuda Sadeh Weinraub <yehudasa@xxxxxxxxx>:
>> +        while  (s->queuesize > MAX_QUEUE_SIZE) {
>> +            qemu_mutex_lock(s->queue_mutex);
>> +            qemu_cond_wait(s->queue_threshold, s->queue_mutex);
>> +            qemu_mutex_unlock(s->queue_mutex);
>> +        }
>
> Actually we shouldn't be waiting inside the aio handler. We should
> probably be feeding the request into some wait queue and have a
> separate thread that drains all those requests out. Though this
> wouldn't help us in throttling the client memory, it's probably a more
> correct way to handle the problem.

I agree with you, however I don't think that this is worth the effort.
I've never seen this occur when running a virtual machine, only when
running qemu-io.

@Kevin: Did you follow this thread? What is your opinion?

Christian
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux