Re: [alsa-devel] snd_usb_caiaq bug?

11 Jan 2009

      At Sat, 10 Jan 2009 13:53:24 +0100,
Marco Santamaria wrote:
...
During a Rosegarden session a qsampler crash caused the following output in
dmesg:
[11530.642970] qsampler[22794]: segfault at 00000008 eip b73f52c5 esp bfe77e20
error 4
[13278.161113] ALSA /home/marco/build/alsa-1.0.17/linux-2.6.24-23-rt/driver/
alsa-driver-1.0.17/usb/caiaq/../../alsa-kernel/usb/caiaq/caiaq-midi.c:90:
snd-usb-caiaq log: snd_usb_caiaq_midi_send(f6d52b40): usb_submit_urb()
failed,ret=-22, len=61
[13720.859888] qjackctl.bin[22273]: segfault at 00000314 eip b7b07a7f esp
bfbe8f54 error 4
There are segmentation fault errors from qsampler and qjackctl, but perhaps
there's also a problem in caiaq-midi.c .
Might be, but difficult to judge as now since the error seems
happening after qsampler, and the urb error itself seems
non-critical.
Takashi
...
Regards,
Marco
2009/1/9 Takashi Iwai tiwai@suse.de
At Thu, 8 Jan 2009 23:11:53 +0100,
Daniel Mack wrote:
>
> Hi Takashi,
>
> On Thu, Jan 08, 2009 at 03:37:30PM +0100, Takashi Iwai wrote:
> > > every thing is Ok for me now. No more crash, no more messages in
dmesg.
> > > I just had a warning during compilation, but I think this is not
really
> > > important :
> > >
> > > sound/usb/caiaq/caiaq-midi.c: In function \
u2018snd_usb_caiaq_midi_output_done
> > > \u2019:
> > > sound/usb/caiaq/caiaq-midi.c:170: attention : unused variable \
u2018buf\u2019
> > >
> > > Perhaps this is not the same line (170) as you but, in my file, it
concerns
> > > the last function.
> >
> > OK, it's simple to fix.
> > The below is the latest version I committed now to sound git tree.
> > It'll be pushed to 2.6.29 (and stable kernels) later.
>
> Thanks a lot for your help - I was on vacation until now and had very
> limited ways to follow that thread. I would, however, apprectiate the
> driver version number would be increased for this change, as it makes
> tracking of user error reports better trackable.
>
> Do you want me to send in a patch for that or is it easier if you do
> it yourself?

I increased the version to 1.3.10 now.

thanks,

Takashi