Re: [PATCH] block: Remove leading whitespace and trailing newline in elevator switch error message

["markus@trippelsdorf.de" <markus@trippelsdorf.de>]

On 2017.05.08 at 20:22 -0600, Jens Axboe wrote:
> 
> > On May 8, 2017, at 8:18 PM, Elliott, Robert (Persistent Memory)
> > <elliott@hpe.com> wrote:
> >> +        printk(KERN_ERR "elevator: type %s not found\n", name);
> > ...
> >> +    printk(KERN_ERR "elevator: switch to %s failed\n",
> > 
> > That leaves two lines of error prints for a single error, and
> > the second line doesn't convey additional information.
> 
> Yeah good point. Honestly I think we should just kill both lines. None
> of them add value. 

They at least help debugging of udev rules, e.g. on Gentoo they have:

 # set noop on solid state drives
 SUBSYSTEM=="block", ACTION=="add", ATTR{queue/rotational}=="0", ATTR{queue/scheduler}="noop"

And this will of course fail for mq. But how will you notice it without
any dmesg error message?

-- 
Markus