From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: Re: snd_mpu401_uart: hang when trying to play .mid Date: Mon, 06 Sep 2004 22:45:04 +0200 Sender: alsa-devel-admin@lists.sourceforge.net Message-ID: References: <20040905092234.GA3605@masina.coderock.org> <20040906154819.GA7901@nd47.coderock.org> <20040906185838.GA8347@nd47.coderock.org> <1094501904.29921.56.camel@krustophenia.net> Mime-Version: 1.0 (generated by SEMI 1.14.5 - "Awara-Onsen") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: <1094501904.29921.56.camel@krustophenia.net> Errors-To: alsa-devel-admin@lists.sourceforge.net List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Lee Revell Cc: Domen Puncer , alsa-devel List-Id: alsa-devel@alsa-project.org At Mon, 06 Sep 2004 16:18:25 -0400, Lee Revell wrote: > > On Mon, 2004-09-06 at 14:58, Domen Puncer wrote: > > On 06/09/04 18:25 +0200, Takashi Iwai wrote: > > > At Mon, 6 Sep 2004 17:48:19 +0200, > > > Domen Puncer wrote: > > > > On 06/09/04 17:18 +0200, Takashi Iwai wrote: > > > > > At Sun, 5 Sep 2004 11:22:35 +0200, > > > > > Domen Puncer wrote: > > > > > Does your hardware really a MPU401 device? > > > > > > > > I don't know, probably not. > > > > > > > > > If not, you can remove the call of snd_mpu401_uart_new() in > > > > > snd_trident_probe(). > > > > > > > > Or just never start kmid again :-) > > > > My point is: user can hang the computer; this is a bug. > > > > > > Sure. Removing the unneeded creation of mpu401 interface is _the_ > > > solution if you really don't have mpu401. > > > > Umm... you mean removing this for linus tree? Then why it was there > > in the first place? > > No, Takashi-san is telling you to try commenting this out and recompile > and see if the problem goes away. If so then we can figure out how to > detect this MPU-less device and fix it in ALSA CVS. He probably does > not have access to the hardware to test it himself. Exactly. Thanks for interpretation ;) > This is a pretty common issue with ALSA because a lot of cards that > purport to be the same as another have small differences, and the alsa > devs can't possibly test every single device. Yes, indeed, the trident code is shared among several different chips (trident 4D DX, 4D NX and SI7018), and MPU401 handling seems hitting on SI7018 only. I guess SI7018 is equipped mostly on laptops, which unlikely have MPU401 connectors. So, removing the mpu401 support for SI7018 may be the simplest and safest solution. Of course, we can track down the hang-up point (perhaps in the interrupt handler, snd_mpu401_uart_interrupt()) and fix it, too. Takashi ------------------------------------------------------- This SF.Net email is sponsored by BEA Weblogic Workshop FREE Java Enterprise J2EE developer tools! Get your free copy of BEA WebLogic Workshop 8.1 today. http://ads.osdn.com/?ad_id=5047&alloc_id=10808&op=click