[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts from execlist submission (rev2)

2020-12-19 Thread Patchwork
== Series Details ==

Series: series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts 
from execlist submission (rev2)
URL   : https://patchwork.freedesktop.org/series/85105/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
51e53ab625b5 drm/i915/gt: Split logical ring contexts from execlist submission
-:1882: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1882: FILE: drivers/gpu/drm/i915/gt/intel_execlists_submission.c:3406:
+static int virtual_context_pre_pin(struct intel_context *ce,
+struct i915_gem_ww_ctx *ww,

-:1974: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does 
MAINTAINERS need updating?
#1974: 
new file mode 100644

-:2005: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'count' - possible 
side-effects?
#2005: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:27:
+#define LRI(count, flags) ((flags) << 6 | (count) | BUILD_BUG_ON_ZERO(count >= 
BIT(6)))

-:2007: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2007: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:29:
+#define REG(x) (((x) >> 2) | BUILD_BUG_ON_ZERO(x >= 0x200))

-:2008: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:2008: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:6071: WARNING:MEMORY_BARRIER: memory barrier without comment
#6071: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:634:
+   wmb();

-:6100: WARNING:MEMORY_BARRIER: memory barrier without comment
#6100: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:663:
+   wmb();

-:6232: WARNING:MEMORY_BARRIER: memory barrier without comment
#6232: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:795:
+   wmb();

-:6780: WARNING:MEMORY_BARRIER: memory barrier without comment
#6780: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1343:
+   wmb();

-:7194: WARNING:LINE_SPACING: Missing a blank line after declarations
#7194: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1757:
+   struct i915_request *rq;
+   IGT_TIMEOUT(end_time);

total: 1 errors, 6 warnings, 4 checks, 5502 lines checked
e847d1d180bd drm/i915/gt: Provide a utility to create a scratch buffer


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


[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts from execlist submission

2020-12-18 Thread Patchwork
== Series Details ==

Series: series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts 
from execlist submission
URL   : https://patchwork.freedesktop.org/series/85105/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
5dd14c1bb771 drm/i915/gt: Split logical ring contexts from execlist submission
-:1882: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1882: FILE: drivers/gpu/drm/i915/gt/intel_execlists_submission.c:3406:
+static int virtual_context_pre_pin(struct intel_context *ce,
+struct i915_gem_ww_ctx *ww,

-:1974: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does 
MAINTAINERS need updating?
#1974: 
new file mode 100644

-:2005: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'count' - possible 
side-effects?
#2005: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:27:
+#define LRI(count, flags) ((flags) << 6 | (count) | BUILD_BUG_ON_ZERO(count >= 
BIT(6)))

-:2007: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2007: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:29:
+#define REG(x) (((x) >> 2) | BUILD_BUG_ON_ZERO(x >= 0x200))

-:2008: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:2008: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:6071: WARNING:MEMORY_BARRIER: memory barrier without comment
#6071: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:634:
+   wmb();

-:6100: WARNING:MEMORY_BARRIER: memory barrier without comment
#6100: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:663:
+   wmb();

-:6232: WARNING:MEMORY_BARRIER: memory barrier without comment
#6232: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:795:
+   wmb();

-:6780: WARNING:MEMORY_BARRIER: memory barrier without comment
#6780: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1343:
+   wmb();

-:7194: WARNING:LINE_SPACING: Missing a blank line after declarations
#7194: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1757:
+   struct i915_request *rq;
+   IGT_TIMEOUT(end_time);

total: 1 errors, 6 warnings, 4 checks, 5502 lines checked
b9d0ad833b5c drm/i915/gt: Provide a utility to create a scratch buffer


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


[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts from execlist submission

2020-12-18 Thread Patchwork
== Series Details ==

Series: series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts 
from execlist submission
URL   : https://patchwork.freedesktop.org/series/85096/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
15e1c6d96de6 drm/i915/gt: Split logical ring contexts from execlist submission
-:1882: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1882: FILE: drivers/gpu/drm/i915/gt/intel_execlists_submission.c:3406:
+static int virtual_context_pre_pin(struct intel_context *ce,
+struct i915_gem_ww_ctx *ww,

-:1974: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does 
MAINTAINERS need updating?
#1974: 
new file mode 100644

-:2005: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'count' - possible 
side-effects?
#2005: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:27:
+#define LRI(count, flags) ((flags) << 6 | (count) | BUILD_BUG_ON_ZERO(count >= 
BIT(6)))

-:2007: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2007: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:29:
+#define REG(x) (((x) >> 2) | BUILD_BUG_ON_ZERO(x >= 0x200))

-:2008: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:2008: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:6071: WARNING:MEMORY_BARRIER: memory barrier without comment
#6071: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:634:
+   wmb();

-:6100: WARNING:MEMORY_BARRIER: memory barrier without comment
#6100: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:663:
+   wmb();

-:6232: WARNING:MEMORY_BARRIER: memory barrier without comment
#6232: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:795:
+   wmb();

-:6780: WARNING:MEMORY_BARRIER: memory barrier without comment
#6780: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1343:
+   wmb();

-:7194: WARNING:LINE_SPACING: Missing a blank line after declarations
#7194: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1757:
+   struct i915_request *rq;
+   IGT_TIMEOUT(end_time);

total: 1 errors, 6 warnings, 4 checks, 5502 lines checked
0c40aa440b7b drm/i915/gt: Provide a utility to create a scratch buffer


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


[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts from execlist submission

2020-12-18 Thread Patchwork
== Series Details ==

Series: series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts 
from execlist submission
URL   : https://patchwork.freedesktop.org/series/85095/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
0fd013ab16e2 drm/i915/gt: Split logical ring contexts from execlist submission
-:1882: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#1882: FILE: drivers/gpu/drm/i915/gt/intel_execlists_submission.c:3406:
+static int virtual_context_pre_pin(struct intel_context *ce,
+struct i915_gem_ww_ctx *ww,

-:1974: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does 
MAINTAINERS need updating?
#1974: 
new file mode 100644

-:2005: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'count' - possible 
side-effects?
#2005: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:27:
+#define LRI(count, flags) ((flags) << 6 | (count) | BUILD_BUG_ON_ZERO(count >= 
BIT(6)))

-:2007: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2007: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:29:
+#define REG(x) (((x) >> 2) | BUILD_BUG_ON_ZERO(x >= 0x200))

-:2008: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:2008: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#2008: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:2011: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#2011: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:33:
+#define END(total_state_size) 0, (total_state_size)

-:6092: WARNING:MEMORY_BARRIER: memory barrier without comment
#6092: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:634:
+   wmb();

-:6121: WARNING:MEMORY_BARRIER: memory barrier without comment
#6121: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:663:
+   wmb();

-:6253: WARNING:MEMORY_BARRIER: memory barrier without comment
#6253: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:795:
+   wmb();

-:6801: WARNING:MEMORY_BARRIER: memory barrier without comment
#6801: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1343:
+   wmb();

-:7215: WARNING:LINE_SPACING: Missing a blank line after declarations
#7215: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1757:
+   struct i915_request *rq;
+   IGT_TIMEOUT(end_time);

total: 2 errors, 6 warnings, 4 checks, 5523 lines checked
1d13fb4c2906 drm/i915/gt: Provide a utility to create a scratch buffer


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


[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts from execlist submission

2020-12-16 Thread Patchwork
== Series Details ==

Series: series starting with [CI,1/2] drm/i915/gt: Split logical ring contexts 
from execlist submission
URL   : https://patchwork.freedesktop.org/series/85017/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
d46ab2bc6044 drm/i915/gt: Split logical ring contexts from execlist submission
-:1796: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does 
MAINTAINERS need updating?
#1796: 
new file mode 100644

-:1827: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'count' - possible 
side-effects?
#1827: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:27:
+#define LRI(count, flags) ((flags) << 6 | (count) | BUILD_BUG_ON_ZERO(count >= 
BIT(6)))

-:1829: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#1829: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:29:
+#define REG(x) (((x) >> 2) | BUILD_BUG_ON_ZERO(x >= 0x200))

-:1830: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#1830: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:1830: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#1830: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:30:
+#define REG16(x) \
+   (((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x1)), \
+   (((x) >> 2) & 0x7f)

-:1833: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in 
parentheses
#1833: FILE: drivers/gpu/drm/i915/gt/intel_lrc.c:33:
+#define END(total_state_size) 0, (total_state_size)

-:5792: WARNING:MEMORY_BARRIER: memory barrier without comment
#5792: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:637:
+   wmb();

-:5821: WARNING:MEMORY_BARRIER: memory barrier without comment
#5821: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:666:
+   wmb();

-:5953: WARNING:MEMORY_BARRIER: memory barrier without comment
#5953: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:798:
+   wmb();

-:6501: WARNING:MEMORY_BARRIER: memory barrier without comment
#6501: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1346:
+   wmb();

-:6915: WARNING:LINE_SPACING: Missing a blank line after declarations
#6915: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:1760:
+   struct i915_request *rq;
+   IGT_TIMEOUT(end_time);

total: 2 errors, 6 warnings, 3 checks, 5222 lines checked
ec652f702ec9 drm/i915/gt: Provide a utility to create a scratch buffer


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