mbox series

[0/4] block, bfq: series of improvements and small fixes of the injection mechanism

Message ID 20190822152037.15413-1-paolo.valente@linaro.org (mailing list archive)
Headers show
Series block, bfq: series of improvements and small fixes of the injection mechanism | expand

Message

Paolo Valente Aug. 22, 2019, 3:20 p.m. UTC
Hi Jens,
this patch series makes the injection mechanism better at preserving
control on I/O.

Thanks,
Paolo

Paolo Valente (4):
  block, bfq: update inject limit only after injection occurred
  block, bfq: reduce upper bound for inject limit to max_rq_in_driver+1
  block, bfq: increase update frequency of inject limit
  block, bfq: push up injection only after setting service time

 block/bfq-iosched.c | 35 ++++++++++++++++++++++++++---------
 1 file changed, 26 insertions(+), 9 deletions(-)

--
2.20.1

Comments

Paolo Valente Sept. 9, 2019, 5:57 a.m. UTC | #1
Hi Jens,
have you looked into this?

Thanks,
Paolo

> Il giorno 22 ago 2019, alle ore 17:20, Paolo Valente <paolo.valente@linaro.org> ha scritto:
> 
> Hi Jens,
> this patch series makes the injection mechanism better at preserving
> control on I/O.
> 
> Thanks,
> Paolo
> 
> Paolo Valente (4):
>  block, bfq: update inject limit only after injection occurred
>  block, bfq: reduce upper bound for inject limit to max_rq_in_driver+1
>  block, bfq: increase update frequency of inject limit
>  block, bfq: push up injection only after setting service time
> 
> block/bfq-iosched.c | 35 ++++++++++++++++++++++++++---------
> 1 file changed, 26 insertions(+), 9 deletions(-)
> 
> --
> 2.20.1
Oleksandr Natalenko Sept. 9, 2019, 6:03 a.m. UTC | #2
On 22.08.2019 17:20, Paolo Valente wrote:
> Hi Jens,
> this patch series makes the injection mechanism better at preserving
> control on I/O.
> 
> Thanks,
> Paolo
> 
> Paolo Valente (4):
>   block, bfq: update inject limit only after injection occurred
>   block, bfq: reduce upper bound for inject limit to max_rq_in_driver+1
>   block, bfq: increase update frequency of inject limit
>   block, bfq: push up injection only after setting service time
> 
>  block/bfq-iosched.c | 35 ++++++++++++++++++++++++++---------
>  1 file changed, 26 insertions(+), 9 deletions(-)
> 
> --
> 2.20.1

FWIW, Tested-by: Oleksandr Natalenko <oleksandr@natalenko.name> (I run 
it for quite some time already).
Paolo Valente Sept. 16, 2019, 3:32 p.m. UTC | #3
Hi Jens,
can these be considered for 5.4 too?

Thanks,
Paolo

> Il giorno 9 set 2019, alle ore 08:03, Oleksandr Natalenko <oleksandr@natalenko.name> ha scritto:
> 
> On 22.08.2019 17:20, Paolo Valente wrote:
>> Hi Jens,
>> this patch series makes the injection mechanism better at preserving
>> control on I/O.
>> Thanks,
>> Paolo
>> Paolo Valente (4):
>>  block, bfq: update inject limit only after injection occurred
>>  block, bfq: reduce upper bound for inject limit to max_rq_in_driver+1
>>  block, bfq: increase update frequency of inject limit
>>  block, bfq: push up injection only after setting service time
>> block/bfq-iosched.c | 35 ++++++++++++++++++++++++++---------
>> 1 file changed, 26 insertions(+), 9 deletions(-)
>> --
>> 2.20.1
> 
> FWIW, Tested-by: Oleksandr Natalenko <oleksandr@natalenko.name> (I run it for quite some time already).
> 
> -- 
>  Oleksandr Natalenko (post-factum)
Jens Axboe Sept. 16, 2019, 4:17 p.m. UTC | #4
On 8/22/19 9:20 AM, Paolo Valente wrote:
> Hi Jens,
> this patch series makes the injection mechanism better at preserving
> control on I/O.

Applied, thanks.