* [PATCH] drm: Avoid calling dev_printk(.dev = NULL)
@ 2016-08-19 7:37 Chris Wilson
2016-08-19 11:33 ` Sean Paul
0 siblings, 1 reply; 3+ messages in thread
From: Chris Wilson @ 2016-08-19 7:37 UTC (permalink / raw)
To: dri-devel
Since dev_printk likes to print "(NULL device *):" when passed in a NULL
pointer, we have to manually call printk() ourselves.
Fixes: c4e68a583202 ("drm: Introduce DRM_DEV_* log messages")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Eric Engestrom <eric.engestrom@imgtec.com>
Cc: Sean Paul <seanpaul@chromium.org>
---
drivers/gpu/drm/drm_drv.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
index a7f628298365..acf6a5f38920 100644
--- a/drivers/gpu/drm/drm_drv.c
+++ b/drivers/gpu/drm/drm_drv.c
@@ -79,8 +79,11 @@ void drm_dev_printk(const struct device *dev, const char *level,
vaf.fmt = format;
vaf.va = &args;
- dev_printk(level, dev, DRM_PRINTK_FMT, function_name, prefix,
- &vaf);
+ if (dev)
+ dev_printk(level, dev, DRM_PRINTK_FMT, function_name, prefix,
+ &vaf);
+ else
+ printk("%s" DRM_PRINTK_FMT, level, function_name, prefix, &vaf);
va_end(args);
}
--
2.9.3
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm: Avoid calling dev_printk(.dev = NULL)
2016-08-19 7:37 [PATCH] drm: Avoid calling dev_printk(.dev = NULL) Chris Wilson
@ 2016-08-19 11:33 ` Sean Paul
2016-08-22 16:11 ` Sean Paul
0 siblings, 1 reply; 3+ messages in thread
From: Sean Paul @ 2016-08-19 11:33 UTC (permalink / raw)
To: Chris Wilson; +Cc: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 1301 bytes --]
On Aug 19, 2016 12:37 AM, "Chris Wilson" <chris@chris-wilson.co.uk> wrote:
>
> Since dev_printk likes to print "(NULL device *):" when passed in a NULL
> pointer, we have to manually call printk() ourselves.
>
> Fixes: c4e68a583202 ("drm: Introduce DRM_DEV_* log messages")
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Eric Engestrom <eric.engestrom@imgtec.com>
> Cc: Sean Paul <seanpaul@chromium.org>
Thanks for fixing this, Chris.
Reviewed-by: Sean Paul <seanpaul@chromium.org>
> ---
> drivers/gpu/drm/drm_drv.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
> index a7f628298365..acf6a5f38920 100644
> --- a/drivers/gpu/drm/drm_drv.c
> +++ b/drivers/gpu/drm/drm_drv.c
> @@ -79,8 +79,11 @@ void drm_dev_printk(const struct device *dev, const
char *level,
> vaf.fmt = format;
> vaf.va = &args;
>
> - dev_printk(level, dev, DRM_PRINTK_FMT, function_name, prefix,
> - &vaf);
> + if (dev)
> + dev_printk(level, dev, DRM_PRINTK_FMT, function_name,
prefix,
> + &vaf);
> + else
> + printk("%s" DRM_PRINTK_FMT, level, function_name, prefix,
&vaf);
>
> va_end(args);
> }
> --
> 2.9.3
>
[-- Attachment #1.2: Type: text/html, Size: 2056 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm: Avoid calling dev_printk(.dev = NULL)
2016-08-19 11:33 ` Sean Paul
@ 2016-08-22 16:11 ` Sean Paul
0 siblings, 0 replies; 3+ messages in thread
From: Sean Paul @ 2016-08-22 16:11 UTC (permalink / raw)
To: Chris Wilson; +Cc: dri-devel
On Fri, Aug 19, 2016 at 7:33 AM, Sean Paul <seanpaul@chromium.org> wrote:
> On Aug 19, 2016 12:37 AM, "Chris Wilson" <chris@chris-wilson.co.uk> wrote:
>>
>> Since dev_printk likes to print "(NULL device *):" when passed in a NULL
>> pointer, we have to manually call printk() ourselves.
>>
>> Fixes: c4e68a583202 ("drm: Introduce DRM_DEV_* log messages")
>> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>> Cc: Eric Engestrom <eric.engestrom@imgtec.com>
>> Cc: Sean Paul <seanpaul@chromium.org>
>
> Thanks for fixing this, Chris.
>
> Reviewed-by: Sean Paul <seanpaul@chromium.org>
Applied to drm-misc
>
>> ---
>> drivers/gpu/drm/drm_drv.c | 7 +++++--
>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
>> index a7f628298365..acf6a5f38920 100644
>> --- a/drivers/gpu/drm/drm_drv.c
>> +++ b/drivers/gpu/drm/drm_drv.c
>> @@ -79,8 +79,11 @@ void drm_dev_printk(const struct device *dev, const
>> char *level,
>> vaf.fmt = format;
>> vaf.va = &args;
>>
>> - dev_printk(level, dev, DRM_PRINTK_FMT, function_name, prefix,
>> - &vaf);
>> + if (dev)
>> + dev_printk(level, dev, DRM_PRINTK_FMT, function_name,
>> prefix,
>> + &vaf);
>> + else
>> + printk("%s" DRM_PRINTK_FMT, level, function_name, prefix,
>> &vaf);
>>
>> va_end(args);
>> }
>> --
>> 2.9.3
>>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-22 16:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-19 7:37 [PATCH] drm: Avoid calling dev_printk(.dev = NULL) Chris Wilson
2016-08-19 11:33 ` Sean Paul
2016-08-22 16:11 ` Sean Paul
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.