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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <01000160e6e3a905-df1c1554-dde2-41c1-8095-eb58a4ba3a75-000000@email.amazonses.com>
Date:   Thu, 11 Jan 2018 20:22:48 +0000
From:   Jeremy Cline <jeremy@...ine.org>
To:     "Martin K. Petersen" <martin.petersen@...cle.com>
Cc:     "James E . J . Bottomley" <jejb@...ux.vnet.ibm.com>,
        linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] scsi: sd: Use warn for logs about no caching info

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