Re: usb audio race at disconnect time

2012-10-30 Thread Alan Stern
On Tue, 30 Oct 2012, Matthieu CASTET wrote: > But I saw a failure I didn't saw before during stress test [1]. > It is hard to say if it is related to your patches : > - we test on a older kernel version and I backported usbaudio from 3.7 with > your > patches > - the start of the trace is missing

Re: usb audio race at disconnect time

2012-10-30 Thread Takashi Iwai
At Tue, 30 Oct 2012 11:52:33 +0100, Matthieu CASTET wrote: > > > Takashi Iwai a écrit : > >> Thanks, > >> > >> the first tests seems ok. > >> We will do more strees test. > > > > Do you have any good/bad test result? > > If no regression is seen, I'm going to apply the patches for 3.7-rc4. > The

Re: usb audio race at disconnect time

2012-10-29 Thread Takashi Iwai
At Thu, 18 Oct 2012 17:39:05 +0200, Matthieu CASTET wrote: > > Takashi Iwai a écrit : > > At Tue, 16 Oct 2012 18:01:13 +0200, > > Matthieu CASTET wrote: > >> Takashi Iwai a écrit : > >>> At Mon, 15 Oct 2012 19:41:40 +0200, > >>> Matthieu CASTET wrote: > Hi Takashi, > > But I believe

Re: usb audio race at disconnect time

2012-10-19 Thread Takashi Iwai
At Thu, 18 Oct 2012 17:39:05 +0200, Matthieu CASTET wrote: > > Takashi Iwai a écrit : > > At Tue, 16 Oct 2012 18:01:13 +0200, > > Matthieu CASTET wrote: > >> Takashi Iwai a écrit : > >>> At Mon, 15 Oct 2012 19:41:40 +0200, > >>> Matthieu CASTET wrote: > Hi Takashi, > > But I believe

Re: usb audio race at disconnect time

2012-10-18 Thread Matthieu CASTET
Takashi Iwai a écrit : > At Tue, 16 Oct 2012 18:01:13 +0200, > Matthieu CASTET wrote: >> Takashi Iwai a écrit : >>> At Mon, 15 Oct 2012 19:41:40 +0200, >>> Matthieu CASTET wrote: Hi Takashi, But I believe I found other races in the alsa char device handling. With the attac

Re: usb audio race at disconnect time

2012-10-16 Thread Takashi Iwai
At Tue, 16 Oct 2012 18:01:13 +0200, Matthieu CASTET wrote: > > Takashi Iwai a écrit : > > At Mon, 15 Oct 2012 19:41:40 +0200, > > Matthieu CASTET wrote: > >> Hi Takashi, > >> > >> But I believe I found other races in the alsa char device handling. With > >> the > >> attached patch, if you disconn

Re: usb audio race at disconnect time

2012-10-16 Thread Matthieu CASTET
Takashi Iwai a écrit : > At Mon, 15 Oct 2012 19:41:40 +0200, > Matthieu CASTET wrote: >> Hi Takashi, >> >> But I believe I found other races in the alsa char device handling. With the >> attached patch, if you disconnect the usb audio device between "msleep o" and >> "msleep o+", you will free the

Re: usb audio race at disconnect time

2012-10-16 Thread Takashi Iwai
At Mon, 15 Oct 2012 20:45:44 +0200, Takashi Iwai wrote: > > At Mon, 15 Oct 2012 19:41:40 +0200, > Matthieu CASTET wrote: > > > > Hi Takashi, > > > > Takashi Iwai a écrit : > > > At Fri, 12 Oct 2012 17:42:19 +0200, > > > Matthieu CASTET wrote: > > >> Hi, > > >> > > >> > > >> Takashi Iwai a écrit

Re: usb audio race at disconnect time

2012-10-15 Thread Takashi Iwai
At Mon, 15 Oct 2012 19:41:40 +0200, Matthieu CASTET wrote: > > Hi Takashi, > > Takashi Iwai a écrit : > > At Fri, 12 Oct 2012 17:42:19 +0200, > > Matthieu CASTET wrote: > >> Hi, > >> > >> > >> Takashi Iwai a écrit : > >>> [Added Daniel and Clemens in the loop] > >>> > >>> > >>> I don't think this

Re: usb audio race at disconnect time

2012-10-15 Thread Matthieu CASTET
Hi Takashi, Takashi Iwai a écrit : > At Fri, 12 Oct 2012 17:42:19 +0200, > Matthieu CASTET wrote: >> Hi, >> >> >> Takashi Iwai a écrit : >>> [Added Daniel and Clemens in the loop] >>> >>> >>> I don't think this is needed. >>> >>> So... the below is a quick hack I did without testing at all. >>> Ho

Re: usb audio race at disconnect time

2012-10-13 Thread Takashi Iwai
At Fri, 12 Oct 2012 17:42:19 +0200, Matthieu CASTET wrote: > > Hi, > > > Takashi Iwai a écrit : > > [Added Daniel and Clemens in the loop] > > > > > > I don't think this is needed. > > > > So... the below is a quick hack I did without testing at all. > > Hopefully this can give some advance.

Re: usb audio race at disconnect time

2012-10-12 Thread Matthieu CASTET
Hi, Takashi Iwai a écrit : > [Added Daniel and Clemens in the loop] > > > I don't think this is needed. > > So... the below is a quick hack I did without testing at all. > Hopefully this can give some advance. Thanks for the quick patch. The patch didn't apply cleany of linus tree, of which t

Re: [alsa-devel] usb audio race at disconnect time

2012-10-12 Thread Takashi Iwai
At Thu, 11 Oct 2012 18:41:52 +0200, Takashi Iwai wrote: > > [Added Daniel and Clemens in the loop] > > At Thu, 11 Oct 2012 17:17:59 +0200, > Matthieu CASTET wrote: > > > > Hi, > > > > while doing some monkey tests on a product we found races in usb audio code > > when > > the device in unplugg

Re: usb audio race at disconnect time

2012-10-11 Thread Takashi Iwai
[Added Daniel and Clemens in the loop] At Thu, 11 Oct 2012 17:17:59 +0200, Matthieu CASTET wrote: > > Hi, > > while doing some monkey tests on a product we found races in usb audio code > when > the device in unplugged from usb (on linus master tree). > > This can be reproduced with usb_audio_

Re: usb audio race at disconnect time

2012-10-11 Thread Alan Stern
On Thu, 11 Oct 2012, Matthieu CASTET wrote: > Hi, > > while doing some monkey tests on a product we found races in usb audio code > when > the device in unplugged from usb (on linus master tree). > > This can be reproduced with usb_audio_show_race.diff and CONFIG_DEBUG_SLAB. > With this patch,

usb audio race at disconnect time

2012-10-11 Thread Matthieu CASTET
Hi, while doing some monkey tests on a product we found races in usb audio code when the device in unplugged from usb (on linus master tree). This can be reproduced with usb_audio_show_race.diff and CONFIG_DEBUG_SLAB. With this patch, start a stream : # arecord -D hw:0 -r 44100 -c 2 -f S16_LE >