* [igt-dev] [PATCH] [PATCH i-g-t]Macros fixes: Removed unused & redundant macros
@ 2020-04-28 13:31 ranjeet1.kumar
2020-04-29 10:07 ` [igt-dev] ✓ Fi.CI.BAT: success for Macros " Patchwork
2020-04-29 12:07 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
0 siblings, 2 replies; 3+ messages in thread
From: ranjeet1.kumar @ 2020-04-28 13:31 UTC (permalink / raw)
To: igt-dev, ranjeet1.kumar
From: ranjeet kumar <ranjeet1.kumar@intel.com>
Local macros has been declared in several files.
Removed local and unused macros from several files.
used macros defined in drm header.
Cc: Dixit, Ashutosh <ashutosh.dixit@intel.com>
Cc: Tahvanainen Jari <jari.tahvanainen@intel.com>
Signed-off-by: ranjeet kumar <ranjeet1.kumar@intel.com>
---
benchmarks/gem_blt.c | 11 ++++------
benchmarks/gem_busy.c | 12 +++--------
benchmarks/gem_create.c | 7 ++----
benchmarks/gem_exec_ctx.c | 7 ++----
benchmarks/gem_exec_fault.c | 12 +++--------
benchmarks/gem_exec_nop.c | 14 ++++--------
benchmarks/gem_exec_reloc.c | 7 ++----
benchmarks/gem_syslatency.c | 12 +++--------
lib/igt_dummyload.c | 5 +----
tests/i915/gem_ctx_switch.c | 11 ++++------
tests/i915/gem_exec_await.c | 16 +++++---------
tests/i915/gem_exec_create.c | 12 +++--------
tests/i915/gem_exec_latency.c | 12 +++--------
tests/i915/gem_exec_lut_handle.c | 13 ++++-------
tests/i915/gem_exec_nop.c | 36 +++++++++++++------------------
tests/i915/gem_exec_parallel.c | 12 +++--------
tests/i915/gem_exec_reloc.c | 10 ++-------
tests/i915/gem_exec_store.c | 5 +----
tests/i915/gem_exec_suspend.c | 5 +----
tests/i915/gem_exec_whisper.c | 12 +++--------
tests/i915/gem_streaming_writes.c | 16 ++++++--------
tests/i915/gem_sync.c | 24 ++++++++-------------
tests/i915/i915_module_load.c | 6 +-----
23 files changed, 85 insertions(+), 192 deletions(-)
diff --git a/benchmarks/gem_blt.c b/benchmarks/gem_blt.c
index 6fb72394..5c6d28d3 100644
--- a/benchmarks/gem_blt.c
+++ b/benchmarks/gem_blt.c
@@ -42,9 +42,6 @@
#include "drm.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
#define COPY_BLT_CMD (2<<29|0x53<<22|0x6)
#define BLT_WRITE_ALPHA (1<<21)
#define BLT_WRITE_RGB (1<<20)
@@ -229,7 +226,7 @@ static int run(int object, int batch, int time, int reps, int ncpus, unsigned fl
execbuf.buffer_count = 3;
execbuf.batch_len = len;
execbuf.flags = ring;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
if (__gem_execbuf(fd, &execbuf)) {
gem_set_domain(fd, handle, I915_GEM_DOMAIN_CPU, I915_GEM_DOMAIN_CPU);
@@ -241,7 +238,7 @@ static int run(int object, int batch, int time, int reps, int ncpus, unsigned fl
gem_sync(fd, handle);
if (batch > 1) {
- if (execbuf.flags & LOCAL_I915_EXEC_HANDLE_LUT) {
+ if (execbuf.flags & I915_EXEC_HANDLE_LUT) {
src = 0;
dst = 1;
}
@@ -258,8 +255,8 @@ static int run(int object, int batch, int time, int reps, int ncpus, unsigned fl
gem_execbuf(fd, &execbuf);
gem_sync(fd, handle);
}
- if (execbuf.flags & LOCAL_I915_EXEC_HANDLE_LUT)
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ if (execbuf.flags & I915_EXEC_HANDLE_LUT)
+ execbuf.flags |= I915_EXEC_NO_RELOC;
/* Guess how many loops we need for 0.1s */
count = baseline((uint64_t)object * batch, 100) / ncpus;
diff --git a/benchmarks/gem_busy.c b/benchmarks/gem_busy.c
index 405cc9b0..41e86d2b 100644
--- a/benchmarks/gem_busy.c
+++ b/benchmarks/gem_busy.c
@@ -47,13 +47,7 @@
#include "igt_stats.h"
#include "i915/gem_mman.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define WRITE 0x1
#define IDLE 0x2
@@ -193,8 +187,8 @@ static int loop(unsigned ring, int reps, int ncpus, unsigned flags)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(obj);
execbuf.buffer_count = 2;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
if (__gem_execbuf(fd, &execbuf))
diff --git a/benchmarks/gem_create.c b/benchmarks/gem_create.c
index bb3e7b64..5dd996c9 100644
--- a/benchmarks/gem_create.c
+++ b/benchmarks/gem_create.c
@@ -47,9 +47,6 @@
#define OBJECT_SIZE (1<<23)
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
static double elapsed(const struct timespec *start,
const struct timespec *end)
{
@@ -70,8 +67,8 @@ static void make_busy(int fd, uint32_t handle)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = (uintptr_t)&gem_exec;
execbuf.buffer_count = 1;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
diff --git a/benchmarks/gem_exec_ctx.c b/benchmarks/gem_exec_ctx.c
index e55275f3..a2e8d93e 100644
--- a/benchmarks/gem_exec_ctx.c
+++ b/benchmarks/gem_exec_ctx.c
@@ -48,9 +48,6 @@
enum mode { NOP, CREATE, SWITCH, DEFAULT };
#define SYNC 0x1
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
static double elapsed(const struct timespec *start,
const struct timespec *end)
{
@@ -99,8 +96,8 @@ static int loop(unsigned ring,
execbuf.buffers_ptr = (uintptr_t)&obj;
execbuf.buffer_count = 1;
execbuf.flags = ring;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (mode != DEFAULT) {
execbuf.rsvd1 = __gem_context_create_local(fd);
if (execbuf.rsvd1 == 0)
diff --git a/benchmarks/gem_exec_fault.c b/benchmarks/gem_exec_fault.c
index 3f58c77d..4bd05372 100644
--- a/benchmarks/gem_exec_fault.c
+++ b/benchmarks/gem_exec_fault.c
@@ -47,13 +47,7 @@
#include "intel_reg.h"
#include "igt_stats.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
static double elapsed(const struct timespec *start,
const struct timespec *end)
@@ -88,8 +82,8 @@ static int loop(uint64_t size, unsigned ring, int reps, int ncpus, unsigned flag
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = (uintptr_t)&obj;
execbuf.buffer_count = 1;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
if (__gem_execbuf(fd, &execbuf))
diff --git a/benchmarks/gem_exec_nop.c b/benchmarks/gem_exec_nop.c
index b327e2e8..0f17d232 100644
--- a/benchmarks/gem_exec_nop.c
+++ b/benchmarks/gem_exec_nop.c
@@ -45,13 +45,7 @@
#include "intel_reg.h"
#include "igt_stats.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define SYNC 0x1
#define WRITE 0x2
@@ -95,15 +89,15 @@ static int loop(unsigned ring, int reps, int ncpus, unsigned flags)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = (uintptr_t)obj;
execbuf.buffer_count = 2;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
if (__gem_execbuf(fd, &execbuf))
return 77;
}
- if (flags & WRITE && !(execbuf.flags & LOCAL_I915_EXEC_HANDLE_LUT))
+ if (flags & WRITE && !(execbuf.flags & I915_EXEC_HANDLE_LUT))
return 77;
all_nengine = 0;
diff --git a/benchmarks/gem_exec_reloc.c b/benchmarks/gem_exec_reloc.c
index 0a2454cb..a0736b51 100644
--- a/benchmarks/gem_exec_reloc.c
+++ b/benchmarks/gem_exec_reloc.c
@@ -42,9 +42,6 @@
#include "drmtest.h"
#include "i915/gem_mman.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
#define SKIP_RELOC 0x1
#define NO_RELOC 0x2
#define CYCLE_BATCH 0x4
@@ -131,9 +128,9 @@ static int run(unsigned batch_size,
execbuf.buffers_ptr = (uintptr_t)objects;
execbuf.buffer_count = num_objects + 1;
if (flags & LUT)
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
if (flags & NO_RELOC)
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
for (n = 0; n < num_relocs; n++) {
target[n] = hars_petruska_f54_1_random() % num_objects;
diff --git a/benchmarks/gem_syslatency.c b/benchmarks/gem_syslatency.c
index 40230630..456b1f4b 100644
--- a/benchmarks/gem_syslatency.c
+++ b/benchmarks/gem_syslatency.c
@@ -71,13 +71,7 @@ static void force_low_latency(void)
strerror(errno));
}
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
static void *gem_busyspin(void *arg)
{
@@ -111,8 +105,8 @@ static void *gem_busyspin(void *arg)
execbuf.buffers_ptr = (uintptr_t)&obj[1];
execbuf.buffer_count = 1;
}
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
diff --git a/lib/igt_dummyload.c b/lib/igt_dummyload.c
index ae0fb937..da15db96 100644
--- a/lib/igt_dummyload.c
+++ b/lib/igt_dummyload.c
@@ -57,10 +57,7 @@
* consume exactly a specific amount of time.
*/
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define MI_ARB_CHK (0x5 << 23)
diff --git a/tests/i915/gem_ctx_switch.c b/tests/i915/gem_ctx_switch.c
index 5f94dac8..54da9b96 100644
--- a/tests/i915/gem_ctx_switch.c
+++ b/tests/i915/gem_ctx_switch.c
@@ -41,9 +41,6 @@
#include <time.h>
#include "drm.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
#define INTERRUPTIBLE 0x1
#define QUEUE 0x2
@@ -152,8 +149,8 @@ static void single(int fd, uint32_t handle,
execbuf.buffer_count = 1;
execbuf.rsvd1 = contexts[0];
execbuf.flags = e2->flags;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
igt_require(__gem_execbuf(fd, &execbuf) == 0);
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = e2->flags;
@@ -244,8 +241,8 @@ static void all(int fd, uint32_t handle, unsigned flags, int timeout)
execbuf.buffers_ptr = to_user_pointer(obj + 1);
execbuf.buffer_count = 1;
execbuf.rsvd1 = contexts[0];
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
igt_require(__gem_execbuf(fd, &execbuf) == 0);
gem_sync(fd, handle);
diff --git a/tests/i915/gem_exec_await.c b/tests/i915/gem_exec_await.c
index 7817b483..af70a136 100644
--- a/tests/i915/gem_exec_await.c
+++ b/tests/i915/gem_exec_await.c
@@ -31,13 +31,7 @@
#include <sys/ioctl.h>
#include <sys/signal.h>
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
static double elapsed(const struct timespec *start, const struct timespec *end)
{
@@ -107,8 +101,8 @@ static void wide(int fd, int ring_size, int timeout, unsigned int flags)
exec[e].execbuf.buffers_ptr = to_user_pointer(exec[e].exec);
exec[e].execbuf.buffer_count = 1;
exec[e].execbuf.flags = (engines[e] |
- LOCAL_I915_EXEC_NO_RELOC |
- LOCAL_I915_EXEC_HANDLE_LUT);
+ I915_EXEC_NO_RELOC |
+ I915_EXEC_HANDLE_LUT);
if (flags & CONTEXTS) {
exec[e].execbuf.rsvd1 = gem_context_create(fd);
@@ -195,8 +189,8 @@ static void wide(int fd, int ring_size, int timeout, unsigned int flags)
clock_gettime(CLOCK_MONOTONIC, &start);
for (unsigned e = 0; e < nengine; e++) {
execbuf.flags = (engines[e] |
- LOCAL_I915_EXEC_NO_RELOC |
- LOCAL_I915_EXEC_HANDLE_LUT);
+ I915_EXEC_NO_RELOC |
+ I915_EXEC_HANDLE_LUT);
gem_execbuf(fd, &execbuf);
}
clock_gettime(CLOCK_MONOTONIC, &now);
diff --git a/tests/i915/gem_exec_create.c b/tests/i915/gem_exec_create.c
index e3ceb3f5..8e11c9ed 100644
--- a/tests/i915/gem_exec_create.c
+++ b/tests/i915/gem_exec_create.c
@@ -40,13 +40,7 @@
#include <time.h>
#include "drm.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
static double elapsed(const struct timespec *start, const struct timespec *end)
{
@@ -75,8 +69,8 @@ static void all(int fd, unsigned flags, int timeout, int ncpus)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
diff --git a/tests/i915/gem_exec_latency.c b/tests/i915/gem_exec_latency.c
index d2159f31..c123b0f7 100644
--- a/tests/i915/gem_exec_latency.c
+++ b/tests/i915/gem_exec_latency.c
@@ -47,13 +47,7 @@
#include "i915/gem_ring.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define LIVE 0x1
#define CORK 0x2
@@ -136,7 +130,7 @@ static void latency_on_ring(int fd,
execbuf.buffers_ptr = to_user_pointer(&obj[1]);
execbuf.buffer_count = 2;
execbuf.flags = ring;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC | LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC | I915_EXEC_HANDLE_LUT;
memset(obj, 0, sizeof(obj));
obj[1].handle = gem_create(fd, 4096);
@@ -288,7 +282,7 @@ static void latency_from_ring(int fd,
execbuf.buffers_ptr = to_user_pointer(&obj[1]);
execbuf.buffer_count = 2;
execbuf.flags = ring;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC | LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC | I915_EXEC_HANDLE_LUT;
execbuf.rsvd1 = ctx[1];
memset(obj, 0, sizeof(obj));
diff --git a/tests/i915/gem_exec_lut_handle.c b/tests/i915/gem_exec_lut_handle.c
index 6f227628..f64285db 100644
--- a/tests/i915/gem_exec_lut_handle.c
+++ b/tests/i915/gem_exec_lut_handle.c
@@ -41,11 +41,6 @@
IGT_TEST_DESCRIPTION("Exercises the basic execbuffer using the handle LUT"
" interface.");
-#define BATCH_SIZE (1024*1024)
-
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
#define MAX_NUM_EXEC 2048
#define MAX_NUM_RELOC 4096
@@ -74,7 +69,7 @@ static int has_exec_lut(int fd)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer((gem_exec + MAX_NUM_EXEC));
execbuf.buffer_count = 1;
- execbuf.flags = LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags = I915_EXEC_HANDLE_LUT;
return __gem_execbuf(fd, &execbuf) == 0;
}
@@ -150,9 +145,9 @@ igt_simple_main
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(objects);
execbuf.buffer_count = n + 1;
- execbuf.flags = LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags = I915_EXEC_HANDLE_LUT;
if (p->flags & NO_RELOC)
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
for (j = 0; j < m; j++) {
target[j] = hars_petruska_f54_1_random() % n;
@@ -181,7 +176,7 @@ igt_simple_main
gem_exec[MAX_NUM_EXEC].handle = cycle[c];
elapsed[i][1] = ELAPSED(&start, &end);
- execbuf.flags &= ~LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags &= ~I915_EXEC_HANDLE_LUT;
for (j = 0; j < m; j++)
reloc[j].target_handle = objects[target[j]].handle;
diff --git a/tests/i915/gem_exec_nop.c b/tests/i915/gem_exec_nop.c
index 64ddb3db..6b255b55 100644
--- a/tests/i915/gem_exec_nop.c
+++ b/tests/i915/gem_exec_nop.c
@@ -45,13 +45,7 @@
#include <time.h>
#include "drm.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_FLAGS (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_FLAGS (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define MAX_PRIO LOCAL_I915_CONTEXT_MAX_USER_PRIORITY
#define MIN_PRIO LOCAL_I915_CONTEXT_MIN_USER_PRIORITY
@@ -83,8 +77,8 @@ static double nop_on_ring(int fd, uint32_t handle,
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
execbuf.flags = e->flags;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = e->flags;
gem_execbuf(fd, &execbuf);
@@ -462,8 +456,8 @@ static void parallel(int fd, uint32_t handle, int timeout)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
@@ -524,8 +518,8 @@ static void independent(int fd, uint32_t handle, int timeout)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
@@ -579,8 +573,8 @@ static void multiple(int fd,
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
execbuf.flags = e->flags;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = e->flags;
gem_execbuf(fd, &execbuf);
@@ -651,8 +645,8 @@ static void series(int fd, uint32_t handle, int timeout)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
@@ -747,8 +741,8 @@ static void sequential(int fd, uint32_t handle, unsigned flags, int timeout)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(obj);
execbuf.buffer_count = 2;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
igt_require(__gem_execbuf(fd, &execbuf) == 0);
if (flags & CONTEXT) {
@@ -939,8 +933,8 @@ static void preempt(int fd, uint32_t handle,
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
execbuf.flags = e->flags;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = e->flags;
gem_execbuf(fd, &execbuf);
diff --git a/tests/i915/gem_exec_parallel.c b/tests/i915/gem_exec_parallel.c
index 66fe1853..94d35f85 100644
--- a/tests/i915/gem_exec_parallel.c
+++ b/tests/i915/gem_exec_parallel.c
@@ -32,13 +32,7 @@
#include "igt.h"
#include "igt_gt.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define VERIFY 0
@@ -121,8 +115,8 @@ static void *thread(void *data)
execbuf.buffers_ptr = to_user_pointer(obj);
execbuf.buffer_count = 2;
execbuf.flags = t->engine;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (t->gen < 6)
execbuf.flags |= I915_EXEC_SECURE;
if (t->flags & CONTEXTS) {
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index 4112e145..a9cd5cfd 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -29,13 +29,7 @@
IGT_TEST_DESCRIPTION("Basic sanity check of execbuf-ioctl relocations.");
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
static uint32_t find_last_set(uint64_t x)
{
@@ -407,7 +401,7 @@ static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
execbuf.buffers_ptr = to_user_pointer(&obj);
execbuf.buffer_count = 1;
if (flags & NORELOC)
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
for (reloc_offset = 4096 - 8; reloc_offset <= 4096 + 8; reloc_offset += 4) {
igt_spin_t *spin = NULL;
diff --git a/tests/i915/gem_exec_store.c b/tests/i915/gem_exec_store.c
index 677db6f6..a98e9126 100644
--- a/tests/i915/gem_exec_store.c
+++ b/tests/i915/gem_exec_store.c
@@ -32,10 +32,7 @@
#include "igt_gt.h"
#include <strings.h>
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
static void store_dword(int fd, const struct intel_execution_engine2 *e)
{
diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c
index f0c53daf..d2bc1c8f 100644
--- a/tests/i915/gem_exec_suspend.c
+++ b/tests/i915/gem_exec_suspend.c
@@ -43,10 +43,7 @@
#define HIBERNATE 5
#define mode(x) ((x) & 0xff)
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define UNCACHED (0<<8)
#define CACHED (1<<8)
diff --git a/tests/i915/gem_exec_whisper.c b/tests/i915/gem_exec_whisper.c
index 153854d1..e6ca47f2 100644
--- a/tests/i915/gem_exec_whisper.c
+++ b/tests/i915/gem_exec_whisper.c
@@ -34,13 +34,7 @@
#include "igt_rand.h"
#include "igt_sysfs.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
#define VERIFY 0
@@ -273,8 +267,8 @@ static void whisper(int fd, unsigned engine, unsigned flags)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(tmp);
execbuf.buffer_count = 2;
- execbuf.flags = LOCAL_I915_EXEC_HANDLE_LUT;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
+ execbuf.flags = I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
if (gen < 6)
execbuf.flags |= I915_EXEC_SECURE;
igt_require(__gem_execbuf(fd, &execbuf) == 0);
diff --git a/tests/i915/gem_streaming_writes.c b/tests/i915/gem_streaming_writes.c
index e83d69de..2d6bf3a8 100644
--- a/tests/i915/gem_streaming_writes.c
+++ b/tests/i915/gem_streaming_writes.c
@@ -46,8 +46,6 @@
#define BLT_WRITE_RGB (1<<20)
#define BLT_WRITE_ARGB (BLT_WRITE_ALPHA | BLT_WRITE_RGB)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
IGT_TEST_DESCRIPTION("Test of streaming writes into active GPU sources");
#define SRC 0
@@ -102,7 +100,7 @@ static void test_streaming(int fd, int mode, int sync)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(exec);
execbuf.buffer_count = 2;
- execbuf.flags = LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags = I915_EXEC_HANDLE_LUT;
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
igt_require(__gem_execbuf(fd, &execbuf) == 0);
@@ -115,7 +113,7 @@ static void test_streaming(int fd, int mode, int sync)
for (i = 0; i < 64; i++) {
reloc[2*i+0].offset = 64*i + 4 * sizeof(uint32_t);
reloc[2*i+0].delta = 0;
- reloc[2*i+0].target_handle = execbuf.flags & LOCAL_I915_EXEC_HANDLE_LUT ? DST : dst;
+ reloc[2*i+0].target_handle = execbuf.flags & I915_EXEC_HANDLE_LUT ? DST : dst;
reloc[2*i+0].presumed_offset = dst_offset;
reloc[2*i+0].read_domains = I915_GEM_DOMAIN_RENDER;
reloc[2*i+0].write_domain = I915_GEM_DOMAIN_RENDER;
@@ -124,7 +122,7 @@ static void test_streaming(int fd, int mode, int sync)
if (has_64bit_reloc)
reloc[2*i+1].offset += sizeof(uint32_t);
reloc[2*i+1].delta = 0;
- reloc[2*i+1].target_handle = execbuf.flags & LOCAL_I915_EXEC_HANDLE_LUT ? SRC : src;
+ reloc[2*i+1].target_handle = execbuf.flags & I915_EXEC_HANDLE_LUT ? SRC : src;
reloc[2*i+1].presumed_offset = src_offset;
reloc[2*i+1].read_domains = I915_GEM_DOMAIN_RENDER;
reloc[2*i+1].write_domain = 0;
@@ -256,7 +254,7 @@ static void test_batch(int fd, int mode, int reverse)
memset(reloc, 0, sizeof(reloc));
reloc[0].offset = 4 * sizeof(uint32_t);
reloc[0].delta = 0;
- reloc[0].target_handle = execbuf.flags & LOCAL_I915_EXEC_HANDLE_LUT ? DST : dst;
+ reloc[0].target_handle = execbuf.flags & I915_EXEC_HANDLE_LUT ? DST : dst;
reloc[0].presumed_offset = dst_offset;
reloc[0].read_domains = I915_GEM_DOMAIN_RENDER;
reloc[0].write_domain = I915_GEM_DOMAIN_RENDER;
@@ -265,7 +263,7 @@ static void test_batch(int fd, int mode, int reverse)
if (has_64bit_reloc)
reloc[1].offset += sizeof(uint32_t);
reloc[1].delta = 0;
- reloc[1].target_handle = execbuf.flags & LOCAL_I915_EXEC_HANDLE_LUT ? SRC : src;
+ reloc[1].target_handle = execbuf.flags & I915_EXEC_HANDLE_LUT ? SRC : src;
reloc[1].presumed_offset = src_offset;
reloc[1].read_domains = I915_GEM_DOMAIN_RENDER;
reloc[1].write_domain = 0;
@@ -296,11 +294,11 @@ static void test_batch(int fd, int mode, int reverse)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(exec);
execbuf.buffer_count = 3;
- execbuf.flags = LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags = I915_EXEC_HANDLE_LUT;
if (gem_has_blt(fd))
execbuf.flags |= I915_EXEC_BLT;
if (__gem_execbuf(fd, &execbuf)) {
- execbuf.flags &= ~LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags &= ~I915_EXEC_HANDLE_LUT;
gem_execbuf(fd, &execbuf);
}
execbuf.flags |= I915_EXEC_NO_RELOC;
diff --git a/tests/i915/gem_sync.c b/tests/i915/gem_sync.c
index b50ad4f2..feca7447 100644
--- a/tests/i915/gem_sync.c
+++ b/tests/i915/gem_sync.c
@@ -30,16 +30,10 @@
#include "igt.h"
#include "igt_sysfs.h"
-#define LOCAL_I915_EXEC_NO_RELOC (1<<11)
-#define LOCAL_I915_EXEC_HANDLE_LUT (1<<12)
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
#define MAX_PRIO LOCAL_I915_CONTEXT_MAX_USER_PRIORITY
#define MIN_PRIO LOCAL_I915_CONTEXT_MIN_USER_PRIORITY
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
IGT_TEST_DESCRIPTION("Basic check of ring<->ring write synchronisation.");
@@ -516,8 +510,8 @@ store_ring(int fd, unsigned ring, int num_children, int timeout)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(object);
execbuf.flags = ied_flags(&ied, child);
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
if (gen < 6)
execbuf.flags |= I915_EXEC_SECURE;
@@ -620,8 +614,8 @@ switch_ring(int fd, unsigned ring, int num_children, int timeout)
memset(&c->execbuf, 0, sizeof(c->execbuf));
c->execbuf.buffers_ptr = to_user_pointer(c->object);
c->execbuf.flags = ied_flags(&ied, child);
- c->execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
- c->execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
+ c->execbuf.flags |= I915_EXEC_NO_RELOC;
+ c->execbuf.flags |= I915_EXEC_HANDLE_LUT;
if (gen < 6)
c->execbuf.flags |= I915_EXEC_SECURE;
c->execbuf.rsvd1 = gem_context_create(fd);
@@ -784,8 +778,8 @@ __store_many(int fd, unsigned ring, int timeout, unsigned long *cycles)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(object);
execbuf.flags = ring;
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
if (gen < 6)
execbuf.flags |= I915_EXEC_SECURE;
@@ -994,8 +988,8 @@ store_all(int fd, int num_children, int timeout)
memset(&execbuf, 0, sizeof(execbuf));
execbuf.buffers_ptr = to_user_pointer(object);
- execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
- execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
+ execbuf.flags |= I915_EXEC_NO_RELOC;
+ execbuf.flags |= I915_EXEC_HANDLE_LUT;
if (gen < 6)
execbuf.flags |= I915_EXEC_SECURE;
diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
index 7d9a5cfd..77aaac5c 100644
--- a/tests/i915/i915_module_load.c
+++ b/tests/i915/i915_module_load.c
@@ -36,11 +36,7 @@
#include <sys/ioctl.h>
#include <fcntl.h>
-
-#define LOCAL_I915_EXEC_BSD_SHIFT (13)
-#define LOCAL_I915_EXEC_BSD_MASK (3 << LOCAL_I915_EXEC_BSD_SHIFT)
-
-#define ENGINE_MASK (I915_EXEC_RING_MASK | LOCAL_I915_EXEC_BSD_MASK)
+#define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
static void store_dword(int fd, unsigned ring)
{
--
2.26.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for Macros fixes: Removed unused & redundant macros
2020-04-28 13:31 [igt-dev] [PATCH] [PATCH i-g-t]Macros fixes: Removed unused & redundant macros ranjeet1.kumar
@ 2020-04-29 10:07 ` Patchwork
2020-04-29 12:07 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
1 sibling, 0 replies; 3+ messages in thread
From: Patchwork @ 2020-04-29 10:07 UTC (permalink / raw)
To: ranjeet1.kumar; +Cc: igt-dev
== Series Details ==
Series: Macros fixes: Removed unused & redundant macros
URL : https://patchwork.freedesktop.org/series/76710/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8390 -> IGTPW_4515
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/index.html
Changes
-------
No changes found
Participating hosts (48 -> 42)
------------------------------
Missing (6): fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-kbl-7500u fi-ctg-p8600 fi-byt-clapper
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5614 -> IGTPW_4515
CI-20190529: 20190529
CI_DRM_8390: 89473e10666c78c4df9e92c9caf03d7311c291cb @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_4515: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/index.html
IGT_5614: d095827add11d4e8158b87683971ee659749d9a4 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/index.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 3+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for Macros fixes: Removed unused & redundant macros
2020-04-28 13:31 [igt-dev] [PATCH] [PATCH i-g-t]Macros fixes: Removed unused & redundant macros ranjeet1.kumar
2020-04-29 10:07 ` [igt-dev] ✓ Fi.CI.BAT: success for Macros " Patchwork
@ 2020-04-29 12:07 ` Patchwork
1 sibling, 0 replies; 3+ messages in thread
From: Patchwork @ 2020-04-29 12:07 UTC (permalink / raw)
To: ranjeet1.kumar; +Cc: igt-dev
== Series Details ==
Series: Macros fixes: Removed unused & redundant macros
URL : https://patchwork.freedesktop.org/series/76710/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8390_full -> IGTPW_4515_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/index.html
Known issues
------------
Here are the changes found in IGTPW_4515_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_cursor_crc@pipe-a-cursor-128x128-sliding:
- shard-apl: [PASS][1] -> [FAIL][2] ([i915#54] / [i915#95]) +2 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl8/igt@kms_cursor_crc@pipe-a-cursor-128x128-sliding.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl7/igt@kms_cursor_crc@pipe-a-cursor-128x128-sliding.html
* igt@kms_cursor_crc@pipe-a-cursor-256x256-offscreen:
- shard-kbl: [PASS][3] -> [FAIL][4] ([i915#54] / [i915#93] / [i915#95]) +1 similar issue
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl3/igt@kms_cursor_crc@pipe-a-cursor-256x256-offscreen.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl4/igt@kms_cursor_crc@pipe-a-cursor-256x256-offscreen.html
* igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic:
- shard-glk: [PASS][5] -> [FAIL][6] ([i915#72])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-glk7/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-glk8/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html
* igt@kms_cursor_legacy@all-pipes-torture-bo:
- shard-hsw: [PASS][7] -> [DMESG-WARN][8] ([i915#128])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-hsw4/igt@kms_cursor_legacy@all-pipes-torture-bo.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-hsw4/igt@kms_cursor_legacy@all-pipes-torture-bo.html
* igt@kms_cursor_legacy@flip-vs-cursor-crc-legacy:
- shard-kbl: [PASS][9] -> [FAIL][10] ([i915#1566] / [i915#93] / [i915#95])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl4/igt@kms_cursor_legacy@flip-vs-cursor-crc-legacy.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl1/igt@kms_cursor_legacy@flip-vs-cursor-crc-legacy.html
* igt@kms_draw_crc@draw-method-rgb565-render-ytiled:
- shard-glk: [PASS][11] -> [FAIL][12] ([i915#52] / [i915#54]) +3 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-glk5/igt@kms_draw_crc@draw-method-rgb565-render-ytiled.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-glk6/igt@kms_draw_crc@draw-method-rgb565-render-ytiled.html
* igt@kms_flip_tiling@flip-changes-tiling-yf:
- shard-kbl: [PASS][13] -> [FAIL][14] ([i915#699] / [i915#93] / [i915#95])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl2/igt@kms_flip_tiling@flip-changes-tiling-yf.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl3/igt@kms_flip_tiling@flip-changes-tiling-yf.html
- shard-apl: [PASS][15] -> [FAIL][16] ([i915#95])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl2/igt@kms_flip_tiling@flip-changes-tiling-yf.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl3/igt@kms_flip_tiling@flip-changes-tiling-yf.html
* igt@kms_frontbuffer_tracking@fbc-suspend:
- shard-apl: [PASS][17] -> [DMESG-WARN][18] ([i915#180] / [i915#95])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl6/igt@kms_frontbuffer_tracking@fbc-suspend.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl6/igt@kms_frontbuffer_tracking@fbc-suspend.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
- shard-apl: [PASS][19] -> [DMESG-WARN][20] ([i915#180])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c.html
* igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes:
- shard-kbl: [PASS][21] -> [DMESG-WARN][22] ([i915#180]) +1 similar issue
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl3/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
* igt@kms_psr@psr2_cursor_blt:
- shard-iclb: [PASS][23] -> [SKIP][24] ([fdo#109441]) +1 similar issue
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-iclb2/igt@kms_psr@psr2_cursor_blt.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-iclb4/igt@kms_psr@psr2_cursor_blt.html
#### Possible fixes ####
* igt@gem_workarounds@suspend-resume-context:
- shard-apl: [DMESG-WARN][25] ([i915#180]) -> [PASS][26] +2 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl4/igt@gem_workarounds@suspend-resume-context.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl6/igt@gem_workarounds@suspend-resume-context.html
* igt@kms_cursor_crc@pipe-a-cursor-128x128-offscreen:
- shard-kbl: [FAIL][27] ([i915#54] / [i915#93] / [i915#95]) -> [PASS][28] +1 similar issue
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl1/igt@kms_cursor_crc@pipe-a-cursor-128x128-offscreen.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl2/igt@kms_cursor_crc@pipe-a-cursor-128x128-offscreen.html
* igt@kms_cursor_legacy@flip-vs-cursor-crc-atomic:
- shard-kbl: [FAIL][29] ([i915#1566] / [i915#93] / [i915#95]) -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl3/igt@kms_cursor_legacy@flip-vs-cursor-crc-atomic.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl1/igt@kms_cursor_legacy@flip-vs-cursor-crc-atomic.html
* igt@kms_draw_crc@draw-method-rgb565-pwrite-xtiled:
- shard-glk: [FAIL][31] ([i915#52] / [i915#54]) -> [PASS][32] +1 similar issue
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-glk5/igt@kms_draw_crc@draw-method-rgb565-pwrite-xtiled.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-glk5/igt@kms_draw_crc@draw-method-rgb565-pwrite-xtiled.html
* igt@kms_draw_crc@draw-method-xrgb8888-mmap-cpu-untiled:
- shard-kbl: [FAIL][33] ([i915#177] / [i915#52] / [i915#54] / [i915#93] / [i915#95]) -> [PASS][34] +1 similar issue
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl2/igt@kms_draw_crc@draw-method-xrgb8888-mmap-cpu-untiled.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl4/igt@kms_draw_crc@draw-method-xrgb8888-mmap-cpu-untiled.html
- shard-apl: [FAIL][35] ([i915#52] / [i915#54] / [i915#95]) -> [PASS][36] +1 similar issue
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl4/igt@kms_draw_crc@draw-method-xrgb8888-mmap-cpu-untiled.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl1/igt@kms_draw_crc@draw-method-xrgb8888-mmap-cpu-untiled.html
* igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled:
- shard-kbl: [FAIL][37] ([fdo#108145] / [i915#177] / [i915#52] / [i915#54] / [i915#93] / [i915#95]) -> [PASS][38]
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl2/igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl4/igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled.html
- shard-apl: [FAIL][39] ([fdo#108145] / [i915#52] / [i915#54] / [i915#95]) -> [PASS][40]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl2/igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl1/igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-untiled.html
* {igt@kms_flip@flip-vs-expired-vblank-interruptible@a-hdmi-a2}:
- shard-glk: [FAIL][41] ([i915#79]) -> [PASS][42] +1 similar issue
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-glk5/igt@kms_flip@flip-vs-expired-vblank-interruptible@a-hdmi-a2.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-glk8/igt@kms_flip@flip-vs-expired-vblank-interruptible@a-hdmi-a2.html
* {igt@kms_flip@flip-vs-suspend@c-dp1}:
- shard-kbl: [DMESG-WARN][43] ([i915#180]) -> [PASS][44] +8 similar issues
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl2/igt@kms_flip@flip-vs-suspend@c-dp1.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl2/igt@kms_flip@flip-vs-suspend@c-dp1.html
* igt@kms_flip_tiling@flip-changes-tiling-y:
- shard-kbl: [FAIL][45] ([i915#699] / [i915#93] / [i915#95]) -> [PASS][46]
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl7/igt@kms_flip_tiling@flip-changes-tiling-y.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl1/igt@kms_flip_tiling@flip-changes-tiling-y.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-b:
- shard-snb: [SKIP][47] ([fdo#109271]) -> [PASS][48] +1 similar issue
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-snb4/igt@kms_pipe_crc_basic@hang-read-crc-pipe-b.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-snb5/igt@kms_pipe_crc_basic@hang-read-crc-pipe-b.html
* igt@kms_plane_cursor@pipe-a-viewport-size-64:
- shard-kbl: [FAIL][49] ([i915#1559] / [i915#93] / [i915#95]) -> [PASS][50]
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl4/igt@kms_plane_cursor@pipe-a-viewport-size-64.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl2/igt@kms_plane_cursor@pipe-a-viewport-size-64.html
- shard-apl: [FAIL][51] ([i915#1559] / [i915#95]) -> [PASS][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl8/igt@kms_plane_cursor@pipe-a-viewport-size-64.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl7/igt@kms_plane_cursor@pipe-a-viewport-size-64.html
* igt@kms_psr@psr2_sprite_plane_move:
- shard-iclb: [SKIP][53] ([fdo#109441]) -> [PASS][54] +3 similar issues
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-iclb1/igt@kms_psr@psr2_sprite_plane_move.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html
* igt@perf@polling:
- shard-glk: [SKIP][55] ([fdo#109271]) -> [PASS][56]
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-glk1/igt@perf@polling.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-glk5/igt@perf@polling.html
- shard-tglb: [SKIP][57] ([i915#405]) -> [PASS][58]
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-tglb1/igt@perf@polling.html
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-tglb1/igt@perf@polling.html
- shard-apl: [SKIP][59] ([fdo#109271]) -> [PASS][60]
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl2/igt@perf@polling.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl6/igt@perf@polling.html
- shard-kbl: [SKIP][61] ([fdo#109271]) -> [PASS][62]
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl2/igt@perf@polling.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl3/igt@perf@polling.html
- shard-hsw: [SKIP][63] ([fdo#109271]) -> [PASS][64]
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-hsw7/igt@perf@polling.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-hsw2/igt@perf@polling.html
- shard-iclb: [SKIP][65] ([i915#405]) -> [PASS][66]
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-iclb3/igt@perf@polling.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-iclb1/igt@perf@polling.html
#### Warnings ####
* igt@i915_pm_dc@dc6-psr:
- shard-snb: [INCOMPLETE][67] ([i915#82]) -> [SKIP][68] ([fdo#109271])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-snb6/igt@i915_pm_dc@dc6-psr.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-snb1/igt@i915_pm_dc@dc6-psr.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-7efc:
- shard-kbl: [FAIL][69] ([fdo#108145] / [i915#265]) -> [FAIL][70] ([fdo#108145] / [i915#265] / [i915#93] / [i915#95])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-kbl3/igt@kms_plane_alpha_blend@pipe-a-alpha-7efc.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-kbl3/igt@kms_plane_alpha_blend@pipe-a-alpha-7efc.html
* igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
- shard-apl: [FAIL][71] ([fdo#108145] / [i915#265]) -> [FAIL][72] ([fdo#108145] / [i915#265] / [i915#95])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8390/shard-apl7/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/shard-apl8/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[i915#128]: https://gitlab.freedesktop.org/drm/intel/issues/128
[i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542
[i915#1559]: https://gitlab.freedesktop.org/drm/intel/issues/1559
[i915#1566]: https://gitlab.freedesktop.org/drm/intel/issues/1566
[i915#177]: https://gitlab.freedesktop.org/drm/intel/issues/177
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#405]: https://gitlab.freedesktop.org/drm/intel/issues/405
[i915#52]: https://gitlab.freedesktop.org/drm/intel/issues/52
[i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
[i915#699]: https://gitlab.freedesktop.org/drm/intel/issues/699
[i915#72]: https://gitlab.freedesktop.org/drm/intel/issues/72
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
[i915#82]: https://gitlab.freedesktop.org/drm/intel/issues/82
[i915#93]: https://gitlab.freedesktop.org/drm/intel/issues/93
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
Participating hosts (10 -> 8)
------------------------------
Missing (2): pig-skl-6260u pig-glk-j5005
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5614 -> IGTPW_4515
* Piglit: piglit_4509 -> None
CI-20190529: 20190529
CI_DRM_8390: 89473e10666c78c4df9e92c9caf03d7311c291cb @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_4515: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/index.html
IGT_5614: d095827add11d4e8158b87683971ee659749d9a4 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4515/index.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-04-29 12:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-28 13:31 [igt-dev] [PATCH] [PATCH i-g-t]Macros fixes: Removed unused & redundant macros ranjeet1.kumar
2020-04-29 10:07 ` [igt-dev] ✓ Fi.CI.BAT: success for Macros " Patchwork
2020-04-29 12:07 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
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.