[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/6] block: add optional 'speed' parameter to
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/6] block: add optional 'speed' parameter to block-stream |
Date: |
Wed, 25 Apr 2012 11:57:48 +0100 |
On Tue, Apr 24, 2012 at 8:31 PM, Luiz Capitulino <address@hidden> wrote:
> On Tue, 24 Apr 2012 14:53:54 +0100
> Stefan Hajnoczi <address@hidden> wrote:
>
>> Eric Blake raised concerns about the inability to start block jobs with a
>> speed
>> limit. Current the user needs to follow up the block-stream command with
>> block-job-set-speed. There is a window of time while the new block job is
>> running but block-job-set-speed has not been processed yet.
>>
>> This series adds an optional 'speed' parameter to block-stream so streaming
>> can
>> be started with a speed limit that takes effect immediately.
>>
>> For consistency it also renames the block-job-set-speed parameter from
>> 'value'
>> to 'speed'. This is externally visible, but this API has not been in a QEMU
>> release yet so we can still do this.
>>
>> I considered several other approaches, including adding a
>> default_block_job_speed field to BlockDriverState but ultimately the cleanest
>> solution is to pass in a speed parameter on job creation. This way we do not
>> change semantics of existing commands, we only add an optional parameter. We
>> also do not need to add state to BlockDriverState, which is already huge and
>> messy.
>>
>> The last patch fixes a small bug I found in qemu-iotests 030.
>
> Looks good to me. Is this expected to go through my tree?
It probably should. The "block:" prefix to this series would probably
best be "qmp:" :).
I will send out a v3 including the cleanups that have been suggested.
Stefan
- [Qemu-devel] [PATCH v2 2/6] block: use Error mechanism instead of -errno for block_job_set_speed(), (continued)
- [Qemu-devel] [PATCH v2 2/6] block: use Error mechanism instead of -errno for block_job_set_speed(), Stefan Hajnoczi, 2012/04/24
- [Qemu-devel] [PATCH v2 1/6] block: use Error mechanism instead of -errno for block_job_create(), Stefan Hajnoczi, 2012/04/24
- [Qemu-devel] [PATCH v2 5/6] qemu-iotests: add block-stream with invalid speed value test, Stefan Hajnoczi, 2012/04/24
- [Qemu-devel] [PATCH v2 6/6] qemu-iotests: fix missing 'result' variable assignment in 030, Stefan Hajnoczi, 2012/04/24
- Re: [Qemu-devel] [PATCH v2 0/6] block: add optional 'speed' parameter to block-stream, Luiz Capitulino, 2012/04/24
- Re: [Qemu-devel] [PATCH v2 0/6] block: add optional 'speed' parameter to block-stream,
Stefan Hajnoczi <=