lkml.org 
[lkml]   [1998]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: knfsd problem in late 2.1
ak@muc.de wrote:
>
> Ok, this patch fixes knfsd. It does the following:
>
> - Changes all internal interfaces to use kdev_t (including the
> nfs filehandle cookie).
> - Convert dev_t to kdev_t in all interfaces to usermode (nfsctl()).
> The only exception is in the nfsfh cookie that is passed for a mount
> request, but mountd should treat that as opaque anyways so it should
> be no problem.
> - Add some KERN_DEBUGs to printk()s. There are still some missing, and
> far too many of them too (currently when I kill all nfsds in my shutdown
> script I get half a screenfull of crap)
> - Make linux/sunrpc/sched.h compile without -D__KERNEL__ [needed by
> the user tools].

Hi Andi,

Thanks for going through and making the kdev_t changes.

With regards to the KERN_DEBUG changes, most of them are OK, but I don't
agree with changing the classification of a few of them. I often mark
unexpected error conditions with a ! at the end, and these should be
classified as KERN_ERR to make sure they get logged (and hopefully
reported.)

> if (!fhp->fh_dverified) {
> - printk("fh_update: fh not verified!\n");
> + printk(KERN_DEBUG "fh_update: fh not verified!\n");
> goto out;

> if (!inode) {
> - printk("fh_update: %s/%s still negative!\n",
> + printk(KERN_DEBUG "fh_update: %s/%s still negative!\n",
> dentry->d_parent->d_name.name, dentry->d_name.name);

> fhp->fh_dverified = 0;
> if (!dentry->d_count) {
> - printk("fh_put: %s/%s has d_count 0!\n",
> + printk(KERN_DEBUG "fh_put: %s/%s has d_count 0!\n",
> dentry->d_parent->d_name.name, dentry->d_name.name);
> return;

Regards,
Bill

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu

\
 
 \ /
  Last update: 2005-03-22 13:42    [W:2.104 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site