Re: [Bulk] Re: [Bugme-new] [Bug 7482] New: usb ehci driver crash on system with more tahn 256MByte memory

> -----Original Message-----
> From: David Brownell [mailto:[email protected]]
> Sent: Friday, May 18, 2007 6:49 PM
> To: Andrew Morton
> Cc: Stefan Meyer; [email protected]; Alan Stern;
Greg KH;
> [email protected]; Natalie Protasevich; Li Yang-r58472
> Subject: Re: [Bulk] Re: [Bugme-new] [Bug 7482] New: usb ehci driver
crash on system
> with more tahn 256MByte memory
> 
> On Friday 18 May 2007, Andrew Morton wrote:
> > On Fri, 10 Nov 2006 00:47:59 -0800 David Brownell
<[email protected]> wrote:
> >
> > > On Thursday 09 November 2006 11:56 pm, Stefan Meyer wrote:
> > >
> > > > Changing it to 0x0000001C creates a 512MByte windows (this was
of course a
> > > > non-generic quick fix).
> > > > Running with anything up to 256MByte is fine.
> > > > My system has 512MByte and it took quite a bit of chasing to
find this
> > > > issue.
> > >
> > > A better patch would (a) fix the comment which says both lines of
> > > code affects the PHY, (b) not hard-wire the amount of memory on
> > > the system.
> > >
> > > Assuming it's possible to put non-cacheable memory into that chunk
> > > of address space (memory mapped peripherals etc) then it's
dangerous
> > > to hard-wire _any_ value.
> > >
> >
> > So, did this ever get fixed? (Full history at
> > http://bugzilla.kernel.org/show_bug.cgi?id=7482)
> 
> Not from what I can tell.  I cc'd the Freescale person who seems
> to be maintaining EHCI on that platform ...

I see.  The complete fix that I expect is to set snooping on the whole
4G memory space as fsl_usb2_udc did.  Also, we need to take care of the
case that the Soc used on architectures other than powerpc which don't
have snooping mechanism.

Shall I provide a patch to fix this?

- Leo

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel