Re: [PATCH] libata: Add multi-count transfer debugging

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Carl-Daniel Hailfinger wrote:
On my Asus M2A-VM (SB600) board, the SATA disks use multi-count
transfers only with some BIOS variants.
Dumping the relevant config words helps debugging a lot.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@xxxxxxx>

--- a/drivers/ata/libata-core.c	2009-05-06 14:24:32.000000000 +0200
+++ b/drivers/ata/libata-core.c	2009-05-06 15:27:06.000000000 +0200
@@ -2390,11 +2390,11 @@
 	/* print device capabilities */
 	if (ata_msg_probe(ap))
 		ata_dev_printk(dev, KERN_DEBUG,
-			       "%s: cfg 49:%04x 82:%04x 83:%04x 84:%04x "
-			       "85:%04x 86:%04x 87:%04x 88:%04x\n",
+			       "%s: cfg 47:%04x 49:%04x 59:%04x 82:%04x 83:%04x"
+			       " 84:%04x 85:%04x 86:%04x 87:%04x 88:%04x\n",
 			       __func__,
-			       id[49], id[82], id[83], id[84],
-			       id[85], id[86], id[87], id[88]);
+			       id[47], id[49], id[59], id[82], id[83],
+			       id[84], id[85], id[86], id[87], id[88]);
/* initialize to-be-configured parameters */
 	dev->flags &= ~ATA_DFLAG_CFG_MASK;


--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Keep in mind that multi-count transfers are only meaningful for PIO read/write commands (it controls the number of sectors transferred for each DRQ assertion), which would only be used if the device didn't support DMA - unlikely for SATA..
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux