Re: [patch 4/4] make pr_debug() dynamic - update docs

2008-02-08 Thread Jason Baron
On Thu, Feb 07, 2008 at 01:26:04PM -0800, Randy Dunlap wrote:
> > --- a/Documentation/kernel-parameters.txt
> > +++ b/Documentation/kernel-parameters.txt
> > @@ -1494,6 +1494,11 @@ and is between 256 and 4096 characters. It is 
> > defined in the file
> > autoconfiguration.
> > Ranges are in pairs (memory base and size).
> >  
> > +   pr_debug
> > +   Enables pr_debug() calls if the immediate
> > +   infrastructure has been enabled. These can also be
> > +   switched on/off via /proc/sys/debug/pr_debug.
> 
> Is this actually Enables/disables?
> I.e., is this actually pr_debug= ?
> If so, please document that also.  If not, why not?
> 

The intention is that pr_debug is off by default. If you pass 'pr_debug' at the
command line, it is turned on early in the boot process. I orginally 
implemented it as pr_debug=, but I believe just the string is simpler.

thanks,

-Jason


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [patch 4/4] make pr_debug() dynamic - update docs

2008-02-07 Thread Randy Dunlap
On Thu, 7 Feb 2008 16:13:49 -0500 Jason Baron wrote:

> 
> -add documentation about pr_debug kernel-parameters.txt
> 
> Signed-off-by: Jason Baron <[EMAIL PROTECTED]>
> ---
> 
>  Documentation/kernel-parameters.txt |5 +
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> 
> diff --git a/Documentation/kernel-parameters.txt 
> b/Documentation/kernel-parameters.txt
> index cf38689..6d524bb 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -1494,6 +1494,11 @@ and is between 256 and 4096 characters. It is defined 
> in the file
>   autoconfiguration.
>   Ranges are in pairs (memory base and size).
>  
> + pr_debug
> + Enables pr_debug() calls if the immediate
> + infrastructure has been enabled. These can also be
> + switched on/off via /proc/sys/debug/pr_debug.

Is this actually Enables/disables?
I.e., is this actually pr_debug= ?
If so, please document that also.  If not, why not?

>   print-fatal-signals=
>   [KNL] debug: print fatal signals
>   print-fatal-signals=1: print segfault info to



---
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/