* [RFC 1/6] drm/armada: Simplify drm_dev_init error log
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
@ 2018-01-24 16:18 ` Tvrtko Ursulin
2018-01-24 17:44 ` Russell King - ARM Linux
2018-01-24 16:18 ` [RFC 2/6] drm: Introduce unexported DRM_LOG_NAME define for logging Tvrtko Ursulin
` (7 subsequent siblings)
8 siblings, 1 reply; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-24 16:18 UTC (permalink / raw)
To: Intel-gfx; +Cc: David Airlie, Russell King, dri-devel
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
dev_err will log the device in question and since there is a single caller
to drm_dev_init inside this driver, the drm prefix and the function name
can both also be safely dropped.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Russell King <linux@armlinux.org.uk>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/armada/armada_drv.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/armada/armada_drv.c b/drivers/gpu/drm/armada/armada_drv.c
index 4b11b6b52f1d..76a06bf6ebcd 100644
--- a/drivers/gpu/drm/armada/armada_drv.c
+++ b/drivers/gpu/drm/armada/armada_drv.c
@@ -115,8 +115,7 @@ static int armada_drm_bind(struct device *dev)
ret = drm_dev_init(&priv->drm, &armada_drm_driver, dev);
if (ret) {
- dev_err(dev, "[" DRM_NAME ":%s] drm_dev_init failed: %d\n",
- __func__, ret);
+ dev_err(dev, "drm_dev_init failed: %d\n", ret);
kfree(priv);
return ret;
}
--
2.14.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [RFC 1/6] drm/armada: Simplify drm_dev_init error log
2018-01-24 16:18 ` [RFC 1/6] drm/armada: Simplify drm_dev_init error log Tvrtko Ursulin
@ 2018-01-24 17:44 ` Russell King - ARM Linux
0 siblings, 0 replies; 16+ messages in thread
From: Russell King - ARM Linux @ 2018-01-24 17:44 UTC (permalink / raw)
To: Tvrtko Ursulin; +Cc: David Airlie, Intel-gfx, dri-devel
On Wed, Jan 24, 2018 at 04:18:16PM +0000, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> dev_err will log the device in question and since there is a single caller
> to drm_dev_init inside this driver, the drm prefix and the function name
> can both also be safely dropped.
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Acked-by: Russell King <rmk+kernel@armlinux.org.uk>
Thanks.
> Cc: David Airlie <airlied@linux.ie>
> Cc: dri-devel@lists.freedesktop.org
> ---
> drivers/gpu/drm/armada/armada_drv.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/armada/armada_drv.c b/drivers/gpu/drm/armada/armada_drv.c
> index 4b11b6b52f1d..76a06bf6ebcd 100644
> --- a/drivers/gpu/drm/armada/armada_drv.c
> +++ b/drivers/gpu/drm/armada/armada_drv.c
> @@ -115,8 +115,7 @@ static int armada_drm_bind(struct device *dev)
>
> ret = drm_dev_init(&priv->drm, &armada_drm_driver, dev);
> if (ret) {
> - dev_err(dev, "[" DRM_NAME ":%s] drm_dev_init failed: %d\n",
> - __func__, ret);
> + dev_err(dev, "drm_dev_init failed: %d\n", ret);
> kfree(priv);
> return ret;
> }
> --
> 2.14.1
>
--
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 8.8Mbps down 630kbps up
According to speedtest.net: 8.21Mbps down 510kbps up
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 16+ messages in thread
* [RFC 2/6] drm: Introduce unexported DRM_LOG_NAME define for logging
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
2018-01-24 16:18 ` [RFC 1/6] drm/armada: Simplify drm_dev_init error log Tvrtko Ursulin
@ 2018-01-24 16:18 ` Tvrtko Ursulin
2018-01-24 16:18 ` [RFC 3/6] drm/i915: Give our log messages our name Tvrtko Ursulin
` (6 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-24 16:18 UTC (permalink / raw)
To: Intel-gfx; +Cc: David Airlie, dri-devel, Daniel Vetter
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Instead of having DRM_NAME which is defined in the uAPI headers used for
kernel log prefix messages, introduct a new DRM_LOG_NAME define defined
privately in kernel space.
Leave the DRM_NAME around, but mark it as obsolete in case there is some
userspace code depending on its presence.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Gustavo Padovan <gustavo@padovan.org>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/drm_print.c | 6 +++---
drivers/gpu/drm/i915/i915_drv.c | 2 +-
include/drm/drm_print.h | 9 ++++++++-
include/uapi/drm/drm.h | 2 +-
4 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/drm_print.c b/drivers/gpu/drm/drm_print.c
index 781518fd88e3..27244f11c9a3 100644
--- a/drivers/gpu/drm/drm_print.c
+++ b/drivers/gpu/drm/drm_print.c
@@ -38,7 +38,7 @@ EXPORT_SYMBOL(__drm_printfn_seq_file);
void __drm_printfn_info(struct drm_printer *p, struct va_format *vaf)
{
- dev_info(p->arg, "[" DRM_NAME "] %pV", vaf);
+ dev_info(p->arg, "[" DRM_LOG_NAME "] %pV", vaf);
}
EXPORT_SYMBOL(__drm_printfn_info);
@@ -63,7 +63,7 @@ void drm_printf(struct drm_printer *p, const char *f, ...)
}
EXPORT_SYMBOL(drm_printf);
-#define DRM_PRINTK_FMT "[" DRM_NAME ":%s]%s %pV"
+#define DRM_PRINTK_FMT "[" DRM_LOG_NAME ":%s]%s %pV"
void drm_dev_printk(const struct device *dev, const char *level,
unsigned int category, const char *function_name,
@@ -102,7 +102,7 @@ void drm_printk(const char *level, unsigned int category,
vaf.fmt = format;
vaf.va = &args;
- printk("%s" "[" DRM_NAME ":%ps]%s %pV",
+ printk("%s" "[" DRM_LOG_NAME ":%ps]%s %pV",
level, __builtin_return_address(0),
strcmp(level, KERN_ERR) == 0 ? " *ERROR*" : "", &vaf);
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index 1fbe37889d92..915e43aacaa5 100644
--- a/drivers/gpu/drm/i915/i915_drv.c
+++ b/drivers/gpu/drm/i915/i915_drv.c
@@ -94,7 +94,7 @@ __i915_printk(struct drm_i915_private *dev_priv, const char *level,
vaf.fmt = fmt;
vaf.va = &args;
- dev_printk(level, kdev, "[" DRM_NAME ":%ps] %pV",
+ dev_printk(level, kdev, "[" DRM_LOG_NAME ":%ps] %pV",
__builtin_return_address(0), &vaf);
if (is_error && !shown_bug_once) {
diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
index 2a4a42e59a47..d6901128faf2 100644
--- a/include/drm/drm_print.h
+++ b/include/drm/drm_print.h
@@ -60,6 +60,13 @@
* }
*/
+/**
+ * String output in log messages - can be overriden by driver code.
+ */
+#ifndef DRM_LOG_NAME
+#define DRM_LOG_NAME "drm"
+#endif
+
/**
* struct drm_printer - drm output "stream"
*
@@ -208,7 +215,7 @@ void drm_printk(const char *level, unsigned int category,
#define _DRM_PRINTK(once, level, fmt, ...) \
do { \
- printk##once(KERN_##level "[" DRM_NAME "] " fmt, \
+ printk##once(KERN_##level "[" DRM_LOG_NAME "] " fmt, \
##__VA_ARGS__); \
} while (0)
diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h
index 6fdff5945c8a..7b512673c220 100644
--- a/include/uapi/drm/drm.h
+++ b/include/uapi/drm/drm.h
@@ -69,7 +69,7 @@ typedef unsigned long drm_handle_t;
extern "C" {
#endif
-#define DRM_NAME "drm" /**< Name in kernel, /dev, and /proc */
+#define DRM_NAME "drm" /**< Deprecated. */
#define DRM_MIN_ORDER 5 /**< At least 2^5 bytes = 32 bytes */
#define DRM_MAX_ORDER 22 /**< Up to 2^22 bytes = 4MB */
#define DRM_RAM_PERCENT 10 /**< How much system ram can we lock? */
--
2.14.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 16+ messages in thread* [RFC 3/6] drm/i915: Give our log messages our name
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
2018-01-24 16:18 ` [RFC 1/6] drm/armada: Simplify drm_dev_init error log Tvrtko Ursulin
2018-01-24 16:18 ` [RFC 2/6] drm: Introduce unexported DRM_LOG_NAME define for logging Tvrtko Ursulin
@ 2018-01-24 16:18 ` Tvrtko Ursulin
2018-01-26 13:10 ` [Intel-gfx] " Michal Wajdeczko
2018-01-24 16:18 ` [RFC 4/6] drm: Respect driver set DRM_LOG_NAME in drm_printk Tvrtko Ursulin
` (5 subsequent siblings)
8 siblings, 1 reply; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-24 16:18 UTC (permalink / raw)
To: Intel-gfx; +Cc: dri-devel
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Define DRM_LOG_NAME to i915 so that the log messages we output change
from:
[drm] RC6 on
to:
[i915] RC6 on
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/i915/i915_drv.h | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 8333692dac5a..2b48a7d2d129 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -30,6 +30,11 @@
#ifndef _I915_DRV_H_
#define _I915_DRV_H_
+#ifdef DRM_LOG_NAME
+#undef DRM_LOG_NAME
+#endif
+#define DRM_LOG_NAME "i915"
+
#include <uapi/drm/i915_drm.h>
#include <uapi/drm/drm_fourcc.h>
--
2.14.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [Intel-gfx] [RFC 3/6] drm/i915: Give our log messages our name
2018-01-24 16:18 ` [RFC 3/6] drm/i915: Give our log messages our name Tvrtko Ursulin
@ 2018-01-26 13:10 ` Michal Wajdeczko
2018-01-30 10:53 ` Tvrtko Ursulin
0 siblings, 1 reply; 16+ messages in thread
From: Michal Wajdeczko @ 2018-01-26 13:10 UTC (permalink / raw)
To: Intel-gfx, Tvrtko Ursulin; +Cc: dri-devel
On Wed, 24 Jan 2018 17:18:18 +0100, Tvrtko Ursulin <tursulin@ursulin.net>
wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> Define DRM_LOG_NAME to i915 so that the log messages we output change
> from:
>
> [drm] RC6 on
>
> to:
>
> [i915] RC6 on
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Cc: dri-devel@lists.freedesktop.org
> ---
> drivers/gpu/drm/i915/i915_drv.h | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.h
> b/drivers/gpu/drm/i915/i915_drv.h
> index 8333692dac5a..2b48a7d2d129 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -30,6 +30,11 @@
> #ifndef _I915_DRV_H_
> #define _I915_DRV_H_
> +#ifdef DRM_LOG_NAME
> +#undef DRM_LOG_NAME
> +#endif
> +#define DRM_LOG_NAME "i915"
Maybe better option would be to add this definition to our Makefile
subdir-ccflags-y += -DDRM_LOG_NAME=\"i915\"
Note that drm_print.h (patch 2/6) already has proper #ifndef
Michal
> +
> #include <uapi/drm/i915_drm.h>
> #include <uapi/drm/drm_fourcc.h>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [RFC 3/6] drm/i915: Give our log messages our name
2018-01-26 13:10 ` [Intel-gfx] " Michal Wajdeczko
@ 2018-01-30 10:53 ` Tvrtko Ursulin
0 siblings, 0 replies; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-30 10:53 UTC (permalink / raw)
To: Michal Wajdeczko, Intel-gfx, Tvrtko Ursulin; +Cc: dri-devel
On 26/01/2018 13:10, Michal Wajdeczko wrote:
> On Wed, 24 Jan 2018 17:18:18 +0100, Tvrtko Ursulin
> <tursulin@ursulin.net> wrote:
>
>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>
>> Define DRM_LOG_NAME to i915 so that the log messages we output change
>> from:
>>
>> [drm] RC6 on
>>
>> to:
>>
>> [i915] RC6 on
>>
>> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>> Cc: dri-devel@lists.freedesktop.org
>> ---
>> drivers/gpu/drm/i915/i915_drv.h | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_drv.h
>> b/drivers/gpu/drm/i915/i915_drv.h
>> index 8333692dac5a..2b48a7d2d129 100644
>> --- a/drivers/gpu/drm/i915/i915_drv.h
>> +++ b/drivers/gpu/drm/i915/i915_drv.h
>> @@ -30,6 +30,11 @@
>> #ifndef _I915_DRV_H_
>> #define _I915_DRV_H_
>> +#ifdef DRM_LOG_NAME
>> +#undef DRM_LOG_NAME
>> +#endif
>> +#define DRM_LOG_NAME "i915"
>
> Maybe better option would be to add this definition to our Makefile
>
> subdir-ccflags-y += -DDRM_LOG_NAME=\"i915\"
>
> Note that drm_print.h (patch 2/6) already has proper #ifndef
So that is always guaranteed to be included first? Sounds attractive, yep.
Unfortunately series does not seem to improve things sufficiently to
garner any interest so I am reluctant to respin with that change.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 16+ messages in thread
* [RFC 4/6] drm: Respect driver set DRM_LOG_NAME in drm_printk
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
` (2 preceding siblings ...)
2018-01-24 16:18 ` [RFC 3/6] drm/i915: Give our log messages our name Tvrtko Ursulin
@ 2018-01-24 16:18 ` Tvrtko Ursulin
2018-01-24 16:18 ` [RFC 5/6] drm: Respect driver set DRM_LOG_NAME in drm_dev_printk Tvrtko Ursulin
` (4 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-24 16:18 UTC (permalink / raw)
To: Intel-gfx; +Cc: David Airlie, dri-devel, Daniel Vetter
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Since drm_printk is built inside the DRM core module, it will not respect
the potentially overriden by the driver DRM_LOG_NAME.
Make it tale the driver prefix and change it's wrappers so it is passed
in from the driver code. This makes the driver log messages correctly
identify from which driver they originated.
For instance i915, instead of:
[drm:edp_panel_off [i915]] Wait for panel power off time
would now log debug messages as:
[i915:edp_panel_off [i915]] Wait for panel power off time
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Gustavo Padovan <gustavo@padovan.org>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/drm_print.c | 6 +++---
drivers/gpu/drm/i915/intel_display.c | 3 ++-
include/drm/drm_print.h | 20 ++++++++++----------
3 files changed, 15 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/drm_print.c b/drivers/gpu/drm/drm_print.c
index 27244f11c9a3..90d38c830392 100644
--- a/drivers/gpu/drm/drm_print.c
+++ b/drivers/gpu/drm/drm_print.c
@@ -89,7 +89,7 @@ void drm_dev_printk(const struct device *dev, const char *level,
}
EXPORT_SYMBOL(drm_dev_printk);
-void drm_printk(const char *level, unsigned int category,
+void drm_printk(const char *level, unsigned int category, const char *driver,
const char *format, ...)
{
struct va_format vaf;
@@ -102,8 +102,8 @@ void drm_printk(const char *level, unsigned int category,
vaf.fmt = format;
vaf.va = &args;
- printk("%s" "[" DRM_LOG_NAME ":%ps]%s %pV",
- level, __builtin_return_address(0),
+ printk("%s[%s:%ps]%s %pV",
+ level, driver, __builtin_return_address(0),
strcmp(level, KERN_ERR) == 0 ? " *ERROR*" : "", &vaf);
va_end(args);
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index d585ce4c8732..51fea2eb1b2e 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -10973,7 +10973,8 @@ pipe_config_err(bool adjust, const char *name, const char *format, ...)
vaf.fmt = format;
vaf.va = &args;
- drm_printk(level, category, "mismatch in %s %pV", name, &vaf);
+ drm_printk(level, category, DRM_LOG_NAME, "mismatch in %s %pV", name,
+ &vaf);
va_end(args);
}
diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
index d6901128faf2..c6047de9cbcc 100644
--- a/include/drm/drm_print.h
+++ b/include/drm/drm_print.h
@@ -207,8 +207,8 @@ __printf(6, 7)
void drm_dev_printk(const struct device *dev, const char *level,
unsigned int category, const char *function_name,
const char *prefix, const char *format, ...);
-__printf(3, 4)
-void drm_printk(const char *level, unsigned int category,
+__printf(4, 5)
+void drm_printk(const char *level, unsigned int category, const char *driver,
const char *format, ...);
/* Macros to make printk easier */
@@ -243,7 +243,7 @@ void drm_printk(const char *level, unsigned int category,
drm_dev_printk(dev, KERN_ERR, DRM_UT_NONE, __func__, " *ERROR*",\
fmt, ##__VA_ARGS__)
#define DRM_ERROR(fmt, ...) \
- drm_printk(KERN_ERR, DRM_UT_NONE, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_ERR, DRM_UT_NONE, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
/**
* Rate limited error output. Like DRM_ERROR() but won't flood the log.
@@ -286,40 +286,40 @@ void drm_printk(const char *level, unsigned int category,
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_CORE, __func__, "", fmt, \
##args)
#define DRM_DEBUG(fmt, ...) \
- drm_printk(KERN_DEBUG, DRM_UT_CORE, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_DEBUG, DRM_UT_CORE, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_DRIVER(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_DRIVER, __func__, "", \
fmt, ##args)
#define DRM_DEBUG_DRIVER(fmt, ...) \
- drm_printk(KERN_DEBUG, DRM_UT_DRIVER, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_DEBUG, DRM_UT_DRIVER, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_KMS(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_KMS, __func__, "", fmt, \
##args)
#define DRM_DEBUG_KMS(fmt, ...) \
- drm_printk(KERN_DEBUG, DRM_UT_KMS, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_DEBUG, DRM_UT_KMS, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_PRIME(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_PRIME, __func__, "", \
fmt, ##args)
#define DRM_DEBUG_PRIME(fmt, ...) \
- drm_printk(KERN_DEBUG, DRM_UT_PRIME, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_DEBUG, DRM_UT_PRIME, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_ATOMIC(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_ATOMIC, __func__, "", \
fmt, ##args)
#define DRM_DEBUG_ATOMIC(fmt, ...) \
- drm_printk(KERN_DEBUG, DRM_UT_ATOMIC, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_DEBUG, DRM_UT_ATOMIC, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_VBL(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_VBL, __func__, "", fmt, \
##args)
#define DRM_DEBUG_VBL(fmt, ...) \
- drm_printk(KERN_DEBUG, DRM_UT_VBL, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_DEBUG, DRM_UT_VBL, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEBUG_LEASE(fmt, ...) \
- drm_printk(KERN_DEBUG, DRM_UT_LEASE, fmt, ##__VA_ARGS__)
+ drm_printk(KERN_DEBUG, DRM_UT_LEASE, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define _DRM_DEV_DEFINE_DEBUG_RATELIMITED(dev, level, fmt, args...) \
({ \
--
2.14.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 16+ messages in thread* [RFC 5/6] drm: Respect driver set DRM_LOG_NAME in drm_dev_printk
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
` (3 preceding siblings ...)
2018-01-24 16:18 ` [RFC 4/6] drm: Respect driver set DRM_LOG_NAME in drm_printk Tvrtko Ursulin
@ 2018-01-24 16:18 ` Tvrtko Ursulin
2018-01-24 16:18 ` [RFC 6/6] drm: Respect driver set DRM_LOG_NAME in drm_info_printer Tvrtko Ursulin
` (3 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-24 16:18 UTC (permalink / raw)
To: Intel-gfx; +Cc: David Airlie, dri-devel, Daniel Vetter
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Same as the previous patch did for drm_printk, allow for the logging
macros to pass in the driver set DRM_LOG_NAME for drm_dev_printk.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Gustavo Padovan <gustavo@padovan.org>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/drm_print.c | 12 ++++++------
include/drm/drm_print.h | 31 ++++++++++++++++---------------
2 files changed, 22 insertions(+), 21 deletions(-)
diff --git a/drivers/gpu/drm/drm_print.c b/drivers/gpu/drm/drm_print.c
index 90d38c830392..f97bfcb7d882 100644
--- a/drivers/gpu/drm/drm_print.c
+++ b/drivers/gpu/drm/drm_print.c
@@ -63,11 +63,10 @@ void drm_printf(struct drm_printer *p, const char *f, ...)
}
EXPORT_SYMBOL(drm_printf);
-#define DRM_PRINTK_FMT "[" DRM_LOG_NAME ":%s]%s %pV"
-
void drm_dev_printk(const struct device *dev, const char *level,
unsigned int category, const char *function_name,
- const char *prefix, const char *format, ...)
+ const char *prefix, const char *driver,
+ const char *format, ...)
{
struct va_format vaf;
va_list args;
@@ -80,10 +79,11 @@ void drm_dev_printk(const struct device *dev, const char *level,
vaf.va = &args;
if (dev)
- dev_printk(level, dev, DRM_PRINTK_FMT, function_name, prefix,
- &vaf);
+ dev_printk(level, dev, "[%s:%s]%s %pV", driver, function_name,
+ prefix, &vaf);
else
- printk("%s" DRM_PRINTK_FMT, level, function_name, prefix, &vaf);
+ printk("%s[%s:%s]%s %pV", level, driver, function_name, prefix,
+ &vaf);
va_end(args);
}
diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
index c6047de9cbcc..c4dfcd217bce 100644
--- a/include/drm/drm_print.h
+++ b/include/drm/drm_print.h
@@ -203,10 +203,11 @@ static inline struct drm_printer drm_debug_printer(const char *prefix)
#define DRM_UT_STATE 0x40
#define DRM_UT_LEASE 0x80
-__printf(6, 7)
+__printf(7, 8)
void drm_dev_printk(const struct device *dev, const char *level,
unsigned int category, const char *function_name,
- const char *prefix, const char *format, ...);
+ const char *prefix, const char *driver, const char *format,
+ ...);
__printf(4, 5)
void drm_printk(const char *level, unsigned int category, const char *driver,
const char *format, ...);
@@ -241,7 +242,7 @@ void drm_printk(const char *level, unsigned int category, const char *driver,
*/
#define DRM_DEV_ERROR(dev, fmt, ...) \
drm_dev_printk(dev, KERN_ERR, DRM_UT_NONE, __func__, " *ERROR*",\
- fmt, ##__VA_ARGS__)
+ DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_ERROR(fmt, ...) \
drm_printk(KERN_ERR, DRM_UT_NONE, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
@@ -264,8 +265,8 @@ void drm_printk(const char *level, unsigned int category, const char *driver,
DRM_DEV_ERROR_RATELIMITED(NULL, fmt, ##__VA_ARGS__)
#define DRM_DEV_INFO(dev, fmt, ...) \
- drm_dev_printk(dev, KERN_INFO, DRM_UT_NONE, __func__, "", fmt, \
- ##__VA_ARGS__)
+ drm_dev_printk(dev, KERN_INFO, DRM_UT_NONE, __func__, "", \
+ DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_INFO_ONCE(dev, fmt, ...) \
({ \
@@ -283,38 +284,38 @@ void drm_printk(const char *level, unsigned int category, const char *driver,
* @fmt: printf() like format string.
*/
#define DRM_DEV_DEBUG(dev, fmt, args...) \
- drm_dev_printk(dev, KERN_DEBUG, DRM_UT_CORE, __func__, "", fmt, \
- ##args)
+ drm_dev_printk(dev, KERN_DEBUG, DRM_UT_CORE, __func__, "", \
+ DRM_LOG_NAME, fmt, ##args)
#define DRM_DEBUG(fmt, ...) \
drm_printk(KERN_DEBUG, DRM_UT_CORE, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_DRIVER(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_DRIVER, __func__, "", \
- fmt, ##args)
+ DRM_LOG_NAME, fmt, ##args)
#define DRM_DEBUG_DRIVER(fmt, ...) \
drm_printk(KERN_DEBUG, DRM_UT_DRIVER, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_KMS(dev, fmt, args...) \
- drm_dev_printk(dev, KERN_DEBUG, DRM_UT_KMS, __func__, "", fmt, \
- ##args)
+ drm_dev_printk(dev, KERN_DEBUG, DRM_UT_KMS, __func__, "", \
+ DRM_LOG_NAME, fmt, ##args)
#define DRM_DEBUG_KMS(fmt, ...) \
drm_printk(KERN_DEBUG, DRM_UT_KMS, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_PRIME(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_PRIME, __func__, "", \
- fmt, ##args)
+ DRM_LOG_NAME, fmt, ##args)
#define DRM_DEBUG_PRIME(fmt, ...) \
drm_printk(KERN_DEBUG, DRM_UT_PRIME, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_ATOMIC(dev, fmt, args...) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_ATOMIC, __func__, "", \
- fmt, ##args)
+ DRM_LOG_NAME, fmt, ##args)
#define DRM_DEBUG_ATOMIC(fmt, ...) \
drm_printk(KERN_DEBUG, DRM_UT_ATOMIC, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
#define DRM_DEV_DEBUG_VBL(dev, fmt, args...) \
- drm_dev_printk(dev, KERN_DEBUG, DRM_UT_VBL, __func__, "", fmt, \
- ##args)
+ drm_dev_printk(dev, KERN_DEBUG, DRM_UT_VBL, __func__, "" \
+ DRM_LOG_NAME, fmt, ##args)
#define DRM_DEBUG_VBL(fmt, ...) \
drm_printk(KERN_DEBUG, DRM_UT_VBL, DRM_LOG_NAME, fmt, ##__VA_ARGS__)
@@ -328,7 +329,7 @@ void drm_printk(const char *level, unsigned int category, const char *driver,
DEFAULT_RATELIMIT_BURST); \
if (__ratelimit(&_rs)) \
drm_dev_printk(dev, KERN_DEBUG, DRM_UT_ ## level, \
- __func__, "", fmt, ##args); \
+ __func__, "", DRM_LOG_NAME, fmt, ##args);\
})
/**
--
2.14.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 16+ messages in thread* [RFC 6/6] drm: Respect driver set DRM_LOG_NAME in drm_info_printer
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
` (4 preceding siblings ...)
2018-01-24 16:18 ` [RFC 5/6] drm: Respect driver set DRM_LOG_NAME in drm_dev_printk Tvrtko Ursulin
@ 2018-01-24 16:18 ` Tvrtko Ursulin
2018-01-24 16:23 ` [Intel-gfx] [RFC v2 0/6] DRM logging tidy Chris Wilson
` (2 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-24 16:18 UTC (permalink / raw)
To: Intel-gfx; +Cc: David Airlie, dri-devel, Daniel Vetter
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Same as the previous patch did for drm_printk, allow for the logging
macros to pass in the driver set DRM_LOG_NAME for drm_info_printer.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Gustavo Padovan <gustavo@padovan.org>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/drm_print.c | 2 +-
include/drm/drm_print.h | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_print.c b/drivers/gpu/drm/drm_print.c
index f97bfcb7d882..1fcc08876f87 100644
--- a/drivers/gpu/drm/drm_print.c
+++ b/drivers/gpu/drm/drm_print.c
@@ -38,7 +38,7 @@ EXPORT_SYMBOL(__drm_printfn_seq_file);
void __drm_printfn_info(struct drm_printer *p, struct va_format *vaf)
{
- dev_info(p->arg, "[" DRM_LOG_NAME "] %pV", vaf);
+ dev_info(p->arg, "[%s] %pV", p->prefix, vaf);
}
EXPORT_SYMBOL(__drm_printfn_info);
diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
index c4dfcd217bce..1f613b49c8f2 100644
--- a/include/drm/drm_print.h
+++ b/include/drm/drm_print.h
@@ -139,6 +139,7 @@ static inline struct drm_printer drm_info_printer(struct device *dev)
struct drm_printer p = {
.printfn = __drm_printfn_info,
.arg = dev,
+ .prefix = DRM_LOG_NAME,
};
return p;
}
--
2.14.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [Intel-gfx] [RFC v2 0/6] DRM logging tidy
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
` (5 preceding siblings ...)
2018-01-24 16:18 ` [RFC 6/6] drm: Respect driver set DRM_LOG_NAME in drm_info_printer Tvrtko Ursulin
@ 2018-01-24 16:23 ` Chris Wilson
2018-01-24 16:48 ` Tvrtko Ursulin
2018-01-24 16:42 ` ✓ Fi.CI.BAT: success for DRM logging tidy (rev2) Patchwork
2018-01-24 20:12 ` ✗ Fi.CI.IGT: warning " Patchwork
8 siblings, 1 reply; 16+ messages in thread
From: Chris Wilson @ 2018-01-24 16:23 UTC (permalink / raw)
To: Tvrtko Ursulin, Intel-gfx; +Cc: dri-devel
Quoting Tvrtko Ursulin (2018-01-24 16:18:15)
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> This series tries to solve a few issues in the current DRM logging code to
> primarily make it clearer which messages belong to which driver.
>
> Main problem is that currently some logging functions allow individual drivers
> to override the log prefix (since they are defined as macros, or static
> inlines), while other hardcode the "drm" prefix into them due being situated in
> the DRM core modules.
>
> Another thing is that I noticed the DRM_NAME macro which is used for this is
> defined in the uAPI header and had a comment which looked outdated.
>
> Therefore I introduce a new define, called, DRM_LOG_NAME, this time defined
> internally in the kernel headers and not exported in the uAPI.
>
> I also refactored some logging functions to take this string as a parameter
> instead of hardcoding it.
>
> Individual drivers can then override this define to make DRM logging functions
> prefix their message with the respective driver prefix.
>
> End result in the case of the i915 driver looks like this:
>
> Old log:
>
> [drm] Found 128MB of eDRAM
> [drm:skl_enable_dc6 [i915]] Enabling DC6
>
> New log:
>
> [i915] Found 128MB of eDRAM
> [i915:skl_enable_dc6 [i915]] Enabling DC6
And still not conforming to the standard logging string. DRM_LOG should
be killed off as an anachronistic OS compat layer.
-Chris
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 16+ messages in thread* Re: [RFC v2 0/6] DRM logging tidy
2018-01-24 16:23 ` [Intel-gfx] [RFC v2 0/6] DRM logging tidy Chris Wilson
@ 2018-01-24 16:48 ` Tvrtko Ursulin
2018-01-25 11:32 ` Jani Nikula
0 siblings, 1 reply; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-24 16:48 UTC (permalink / raw)
To: Chris Wilson, Tvrtko Ursulin, Intel-gfx; +Cc: dri-devel
On 24/01/2018 16:23, Chris Wilson wrote:
> Quoting Tvrtko Ursulin (2018-01-24 16:18:15)
>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>
>> This series tries to solve a few issues in the current DRM logging code to
>> primarily make it clearer which messages belong to which driver.
>>
>> Main problem is that currently some logging functions allow individual drivers
>> to override the log prefix (since they are defined as macros, or static
>> inlines), while other hardcode the "drm" prefix into them due being situated in
>> the DRM core modules.
>>
>> Another thing is that I noticed the DRM_NAME macro which is used for this is
>> defined in the uAPI header and had a comment which looked outdated.
>>
>> Therefore I introduce a new define, called, DRM_LOG_NAME, this time defined
>> internally in the kernel headers and not exported in the uAPI.
>>
>> I also refactored some logging functions to take this string as a parameter
>> instead of hardcoding it.
>>
>> Individual drivers can then override this define to make DRM logging functions
>> prefix their message with the respective driver prefix.
>>
>> End result in the case of the i915 driver looks like this:
>>
>> Old log:
>>
>> [drm] Found 128MB of eDRAM
>> [drm:skl_enable_dc6 [i915]] Enabling DC6
>>
>> New log:
>>
>> [i915] Found 128MB of eDRAM
>> [i915:skl_enable_dc6 [i915]] Enabling DC6
>
> And still not conforming to the standard logging string. DRM_LOG should
What is the standard logging string? the dev_ one?
> be killed off as an anachronistic OS compat layer.
You mean only dev variants should be kept?
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [RFC v2 0/6] DRM logging tidy
2018-01-24 16:48 ` Tvrtko Ursulin
@ 2018-01-25 11:32 ` Jani Nikula
2018-01-25 13:38 ` [Intel-gfx] " Tvrtko Ursulin
0 siblings, 1 reply; 16+ messages in thread
From: Jani Nikula @ 2018-01-25 11:32 UTC (permalink / raw)
To: Tvrtko Ursulin, Chris Wilson, Tvrtko Ursulin, Intel-gfx; +Cc: dri-devel
On Wed, 24 Jan 2018, Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com> wrote:
> On 24/01/2018 16:23, Chris Wilson wrote:
>> Quoting Tvrtko Ursulin (2018-01-24 16:18:15)
>>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>>
>>> This series tries to solve a few issues in the current DRM logging code to
>>> primarily make it clearer which messages belong to which driver.
>>>
>>> Main problem is that currently some logging functions allow individual drivers
>>> to override the log prefix (since they are defined as macros, or static
>>> inlines), while other hardcode the "drm" prefix into them due being situated in
>>> the DRM core modules.
>>>
>>> Another thing is that I noticed the DRM_NAME macro which is used for this is
>>> defined in the uAPI header and had a comment which looked outdated.
>>>
>>> Therefore I introduce a new define, called, DRM_LOG_NAME, this time defined
>>> internally in the kernel headers and not exported in the uAPI.
>>>
>>> I also refactored some logging functions to take this string as a parameter
>>> instead of hardcoding it.
>>>
>>> Individual drivers can then override this define to make DRM logging functions
>>> prefix their message with the respective driver prefix.
>>>
>>> End result in the case of the i915 driver looks like this:
>>>
>>> Old log:
>>>
>>> [drm] Found 128MB of eDRAM
>>> [drm:skl_enable_dc6 [i915]] Enabling DC6
>>>
>>> New log:
>>>
>>> [i915] Found 128MB of eDRAM
>>> [i915:skl_enable_dc6 [i915]] Enabling DC6
>>
>> And still not conforming to the standard logging string. DRM_LOG should
>
> What is the standard logging string? the dev_ one?
>
>> be killed off as an anachronistic OS compat layer.
>
> You mean only dev variants should be kept?
I think the DRM_LOG_NAME override mechanism is too fragile, as it
depends on #include ordering. For our driver, I think it basically means
always including one of our headers (i915_drv.h) first everywhere (to
have a single point of truth for DRM_LOG_NAME), and including
drm_print.h first from there. That's not currently true, and I don't
want to see a massive #include reordering patchset to make it so.
This is like pr_fmt() which I think has been a mistake and should not be
repeated.
I think the direction to go is using dev_printk, dev_dbg, dev_err,
dev_warn, and friends, which use dev_driver_string internally. We
already have some drm wrappers for those. The problem with them is
passing dev, and I think that's the problem we should think about.
We also seem to have opted to use drm_dev_printk (which calls dev_printk
or printk) for DRM_DEV_DEBUG and friends. This is arguably a bad choice,
because using dev_dbg would let us make use of dynamic debug.
BR,
Jani.
>
> Regards,
>
> Tvrtko
>
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Intel-gfx] [RFC v2 0/6] DRM logging tidy
2018-01-25 11:32 ` Jani Nikula
@ 2018-01-25 13:38 ` Tvrtko Ursulin
0 siblings, 0 replies; 16+ messages in thread
From: Tvrtko Ursulin @ 2018-01-25 13:38 UTC (permalink / raw)
To: Jani Nikula, Chris Wilson, Tvrtko Ursulin, Intel-gfx; +Cc: dri-devel
On 25/01/2018 11:32, Jani Nikula wrote:
> On Wed, 24 Jan 2018, Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com> wrote:
>> On 24/01/2018 16:23, Chris Wilson wrote:
>>> Quoting Tvrtko Ursulin (2018-01-24 16:18:15)
>>>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>>>
>>>> This series tries to solve a few issues in the current DRM logging code to
>>>> primarily make it clearer which messages belong to which driver.
>>>>
>>>> Main problem is that currently some logging functions allow individual drivers
>>>> to override the log prefix (since they are defined as macros, or static
>>>> inlines), while other hardcode the "drm" prefix into them due being situated in
>>>> the DRM core modules.
>>>>
>>>> Another thing is that I noticed the DRM_NAME macro which is used for this is
>>>> defined in the uAPI header and had a comment which looked outdated.
>>>>
>>>> Therefore I introduce a new define, called, DRM_LOG_NAME, this time defined
>>>> internally in the kernel headers and not exported in the uAPI.
>>>>
>>>> I also refactored some logging functions to take this string as a parameter
>>>> instead of hardcoding it.
>>>>
>>>> Individual drivers can then override this define to make DRM logging functions
>>>> prefix their message with the respective driver prefix.
>>>>
>>>> End result in the case of the i915 driver looks like this:
>>>>
>>>> Old log:
>>>>
>>>> [drm] Found 128MB of eDRAM
>>>> [drm:skl_enable_dc6 [i915]] Enabling DC6
>>>>
>>>> New log:
>>>>
>>>> [i915] Found 128MB of eDRAM
>>>> [i915:skl_enable_dc6 [i915]] Enabling DC6
>>>
>>> And still not conforming to the standard logging string. DRM_LOG should
>>
>> What is the standard logging string? the dev_ one?
>>
>>> be killed off as an anachronistic OS compat layer.
>>
>> You mean only dev variants should be kept?
>
> I think the DRM_LOG_NAME override mechanism is too fragile, as it
> depends on #include ordering. For our driver, I think it basically means
> always including one of our headers (i915_drv.h) first everywhere (to
> have a single point of truth for DRM_LOG_NAME), and including
> drm_print.h first from there. That's not currently true, and I don't
> want to see a massive #include reordering patchset to make it so.
>
> This is like pr_fmt() which I think has been a mistake and should not be
> repeated.
>
> I think the direction to go is using dev_printk, dev_dbg, dev_err,
> dev_warn, and friends, which use dev_driver_string internally. We
> already have some drm wrappers for those. The problem with them is
> passing dev, and I think that's the problem we should think about.
>
> We also seem to have opted to use drm_dev_printk (which calls dev_printk
> or printk) for DRM_DEV_DEBUG and friends. This is arguably a bad choice,
> because using dev_dbg would let us make use of dynamic debug.
I agree the current state is messy and that a better improvement could
be made. And that the macro string approach is ugly.
To my defence, it sounded like a smaller amount of work to make it at
least a little bit better. The way I implemented it, as long as the
define is before any include directives it will work as expected. And on
my T460p which has both nouveau and i915 it makes the kernel log a bit
less confusing. But yeah, it is a marginal user base.
Scratch this then, previous posting was only two years ago so it can
wait some more for a more thorough etc rework.
Regards,
Tvrtko
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 16+ messages in thread
* ✓ Fi.CI.BAT: success for DRM logging tidy (rev2)
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
` (6 preceding siblings ...)
2018-01-24 16:23 ` [Intel-gfx] [RFC v2 0/6] DRM logging tidy Chris Wilson
@ 2018-01-24 16:42 ` Patchwork
2018-01-24 20:12 ` ✗ Fi.CI.IGT: warning " Patchwork
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2018-01-24 16:42 UTC (permalink / raw)
To: Tvrtko Ursulin; +Cc: intel-gfx
== Series Details ==
Series: DRM logging tidy (rev2)
URL : https://patchwork.freedesktop.org/series/16439/
State : success
== Summary ==
Series 16439v2 DRM logging tidy
https://patchwork.freedesktop.org/api/1.0/series/16439/revisions/2/mbox/
Test debugfs_test:
Subgroup read_all_entries:
dmesg-warn -> DMESG-FAIL (fi-elk-e7500) fdo#103989
Test gem_mmap_gtt:
Subgroup basic-small-bo-tiledx:
pass -> FAIL (fi-gdg-551) fdo#102575
Test kms_pipe_crc_basic:
Subgroup suspend-read-crc-pipe-b:
pass -> INCOMPLETE (fi-snb-2520m) fdo#103713
fdo#103989 https://bugs.freedesktop.org/show_bug.cgi?id=103989
fdo#102575 https://bugs.freedesktop.org/show_bug.cgi?id=102575
fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
fi-bdw-5557u total:288 pass:267 dwarn:0 dfail:0 fail:0 skip:21 time:419s
fi-bdw-gvtdvm total:288 pass:264 dwarn:0 dfail:0 fail:0 skip:24 time:430s
fi-blb-e6850 total:288 pass:223 dwarn:1 dfail:0 fail:0 skip:64 time:375s
fi-bsw-n3050 total:288 pass:242 dwarn:0 dfail:0 fail:0 skip:46 time:485s
fi-bwr-2160 total:288 pass:183 dwarn:0 dfail:0 fail:0 skip:105 time:281s
fi-bxt-dsi total:288 pass:258 dwarn:0 dfail:0 fail:0 skip:30 time:483s
fi-bxt-j4205 total:288 pass:259 dwarn:0 dfail:0 fail:0 skip:29 time:486s
fi-byt-j1900 total:288 pass:253 dwarn:0 dfail:0 fail:0 skip:35 time:469s
fi-byt-n2820 total:288 pass:249 dwarn:0 dfail:0 fail:0 skip:39 time:456s
fi-elk-e7500 total:224 pass:168 dwarn:9 dfail:1 fail:0 skip:45
fi-gdg-551 total:288 pass:179 dwarn:0 dfail:0 fail:1 skip:108 time:278s
fi-glk-1 total:288 pass:260 dwarn:0 dfail:0 fail:0 skip:28 time:512s
fi-hsw-4770 total:288 pass:261 dwarn:0 dfail:0 fail:0 skip:27 time:399s
fi-hsw-4770r total:288 pass:261 dwarn:0 dfail:0 fail:0 skip:27 time:400s
fi-ilk-650 total:288 pass:228 dwarn:0 dfail:0 fail:0 skip:60 time:411s
fi-ivb-3520m total:288 pass:259 dwarn:0 dfail:0 fail:0 skip:29 time:447s
fi-ivb-3770 total:288 pass:255 dwarn:0 dfail:0 fail:0 skip:33 time:413s
fi-kbl-7500u total:288 pass:263 dwarn:1 dfail:0 fail:0 skip:24 time:459s
fi-kbl-7560u total:288 pass:269 dwarn:0 dfail:0 fail:0 skip:19 time:498s
fi-kbl-7567u total:288 pass:268 dwarn:0 dfail:0 fail:0 skip:20 time:457s
fi-kbl-r total:288 pass:261 dwarn:0 dfail:0 fail:0 skip:27 time:508s
fi-pnv-d510 total:288 pass:222 dwarn:1 dfail:0 fail:0 skip:65 time:596s
fi-skl-6260u total:288 pass:268 dwarn:0 dfail:0 fail:0 skip:20 time:430s
fi-skl-6600u total:288 pass:261 dwarn:0 dfail:0 fail:0 skip:27 time:510s
fi-skl-6700hq total:288 pass:262 dwarn:0 dfail:0 fail:0 skip:26 time:525s
fi-skl-6700k2 total:288 pass:264 dwarn:0 dfail:0 fail:0 skip:24 time:489s
fi-skl-6770hq total:288 pass:268 dwarn:0 dfail:0 fail:0 skip:20 time:479s
fi-skl-guc total:288 pass:260 dwarn:0 dfail:0 fail:0 skip:28 time:415s
fi-skl-gvtdvm total:288 pass:265 dwarn:0 dfail:0 fail:0 skip:23 time:432s
fi-snb-2520m total:245 pass:211 dwarn:0 dfail:0 fail:0 skip:33
fi-snb-2600 total:288 pass:248 dwarn:0 dfail:0 fail:0 skip:40 time:400s
Blacklisted hosts:
fi-cfl-s2 total:288 pass:262 dwarn:0 dfail:0 fail:0 skip:26 time:567s
fi-glk-dsi total:288 pass:258 dwarn:0 dfail:0 fail:0 skip:30 time:468s
92c5caeeb0cae3952f5e01d0305be09559ce49b4 drm-tip: 2018y-01m-24d-15h-29m-34s UTC integration manifest
641f30a6ad61 drm: Respect driver set DRM_LOG_NAME in drm_info_printer
3260202bbbd8 drm: Respect driver set DRM_LOG_NAME in drm_dev_printk
e4053d27154d drm: Respect driver set DRM_LOG_NAME in drm_printk
95c203daf028 drm/i915: Give our log messages our name
c3494a30403b drm: Introduce unexported DRM_LOG_NAME define for logging
2507afa2f0d7 drm/armada: Simplify drm_dev_init error log
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_7769/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 16+ messages in thread* ✗ Fi.CI.IGT: warning for DRM logging tidy (rev2)
2018-01-24 16:18 [RFC v2 0/6] DRM logging tidy Tvrtko Ursulin
` (7 preceding siblings ...)
2018-01-24 16:42 ` ✓ Fi.CI.BAT: success for DRM logging tidy (rev2) Patchwork
@ 2018-01-24 20:12 ` Patchwork
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2018-01-24 20:12 UTC (permalink / raw)
To: Tvrtko Ursulin; +Cc: intel-gfx
== Series Details ==
Series: DRM logging tidy (rev2)
URL : https://patchwork.freedesktop.org/series/16439/
State : warning
== Summary ==
Test kms_setmode:
Subgroup basic:
fail -> PASS (shard-hsw) fdo#99912
Test perf:
Subgroup enable-disable:
pass -> FAIL (shard-apl) fdo#103715
Subgroup oa-exponents:
fail -> PASS (shard-apl) fdo#102254
Test kms_frontbuffer_tracking:
Subgroup fbc-1p-primscrn-shrfb-msflip-blt:
pass -> SKIP (shard-apl) fdo#101623 +1
Subgroup fbc-1p-primscrn-cur-indfb-draw-blt:
pass -> FAIL (shard-apl) fdo#103167
Test pm_rpm:
Subgroup i2c:
pass -> SKIP (shard-apl)
Test kms_chv_cursor_fail:
Subgroup pipe-c-256x256-left-edge:
pass -> SKIP (shard-apl)
Subgroup pipe-b-256x256-bottom-edge:
pass -> SKIP (shard-apl)
Test kms_universal_plane:
Subgroup universal-plane-pipe-b-sanity:
pass -> SKIP (shard-apl)
Test kms_plane:
Subgroup plane-panning-bottom-right-suspend-pipe-c-planes:
pass -> DMESG-WARN (shard-apl) fdo#104164
Test kms_flip:
Subgroup flip-vs-absolute-wf_vblank-interruptible:
pass -> SKIP (shard-apl) fdo#100368
Subgroup flip-vs-rmfb-interruptible:
pass -> SKIP (shard-apl)
Subgroup 2x-plain-flip-ts-check-interruptible:
fail -> PASS (shard-hsw)
Test kms_pipe_crc_basic:
Subgroup read-crc-pipe-b-frame-sequence:
pass -> SKIP (shard-apl) fdo#103481
Test gem_eio:
Subgroup in-flight:
pass -> DMESG-WARN (shard-snb) fdo#104058
Test kms_rotation_crc:
Subgroup sprite-rotation-90:
pass -> FAIL (shard-apl) fdo#103356
Test drv_suspend:
Subgroup forcewake-hibernate:
skip -> FAIL (shard-snb) fdo#103375
fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912
fdo#103715 https://bugs.freedesktop.org/show_bug.cgi?id=103715
fdo#102254 https://bugs.freedesktop.org/show_bug.cgi?id=102254
fdo#101623 https://bugs.freedesktop.org/show_bug.cgi?id=101623
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fdo#104164 https://bugs.freedesktop.org/show_bug.cgi?id=104164
fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
fdo#103481 https://bugs.freedesktop.org/show_bug.cgi?id=103481
fdo#104058 https://bugs.freedesktop.org/show_bug.cgi?id=104058
fdo#103356 https://bugs.freedesktop.org/show_bug.cgi?id=103356
fdo#103375 https://bugs.freedesktop.org/show_bug.cgi?id=103375
shard-apl total:2838 pass:1741 dwarn:2 dfail:0 fail:25 skip:1070 time:12451s
shard-hsw total:2838 pass:1735 dwarn:1 dfail:0 fail:10 skip:1091 time:11767s
shard-snb total:2838 pass:1328 dwarn:2 dfail:0 fail:11 skip:1497 time:6577s
Blacklisted hosts:
shard-kbl total:2795 pass:1834 dwarn:15 dfail:1 fail:23 skip:921 time:9149s
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_7769/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 16+ messages in thread