[PATCH i-g-t 1/2] lib: Generalise rapl interface

[PATCH i-g-t 1/2] lib: Generalise rapl interface

[Chris Wilson]

We can use our existing rapl interface that monitors gpu power, to also
sample the other rapl domains such as package, cores and ram.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Andi Shyti <andi.shyti@intel.com>
---
 lib/Makefile.sources                |  4 +-
 lib/igt_gpu_power.c                 | 87 -----------------------------
 lib/igt_rapl.c                      | 69 +++++++++++++++++++++++
 lib/{igt_gpu_power.h => igt_rapl.h} | 69 ++++++++++++++++-------
 lib/meson.build                     |  2 +-
 tests/i915/gem_exec_schedule.c      | 39 ++++++++-----
 tests/i915/gem_exec_whisper.c       | 15 ++---
 7 files changed, 155 insertions(+), 130 deletions(-)
 delete mode 100644 lib/igt_gpu_power.c
 create mode 100644 lib/igt_rapl.c
 rename lib/{igt_gpu_power.h => igt_rapl.h} (52%)

diff --git a/lib/Makefile.sources b/lib/Makefile.sources
index cf094ab89..34e0c012d 100644
--- a/lib/Makefile.sources
+++ b/lib/Makefile.sources
@@ -33,8 +33,6 @@ lib_source_list =	 	\
 	igt_edid.h		\
 	igt_eld.c		\
 	igt_eld.h		\
-	igt_gpu_power.c		\
-	igt_gpu_power.h		\
 	igt_gt.c		\
 	igt_gt.h		\
 	igt_gvt.c		\
@@ -49,6 +47,8 @@ lib_source_list =	 	\
 	igt_primes.h		\
 	igt_rand.c		\
 	igt_rand.h		\
+	igt_rapl.c		\
+	igt_rapl.h		\
 	igt_rc.h		\
 	igt_stats.c		\
 	igt_stats.h		\
diff --git a/lib/igt_gpu_power.c b/lib/igt_gpu_power.c
deleted file mode 100644
index 7092b75b3..000000000
--- a/lib/igt_gpu_power.c
+++ /dev/null
@@ -1,87 +0,0 @@
-#include <ctype.h>
-#include <errno.h>
-#include <fcntl.h>
-#include <locale.h>
-#include <math.h>
-#include <unistd.h>
-#include <inttypes.h>
-
-#include "igt_gpu_power.h"
-#include "igt_perf.h"
-#include "igt_sysfs.h"
-
-struct rapl {
-	uint64_t power, type;
-	double scale;
-};
-
-static int rapl_parse(struct rapl *r)
-{
-	locale_t locale, oldlocale;
-	bool result;
-	int dir;
-
-	memset(r, 0, sizeof(*r));
-
-	dir = open("/sys/devices/power", O_RDONLY);
-	if (dir < 0)
-		return -errno;
-
-	/* Replace user environment with plain C to match kernel format */
-	locale = newlocale(LC_ALL, "C", 0);
-	oldlocale = uselocale(locale);
-
-	result = true;
-	result &= igt_sysfs_scanf(dir, "type",
-				  "%"PRIu64, &r->type) == 1;
-	result &= igt_sysfs_scanf(dir, "events/energy-gpu",
-				  "event=%"PRIx64, &r->power) == 1;
-	result &= igt_sysfs_scanf(dir, "events/energy-gpu.scale",
-				  "%lf", &r->scale) == 1;
-
-	uselocale(oldlocale);
-	freelocale(locale);
-
-	close(dir);
-
-	if (!result)
-		return -EINVAL;
-
-	if (isnan(r->scale) || !r->scale)
-		return -ERANGE;
-
-	return 0;
-}
-
-int gpu_power_open(struct gpu_power *power)
-{
-	struct rapl r;
-
-	power->fd = rapl_parse(&r);
-	if (power->fd < 0)
-		goto err;
-
-	power->fd = igt_perf_open(r.type, r.power);
-	if (power->fd < 0) {
-		power->fd = -errno;
-		goto err;
-	}
-
-	power->scale = r.scale;
-
-	return 0;
-
-err:
-	errno = 0;
-	return power->fd;
-}
-
-bool gpu_power_read(struct gpu_power *power, struct gpu_power_sample *s)
-{
-	return read(power->fd, s, sizeof(*s)) == sizeof(*s);
-}
-
-void gpu_power_close(struct gpu_power *power)
-{
-	close(power->fd);
-}
diff --git a/lib/igt_rapl.c b/lib/igt_rapl.c
new file mode 100644
index 000000000..03e492260
--- /dev/null
+++ b/lib/igt_rapl.c
@@ -0,0 +1,69 @@
+#include <ctype.h>
+#include <errno.h>
+#include <fcntl.h>
+#include <locale.h>
+#include <math.h>
+#include <unistd.h>
+#include <inttypes.h>
+
+#include "igt_perf.h"
+#include "igt_rapl.h"
+#include "igt_sysfs.h"
+
+static int rapl_parse(struct rapl *r, const char *str)
+{
+	locale_t locale, oldlocale;
+	bool result = true;
+	char buf[128];
+	int dir;
+
+	memset(r, 0, sizeof(*r));
+
+	dir = open("/sys/devices/power", O_RDONLY);
+	if (dir < 0)
+		return -errno;
+
+	/* Replace user environment with plain C to match kernel format */
+	locale = newlocale(LC_ALL, "C", 0);
+	oldlocale = uselocale(locale);
+
+	result &= igt_sysfs_scanf(dir, "type", "%"PRIu64, &r->type) == 1;
+
+	snprintf(buf, sizeof(buf), "events/energy-%s", str);
+	result &= igt_sysfs_scanf(dir, buf, "event=%"PRIx64, &r->power) == 1;
+
+	snprintf(buf, sizeof(buf), "events/energy-%s.scale", str);
+	result &= igt_sysfs_scanf(dir, buf, "%lf", &r->scale) == 1;
+
+	uselocale(oldlocale);
+	freelocale(locale);
+
+	close(dir);
+
+	if (!result)
+		return -EINVAL;
+
+	if (isnan(r->scale) || !r->scale)
+		return -ERANGE;
+
+	return 0;
+}
+
+int rapl_open(struct rapl *r, const char *domain)
+{
+	r->fd = rapl_parse(r, domain);
+	if (r->fd < 0)
+		goto err;
+
+	r->fd = igt_perf_open(r->type, r->power);
+	if (r->fd < 0) {
+		r->fd = -errno;
+		goto err;
+	}
+
+	return 0;
+
+err:
+	errno = 0;
+	return r->fd;
+}
diff --git a/lib/igt_gpu_power.h b/lib/igt_rapl.h
similarity index 52%
rename from lib/igt_gpu_power.h
rename to lib/igt_rapl.h
index a578a02a0..55c46198e 100644
--- a/lib/igt_gpu_power.h
+++ b/lib/igt_rapl.h
@@ -22,45 +22,74 @@
  *
  */
 
-#ifndef IGT_GPU_POWER_H
-#define IGT_GPU_POWER_H
+#ifndef IGT_RAPL_H
+#define IGT_RAPL_H
 
 #include <stdbool.h>
 #include <stdint.h>
 
-struct gpu_power {
-	int fd;
+struct rapl {
+	uint64_t power, type;
 	double scale;
+	int fd;
 };
 
-struct gpu_power_sample {
+struct power_sample {
 	uint64_t energy;
 	uint64_t time;
 };
 
-int gpu_power_open(struct gpu_power *power);
-bool gpu_power_read(struct gpu_power *power, struct gpu_power_sample *s);
-void gpu_power_close(struct gpu_power *power);
+int rapl_open(struct rapl *r, const char *domain);
+
+static inline int cpu_power_open(struct rapl *r)
+{
+	return rapl_open(r, "cpu");
+}
+
+static inline int gpu_power_open(struct rapl *r)
+{
+	return rapl_open(r, "gpu");
+}
+
+static inline int pkg_power_open(struct rapl *r)
+{
+	return rapl_open(r, "pkg");
+}
+
+static inline int ram_power_open(struct rapl *r)
+{
+	return rapl_open(r, "ram");
+}
+
+static inline bool rapl_read(struct rapl *r, struct power_sample *s)
+{
+	return read(r->fd, s, sizeof(*s)) == sizeof(*s);
+}
+
+static inline void rapl_close(struct rapl *r)
+{
+	close(r->fd);
+}
 
-static inline double gpu_power_J(const struct gpu_power *p,
-				 const struct gpu_power_sample *p0,
-				 const struct gpu_power_sample *p1)
+static inline double power_J(const struct rapl *r,
+			     const struct power_sample *p0,
+			     const struct power_sample *p1)
 {
-	return (p1->energy - p0->energy) * p->scale;
+	return (p1->energy - p0->energy) * r->scale;
 }
 
-static inline double gpu_power_s(const struct gpu_power *p,
-				 const struct gpu_power_sample *p0,
-				 const struct gpu_power_sample *p1)
+static inline double power_s(const struct rapl *r,
+			     const struct power_sample *p0,
+			     const struct power_sample *p1)
 {
 	return (p1->time - p0->time) * 1e-9;
 }
 
-static inline double gpu_power_W(const struct gpu_power *p,
-				 const struct gpu_power_sample *p0,
-				 const struct gpu_power_sample *p1)
+static inline double power_W(const struct rapl *r,
+			     const struct power_sample *p0,
+			     const struct power_sample *p1)
 {
-	return gpu_power_J(p, p0, p1) / gpu_power_s(p, p0, p1);
+	return power_J(r, p0, p1) / power_s(r, p0, p1);
 }
 
-#endif /* IGT_GPU_POWER_H */
+#endif /* IGT_RAPL_H */
diff --git a/lib/meson.build b/lib/meson.build
index 221ae28c0..fbc0c8d14 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -11,7 +11,6 @@ lib_sources = [
 	'igt_debugfs.c',
 	'igt_device.c',
 	'igt_aux.c',
-	'igt_gpu_power.c',
 	'igt_gt.c',
 	'igt_gvt.c',
 	'igt_halffloat.c',
@@ -19,6 +18,7 @@ lib_sources = [
 	'igt_perf.c',
 	'igt_primes.c',
 	'igt_rand.c',
+	'igt_rapl.c',
 	'igt_stats.c',
 	'igt_syncobj.c',
 	'igt_sysfs.c',
diff --git a/tests/i915/gem_exec_schedule.c b/tests/i915/gem_exec_schedule.c
index ddcb1f21a..9a2f2352b 100644
--- a/tests/i915/gem_exec_schedule.c
+++ b/tests/i915/gem_exec_schedule.c
@@ -29,8 +29,8 @@
 #include <signal.h>
 
 #include "igt.h"
-#include "igt_gpu_power.h"
 #include "igt_rand.h"
+#include "igt_rapl.h"
 #include "igt_sysfs.h"
 #include "igt_vgem.h"
 #include "i915/gem_ring.h"
@@ -1608,14 +1608,16 @@ static void test_pi_ringfull(int fd, unsigned int engine)
 
 static void measure_semaphore_power(int i915)
 {
-	struct gpu_power power;
 	unsigned int engine, signaler;
+	struct rapl gpu, pkg;
 
-	igt_require(gpu_power_open(&power) == 0);
+	igt_require(pkg_power_open(&pkg) == 0);
+	igt_require(gpu_power_open(&gpu) == 0);
 
 	for_each_physical_engine(i915, signaler) {
-		struct gpu_power_sample s_spin[2];
-		struct gpu_power_sample s_sema[2];
+		struct {
+			struct power_sample pkg, gpu;
+		} s_spin[2], s_sema[2];
 		double baseline, total;
 		int64_t jiffie = 1;
 		igt_spin_t *spin;
@@ -1626,9 +1628,11 @@ static void measure_semaphore_power(int i915)
 		gem_wait(i915, spin->handle, &jiffie); /* waitboost */
 		igt_spin_busywait_until_started(spin);
 
-		gpu_power_read(&power, &s_spin[0]);
+		rapl_read(&pkg, &s_spin[0].pkg);
+		rapl_read(&gpu, &s_spin[0].gpu);
 		usleep(100*1000);
-		gpu_power_read(&power, &s_spin[1]);
+		rapl_read(&gpu, &s_spin[1].gpu);
+		rapl_read(&pkg, &s_spin[1].pkg);
 
 		/* Add a waiter to each engine */
 		for_each_physical_engine(i915, engine) {
@@ -1645,23 +1649,32 @@ static void measure_semaphore_power(int i915)
 		}
 		usleep(10); /* just give the tasklets a chance to run */
 
-		gpu_power_read(&power, &s_sema[0]);
+		rapl_read(&pkg, &s_sema[0].pkg);
+		rapl_read(&gpu, &s_sema[0].gpu);
 		usleep(100*1000);
-		gpu_power_read(&power, &s_sema[1]);
+		rapl_read(&gpu, &s_sema[1].gpu);
+		rapl_read(&pkg, &s_sema[1].pkg);
 
 		igt_spin_free(i915, spin);
 
-		baseline = gpu_power_W(&power, &s_spin[0], &s_spin[1]);
-		total = gpu_power_W(&power, &s_sema[0], &s_sema[1]);
-
+		baseline = power_W(&gpu, &s_spin[0].gpu, &s_spin[1].gpu);
+		total = power_W(&gpu, &s_sema[0].gpu, &s_sema[1].gpu);
 		igt_info("%s: %.1fmW + %.1fmW (total %1.fmW)\n",
 			 e__->name,
 			 1e3 * baseline,
 			 1e3 * (total - baseline),
 			 1e3 * total);
+
+		baseline = power_W(&pkg, &s_spin[0].pkg, &s_spin[1].pkg);
+		total = power_W(&pkg, &s_sema[0].pkg, &s_sema[1].pkg);
+		igt_info("pkg: %.1fmW + %.1fmW (total %1.fmW)\n",
+			 1e3 * baseline,
+			 1e3 * (total - baseline),
+			 1e3 * total);
 	}
 
-	gpu_power_close(&power);
+	rapl_close(&gpu);
+	rapl_close(&pkg);
 }
 
 igt_main
diff --git a/tests/i915/gem_exec_whisper.c b/tests/i915/gem_exec_whisper.c
index de7a14dad..1135c6497 100644
--- a/tests/i915/gem_exec_whisper.c
+++ b/tests/i915/gem_exec_whisper.c
@@ -29,7 +29,7 @@
 
 #include "igt.h"
 #include "igt_debugfs.h"
-#include "igt_gpu_power.h"
+#include "igt_rapl.h"
 #include "igt_gt.h"
 #include "igt_rand.h"
 #include "igt_sysfs.h"
@@ -186,8 +186,8 @@ static void whisper(int fd, unsigned engine, unsigned flags)
 	unsigned int reloc_migrations = 0;
 	unsigned int reloc_interruptions = 0;
 	unsigned int eb_migrations = 0;
-	struct gpu_power_sample sample[2];
-	struct gpu_power power;
+	struct power_sample sample[2];
+	struct rapl rapl;
 	uint64_t old_offset;
 	int i, n, loc;
 	int debugfs;
@@ -199,7 +199,7 @@ static void whisper(int fd, unsigned engine, unsigned flags)
 	}
 
 	debugfs = igt_debugfs_dir(fd);
-	gpu_power_open(&power);
+	gpu_power_open(&rapl);
 
 	nengine = 0;
 	if (engine == ALL_ENGINES) {
@@ -234,7 +234,7 @@ static void whisper(int fd, unsigned engine, unsigned flags)
 		nchild *= nengine;
 
 	intel_detect_and_clear_missed_interrupts(fd);
-	gpu_power_read(&power, &sample[0]);
+	rapl_read(&rapl, &sample[0]);
 	igt_fork(child, nchild) {
 		unsigned int pass;
 
@@ -510,11 +510,12 @@ static void whisper(int fd, unsigned engine, unsigned flags)
 		fini_hang(&hang);
 	else
 		igt_assert_eq(intel_detect_and_clear_missed_interrupts(fd), 0);
-	if (gpu_power_read(&power, &sample[1]))  {
+	if (rapl_read(&rapl, &sample[1]))  {
 		igt_info("Total energy used: %.1fmJ\n",
-			 gpu_power_J(&power, &sample[0], &sample[1]) * 1e3);
+			 power_J(&rapl, &sample[0], &sample[1]) * 1e3);
 	}
 
+	rapl_close(&rapl);
 	close(debugfs);
 }
 
-- 
2.23.0

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

[PATCH i-g-t 2/2] overlay: Show total package power

[Chris Wilson]

Add the total package power after the GPU package power, for reference.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 overlay/overlay.c | 18 ++++++----
 overlay/power.c   | 88 +++++++++++++++++++----------------------------
 overlay/power.h   | 21 +++++------
 3 files changed, 59 insertions(+), 68 deletions(-)

diff --git a/overlay/overlay.c b/overlay/overlay.c
index eae5ddfa8..dd4fca29e 100644
--- a/overlay/overlay.c
+++ b/overlay/overlay.c
@@ -609,12 +609,12 @@ static void show_gpu_freq(struct overlay_context *ctx, struct overlay_gpu_freq *
 	}
 
 	if (has_power) {
-		chart_add_sample(&gf->power_chart, gf->power.power_mW);
-		if (gf->power.new_sample) {
-			if (gf->power.power_mW > gf->power_max)
-				gf->power_max = gf->power.power_mW;
+		chart_add_sample(&gf->power_chart, gf->power.gpu.power_mW);
+		if (gf->power.gpu.new_sample) {
+			if (gf->power.gpu.power_mW > gf->power_max)
+				gf->power_max = gf->power.gpu.power_mW;
 			chart_set_range(&gf->power_chart, 0, gf->power_max);
-			gf->power.new_sample = 0;
+			gf->power.gpu.new_sample = 0;
 		}
 		chart_draw(&gf->power_chart, ctx->cr);
 	}
@@ -700,8 +700,14 @@ static void show_gpu_freq(struct overlay_context *ctx, struct overlay_gpu_freq *
 	}
 
 	if (has_power) {
-		sprintf(buf, "Power: %llumW", (long long unsigned)gf->power.power_mW);
 		cairo_set_source_rgba(ctx->cr, 1, 1, 1, 1);
+
+		sprintf(buf, "Power: %llumW", (long long unsigned)gf->power.gpu.power_mW);
+		cairo_move_to(ctx->cr, PAD, y);
+		cairo_show_text(ctx->cr, buf);
+		y += 14;
+
+		sprintf(buf, "Package: %llumW", (long long unsigned)gf->power.pkg.power_mW);
 		cairo_move_to(ctx->cr, PAD, y);
 		cairo_show_text(ctx->cr, buf);
 		y += 14;
diff --git a/overlay/power.c b/overlay/power.c
index 0f99e2a4a..76fafea91 100644
--- a/overlay/power.c
+++ b/overlay/power.c
@@ -77,15 +77,6 @@ static uint64_t filename_to_u64(const char *filename, int base)
 	return strtoull(b, NULL, base);
 }
 
-static uint64_t debugfs_file_to_u64(const char *name)
-{
-	char buf[1024];
-
-	snprintf(buf, sizeof(buf), "%s/%s", debugfs_dri_path, name);
-
-	return filename_to_u64(buf, 0);
-}
-
 static uint64_t rapl_type_id(void)
 {
 	return filename_to_u64("/sys/devices/power/type", 10);
@@ -96,6 +87,11 @@ static uint64_t rapl_gpu_power(void)
 	return filename_to_u64("/sys/devices/power/events/energy-gpu", 0);
 }
 
+static uint64_t rapl_pkg_power(void)
+{
+	return filename_to_u64("/sys/devices/power/events/energy-pkg", 0);
+}
+
 static double filename_to_double(const char *filename)
 {
 	char *oldlocale;
@@ -117,70 +113,58 @@ static double rapl_gpu_power_scale(void)
 	return filename_to_double("/sys/devices/power/events/energy-gpu.scale");
 }
 
-int power_init(struct power *power)
+static double rapl_pkg_power_scale(void)
 {
-	uint64_t val;
+	return filename_to_double("/sys/devices/power/events/energy-pkg.scale");
+}
 
+int power_init(struct power *power)
+{
 	memset(power, 0, sizeof(*power));
 
-	power->fd = igt_perf_open(rapl_type_id(), rapl_gpu_power());
-	if (power->fd >= 0) {
-		power->rapl_scale = rapl_gpu_power_scale();
-
-		if (power->rapl_scale != NAN) {
-			power->rapl_scale *= 1e3; /* from nano to micro */
-			return 0;
-		}
-	}
+	power->gpu.fd = igt_perf_open(rapl_type_id(), rapl_gpu_power());
+	if (power->gpu.fd < 0)
+		return power->error = ENOENT;
+	power->gpu.scale = rapl_gpu_power_scale() * 1e3; /* to milli */
 
-	val = debugfs_file_to_u64("i915_energy_uJ");
-	if (val == 0)
-		return power->error = EINVAL;
+	power->pkg.fd = igt_perf_open(rapl_type_id(), rapl_pkg_power());
+	power->pkg.scale = rapl_pkg_power_scale() *1e3; /* to milli */
 
 	return 0;
 }
 
-static uint64_t clock_ms_to_u64(void)
+static void __power_update(struct power_domain *pd, int count)
 {
-	struct timespec tv;
+	struct power_stat *s = &pd->stat[count & 1];
+	struct power_stat *d = &pd->stat[(count + 1) & 1];
+	uint64_t data[2], d_time;
+	int len;
 
-	if (clock_gettime(CLOCK_MONOTONIC, &tv) < 0)
-		return 0;
+	len = read(pd->fd, data, sizeof(data));
+	if (len != sizeof(data))
+		return;
 
-	return (uint64_t)tv.tv_sec * 1e3 + tv.tv_nsec / 1e6;
+	s->energy = llround((double)data[0] * pd->scale);
+	s->timestamp = data[1];
+
+	if (!count)
+		return;
+
+	d_time = s->timestamp - d->timestamp;
+	pd->power_mW = round((s->energy - d->energy) * 1e9 / d_time);
+	pd->new_sample = 1;
 }
 
 int power_update(struct power *power)
 {
-	struct power_stat *s = &power->stat[power->count++ & 1];
-	struct power_stat *d = &power->stat[power->count & 1];
-	uint64_t d_time;
-
 	if (power->error)
 		return power->error;
 
-	if (power->fd >= 0) {
-		uint64_t data[2];
-		int len;
-
-		len = read(power->fd, data, sizeof(data));
-		if (len != sizeof(data))
-			return power->error = errno;
-
-		s->energy = llround((double)data[0] * power->rapl_scale);
-		s->timestamp = data[1] / 1e6;
-	} else {
-		s->energy = debugfs_file_to_u64("i915_energy_uJ") / 1e3;
-		s->timestamp = clock_ms_to_u64();
-	}
+	__power_update(&power->gpu, power->count);
+	__power_update(&power->pkg, power->count);
 
-	if (power->count == 1)
+	if (!power->count++)
 		return EAGAIN;
 
-	d_time = s->timestamp - d->timestamp;
-	power->power_mW = round((double)(s->energy - d->energy) *
-				(1e3f / d_time));
-	power->new_sample = 1;
-
 	return 0;
 }
diff --git a/overlay/power.h b/overlay/power.h
index 28abfc322..be9f1e219 100644
--- a/overlay/power.h
+++ b/overlay/power.h
@@ -28,19 +28,20 @@
 #include <stdint.h>
 
 struct power {
-	struct power_stat {
-		uint64_t energy;
-		uint64_t timestamp;
-	} stat[2];
+	struct power_domain {
+		double scale;
+		uint64_t power_mW;
+		int new_sample;
+		int fd;
+
+		struct power_stat {
+			uint64_t energy;
+			uint64_t timestamp;
+		} stat[2];
+	} gpu, pkg;
 
-	int fd;
 	int error;
 	int count;
-	int new_sample;
-
-	uint64_t power_mW;
-
-	double rapl_scale;
 };
 
 int power_init(struct power *power);
-- 
2.23.0

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

Re: [PATCH i-g-t 1/2] lib: Generalise rapl interface

[Andi Shyti]

Hi Chris,

On Sun, Oct 13, 2019 at 02:50:20PM +0100, Chris Wilson wrote:
> We can use our existing rapl interface that monitors gpu power, to also
> sample the other rapl domains such as package, cores and ram.
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Andi Shyti <andi.shyti@intel.com>

looks good,

Reviewed-by: Andi Shyti <andi.shyti@intel.com>

Thanks,
Andi

PS Don't think I missed a few hidden style changes :)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx