On Thursday, September 13, 2012, Marcin Slusarz wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Sep 13, 2012 at 12:09:20PM +1000, Ben Skeggs wrote:<br>
> On Thu, Sep 13, 2012 at 12:50:06AM +0200, Marcin Slusarz wrote:<br>
> > Useful when we don't know which subsystem is failing.<br>
> This is really unnecessary unless I'm missing something. You can pass nouveau.debug=LEVEL (without specifying a subdev name) to set the default already.<br>
<br>
Does not work for me: <a href="http://bpaste.net/show/45252/" target="_blank">http://bpaste.net/show/45252/</a></blockquote><div>nouveau.debug=paranoia ;)</div><div><br></div><div>As mentioned previously, i'll document this somewhere.</div>
<div> <span></span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
(ignore gpio debugging messages - I'm looking for suspend regression)<br>
<br>
> ><br>
> > Signed-off-by: Marcin Slusarz <<a href="javascript:;" onclick="_e(event, 'cvml', 'marcin.slusarz@gmail.com')">marcin.slusarz@gmail.com</a>><br>
> > ---<br>
> > drivers/gpu/drm/nouveau/core/core/option.c | 9 ++++++++-<br>
> > 1 file changed, 8 insertions(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/drivers/gpu/drm/nouveau/core/core/option.c b/drivers/gpu/drm/nouveau/core/core/option.c<br>
> > index 62a432e..fde06b0 100644<br>
> > --- a/drivers/gpu/drm/nouveau/core/core/option.c<br>
> > +++ b/drivers/gpu/drm/nouveau/core/core/option.c<br>
> > @@ -22,9 +22,16 @@<br>
> > * Authors: Ben Skeggs<br>
> > */<br>
> ><br>
> > +#include <linux/module.h><br>
> > #include <core/option.h><br>
> > #include <core/debug.h><br>
> ><br>
> > +MODULE_PARM_DESC(debug_level, "nv_printk level (0-"<br>
> > + __stringify(CONFIG_NOUVEAU_DEBUG) ", default "<br>
> > + __stringify(CONFIG_NOUVEAU_DEBUG_DEFAULT) ")");<br>
> > +static int nouveau_debug_level = CONFIG_NOUVEAU_DEBUG_DEFAULT;<br>
> > +module_param_named(debug_level, nouveau_debug_level, int, 0400);<br>
> > +<br>
> > /* compares unterminated string 'str' with zero-terminated string 'cmp' */<br>
> > static inline int<br>
> > strncasecmpz(const char *str, const char *cmp, size_t len)<br>
> > @@ -86,7 +93,7 @@ nouveau_boolopt(const char *optstr, const char *opt, bool value)<br>
> > int<br>
> > nouveau_dbgopt(const char *optstr, const char *sub)<br>
> > {<br>
> > - int mode = 1, level = CONFIG_NOUVEAU_DEBUG_DEFAULT;<br>
> > + int mode = 1, level = nouveau_debug_level;<br>
> ><br>
> > while (optstr) {<br>
> > int len = strcspn(optstr, ",=");<br>
> > --<br>
> > 1.7.12<br>
> ><br>
</blockquote>