The intent is to move the GuC and HuC firmware images to the top of the address space. This portion is inaccessible during normal GuC operations and should be relatively safe to house both firmware images. By making the move we can re-enable the full gpu reset with GuC enabled. Placing the firmware images above GUC_GGTT_TOP was discussed previously here: https://patchwork.freedesktop.org/patch/273616/ v3: Bindings are only needed for xfer v2: The decision to rename both the uc_fw init and fini functions made it easier to pull the bind/unbind operations out of intel_guc_fw.* and intel_huc_fw.*. The bind/unbind will now take place within the newly repurposed intel_uc_fw_init/fini. All other changes should be called out in their respective patches and should be the direct result of a review comment. Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx> Fernando Pacheco (5): drm/i915/uc: Rename uC firmware init/fini functions drm/i915/uc: Reserve upper range of GGTT drm/i915/uc: Place uC firmware in upper range of GGTT Revert "drm/i915/guc: Disable global reset" drm/i915/selftests: Check that gpu reset is usable from atomic context drivers/gpu/drm/i915/i915_gem_gtt.c | 25 ++-- drivers/gpu/drm/i915/i915_gem_gtt.h | 1 + drivers/gpu/drm/i915/i915_reset.c | 3 - drivers/gpu/drm/i915/intel_guc.c | 41 ++++++- drivers/gpu/drm/i915/intel_guc.h | 2 + drivers/gpu/drm/i915/intel_guc_fw.c | 20 ++-- drivers/gpu/drm/i915/intel_huc.c | 74 +++++++++--- drivers/gpu/drm/i915/intel_huc.h | 6 +- drivers/gpu/drm/i915/intel_huc_fw.c | 49 +++++--- drivers/gpu/drm/i915/intel_uc.c | 23 +++- drivers/gpu/drm/i915/intel_uc_fw.c | 107 +++++++++++++----- drivers/gpu/drm/i915/intel_uc_fw.h | 10 +- .../gpu/drm/i915/selftests/intel_hangcheck.c | 6 +- 13 files changed, 267 insertions(+), 100 deletions(-) -- 2.21.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
