On 10/15/2012 02:19 AM, Mike Snitzer wrote:
On Fri, Oct 12, 2012 at 4:33 AM, Hannes Reinecke <[email protected]> wrote:
This patchset updates the SCSI midlayer to use dev_printk() instead
of the simple printk(). The main objective here is to avoid line-breaks
in syslog output; with the current state it's nearly impossible to match
the output to the occurring device; under high load even the CDB will
be split off into individual bytes, spread randomly across the lines.
Which makes debugging via scsi_logging_level _really_ hard.
In addition we'll be getting the syslog messages nicely prefixed with
the device, which will make userspace logging daemons happy.
Before:
[ 297.300605] sd 2:0:3:2: Send:
[ 297.300607] 0xffff8802348b0980
[ 297.300610] sd 2:0:3:2: CDB:
[ 297.300615] Test Unit Ready: 00 00 00 00 00 00
[ 297.300747] sd 2:0:3:2: Done:
[ 297.300750] 0xffff8802348b0980 SUCCESS
[ 297.300753] sd 2:0:3:2:
[ 297.300755] Result: hostbyte=DID_OK driverbyte=DRIVER_OK
[ 297.300758] sd 2:0:3:2: CDB:
[ 297.300764] Test Unit Ready: 00 00 00 00 00 00
[ 297.300766] sd 2:0:3:2:
[ 297.300769] Sense Key : Unit Attention [current]
[ 297.300771] Info fld=0x0
[ 297.300772] sd 2:0:3:2:
[ 297.300776] Add. Sense: Capacity data has changed
After:
[ 636.683556] sd 2:0:3:2: Send: 0xffff88043145eec0
[ 636.727856] sd 2:0:3:2: CDB: Test Unit Ready: 00 00 00 00 00 00
[ 636.785330] sd 2:0:3:2: Done: 0xffff88043145eec0 SUCCESS
[ 636.838228] sd 2:0:3:2: Result: hostbyte=DID_OK driverbyte=DRIVER_OK
[ 636.899099] sd 2:0:3:2: CDB: Test Unit Ready: 00 00 00 00 00 00
[ 636.955905] sd 2:0:3:2: Sense Key : Unit Attention [current]
[ 637.069179] sd 2:0:3:2: Add. Sense: Capacity data has changed
I know there are a lot of changes here but shouldn't all this get
fixed in stable 3.5+?
If you ask me, yes, of course.
I can easily post a v2 of the patchset (meanwhile I've got another
patch for st :-), Cc'ing stable with it.
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
[email protected] +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
