[i-g-t PATCH] tests: prefix IGT kernel log messages with [IGT]

[i-g-t PATCH] tests: prefix IGT kernel log messages with [IGT]

[Jani Nikula]

Make the IGT logging stand out better and easier to grep.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 lib/igt_core.c   | 6 +++---
 tests/drm_lib.sh | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/lib/igt_core.c b/lib/igt_core.c
index 43db4684cab0..9cd5f98d2014 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -734,7 +734,7 @@ out:
 		exit(ret == -1 ? 0 : IGT_EXIT_INVALID);
 
 	if (!list_subtests) {
-		kmsg(KERN_INFO "%s: executing\n", command_str);
+		kmsg(KERN_INFO "[IGT] %s: executing\n", command_str);
 		print_version();
 
 		oom_adjust_for_doom();
@@ -860,7 +860,7 @@ bool __igt_run_subtest(const char *subtest_name)
 		return false;
 	}
 
-	kmsg(KERN_INFO "%s: starting subtest %s\n", command_str, subtest_name);
+	kmsg(KERN_INFO "[IGT] %s: starting subtest %s\n", command_str, subtest_name);
 	igt_debug("Starting subtest: %s\n", subtest_name);
 
 	_igt_log_buffer_reset();
@@ -1343,7 +1343,7 @@ void igt_exit(void)
 	}
 
 	if (command_str)
-		kmsg(KERN_INFO "%s: exiting, ret=%d\n",
+		kmsg(KERN_INFO "[IGT] %s: exiting, ret=%d\n",
 		     command_str, igt_exitcode);
 	igt_debug("Exiting with status code %d\n", igt_exitcode);
 
diff --git a/tests/drm_lib.sh b/tests/drm_lib.sh
index 113da4c7d645..1abc1580b96a 100755
--- a/tests/drm_lib.sh
+++ b/tests/drm_lib.sh
@@ -20,12 +20,12 @@ kmsg() {
 
 finish() {
 	exitcode=$?
-	kmsg "$KERN_INFO$NAME: exiting, ret=$exitcode"
+	kmsg "${KERN_INFO}[IGT] $NAME: exiting, ret=$exitcode"
 	exit $exitcode
 }
 trap finish EXIT
 
-kmsg "$KERN_INFO$NAME: executing"
+kmsg "${KERN_INFO}[IGT] $NAME: executing"
 
 skip() {
 	echo "$@"
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [i-g-t PATCH] tests: prefix IGT kernel log messages with [IGT]

[Chris Wilson]

On Fri, Oct 14, 2016 at 12:56:39PM +0300, Jani Nikula wrote:
> Make the IGT logging stand out better and easier to grep.

[IGT] [igt] IGT igt i-g-t I-G-T [i-g-t] [I-G-T]

Hobson's choice. Seems a sensible thing to do anyway.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [i-g-t PATCH] tests: prefix IGT kernel log messages with [IGT]

[Jani Nikula]

On Fri, 14 Oct 2016, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> On Fri, Oct 14, 2016 at 12:56:39PM +0300, Jani Nikula wrote:
>> Make the IGT logging stand out better and easier to grep.
>
> [IGT] [igt] IGT igt i-g-t I-G-T [i-g-t] [I-G-T]
>
> Hobson's choice. Seems a sensible thing to do anyway.

You forgot Igt. Pushed.

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx