Commit 9c5308ea authored Oct 30, 2015 by Mika Kuoppala Committed by Ville Syrjälä Nov 09, 2015

drm/i915/skl: Refuse to load outdated dmc firmware

There is known issue on GT interrupt delivery with DC6 and
firmwares <1.21. There is a suspicion that this causes
spurious gpu hangs on driver init and with some workloads,
as upgrading the firmware to 1.21 makes these problems
disappear.

As of now the current version included in distribution
firmware packages is very like to be 1.19. Play it safe and
refuse to load a firmware version that may affect gpu
side stability.

With < 1.23 there is a palette and dmc ram corruption issue
so blacklist anything below that.

v2: Refuse to load fw instead of notifying the user
v3: Rebase on header version changes
v4: Refuse to load anything less than 1.23
v5: Give enough information for user for finding correct fw (Chris)
v6: better url and formatting (Chris)
v7: move error log for each fail path (Mika)
    bail out earlier in load path (Imre)
v8: Fix the version check (Imre)

Cc: Animesh Manna <animesh.manna@intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Dave Gordon <david.s.gordon@intel.com>
Cc: Arun Siluvery <arun.siluvery@linux.intel.com>
Cc: Imre Deak <imre.deak@intel.com>
Cc: Patrik Jakobsson <patrik.jakobsson@linux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@gmail.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
References: https://01.org/linuxgraphics/downloads/skldmcver121
References: https://01.org/linuxgraphics/downloads/skylake-dmc-1.23


Testcase: igt/gem_exec_nop
Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1446220336-32392-1-git-send-email-mika.kuoppala@intel.com


Tested-by: Daniel Stone <daniels@collabora.com> # SKL
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

parent b6e7d894

drivers/gpu/drm/i915/intel_csr.c

+24 −10

Original line number	Diff line number	Diff line
		@@ -47,6 +47,8 @@
		MODULE_FIRMWARE(I915_CSR_SKL);
		MODULE_FIRMWARE(I915_CSR_BXT);

		#define SKL_CSR_VERSION_REQUIRED CSR_VERSION(1, 23)

		/*
		* SKL CSR registers for DC5 and DC6
		*/
		@@ -303,10 +305,8 @@ static void finish_csr_load(const struct firmware fw, void context)
		uint32_t *dmc_payload;
		bool fw_loaded = false;

		if (!fw) {
		i915_firmware_load_error_print(csr->fw_path, 0);
		if (!fw)
		goto out;
		}

		if ((stepping == -ENODATA) \|\| (substepping == -ENODATA)) {
		DRM_ERROR("Unknown stepping info, firmware loading failed\n");
		@@ -324,6 +324,17 @@ static void finish_csr_load(const struct firmware fw, void context)

		csr->version = css_header->version;

		if (IS_SKYLAKE(dev) && csr->version < SKL_CSR_VERSION_REQUIRED) {
		DRM_INFO("Refusing to load old Skylake DMC firmware v%u.%u,"
		" please upgrade to v%u.%u or later"
		" [https://01.org/linuxgraphics/intel-linux-graphics-firmwares].\n",
		CSR_VERSION_MAJOR(csr->version),
		CSR_VERSION_MINOR(csr->version),
		CSR_VERSION_MAJOR(SKL_CSR_VERSION_REQUIRED),
		CSR_VERSION_MINOR(SKL_CSR_VERSION_REQUIRED));
		goto out;
		}

		readcount += sizeof(struct intel_css_header);

		/* Extract Package Header information*/
		@@ -405,17 +416,20 @@ static void finish_csr_load(const struct firmware fw, void context)
		intel_csr_load_program(dev);
		fw_loaded = true;

		out:
		if (fw_loaded) {
		intel_runtime_pm_put(dev_priv);

		DRM_INFO("Finished loading %s (v%u.%u)\n",
		dev_priv->csr.fw_path,
		CSR_VERSION_MAJOR(csr->version),
		CSR_VERSION_MINOR(csr->version));

		out:
		if (fw_loaded)
		intel_runtime_pm_put(dev_priv);
		else
		} else {
		intel_csr_load_status_set(dev_priv, FW_FAILED);

		i915_firmware_load_error_print(csr->fw_path, 0);
		}

		release_firmware(fw);
		}

Admin message