----- Original Message ----- > From: "Greg KH" <[email protected]> > To: "CAI Qian" <[email protected]> > Cc: [email protected], "David Milburn" <[email protected]>, "Jeff > Garzik" <[email protected]> > Sent: Friday, November 30, 2012 9:24:13 AM > Subject: Re: [PATCH] sata_svw: check DMA start bit before reset > > On Tue, Nov 27, 2012 at 11:59:57PM -0500, CAI Qian wrote: > > sata_svw: check DMA start bit before reset > > > > If kdump is triggered with pending IO, controller may not respond > > causing > > kdump to fail. > > > > http://marc.info/?l=linux-ide&m=133032255424658&w=2 > > > > During error recovery ata_do_dev_read_id never completes due hang > > in mmio_insw. > > > > ata_do_dev_read_id > > ata_sff_data_xfer > > ioread16_rep > > mmio_insw > > > > if DMA start bit is cleared before reset, PIO command is successful > > and kdump succeeds. > > > > Signed-off-by: David Milburn <[email protected]> > > Signed-off-by: Jeff Garzik <[email protected]> > > > > Upstream-ID: b03e66a6be91f8389fcd902ab6c1563db1c9c06b > > Stable-trees: 3.0.x, 3.4.x, 3.6.x > > Signed-off-by: CAI Qian <[email protected]> > > Any reason why you sent this twice? Did I? Oh, It may because that it took a day after I received a confirmation for my subscription to the ML for the first time. Therefore, I thought that the first time I sent this patch was lost (never reached due to the subscription delay), so sent it again.
CAI Qian > > greg k-h > -- > To unsubscribe from this list: send the line "unsubscribe stable" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
