On Fri, Oct 12, 2012 at 4:33 AM, Hannes Reinecke <hare@xxxxxxx> 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+? Thanks for sorting all this out Hannes. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
