Another attempt at providing GuC a list of registers to save/restore during engine resets [1]. The list we provide GuC should mirror, with possibly some exceptions, the list of registers applied during execlists_resume/enable_execlists. Any ideas on how to flag any discrepancies (yet flexible enough to handle exceptions) between the two lists e.g. via a selftest? I would like something stronger than a comment that just states the other needs updating. Or should I take this to mean a different approach is needed? Thanks, Fernando [1] https://patchwork.freedesktop.org/patch/161888/ Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Cc: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx> Fernando Pacheco (1): drm/i915/guc: Provide mmio list to be saved/restored on engine reset drivers/gpu/drm/i915/gt/intel_workarounds.c | 23 +++-- .../gpu/drm/i915/gt/intel_workarounds_types.h | 1 + drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 87 +++++++++++++++++++ 3 files changed, 105 insertions(+), 6 deletions(-) -- 2.23.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx