From alsa-devel-owner@alsa.jcu.cz  Wed Jun 10 12:07:45 1998
Received: from entry.jcu.cz (perex@entry.jcu.cz [160.217.1.111])
	by marvin.jcu.cz (8.8.8/8.8.8) with SMTP id MAA13754;
	Wed, 10 Jun 1998 12:03:20 +0200
Date: Wed, 10 Jun 1998 12:03:20 +0200 (MET DST)
From: Jaroslav Kysela <perex@jcu.cz>
To: Frank van de Pol <F.K.W.van.de.Pol@inter.nl.net>
cc: alsa-devel@alsa.jcu.cz
Subject: Re: Usage of snd_info_create_entry()
In-Reply-To: <199806092041.WAA04237@obelix.fvdpol.inter.nl.net>
Message-ID: <Pine.LNX.3.96.980610115358.457C-100000@entry.jcu.cz>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Reply-To: alsa-devel@alsa.jcu.cz
Sender: alsa-devel-owner@alsa.jcu.cz
Precedence: list

On Tue, 9 Jun 1998, Frank van de Pol wrote:

> 
> Hi,
> 
> I've been hacking with the sequencer stuff last weekend, and beefed up a
> test program to check feasibility of creating the client connections. When
> doing so I noticed a weird behavior when registering proc entries in
> /proc/sound using the snd_info_create_entry call:
> 
> Registering works fine, reading from the just created /proc/sound/seqinfo
> works also fine. 
> 
> When I unload the module, the entry is not removed from /proc/sound, even
> though I call snd_info_unregister() when the module is leaving. 
> 
> Next time the module is loaded a new entry is created, and reading from the
> file still works, so the 'fresh' one is used.....
> 
> This is how my /proc/sound looks at this moment:
> lrwxrwxrwx   1 root     root            0 Jun  9 22:36 0 -> card1/
> dr-xr-xr-x   1 root     root            0 Jun  9 22:36 card1/
> -r--r--r--   1 root     root            0 Jun  9 22:36 cards
> -r--r--r--   1 root     root            0 Jun  9 22:36 devices
> -r--r--r--   1 root     root            0 Jun  9 22:36 meminfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 oss-devices
> -r--r--r--   1 root     root            0 Jun  9 22:36 seqinfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 seqinfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 seqinfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 seqinfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 seqinfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 seqinfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 seqinfo
> -r--r--r--   1 root     root            0 Jun  9 22:36 version

Oops... Here is patch:

--- kernel/info.c.old  Mon May 25 14:06:47 1998
+++ kernel/info.c      Wed Jun 10 11:52:47 1998
@@ -1109,7 +1109,7 @@
   if ( !entry || !entry -> p ) return -EINVAL;
   snd_mutex_down_static( info );
   if ( !entry -> card ) {
-    proc_unregister( &proc_root, entry -> p -> low_ino );
+    proc_unregister( snd_proc_root, entry -> p -> low_ino );
     if ( entry == (e = snd_info_entries) ) {
       snd_info_entries = snd_info_entries -> next;
     } else {

I uploaded alsadriver-0.1.4pre1.patch to alsa ftp site. This patch
contains fixes for RAT which should now work again and above patch.

						Jaroslav

-----
Jaroslav Kysela <perex@jcu.cz>
Academic Computer Centre, University of South Bohemia
Branisovska 31, C. Budejovice, CZ-370 05 Czech Republic



