linux-kernel - Re: 2.6.29 regression: ATA bus errors on resume

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4A1C8444.9040605@kernel.org>
Date:	Wed, 27 May 2009 09:07:32 +0900
From:	Tejun Heo <tj@...nel.org>
To:	Niel Lambrechts <niel.lambrechts@...il.com>
CC:	Alan Cox <alan@...rguk.ukuu.org.uk>,
	"linux.kernel" <linux-kernel@...r.kernel.org>,
	Theodore Tso <tytso@....edu>
Subject: Re: 2.6.29 regression: ATA bus errors on resume

Niel Lambrechts wrote:
> On 05/26/2009 03:33 PM, Tejun Heo wrote:
>> Niel Lambrechts wrote:
>>   
>>> If you send some patches I'll make every effort to test, it beats having
>>> to re-install, my installation is just too customized. :)
>>>      
>> First, let's make sure we aren't balking up the wrong tree.  Can you
>> please apply the attached patch and report the kernel log?
> 
> Hi Tejun,
> 
> Okay it took 5 attempts, some of during which I played audio, did 'find
> /'  etc. but I still do not have a clue whether the extra activity
> helped trigger it or not.

Thanks for testing.

 XXX scmd->result=0x8000002 ff_t=4 ff_dev=2 ff_drv=8
 XXX DID_OK
 XXX CHECK_CONDITION, returning ff_dev
 sd 0:0:0:0: [sda] Result: hostbyte=DID_OK driverbyte=DRIVER_SENSE
 sd 0:0:0:0: [sda] Sense Key : Aborted Command [current] [descriptor]
 Descriptor sense data with sense descriptors (in hex):
	 72 0b 00 00 00 00 00 0c 00 0a 80 00 00 00 00 00 
	 09 b8 71 d1 
 sd 0:0:0:0: [sda] Add. Sense: No additional sense information
 end_request: I/O error, dev sda, sector 242190447

The above is the offending failure and all three failfast bits are
set.  This corresponds to the following ATA exception.

 ata1.00: cmd 60/08:18:6f:88:6f/01:00:0e:00:00/40 tag 3 ncq 135168 in
	  res 50/00:40:d1:71:b8/00:00:09:00:00/40 Emask 0x10 (ATA bus error)

It's 33 page long read command.  Looking at the code the only way all
three fastfail bits can be set seems to be if the request is readahead
- the first part of block/blk-core.c::init_request_from_bio().  Now,
the failure of a readahead request isn't supposed to cause any
problem.  If it fails, well, it fails and things should go on as if
nothing happened.

Can you please try the attached patch?  It takes suspend/resume cycle
out of the equation and simply induces artificial failure to readahead
requests.  It's currently set to fail every 40th readahead.  Feel free
to adjust the frequency as you see fit.  catting files into /dev/null
would trigger readahead to kick in.  Can you reproduce filesystem
failure with this alone?

Thanks.

-- 
tejun

View attachment "fail-readahead.patch" of type "text/x-patch" (723 bytes)