From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Dan Carpenter <error27@gmail.com>
Subject: [peterz-queue:locking/core 24/35] drivers/gpu/drm/i915/gt/intel_execlists_submission.c:891 assert_pending_valid() error: uninitialized symbol 'flags'.
Date: Thu, 6 Mar 2025 07:19:10 +0800 [thread overview]
Message-ID: <202503060731.nwgOwXQP-lkp@intel.com> (raw)
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
TO: Marco Elver <elver@google.com>
CC: Peter Zijlstra <peterz@infradead.org>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git locking/core
head: c90cac58f80cc4268e8799fe51a0ced91220428a
commit: 810bdf94357610e2f9fb997a486a16a0acc0efd2 [24/35] compiler-capability-analysis: Remove __cond_lock() function-like helper
:::::: branch date: 32 hours ago
:::::: commit date: 32 hours ago
config: i386-randconfig-141-20250306 (https://download.01.org/0day-ci/archive/20250306/202503060731.nwgOwXQP-lkp@intel.com/config)
compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Closes: https://lore.kernel.org/r/202503060731.nwgOwXQP-lkp@intel.com/
New smatch warnings:
drivers/gpu/drm/i915/gt/intel_execlists_submission.c:891 assert_pending_valid() error: uninitialized symbol 'flags'.
Old smatch warnings:
drivers/gpu/drm/i915/gt/intel_execlists_submission.c:393 __unwind_incomplete_requests() error: uninitialized symbol 'pl'.
drivers/gpu/drm/i915/gt/intel_execlists_submission.c:1607 execlists_dequeue() warn: inconsistent returns '&sched_engine->lock'.
drivers/gpu/drm/i915/gt/intel_execlists_submission.c:3942 execlists_create_virtual() warn: assigning (-2) to unsigned variable 've->base.instance'
drivers/gpu/drm/i915/gt/intel_execlists_submission.c:3943 execlists_create_virtual() warn: assigning (-2) to unsigned variable 've->base.uabi_instance'
vim +/flags +891 drivers/gpu/drm/i915/gt/intel_execlists_submission.c
f1042cc8537713 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-01-29 768
e73263367088bf drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2021-01-13 769 static __maybe_unused noinline bool
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 770 assert_pending_valid(const struct intel_engine_execlists *execlists,
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 771 const char *msg)
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 772 {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 773 struct intel_engine_cs *engine =
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 774 container_of(execlists, typeof(*engine), execlists);
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 775 struct i915_request * const *port, *rq, *prev = NULL;
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 776 struct intel_context *ce = NULL;
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 777 u32 ccid = -1;
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 778
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 779 trace_ports(execlists, msg, execlists->pending);
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 780
f1042cc8537713 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-01-29 781 /* We may be messing around with the lists during reset, lalala */
22916bad07a5d4 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Matthew Brost 2021-06-17 782 if (reset_in_progress(engine))
f1042cc8537713 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-01-29 783 return true;
f1042cc8537713 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-01-29 784
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 785 if (!execlists->pending[0]) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 786 GEM_TRACE_ERR("%s: Nothing pending for promotion!\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 787 engine->name);
df403069029dc6 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-08-16 788 return false;
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 789 }
df403069029dc6 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-08-16 790
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 791 if (execlists->pending[execlists_num_ports(execlists)]) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 792 GEM_TRACE_ERR("%s: Excess pending[%d] for promotion!\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 793 engine->name, execlists_num_ports(execlists));
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 794 return false;
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 795 }
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 796
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 797 for (port = execlists->pending; (rq = *port); port++) {
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 798 unsigned long flags;
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 799 bool ok = true;
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 800
80aac91b27e130 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-03 801 GEM_BUG_ON(!kref_read(&rq->fence.refcount));
80aac91b27e130 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-03 802 GEM_BUG_ON(!i915_request_is_active(rq));
80aac91b27e130 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-03 803
9f3ccd40acf4a3 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-20 804 if (ce == rq->context) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 805 GEM_TRACE_ERR("%s: Dup context:%llx in pending[%zd]\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 806 engine->name,
38098750719cb5 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-01 807 ce->timeline->fence_context,
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 808 port - execlists->pending);
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 809 return false;
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 810 }
9f3ccd40acf4a3 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-20 811 ce = rq->context;
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 812
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 813 if (ccid == ce->lrc.ccid) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 814 GEM_TRACE_ERR("%s: Dup ccid:%x context:%llx in pending[%zd]\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 815 engine->name,
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 816 ccid, ce->timeline->fence_context,
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 817 port - execlists->pending);
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 818 return false;
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 819 }
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 820 ccid = ce->lrc.ccid;
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 821
15db5fcce9c8e8 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-03-02 822 /*
8733a06323d40e drivers/gpu/drm/i915/gt/intel_lrc.c Tvrtko Ursulin 2020-06-07 823 * Sentinels are supposed to be the last request so they flush
8733a06323d40e drivers/gpu/drm/i915/gt/intel_lrc.c Tvrtko Ursulin 2020-06-07 824 * the current execution off the HW. Check that they are the only
8733a06323d40e drivers/gpu/drm/i915/gt/intel_lrc.c Tvrtko Ursulin 2020-06-07 825 * request in the pending submission.
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 826 *
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 827 * NB: Due to the async nature of preempt-to-busy and request
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 828 * cancellation we need to handle the case where request
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 829 * becomes a sentinel in parallel to CSB processing.
15db5fcce9c8e8 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-03-02 830 */
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 831 if (prev && i915_request_has_sentinel(prev) &&
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 832 !READ_ONCE(prev->fence.error)) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 833 GEM_TRACE_ERR("%s: context:%llx after sentinel in pending[%zd]\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 834 engine->name,
15db5fcce9c8e8 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-03-02 835 ce->timeline->fence_context,
15db5fcce9c8e8 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-03-02 836 port - execlists->pending);
15db5fcce9c8e8 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-03-02 837 return false;
15db5fcce9c8e8 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-03-02 838 }
90a79a91dcc0e8 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Tvrtko Ursulin 2021-03-24 839 prev = rq;
15db5fcce9c8e8 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-03-02 840
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 841 /*
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 842 * We want virtual requests to only be in the first slot so
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 843 * that they are never stuck behind a hog and can be immediately
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 844 * transferred onto the next idle engine.
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 845 */
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 846 if (rq->execution_mask != engine->mask &&
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 847 port != execlists->pending) {
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 848 GEM_TRACE_ERR("%s: virtual engine:%llx not in prime position[%zd]\n",
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 849 engine->name,
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 850 ce->timeline->fence_context,
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 851 port - execlists->pending);
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 852 return false;
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 853 }
f81475bb5bb483 drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 854
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 855 /* Hold tightly onto the lock to prevent concurrent retires! */
49e74c8f9ae635 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-03 856 if (!spin_trylock_irqsave(&rq->lock, flags))
49e74c8f9ae635 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-03 857 continue;
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 858
6f0726b4807c1e drivers/gpu/drm/i915/gt/intel_execlists_submission.c Chris Wilson 2020-12-24 859 if (__i915_request_is_complete(rq))
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 860 goto unlock;
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 861
e6ba76480299a0 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-21 862 if (i915_active_is_idle(&ce->active) &&
e6ba76480299a0 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-12-21 863 !intel_context_is_barrier(ce)) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 864 GEM_TRACE_ERR("%s: Inactive context:%llx in pending[%zd]\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 865 engine->name,
38098750719cb5 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-01 866 ce->timeline->fence_context,
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 867 port - execlists->pending);
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 868 ok = false;
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 869 goto unlock;
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 870 }
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 871
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 872 if (!i915_vma_is_pinned(ce->state)) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 873 GEM_TRACE_ERR("%s: Unpinned context:%llx in pending[%zd]\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 874 engine->name,
38098750719cb5 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-01 875 ce->timeline->fence_context,
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 876 port - execlists->pending);
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 877 ok = false;
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 878 goto unlock;
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 879 }
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 880
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 881 if (!i915_vma_is_pinned(ce->ring->vma)) {
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 882 GEM_TRACE_ERR("%s: Unpinned ring:%llx in pending[%zd]\n",
f6a7c21c991062 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2020-04-28 883 engine->name,
38098750719cb5 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-01 884 ce->timeline->fence_context,
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 885 port - execlists->pending);
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 886 ok = false;
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 887 goto unlock;
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 888 }
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 889
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 890 unlock:
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 @891 spin_unlock_irqrestore(&rq->lock, flags);
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 892 if (!ok)
c95d31c3df1b9f drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-11-21 893 return false;
c97fb526ca0666 drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-10-10 894 }
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 895
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 896 return ce;
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 897 }
22b7a426bbe1eb drivers/gpu/drm/i915/gt/intel_lrc.c Chris Wilson 2019-06-20 898
:::::: The code at line 891 was first introduced by commit
:::::: c95d31c3df1b9ff29bcd85c47324e16d430bbd5a drm/i915/execlists: Lock the request while validating it during promotion
:::::: TO: Chris Wilson <chris@chris-wilson.co.uk>
:::::: CC: Chris Wilson <chris@chris-wilson.co.uk>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
reply other threads:[~2025-03-05 23:19 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202503060731.nwgOwXQP-lkp@intel.com \
--to=lkp@intel.com \
--cc=error27@gmail.com \
--cc=oe-kbuild@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.