MIME-Version: 1.0
Subject: RE: [PATCH] sata_nv: fix nmi intr or system hanging in rhel4u6 adma.
Date: Wed, 27 Feb 2008 12:55:08 +0800
Message-ID: <15F501D1A78BD343BE8F4D8DB854566B1BFE2AE7@hkemmail01.nvidia.com>
In-Reply-To: <47C43E27.5070700@garzik.org>
Thread-Topic: [PATCH] sata_nv: fix nmi intr or system hanging in rhel4u6 adma.
Thread-Index: Ach4lKNSNUMFfONEQ/6e3kuickAn9AAXfr7A
References: <15F501D1A78BD343BE8F4D8DB854566B1BFE2AE5@hkemmail01.nvidia.com> <47C42534.1090107@shaw.ca> <47C43E27.5070700@garzik.org>
From: "Kuan Luo" <kluo@nvidia.com>
To: "Jeff Garzik" <jeff@garzik.org>, "Robert Hancock" <hancockr@shaw.ca>
Cc: "linux-kernel" <linux-kernel@vger.kernel.org>,
       "Tejun Heo" <htejun@gmail.com>, "Peer Chen" <pchen@nvidia.com>
Content-class: urn:content-classes:message
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2026
Lines: 49

Jeff wrote:
>robert worte:
> > 
> > This is basically avoiding switching into register mode, 
> right? I don't 
> > think this is a very good solution as the point of the 
> tf_read function 
> > is that it's supposed to read the taskfile provided by the drive to 
> > diagnose the error, so not doing this isn't a good thing.
> 
> Agree with this analysis -- if ->tf_read() is being called, then 
> obviously the core wants a current copy of the device's ATA registers.
> 
> It is not a good solution to simply avoiding returning 
> meaningful data, 
> because -- as Robert notes -- we need tf_read for analysis.
> 
> 	Jeff
>

The driver got one error : "nv_adma_check_cpb: CPB 0, flags=0x11". The
code entered ata_port_abort -> ata_qc_complete
-> fill_result_tf->nv_adma_tf_read.

Firstly, nv_adma_register_mode failed, showing the below messages:
timeout waiting for ADMA IDLE, stat=0x440
timeout waiting for ADMA LEGACY, stat=0x440

Then enter ata_tf_read function. 
I found the system hung at tf->hob_nsect = ioread8(ioaddr->nsect_addr);
Sometimes the screen showed "
CPU0: Machin check Exception 0000000000000004
Bank 4:b200000000070f0f
kernel panic -not syncing: CPU Context corrupt.
"
If nv_adma_register_mode failed, the reg result should be not
meaningful. 
I don't know why the systm hung.
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/