Re: [GIT PULL] Block fixes for 5.7-rc5

[Jens Axboe <axboe@kernel.dk>]

On 5/9/20 1:40 PM, Linus Torvalds wrote:
> On Fri, May 8, 2020 at 8:17 PM Jens Axboe <axboe@kernel.dk> wrote:
>>
>> A few fixes that should go into this series:
> 
> Jens, wtf?
> 
> This doesn't even build. Commit 0f6438fca125 ("bdi: use bdi_dev_name()
> to get device name") results in
> 
>   In file included from ./include/linux/kernel.h:15,
>                    from ./include/linux/list.h:9,
>                    from ./include/linux/module.h:12,
>                    from block/bfq-iosched.c:116:
>   block/bfq-iosched.c: In function ‘bfq_set_next_ioprio_data’:
>   block/bfq-iosched.c:4980:5: error: implicit declaration of function
> ‘bdi_dev_name’; did you mean ‘blkg_dev_name’?
> [-Werror=implicit-function-declaration]
>    4980 |     bdi_dev_name(bfqq->bfqd->queue->backing_dev_info),
>         |     ^~~~~~~~~~~~
>   ./include/linux/printk.h:299:33: note: in definition of macro ‘pr_err’
>     299 |  printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
>         |                                 ^~~~~~~~~~~
>   In file included from ./include/linux/printk.h:7,
>                    from ./include/linux/kernel.h:15,
>                    from ./include/linux/list.h:9,
>                    from ./include/linux/module.h:12,
>                    from block/bfq-iosched.c:116:
>   ./include/linux/kern_levels.h:5:18: warning: format ‘%s’ expects
> argument of type ‘char *’, but argument 2 has type ‘int’ [-Wformat=]
>       5 | #define KERN_SOH "\001"  /* ASCII Start Of Header */
>         |                  ^~~~~~
>   ./include/linux/kern_levels.h:11:18: note: in expansion of macro ‘KERN_SOH’
>      11 | #define KERN_ERR KERN_SOH "3" /* error conditions */
>         |                  ^~~~~~~~
>   ./include/linux/printk.h:299:9: note: in expansion of macro ‘KERN_ERR’
>     299 |  printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)
>         |         ^~~~~~~~
>   block/bfq-iosched.c:4979:3: note: in expansion of macro ‘pr_err’
>    4979 |   pr_err("bdi %s: bfq: bad prio class %d\n",
>         |   ^~~~~~
>   cc1: some warnings being treated as errors
>   make[1]: *** [scripts/Makefile.build:267: block/bfq-iosched.o] Error 1
>   make: *** [Makefile:1722: block] Error 2
> 
> and no, it's not a merge error - at least not one by me. I tested the
> tip-of-tree that you sent me, at commit ded3148fc653.
> 
> So that build error exists in your branch.
> 
> Unpulled. Get testing, and don't send me garbage.

Gah, sorry about that, not sure how this went undetected. I'll
redo the pull request.

-- 
Jens Axboe