[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1355854204.30366.11.camel@joe-AO722>
Date: Tue, 18 Dec 2012 10:10:04 -0800
From: Joe Perches <joe@...ches.com>
To: Steve French <smfrench@...il.com>
Cc: Borislav Petkov <bp@...en8.de>, Jeff Layton <jlayton@...hat.com>,
LKML <linux-kernel@...r.kernel.org>,
Steve French <sfrench@...ba.org>, linux-cifs@...r.kernel.org,
Theodore Tso <tytso@....edu>, Al Viro <viro@...IV.linux.org.uk>
Subject: vfs trace points?: was: cifs: Do not enable debugging code by
default
On Tue, 2012-12-18 at 09:33 -0600, Steve French wrote:
> On Tue, Dec 18, 2012 at 9:19 AM, Borislav Petkov <bp@...en8.de> wrote:
> > On Tue, Dec 18, 2012 at 10:07:02AM -0500, Jeff Layton wrote:
> >> Yeah, not many do, but presumably they'll set it once and forget about
> >> it. Once someone straightens them out, they should rarely break.
> >>
> >> OTOH, the value of defaulting to "N" is pretty low here. Maybe it
> >> would just be best to just drop this patch and have the embedded folks
> >> turn this off on their own.
> >
> > Yeah, sure.
>
> I am ok with disabling it for embedded, but probably easier to drop
> it. The real benefit will come when someone familiar with dynamic
> trace points comes up with some good examples for file system(s) and
> the VFS so we can reduce our dependency on static tracing.
I'm fine with requiring embedded to turn this off
where appropriate.
It seems that keeping cifs KERN_DEBUG output for
normal kernels is pretty cost free. 120KB ain't much
these days.
I do think that Al Viro & Ted Ts'o should be pinged
about their vfs tracing thoughts. (cc'd)
MIT seems to bounce my emails, so maybe you could ask
Ted directly.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists