Messages in this thread Patch in this message |  | | Date | Sun, 25 Nov 2001 08:53:49 -0800 (PST) | From | Alex Davis <> | Subject | Re: change to fs/proc/inode.c breaks ALSA drivers |
| |
Thanks for the info Mike. But can anyone answer why inode.c was changed in the first place??
--- Mike Maravillo <mike.maravillo@q-linux.com> wrote: > On Sat, Nov 24, 2001 at 07:24:47PM -0800, Alex Davis > wrote: > > > > Somewhere between 2.4.15pre6 and 2.4.15 final, > fs/proc/inode.c > > was modified. The change causes all the devices > files that ALSA > > creates in /proc/asound/dev to have a major and > minor number of > > zero. I'm sending a patch to revert the file back > to what it > > was in 2.4.15pre5. > > This change present on alsa-driver cvs fixed the > problem on mine, > at least. > > diff -urN --exclude=CVS > alsa-driver-0.5.12/kernel/info.c > alsa-driver/kernel/info.c > --- alsa-driver-0.5.12/kernel/info.c Wed Jun 28 > 02:02:03 2000 > +++ alsa-driver/kernel/info.c Wed Nov 21 23:28:35 > 2001 > @@ -897,7 +897,9 @@ > if (p) { > snd_info_device_entry_prepare(p, entry); > #ifdef LINUX_2_3 > - p->proc_fops = &snd_fops; > + /* we should not set this - at least * on 2.4.14 > or later it causes > + problems! */ > + /* p->proc_fops = &snd_fops; */ > #else > p->ops = &snd_info_device_inode_operations; > #endif > > -- > .--. Michael J. Maravillo > office://+63.2.894.3592/ > ( () ) Q Linux Solutions, Inc. > mobile://+63.917.897.0919/ > `--\\ A Philippine Open Source Solutions Co. http://www.q-linux.com/
__________________________________________________ Do You Yahoo!? Yahoo! GeoCities - quick and easy web site hosting, just $8.95/month. http://geocities.yahoo.com/ps/info1 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |