Re: ASUS P5W motherboard PMP

Re: ASUS P5W motherboard PMP

[David Madsen]

Yes, I patched the 2.6.22.1 vanilla kernel with the libata patch from here.

http://home-tj.org/files/libata-tj-stable/libata-tj-2.6.22.1-20070803.tar.bz2

--David Madsen

On 8/3/07, Tejun Heo <htejun@gmail.com> wrote:
> David Madsen wrote:
> > * The long boot delay on ASUS boards with on-board PMP chips should be
> >   fixed by this patchset but I don't have such a board or Sil4723
> >   which is used on those boards, so I'm not sure.  Anyone up for
> >
> >   testing?
> >
> > Hi Tejun,
> >
> >   I have one of the above mentioned ASUS motherboards that has a PMP device connected
> > to one of the SATA ports on the ICH7R southbridge.  I came across your libata patches
> > and mention of a tester for this motherboard so I downloaded the patchset and tried it
> >
> > on my motherboard.  Booting with the new kernel appears to have the same errors and delay
> > that the previous kernel had.
> >
> > I have attached a tar file with the dmesg output as well as my kernel's .config.  I'm not
> >
> > sure what else may be of use to you to help with the debugging so let me know if I can
> > provide you with any more info.
>
> Please cc linux-ide@vger.kernel.org next time.
>
> You patched the kernel with PMP patches, right?
>
> --
> tejun
>

Re: ASUS P5W motherboard PMP

[Tejun Heo]

[-- Attachment #1: Type: text/plain, Size: 293 bytes --]

David Madsen wrote:
> Yes, I patched the 2.6.22.1 vanilla kernel with the libata patch from here.
> 
> http://home-tj.org/files/libata-tj-stable/libata-tj-2.6.22.1-20070803.tar.bz2

Hmmm... Can you please apply the attached patch on top of libata-tj and
report the kernel boot log?

-- 
tejun

[-- Attachment #2: debug.patch --]
[-- Type: text/x-patch, Size: 1626 bytes --]

---
 drivers/ata/ahci.c        |    5 ++++-
 drivers/ata/libata-core.c |    6 +++---
 2 files changed, 7 insertions(+), 4 deletions(-)

Index: work/drivers/ata/libata-core.c
===================================================================
--- work.orig/drivers/ata/libata-core.c
+++ work/drivers/ata/libata-core.c
@@ -691,17 +691,17 @@ unsigned int ata_dev_classify(const stru
 	 * identifies a port multiplier and 0x3c/0xc3 a SEMB device.
 	 */
 	if ((tf->lbam == 0) && (tf->lbah == 0)) {
-		DPRINTK("found ATA device by sig\n");
+		printk("found ATA device by sig\n");
 		return ATA_DEV_ATA;
 	}
 
 	if ((tf->lbam == 0x14) && (tf->lbah == 0xeb)) {
-		DPRINTK("found ATAPI device by sig\n");
+		printk("found ATAPI device by sig\n");
 		return ATA_DEV_ATAPI;
 	}
 
 	if ((tf->lbam == 0x69) && (tf->lbah == 0x96)) {
-		DPRINTK("found PMP device by sig\n");
+		printk("found PMP device by sig\n");
 		return ATA_DEV_PMP;
 	}
 
Index: work/drivers/ata/ahci.c
===================================================================
--- work.orig/drivers/ata/ahci.c
+++ work/drivers/ata/ahci.c
@@ -972,6 +972,9 @@ static unsigned int ahci_dev_classify(st
 	tf.lbal		= (tmp >> 8)	& 0xff;
 	tf.nsect	= (tmp)		& 0xff;
 
+	ata_port_printk(ap, KERN_INFO, "XXX sig %02x:%02x:%02x %02x\n",
+			tf.lbal, tf.lbam, tf.lbah, tf.nsect);
+
 	return ata_dev_classify(&tf);
 }
 
@@ -1072,7 +1075,7 @@ static int ahci_do_softreset(struct ata_
 	struct ata_taskfile tf;
 	int rc;
 
-	DPRINTK("ENTER\n");
+	ata_link_printk(link, KERN_INFO, "XXX softresetting pmp=%d\n", pmp);
 
 	if (ata_link_offline(link)) {
 		DPRINTK("PHY reports no device\n");

Re: ASUS P5W motherboard PMP

[David Madsen]

[-- Attachment #1: Type: text/plain, Size: 434 bytes --]

The updated kernel log with the debug patch is attached.

--David Madsen

On 8/4/07, Tejun Heo <htejun@gmail.com> wrote:
> David Madsen wrote:
> > Yes, I patched the 2.6.22.1 vanilla kernel with the libata patch from here.
> >
> > http://home-tj.org/files/libata-tj-stable/libata-tj-2.6.22.1-20070803.tar.bz2
>
> Hmmm... Can you please apply the attached patch on top of libata-tj and
> report the kernel boot log?
>
> --
> tejun
>
>

[-- Attachment #2: ASUS_P5W_debug.dmesg.bz2 --]
[-- Type: application/x-bzip2, Size: 7640 bytes --]

Re: ASUS P5W motherboard PMP

[Tejun Heo]

Also, please report the result of 'lspci -nn' and 'dmidecode'.

-- 
tejun