]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
[S390] dasd fix dump_sense_dbf
authorStefan Haberland <stefan.haberland@de.ibm.com>
Mon, 25 Oct 2010 14:10:24 +0000 (16:10 +0200)
committerMartin Schwidefsky <sky@mschwide.boeblingen.de.ibm.com>
Mon, 25 Oct 2010 14:10:17 +0000 (16:10 +0200)
The dasd_eckd_dump_sense_dbf function uses a macro for s390 debug
feature that can handle up to 8 parameters (for the DASD device
driver).
Fix the function to use only the maximum number of parameters.

Signed-off-by: Stefan Haberland <stefan.haberland@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
drivers/s390/block/dasd_eckd.c

index 59b4ecfb967b956e7c8d626cad5794ab300cede6..ea0e565ebc9d8ee24ade58ce558b2ae36caac264 100644 (file)
@@ -3093,16 +3093,20 @@ dasd_eckd_dump_sense_dbf(struct dasd_device *device, struct irb *irb,
                         char *reason)
 {
        u64 *sense;
+       u32 stat;
 
        sense = (u64 *) dasd_get_sense(irb);
+       stat = scsw_cstat(&irb->scsw);
+       stat <<= 8;
+       stat |= scsw_dstat(&irb->scsw);
+       stat <<= 8;
+       stat |= scsw_cc(&irb->scsw);
+
        if (sense) {
                DBF_DEV_EVENT(DBF_EMERG, device,
-                             "%s: %s %02x%02x%02x %016llx %016llx %016llx "
-                             "%016llx", reason,
-                             scsw_is_tm(&irb->scsw) ? "t" : "c",
-                             scsw_cc(&irb->scsw), scsw_cstat(&irb->scsw),
-                             scsw_dstat(&irb->scsw), sense[0], sense[1],
-                             sense[2], sense[3]);
+                             "%s: %s %06x %016llx %016llx %016llx %016llx",
+                             reason, scsw_is_tm(&irb->scsw) ? "t" : "c", stat,
+                             sense[0], sense[1], sense[2], sense[3]);
        } else {
                DBF_DEV_EVENT(DBF_EMERG, device, "%s",
                              "SORRY - NO VALID SENSE AVAILABLE\n");