Messages in this thread Patch in this message | | | Date | Fri, 18 Apr 2003 18:21:01 +0200 | From | Christoph Hellwig <> | Subject | Re: [PATCH] devfs (1/7) - fix compilation |
| |
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/
| |