[Nouveau] [PATCH] drm/nouveau: add default debug level override
Ben Skeggs
skeggsb at gmail.com
Wed Sep 12 19:09:20 PDT 2012
On Thu, Sep 13, 2012 at 12:50:06AM +0200, Marcin Slusarz wrote:
> Useful when we don't know which subsystem is failing.
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.
>
> Signed-off-by: Marcin Slusarz <marcin.slusarz at gmail.com>
> ---
> drivers/gpu/drm/nouveau/core/core/option.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/nouveau/core/core/option.c b/drivers/gpu/drm/nouveau/core/core/option.c
> index 62a432e..fde06b0 100644
> --- a/drivers/gpu/drm/nouveau/core/core/option.c
> +++ b/drivers/gpu/drm/nouveau/core/core/option.c
> @@ -22,9 +22,16 @@
> * Authors: Ben Skeggs
> */
>
> +#include <linux/module.h>
> #include <core/option.h>
> #include <core/debug.h>
>
> +MODULE_PARM_DESC(debug_level, "nv_printk level (0-"
> + __stringify(CONFIG_NOUVEAU_DEBUG) ", default "
> + __stringify(CONFIG_NOUVEAU_DEBUG_DEFAULT) ")");
> +static int nouveau_debug_level = CONFIG_NOUVEAU_DEBUG_DEFAULT;
> +module_param_named(debug_level, nouveau_debug_level, int, 0400);
> +
> /* compares unterminated string 'str' with zero-terminated string 'cmp' */
> static inline int
> strncasecmpz(const char *str, const char *cmp, size_t len)
> @@ -86,7 +93,7 @@ nouveau_boolopt(const char *optstr, const char *opt, bool value)
> int
> nouveau_dbgopt(const char *optstr, const char *sub)
> {
> - int mode = 1, level = CONFIG_NOUVEAU_DEBUG_DEFAULT;
> + int mode = 1, level = nouveau_debug_level;
>
> while (optstr) {
> int len = strcspn(optstr, ",=");
> --
> 1.7.12
>
More information about the Nouveau
mailing list