Re: [2.6 PATCH] visor: Always do generic_startup

From: Simon Fowler
Date: Sun Nov 21 2004 - 02:16:51 EST


On Sat, Nov 20, 2004 at 08:08:56PM -0800, Greg KH wrote:
> On Sun, Nov 21, 2004 at 12:23:53PM +1100, Simon Fowler wrote:
> > On Fri, Nov 19, 2004 at 09:44:05AM -0800, Greg KH wrote:
> > > On Tue, Nov 16, 2004 at 04:49:43PM +0100, Roger Luethi wrote:
> > > > generic_startup in visor.c was not called for some hardware, resulting
> > > > in attempts to access memory that had never been allocated, which in
> > > > turn caused the problem several people reported with recent (2.6.10ish)
> > > > kernels.
> > > >
> > > > Signed-off-by: Roger Luethi <rl@xxxxxxxxxxx>
> > >
> > > Thanks for finding this.
> > >
> > > Applied.
> > >
> > This patch fixes the oops, but after applying it I can no longer
> > sync my palm 5 - it starts, but part way through the connection is
> > lost.
>
> Can you enable debugging in the visor driver (either through the
> modprobe paramater, or through the /sys/module/paramater/debug file, and
> send it to us?
>
I've attached the gzipped log file, from the point it the new USB
device is registered to the point the device nodes are deleted by
udev.

Simon

--
PGP public key Id 0x144A991C, or http://himi.org/stuff/himi.asc
(crappy) Homepage: http://himi.org
doe #237 (see http://www.lemuria.org/DeCSS)
My DeCSS mirror: ftp://himi.org/pub/mirrors/css/

Attachment: visor-debug.gz
Description: Binary data

Attachment: signature.asc
Description: Digital signature