lkml.org 
[lkml]   [2016]   [Feb]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [RFC PATCH] sata_via: Apply WD workaround only when needed
Date


On Tuesday 16 February 2016 22:42:53 Tejun Heo wrote:
> Hello, Ondrej.
>
> On Mon, Feb 15, 2016 at 10:01:47PM +0100, Ondrej Zary wrote:
> > The first hdparm is slower because of the error:
> > [ 80.964060] ata5.00: exception Emask 0x12 SAct 0x0 SErr 0x1000500 action 0x6
> > [ 80.964095] ata5.00: BMDMA stat 0x5
> > [ 80.964108] ata5: SError: { UnrecovData Proto TrStaTrns }
> > [ 80.964125] ata5.00: failed command: READ DMA EXT
> > [ 80.964143] ata5.00: cmd 25/00:90:00:00:00/00:04:00:00:00/e0 tag 0 dma 598016 in
> > res 51/84:af:df:00:00/84:03:00:00:00/e0 Emask 0x12 (ATA bus error)
> > [ 80.964173] ata5.00: status: { DRDY ERR }
> > [ 80.964185] ata5.00: error: { ICRC ABRT }
> > [ 80.964209] ata5: hard resetting link
> > [ 81.284056] ata5: SATA link up 1.5 Gbps (SStatus 113 SControl 310)
> > [ 81.300531] ata5.00: configured for UDMA/133
> > [ 81.300569] ata5: Incompatible drive (WD?): enabling workaround
> > [ 81.300598] ata5: EH complete
>
> Hmm... I like the workaround but wish the kernel weren't generating
> the above output. How about doing something like the following?

I wish that too but wasn't sure it was possible. Thanks for idea.

> svia_error_handler()
> {
> if (workaround hasn't been applied yet &&
> the conditions match for the workaround) {
> apply the workaround;
> mark as such;
> print informational message;
> set ATA_EHI_QUIET;
> }
>
> ata_sff_error_handler(ap);
> }
>

Guess that I need to read SErr manually?

--
Ondrej Zary

\
 
 \ /
  Last update: 2016-02-17 00:21    [W:1.022 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site