Skip to contentSkip to conversation

New: Lower log level for "No Caching mode page found" and "Assuming drive cache: write through"

Bug 106871 [email protected] 1 participant 2 messages Oct 29, 2015 – Oct 31, 2015 SCSI

lore
Share & tools
lore
#0 lore raw

https://bugzilla.kernel.org/show_bug.cgi?id=106871

            Bug ID: 106871
           Summary: Lower log level for "No Caching mode page found" and
                    "Assuming drive cache: write through"
           Product: IO/Storage
           Version: 2.5
    Kernel Version: 4.2.4
          Hardware: All
                OS: Linux
              Tree: Mainline
            Status: NEW
          Severity: normal
          Priority: P1
         Component: SCSI
          Assignee: [email protected]
          Reporter: [email protected]
        Regression: No

Is there any particular reason that these two should be of error level? I don't see why we should pay that much of attention to them since it's just how vendors implemented the drive. Neither can the users or should they do anything to fix it. IMHO they should be of notice level or at most warning.

Signature · 2 lines
You are receiving this mail because:
You are the assignee for the bug.
#1 lore raw

[Bug 106871] Lower log level for "No Caching mode page found" and "Assuming drive cache: write through"

https://bugzilla.kernel.org/show_bug.cgi?id=106871

--- Comment #1 from Tom Yan <[email protected]> ---
https://bugzilla.kernel.org/show_bug.cgi?id=16490#c8

The question is, why are we even using "write through" as fallback when if it can be dangerous? What's wrong with using "write back" as fallback instead? Would a SYNCHRONIZE CACHE command ever be a threat to devices without write cache or caching mode page?

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=b81478d82e389dd0961760f5ff6f56b50d29db6d
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=eaa05dfcdb12cf3a7bedf8918dc8699c00944384

Seems like we consider "write back" an safer option everywhere else but we just somehow require users to use a quirk to switch to that for no reason. This just looks silly to me.

Signature · 2 lines
You are receiving this mail because:
You are the assignee for the bug.