Re: [Bug #12263] Sata soft reset filling log

Justin Madru
2009-02-16T21:40:01Z

Sergei Shtylyov wrote:

> Hello.
>> Justin Madru wrote:
>>> After referring to the SCSI command codes "cdb 0x1e" means ALLOW
>>> MEDIUM REMOVAL command -- which
>>> could hardly be addressed to an usual hard disk. So, it looks like
>>> we had a case of the confused bug report which
>>> has a lot of info on the hard disk while errors were most probably
>>> happening with a CD/DVD drive.
>>
>> Yes, I originally thought it was my hard disk because the kernel logs
>> showed ata2.
>> But, Tejun Heo figured out it was my DVD drive (ATAPI) that was on
>> the ata2 link.
>> (see http://marc.info/?l=linux-kernel&m=122993014109646&w=2)
>> I tried to bisect it, but around .28-rc1 I began to get numerous
>> compile errors, so couldn't continue.
>> I also tried patches that Tejun sent me, but non of them worked, it
>> just slightly change the error message.
>> So, yes this is a regression that was introduced in the .28 merge
>> window, and I still think that bug #12609 is a duplicate of my bug.
>
> If 12609 is truly a post-2.6.28 regression and 12263 is post-2.6.27
> regresssion, this just cannot be.

Maybe the reporter of #12609 didn't notice/test kernels 28-rc1 to 28. Or
maybe the difference in hardware is
the issue, but the bug is still the same. Don't know.

>> I don't see this bug on tip/master and this is the diff of origin and
>> tip at the time I tested.
>> $ git diff origin/master..tip/master drivers/ata/
>
> What tree is that?

This is what I have in .git/config and I get the same diff if I run:
git diff master..tip drivers/ata/ or git diff master...tip drivers/ata/

[core]
repositoryformatversion = 0
filemode = true
bare = false
logallrefupdates = true
[remote "origin"]
url =
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git
fetch = +refs/heads/*:refs/remotes/origin/*
[branch "master"]
remote = origin
merge = refs/heads/master
[remote "tip"]
url =
git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip.git
fetch = +refs/heads/*:refs/remotes/tip/*
[branch "tip"]
remote = tip
merge = refs/heads/master

Justin Madru