I think i found the problem - will trybot next before reposting a new rev. was a terribly careless typo when rebasing from internal for drmtip: guc_context_sched_disable called do_sched_disable which was supposed to call __guc_context_sched_disable (if the context was really meant to be finally disabled) but instead called guc_context_sched_disable (no underscore) causing an infinite loop for mid-disabling.
...alan On Fri, 2022-08-19 at 16:47 +0000, Teres Alexis, Alan Previn wrote: > Will look into this - apologies for the trouble Matt. > ...alan > > -----Original Message----- > From: Harrison, John C <john.c.harri...@intel.com> > Sent: Friday, August 19, 2022 8:46 AM > To: Auld, Matthew <matthew.a...@intel.com>; intel-gfx@lists.freedesktop.org > Cc: Brost, Matthew <matthew.br...@intel.com>; Teres Alexis, Alan Previn > <alan.previn.teres.ale...@intel.com> > Subject: Re: [PATCH] Revert "drm/i915/guc: Add delay to disable scheduling > after pin count goes to zero" > > On 8/19/2022 05:39, Matthew Auld wrote: > > This reverts commit 6a079903847cce1dd06345127d2a32f26d2cd9c6. > > > > Everything in CI using GuC is now timing out[1], and killing the > > machine with this change (perhaps a deadlock?). CI was recently on > > fire due to some changes coming in from -rc1, so likely the pre-merge > > CI results for this series were invalid? For now just revert, unless > > GuC experts already have a fix in mind. > > > > [1] https://intel-gfx-ci.01.org/tree/drm-tip/index.html? > > > > Signed-off-by: Matthew Auld <matthew.a...@intel.com> > > Cc: Matthew Brost <matthew.br...@intel.com> > > Cc: Alan Previn <alan.previn.teres.ale...@intel.com> > > Cc: John Harrison <john.c.harri...@intel.com> > Reviewed-by: John Harrison <john.c.harri...@intel.com> > > Given that CI was claiming a pass for the original patch set, no we don't > have a fix in mind. It is most frustrating when CI says all green if the > entire universe is so broken that no tests were even running :(. > > John. > > > > --- > > drivers/gpu/drm/i915/gem/i915_gem_context.c | 2 +- > > drivers/gpu/drm/i915/gt/intel_context.h | 8 - > > drivers/gpu/drm/i915/gt/intel_context_types.h | 7 - > > drivers/gpu/drm/i915/gt/uc/intel_guc.h | 17 +- > > .../gpu/drm/i915/gt/uc/intel_guc_debugfs.c | 60 ------- > > .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 154 +++--------------- > > drivers/gpu/drm/i915/i915_selftest.h | 2 - > > 7 files changed, 27 insertions(+), 223 deletions(-) > >
