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

[Justin Madru]

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/