Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-24 Thread Bart Van Assche
On Wed, 2017-02-22 at 22:29 +0100, Paolo Valente wrote: > thanks for this second attempt of yours. Although, unfortunately, not > providing some clear indication of the exact cause of your hang (apart > from a possible deadlock), your log helped me notice another bug. > > At any rate, as I have

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-22 Thread Paolo Valente
> Il giorno 13 feb 2017, alle ore 23:38, Bart Van Assche > ha scritto: > > On Mon, 2017-02-13 at 22:07 +, Bart Van Assche wrote: >> On Mon, 2017-02-13 at 22:07 +0100, Paolo Valente wrote: >>> but what do you think about trying this fix? >> >> Sorry but with ...

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-13 Thread Bart Van Assche
On Mon, 2017-02-13 at 22:07 +, Bart Van Assche wrote: > On Mon, 2017-02-13 at 22:07 +0100, Paolo Valente wrote: > > but what do you think about trying this fix? > > Sorry but with ... the same server I used for the previous test still > didn't boot up properly. A screenshot is available at >

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-13 Thread Jens Axboe
On 02/13/2017 02:09 PM, Paolo Valente wrote: > >> Il giorno 07 feb 2017, alle ore 18:24, Paolo Valente >> ha scritto: >> >> Hi, >> >> I have finally pushed here [1] the current WIP branch of bfq for >> blk-mq, which I have tentatively named bfq-mq. >> >> This branch

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-13 Thread Bart Van Assche
On Mon, 2017-02-13 at 22:07 +0100, Paolo Valente wrote: > but what do you think about trying this fix? Sorry but with ... the same server I used for the previous test still didn't boot up properly. A screenshot is available at https://goo.gl/photos/Za9QVGCNe2BJBwxVA. > Otherwise, if you have no

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-13 Thread Paolo Valente
> Il giorno 07 feb 2017, alle ore 18:24, Paolo Valente > ha scritto: > > Hi, > > I have finally pushed here [1] the current WIP branch of bfq for > blk-mq, which I have tentatively named bfq-mq. > > This branch *IS NOT* meant for merging into mainline and contain

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-13 Thread Paolo Valente
> Il giorno 10 feb 2017, alle ore 20:49, Paolo Valente > ha scritto: > >> >> Il giorno 10 feb 2017, alle ore 19:13, Bart Van Assche >> ha scritto: >> >> On 02/10/2017 08:49 AM, Paolo Valente wrote: $ grep '^C.*_MQ_' .config

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Paolo Valente
> Il giorno 10 feb 2017, alle ore 19:13, Bart Van Assche > ha scritto: > > On 02/10/2017 08:49 AM, Paolo Valente wrote: >>> $ grep '^C.*_MQ_' .config >>> CONFIG_BLK_MQ_PCI=y >>> CONFIG_MQ_IOSCHED_BFQ=y >>> CONFIG_MQ_IOSCHED_DEADLINE=y >>> CONFIG_MQ_IOSCHED_NONE=y

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Paolo Valente
> Il giorno 10 feb 2017, alle ore 19:34, Bart Van Assche > ha scritto: > > On Tue, 2017-02-07 at 18:24 +0100, Paolo Valente wrote: >> (lock assertions, BUG_ONs, ...). > > Hello Paolo, > > If you are using BUG_ON(), does that mean that you are not aware of Linus' >

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Bart Van Assche
On 02/10/2017 08:49 AM, Paolo Valente wrote: >> $ grep '^C.*_MQ_' .config >> CONFIG_BLK_MQ_PCI=y >> CONFIG_MQ_IOSCHED_BFQ=y >> CONFIG_MQ_IOSCHED_DEADLINE=y >> CONFIG_MQ_IOSCHED_NONE=y >> CONFIG_DEFAULT_MQ_BFQ_MQ=y >> CONFIG_DEFAULT_MQ_IOSCHED="bfq-mq" >> CONFIG_SCSI_MQ_DEFAULT=y >>

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Paolo Valente
> Il giorno 10 feb 2017, alle ore 17:45, Bart Van Assche > ha scritto: > > On Tue, 2017-02-07 at 18:24 +0100, Paolo Valente wrote: >> 2) Enable people to test this first version bfq-mq. > > Hello Paolo, > > I installed this version of bfq-mq on a server that boots

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Paolo Valente
> Il giorno 10 feb 2017, alle ore 17:08, Bart Van Assche > ha scritto: > > On Tue, 2017-02-07 at 18:24 +0100, Paolo Valente wrote: >> [1] https://github.com/Algodev-github/bfq-mq > > Hello Paolo, > > That branch includes two changes of the version suffix

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Bart Van Assche
On Tue, 2017-02-07 at 18:24 +0100, Paolo Valente wrote: > 2) Enable people to test this first version bfq-mq. Hello Paolo, I installed this version of bfq-mq on a server that boots from a SATA disk. That server boots fine with kernel v4.10-rc7 but not with this tree. The first 30 seconds of the

Re: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Bart Van Assche
On Tue, 2017-02-07 at 18:24 +0100, Paolo Valente wrote: > [1] https://github.com/Algodev-github/bfq-mq Hello Paolo, That branch includes two changes of the version suffix (EXTRAVERSION in Makefile). Please don't do that but set CONFIG_LOCALVERSION in .config to add a suffix to the kernel

[WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-07 Thread Paolo Valente
Hi, I have finally pushed here [1] the current WIP branch of bfq for blk-mq, which I have tentatively named bfq-mq. This branch *IS NOT* meant for merging into mainline and contain code that mau easily violate code style, and not only, in many places. Commits implement the following main steps: