Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Content-Type: text/plain;
        charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.8\))
Subject: Re: [PATCH BUGFIX IMPROVEMENT 0/7] boost throughput with synced I/O,
 reduce latency and fix a bandwidth bug
From:   Paolo Valente <paolo.valente@linaro.org>
In-Reply-To: <f3e2d759-911b-f593-9ec5-b6b7a94df71c@csail.mit.edu>
Date:   Tue, 25 Jun 2019 07:11:30 +0200
Cc:     Jens Axboe <axboe@kernel.dk>,
        linux-block <linux-block@vger.kernel.org>,
        kernel list <linux-kernel@vger.kernel.org>,
        Ulf Hansson <ulf.hansson@linaro.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        bfq-iosched@googlegroups.com, oleksandr@natalenko.name,
        bottura.nicola95@gmail.com
Content-Transfer-Encoding: quoted-printable
Message-Id: <3FC8946F-AFB9-4BD8-A778-ECDB1464D9DC@linaro.org>
References: <20190624194042.38747-1-paolo.valente@linaro.org>
 <f3e2d759-911b-f593-9ec5-b6b7a94df71c@csail.mit.edu>
To:     "Srivatsa S. Bhat" <srivatsa@csail.mit.edu>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


> Il giorno 24 giu 2019, alle ore 22:15, Srivatsa S. Bhat =
<srivatsa@csail.mit.edu> ha scritto:
>=20
> On 6/24/19 12:40 PM, Paolo Valente wrote:
>> Hi Jens,
>> this series, based against for-5.3/block, contains:
>> 1) The improvements to recover the throughput loss reported by
>>   Srivatsa [1] (first five patches)
>> 2) A preemption improvement to reduce I/O latency
>> 3) A fix of a subtle bug causing loss of control over I/O bandwidths
>>=20
>=20
> Thanks a lot for these patches, Paolo!
>=20
> Would you mind adding:
>=20
> Reported-by: Srivatsa S. Bhat (VMware) <srivatsa@csail.mit.edu>
> Tested-by: Srivatsa S. Bhat (VMware) <srivatsa@csail.mit.edu>
>=20
> to the first 5 patches, as appropriate?
>=20

With great pleasure!  (sorry for adding you only as tester in the
first place)

Thanks,
Paolo

> Thank you!
>=20
>>=20
>> [1] https://lkml.org/lkml/2019/5/17/755
>>=20
>> Paolo Valente (7):
>>  block, bfq: reset inject limit when think-time state changes
>>  block, bfq: fix rq_in_driver check in bfq_update_inject_limit
>>  block, bfq: update base request service times when possible
>>  block, bfq: bring forward seek&think time update
>>  block, bfq: detect wakers and unconditionally inject their I/O
>>  block, bfq: preempt lower-weight or lower-priority queues
>>  block, bfq: re-schedule empty queues if they deserve I/O plugging
>>=20
>> block/bfq-iosched.c | 952 =
++++++++++++++++++++++++++++++--------------
>> block/bfq-iosched.h |  25 +-
>> 2 files changed, 686 insertions(+), 291 deletions(-)
>>=20
>=20
> Regards,
> Srivatsa
> VMware Photon OS