[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gt: Restore ce->signal flush before releasing virtual engine (rev2)

Patchwork patchwork at emeril.freedesktop.org
Tue Jan 5 16:15:05 UTC 2021


== Series Details ==

Series: drm/i915/gt: Restore ce->signal flush before releasing virtual engine (rev2)
URL   : https://patchwork.freedesktop.org/series/85493/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
7ceba4fd2e81 drm/i915/gt: Restore ce->signal flush before releasing virtual engine
-:14: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit bab0557c8dca ("drm/i915/gt: Remove virtual breadcrumb before transfer")'
#14: 
bab0557c8dca ("drm/i915/gt: Remove virtual breadcrumb before transfer"),

-:34: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#34: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:478:
+       struct i915_request *rq, *rn;$

-:35: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#35: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:479:
+       unsigned long flags;$

-:36: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#36: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:480:
+       bool release;$

-:38: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#38: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:482:
+       if (list_empty(&ce->signals))$

-:38: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#38: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:482:
+       if (list_empty(&ce->signals))
+               return false;

-:39: ERROR:CODE_INDENT: code indent should use tabs where possible
#39: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:483:
+               return false;$

-:39: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#39: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:483:
+               return false;$

-:41: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#41: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:485:
+       spin_lock_irqsave(&ce->signal_lock, flags);$

-:42: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#42: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:486:
+       list_for_each_entry_safe(rq, rn, &ce->signals, signal_link) {$

-:42: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#42: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:486:
+       list_for_each_entry_safe(rq, rn, &ce->signals, signal_link) {
+               GEM_BUG_ON(!__i915_request_is_complete(rq));

-:43: ERROR:CODE_INDENT: code indent should use tabs where possible
#43: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:487:
+               GEM_BUG_ON(!__i915_request_is_complete(rq));$

-:43: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#43: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:487:
+               GEM_BUG_ON(!__i915_request_is_complete(rq));$

-:44: ERROR:CODE_INDENT: code indent should use tabs where possible
#44: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:488:
+               if (!test_and_clear_bit(I915_FENCE_FLAG_SIGNAL,$

-:44: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#44: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:488:
+               if (!test_and_clear_bit(I915_FENCE_FLAG_SIGNAL,$

-:44: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (15, 23)
#44: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:488:
+               if (!test_and_clear_bit(I915_FENCE_FLAG_SIGNAL,
[...]
+                       continue;

-:45: ERROR:CODE_INDENT: code indent should use tabs where possible
#45: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:489:
+                                       &rq->fence.flags))$

-:45: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#45: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:489:
+                                       &rq->fence.flags))$

-:46: ERROR:CODE_INDENT: code indent should use tabs where possible
#46: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:490:
+                       continue;$

-:46: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#46: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:490:
+                       continue;$

-:48: ERROR:CODE_INDENT: code indent should use tabs where possible
#48: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:492:
+               list_del_rcu(&rq->signal_link);$

-:48: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#48: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:492:
+               list_del_rcu(&rq->signal_link);$

-:49: ERROR:CODE_INDENT: code indent should use tabs where possible
#49: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:493:
+               irq_signal_request(rq, b);$

-:49: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#49: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:493:
+               irq_signal_request(rq, b);$

-:50: ERROR:CODE_INDENT: code indent should use tabs where possible
#50: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:494:
+               i915_request_put(rq);$

-:50: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#50: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:494:
+               i915_request_put(rq);$

-:51: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#51: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:495:
+       }$

-:53: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#53: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:497:
+       release = remove_signaling_context(b, ce);$

-:54: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#54: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:498:
+       spin_unlock_irqrestore(&ce->signal_lock, flags);$

-:55: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#55: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:499:
+       if (release)$

-:55: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#55: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:499:
+       if (release)
+               intel_context_put(ce);

-:56: ERROR:CODE_INDENT: code indent should use tabs where possible
#56: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:500:
+               intel_context_put(ce);$

-:56: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#56: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:500:
+               intel_context_put(ce);$

-:58: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#58: FILE: drivers/gpu/drm/i915/gt/intel_breadcrumbs.c:502:
+       return true;$

total: 10 errors, 24 warnings, 0 checks, 104 lines checked




More information about the Intel-gfx mailing list