lkml.org 
[lkml]   [2007]   [Jul]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: Don't miss the ARM-scsi fix.
Date
On Tuesday 03 July 2007 15:21:45 Andrew Morton wrote:
> On Tue, 3 Jul 2007 15:03:56 -0400
>
> Rob Landley <rob@landley.net> wrote:
> > Just making sure this fix winds up in 2.6.23:
>
> 2.6.22?

Er, yes. (After my six red bulls at the Blackthorn party, I've gone cold
turkey on caffeine for a bit to detox. I'm surprised I'm still upright.)

> > http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=4454/1
> >
> > It fixes a regression that occurred between 2.6.20 and 2.6.20-rc1.
> > Without it, qemu-system-arm can't use emulated SCSI drives. It wasn't in
> > -rc7, and the attached patch Works For Me (tm).
>
> Should have cc'ed Russell?

I cc'd the arm list, which he's on. Low enough traffic I expected him to get
it soonish, but I can cc: him directly if you like.

> > Signed-off-by: Rob Landley <rob@landley.net>
> >
> > Rob
> > --
> > "One of my most productive days was throwing away 1000 lines of code."
> > - Ken Thompson.
> >
> >
> > [linux-armfix.patch text/x-diff (1.1KB)]
> > ARM Versatile PCI config reads of one byte width have the lowest two<br
> > /> bits of the address cleared and result in reading from a wrong
> > place<br />
>
> html gunk in changelog

Sorry, that's from my quick grab of the web page above.

I expect this to come through the proper arm channels rather than mine, the
attach was just to be specific about the patch I was talking about.

> > in the config space. This change is to use word size accesses like it is
> > done for halfword reads.
> >
> > Byte reads are used for retrieving the IRQ number of a PCI device and the
> > problem was not exposed until 2.6.20 because the value read was discarded
> > in drivers/pci/setup-irq.c (recently fixed).
> >
> > Signed-off-by: Andrzej Zaborowski &lt;(address hidden)&gt;<br />
> > Acked-by: Paul Brook &lt;(address hidden)&gt;
>
> more

Sorry.

> > Note 2 submitted by Russell King on 02 Jul 2007 12:39:41 (UTC)
> > Moved to applied
> > Applied to git-curr.
> >
> > diff --git a/arch/arm/mach-versatile/pci.c
> > b/arch/arm/mach-versatile/pci.c index ba58223..ca82901 100644
> > --- a/arch/arm/mach-versatile/pci.c
> > +++ b/arch/arm/mach-versatile/pci.c
> > @@ -117,7 +117,10 @@ static int versatile_read_config(struct pci_bus
> > *bus, unsigned int devfn, int wh } else {
> > switch (size) {
> > case 1:
> > - v = __raw_readb(addr);
> > + v = __raw_readl(addr);
> > + if (where & 2) v >>= 16;
> > + if (where & 1) v >>= 8;
>
> Someone's enter key broke?

That's what the patch looked like when I got it. :)

I'm just an intrested party who's been trying to fix this particular problem
on my own (without success) for a couple months now. I hadn't been able to
upgrade Firmware Linux past 2.6.20 because ARM doesn't work on the newer
stuff, and it still doesn't work on 2.6.22-rc7 without this fix.

I found the fix yesterday, and would really really like it not to miss the
actual release. This was my attempt to make puppy eyes at subsystem
maintainers and add a signed-off-by to a small fix that unblocks a project
I'm working on. :)

Rob
--
"One of my most productive days was throwing away 1000 lines of code."
- Ken Thompson.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-07-03 23:49    [W:0.067 / U:0.912 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site