From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 12 Sep 2018 10:58:45 +0200 Message-ID: From: Takashi Iwai To: Jens Axboe Cc: Jeff Moyer , Paolo Valente , linux-block@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [REGRESSION] bfq probe failed on 4.19-rc3 In-Reply-To: <8d5f3ab2-50e8-226c-8ccb-c722872910db@kernel.dk> References: <8b5a7dc9-2dc1-16ee-4b05-a72eb6680c77@kernel.dk> <8d5f3ab2-50e8-226c-8ccb-c722872910db@kernel.dk> MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII List-ID: On Tue, 11 Sep 2018 18:54:19 +0200, Jens Axboe wrote: > > On 9/11/18 10:51 AM, Jeff Moyer wrote: > > Jens Axboe writes: > > > >> On 9/11/18 10:48 AM, Jeff Moyer wrote: > >>> Jens Axboe writes: > >>> > >>>> On 9/11/18 10:03 AM, Takashi Iwai wrote: > >>>>> Hi, > >>>>> > >>>>> I noticed that bfq doesn't appear in the I/O scheduler list on > >>>>> 4.19-rc3. It seems that blkcg_policy_register() at the beginning of > >>>>> bfq_init() returns -ENOSPC, hence the probe aborts silently. > >>>>> > >>>>> Is this already addressed? > >>>> > >>>> Haven't heard about this one before. What kernel did you last use that > >>>> worked? > >>> > >>> I'm guessing you just need to update the maximum number of policies: > >>> > >>> /* > >>> * Maximum number of blkcg policies allowed to be registered concurrently. > >>> * Defined here to simplify include dependency. > >>> */ > >>> #define BLKCG_MAX_POLS 3 > >> > >> That is my guess too, hence the suggestion to try and disable the > >> iolatency policy. Just tried here without, and it works, rebooting > >> with to verify that we're running out of policy slots. > > > > Functions calling this function: blkcg_policy_register > > > > File Function Line > > 0 block/bfq-iosched.c bfq_init 5675 ret = > > blkcg_policy_register(&blkcg_pol > > icy_bfq); > > 1 block/blk-iolatency.c iolatency_init 946 return > > blkcg_policy_register(&blkcg_pol > > icy_iolatency); > > 2 block/blk-throttle.c throtl_init 2511 return > > blkcg_policy_register(&blkcg_pol > > icy_throtl); > > 3 block/cfq-iosched.c cfq_init 4869 ret = > > blkcg_policy_register(&blkcg_pol > > icy_cfq); > > > > And then there were 4... > > Exactly. I'd bump it to 5 to leave room for one more, if we have to grow it > again, probably worth it to make it dynamic. So for, not really needed. Takashi, > can verify that the below fixes it up for you? Yes, it works. Tested-by: Takashi Iwai Thanks! Takashi