Re: [PATCH] scsi: sd: Use warn for logs about no caching info

From: Jeremy Cline
Date: Thu Jan 11 2018 - 15:22:55 EST


On Wed, Jan 03, 2018 at 10:50:29PM -0500, Martin K. Petersen wrote:
>
> Hi Jeremy,
>
> > When no caching mode information can be found for a disk, use the
> > warning log level rather than error. It is common for this to occur
> > with cheap USB sticks.
>
> Just because something is common occurrence doesn't mean it's not an
> error. What's the rationale behind demoting this to a warning?

That's true, that's not a particularly compelling reason. My rationale
is that in this case when it can't find the cache info it drops back to
sane defaults and works fine. It's certainly something to note, but
I wouldn't describe it as an "error condition".

Regards,
Jeremy