Re: [PATCH] devfs (1/7) - fix compilation

From: Christoph Hellwig (hch@lst.de)
Date: Fri Apr 18 2003 - 11:21:01 EST


On Fri, Apr 18, 2003 at 09:19:00AM -0700, Linus Torvalds wrote:
>
> On Fri, 18 Apr 2003, Christoph Hellwig wrote:
> > @@ -1456,8 +1455,8 @@
> > dev_t devnum = 0, dev = MKDEV(major, minor);
> > struct devfs_entry *de;
> >
> > - if (flags)
> > - printk(KERN_ERR "%s called with flags != 0, please fix!\n");
> > + /* we don't accept any flags anymore. prototype will change soon. */
> > + BUG_ON(flags);
>
> PLEASE don't use BUG_ON() except for conditions that you really cannot
> continue from. It's damn impolite (and it makes debugging impossible) to
> kill the kernel startup if somebody has a unconverted driver or similar.

Ok. Is this one better?

diff -Nru a/fs/devfs/base.c b/fs/devfs/base.c
--- a/fs/devfs/base.c Fri Apr 18 15:56:18 2003
+++ b/fs/devfs/base.c Fri Apr 18 15:56:18 2003
@@ -1377,7 +1377,6 @@
  * @uid: The user ID.
  * @gid: The group ID.
  * @fs_info: The filesystem info.
- * @atomic: If TRUE, an atomic allocation is required.
  *
  * Returns %TRUE if an event was queued and devfsd woken up, else %FALSE.
  */
@@ -1423,7 +1422,7 @@
 static void devfsd_notify (struct devfs_entry *de,unsigned short type)
 {
         devfsd_notify_de(de, type, de->mode, current->euid,
- current->egid, &fs_info, 0);
+ current->egid, &fs_info);
 }
 
 
@@ -1456,8 +1455,8 @@
     dev_t devnum = 0, dev = MKDEV(major, minor);
     struct devfs_entry *de;
 
- if (flags)
- printk(KERN_ERR "%s called with flags != 0, please fix!\n");
+ /* we don't accept any flags anymore. prototype will change soon. */
+ WARN_ON(flags);
 
     if (name == NULL)
     {
-
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/



This archive was generated by hypermail 2b29 : Wed Apr 23 2003 - 22:00:24 EST