[Intel-gfx] [PATCH 1/4] drm/i915/params: don't expose inject_probe_failure in debugfs

[Intel-gfx] [PATCH 1/4] drm/i915/params: don't expose inject_probe_failure in debugfs

[Jani Nikula]

The parameter only makes sense as a module parameter only.

Fixes: c43c5a8818d4 ("drm/i915/params: add i915 parameters to debugfs")
Cc: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com>
Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/i915_params.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/i915_params.h b/drivers/gpu/drm/i915/i915_params.h
index 45323732f099..4f21bfffbf0e 100644
--- a/drivers/gpu/drm/i915/i915_params.h
+++ b/drivers/gpu/drm/i915/i915_params.h
@@ -64,7 +64,7 @@ struct drm_printer;
 	param(int, mmio_debug, -IS_ENABLED(CONFIG_DRM_I915_DEBUG_MMIO), 0600) \
 	param(int, edp_vswing, 0, 0400) \
 	param(unsigned int, reset, 3, 0600) \
-	param(unsigned int, inject_probe_failure, 0, 0600) \
+	param(unsigned int, inject_probe_failure, 0, 0) \
 	param(int, fastboot, -1, 0600) \
 	param(int, enable_dpcd_backlight, -1, 0600) \
 	param(char *, force_probe, CONFIG_DRM_I915_FORCE_PROBE, 0400) \
-- 
2.20.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PATCH 2/4] drm/i915/params: fix i915.fake_lmem_start module param sysfs permissions

[Jani Nikula]

fake_lmem_start does not need to be mutable via module param sysfs. It's
only used during driver probe.

Fixes: 1629224324b6 ("drm/i915/lmem: add the fake lmem region")
Cc: Matthew Auld <matthew.auld@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/i915_params.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/i915_params.c b/drivers/gpu/drm/i915/i915_params.c
index add00ec1f787..a3dde770226d 100644
--- a/drivers/gpu/drm/i915/i915_params.c
+++ b/drivers/gpu/drm/i915/i915_params.c
@@ -173,7 +173,7 @@ i915_param_named(enable_gvt, bool, 0400,
 #endif
 
 #if IS_ENABLED(CONFIG_DRM_I915_UNSTABLE_FAKE_LMEM)
-i915_param_named_unsafe(fake_lmem_start, ulong, 0600,
+i915_param_named_unsafe(fake_lmem_start, ulong, 0400,
 	"Fake LMEM start offset (default: 0)");
 #endif
 
-- 
2.20.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH 2/4] drm/i915/params: fix i915.fake_lmem_start module param sysfs permissions

[Rodrigo Vivi]

On Thu, May 14, 2020 at 06:40:04PM +0300, Jani Nikula wrote:
> fake_lmem_start does not need to be mutable via module param sysfs. It's
> only used during driver probe.
> 
> Fixes: 1629224324b6 ("drm/i915/lmem: add the fake lmem region")
> Cc: Matthew Auld <matthew.auld@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  drivers/gpu/drm/i915/i915_params.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_params.c b/drivers/gpu/drm/i915/i915_params.c
> index add00ec1f787..a3dde770226d 100644
> --- a/drivers/gpu/drm/i915/i915_params.c
> +++ b/drivers/gpu/drm/i915/i915_params.c
> @@ -173,7 +173,7 @@ i915_param_named(enable_gvt, bool, 0400,
>  #endif
>  
>  #if IS_ENABLED(CONFIG_DRM_I915_UNSTABLE_FAKE_LMEM)
> -i915_param_named_unsafe(fake_lmem_start, ulong, 0600,
> +i915_param_named_unsafe(fake_lmem_start, ulong, 0400,
>  	"Fake LMEM start offset (default: 0)");
>  #endif
>  
> -- 
> 2.20.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PATCH 3/4] drm/i915/params: prevent changing module params runtime

[Jani Nikula]

Only support runtime changes through the debugfs.

i915.verbose_state_checks remains an exception, and is not exposed via
debugfs.

This depends on IGT having been updated to use the debugfs for modifying
the parameters.

Cc: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com>
Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/i915_params.c | 38 +++++++++++++++++++-----------
 1 file changed, 24 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_params.c b/drivers/gpu/drm/i915/i915_params.c
index a3dde770226d..ace44ad7e6df 100644
--- a/drivers/gpu/drm/i915/i915_params.c
+++ b/drivers/gpu/drm/i915/i915_params.c
@@ -40,6 +40,15 @@ struct i915_params i915_modparams __read_mostly = {
 #undef MEMBER
 };
 
+/*
+ * Note: As a rule, keep module parameter sysfs permissions read-only
+ * 0400. Runtime changes are only supported through i915 debugfs.
+ *
+ * For any exceptions requiring write access and runtime changes through module
+ * parameter sysfs, prevent debugfs file creation by setting the parameter's
+ * debugfs mode to 0.
+ */
+
 i915_param_named(modeset, int, 0400,
 	"Use kernel modesetting [KMS] (0=disable, "
 	"1=on, -1=force vga console preference [default])");
@@ -49,7 +58,7 @@ i915_param_named_unsafe(enable_dc, int, 0400,
 	"(-1=auto [default]; 0=disable; 1=up to DC5; 2=up to DC6; "
 	"3=up to DC5 with DC3CO; 4=up to DC6 with DC3CO)");
 
-i915_param_named_unsafe(enable_fbc, int, 0600,
+i915_param_named_unsafe(enable_fbc, int, 0400,
 	"Enable frame buffer compression for power savings "
 	"(default: -1 (use per-chip default))");
 
@@ -57,7 +66,7 @@ i915_param_named_unsafe(lvds_channel_mode, int, 0400,
 	 "Specify LVDS channel mode "
 	 "(0=probe BIOS [default], 1=single-channel, 2=dual-channel)");
 
-i915_param_named_unsafe(panel_use_ssc, int, 0600,
+i915_param_named_unsafe(panel_use_ssc, int, 0400,
 	"Use Spread Spectrum Clock with panels [LVDS/eDP] "
 	"(default: auto from VBT)");
 
@@ -65,25 +74,25 @@ i915_param_named_unsafe(vbt_sdvo_panel_type, int, 0400,
 	"Override/Ignore selection of SDVO panel mode in the VBT "
 	"(-2=ignore, -1=auto [default], index in VBT BIOS table)");
 
-i915_param_named_unsafe(reset, int, 0600,
+i915_param_named_unsafe(reset, int, 0400,
 	"Attempt GPU resets (0=disabled, 1=full gpu reset, 2=engine reset [default])");
 
 i915_param_named_unsafe(vbt_firmware, charp, 0400,
 	"Load VBT from specified file under /lib/firmware");
 
 #if IS_ENABLED(CONFIG_DRM_I915_CAPTURE_ERROR)
-i915_param_named(error_capture, bool, 0600,
+i915_param_named(error_capture, bool, 0400,
 	"Record the GPU state following a hang. "
 	"This information in /sys/class/drm/card<N>/error is vital for "
 	"triaging and debugging hangs.");
 #endif
 
-i915_param_named_unsafe(enable_hangcheck, bool, 0600,
+i915_param_named_unsafe(enable_hangcheck, bool, 0400,
 	"Periodically check GPU activity for detecting hangs. "
 	"WARNING: Disabling this can cause system wide hangs. "
 	"(default: true)");
 
-i915_param_named_unsafe(enable_psr, int, 0600,
+i915_param_named_unsafe(enable_psr, int, 0400,
 	"Enable PSR "
 	"(0=disabled, 1=enabled) "
 	"Default: -1 (use per-chip default)");
@@ -96,22 +105,22 @@ i915_param_named_unsafe(disable_power_well, int, 0400,
 	"Disable display power wells when possible "
 	"(-1=auto [default], 0=power wells always on, 1=power wells disabled when possible)");
 
-i915_param_named_unsafe(enable_ips, int, 0600, "Enable IPS (default: true)");
+i915_param_named_unsafe(enable_ips, int, 0400, "Enable IPS (default: true)");
 
-i915_param_named(fastboot, int, 0600,
+i915_param_named(fastboot, int, 0400,
 	"Try to skip unnecessary mode sets at boot time "
 	"(0=disabled, 1=enabled) "
 	"Default: -1 (use per-chip default)");
 
-i915_param_named_unsafe(load_detect_test, bool, 0600,
+i915_param_named_unsafe(load_detect_test, bool, 0400,
 	"Force-enable the VGA load detect code for testing (default:false). "
 	"For developers only.");
 
-i915_param_named_unsafe(force_reset_modeset_test, bool, 0600,
+i915_param_named_unsafe(force_reset_modeset_test, bool, 0400,
 	"Force a modeset during gpu reset for testing (default:false). "
 	"For developers only.");
 
-i915_param_named_unsafe(invert_brightness, int, 0600,
+i915_param_named_unsafe(invert_brightness, int, 0400,
 	"Invert backlight brightness "
 	"(-1 force normal, 0 machine defaults, 1 force inversion), please "
 	"report PCI device ID, subsystem vendor and subsystem device ID "
@@ -121,10 +130,11 @@ i915_param_named_unsafe(invert_brightness, int, 0600,
 i915_param_named(disable_display, bool, 0400,
 	"Disable display (default: false)");
 
-i915_param_named(mmio_debug, int, 0600,
+i915_param_named(mmio_debug, int, 0400,
 	"Enable the MMIO debug code for the first N failures (default: off). "
 	"This may negatively affect performance.");
 
+/* Special case writable file */
 i915_param_named(verbose_state_checks, bool, 0600,
 	"Enable verbose logs (ie. WARN_ON()) in case of unexpected hw state conditions.");
 
@@ -155,7 +165,7 @@ i915_param_named_unsafe(huc_firmware_path, charp, 0400,
 i915_param_named_unsafe(dmc_firmware_path, charp, 0400,
 	"DMC firmware path to use instead of the default one");
 
-i915_param_named_unsafe(enable_dp_mst, bool, 0600,
+i915_param_named_unsafe(enable_dp_mst, bool, 0400,
 	"Enable multi-stream transport (MST) for new DisplayPort sinks. (default: true)");
 
 #if IS_ENABLED(CONFIG_DRM_I915_DEBUG)
@@ -163,7 +173,7 @@ i915_param_named_unsafe(inject_probe_failure, uint, 0400,
 	"Force an error after a number of failure check points (0:disabled (default), N:force failure at the Nth failure check point)");
 #endif
 
-i915_param_named(enable_dpcd_backlight, int, 0600,
+i915_param_named(enable_dpcd_backlight, int, 0400,
 	"Enable support for DPCD backlight control"
 	"(-1=use per-VBT LFP backlight type setting [default], 0=disabled, 1=enabled)");
 
-- 
2.20.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH 3/4] drm/i915/params: prevent changing module params runtime

[Juha-Pekka Heikkila]

Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>

On 14.5.2020 18.40, Jani Nikula wrote:
> Only support runtime changes through the debugfs.
> 
> i915.verbose_state_checks remains an exception, and is not exposed via
> debugfs.
> 
> This depends on IGT having been updated to use the debugfs for modifying
> the parameters.
> 
> Cc: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com>
> Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---
>   drivers/gpu/drm/i915/i915_params.c | 38 +++++++++++++++++++-----------
>   1 file changed, 24 insertions(+), 14 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_params.c b/drivers/gpu/drm/i915/i915_params.c
> index a3dde770226d..ace44ad7e6df 100644
> --- a/drivers/gpu/drm/i915/i915_params.c
> +++ b/drivers/gpu/drm/i915/i915_params.c
> @@ -40,6 +40,15 @@ struct i915_params i915_modparams __read_mostly = {
>   #undef MEMBER
>   };
>   
> +/*
> + * Note: As a rule, keep module parameter sysfs permissions read-only
> + * 0400. Runtime changes are only supported through i915 debugfs.
> + *
> + * For any exceptions requiring write access and runtime changes through module
> + * parameter sysfs, prevent debugfs file creation by setting the parameter's
> + * debugfs mode to 0.
> + */
> +
>   i915_param_named(modeset, int, 0400,
>   	"Use kernel modesetting [KMS] (0=disable, "
>   	"1=on, -1=force vga console preference [default])");
> @@ -49,7 +58,7 @@ i915_param_named_unsafe(enable_dc, int, 0400,
>   	"(-1=auto [default]; 0=disable; 1=up to DC5; 2=up to DC6; "
>   	"3=up to DC5 with DC3CO; 4=up to DC6 with DC3CO)");
>   
> -i915_param_named_unsafe(enable_fbc, int, 0600,
> +i915_param_named_unsafe(enable_fbc, int, 0400,
>   	"Enable frame buffer compression for power savings "
>   	"(default: -1 (use per-chip default))");
>   
> @@ -57,7 +66,7 @@ i915_param_named_unsafe(lvds_channel_mode, int, 0400,
>   	 "Specify LVDS channel mode "
>   	 "(0=probe BIOS [default], 1=single-channel, 2=dual-channel)");
>   
> -i915_param_named_unsafe(panel_use_ssc, int, 0600,
> +i915_param_named_unsafe(panel_use_ssc, int, 0400,
>   	"Use Spread Spectrum Clock with panels [LVDS/eDP] "
>   	"(default: auto from VBT)");
>   
> @@ -65,25 +74,25 @@ i915_param_named_unsafe(vbt_sdvo_panel_type, int, 0400,
>   	"Override/Ignore selection of SDVO panel mode in the VBT "
>   	"(-2=ignore, -1=auto [default], index in VBT BIOS table)");
>   
> -i915_param_named_unsafe(reset, int, 0600,
> +i915_param_named_unsafe(reset, int, 0400,
>   	"Attempt GPU resets (0=disabled, 1=full gpu reset, 2=engine reset [default])");
>   
>   i915_param_named_unsafe(vbt_firmware, charp, 0400,
>   	"Load VBT from specified file under /lib/firmware");
>   
>   #if IS_ENABLED(CONFIG_DRM_I915_CAPTURE_ERROR)
> -i915_param_named(error_capture, bool, 0600,
> +i915_param_named(error_capture, bool, 0400,
>   	"Record the GPU state following a hang. "
>   	"This information in /sys/class/drm/card<N>/error is vital for "
>   	"triaging and debugging hangs.");
>   #endif
>   
> -i915_param_named_unsafe(enable_hangcheck, bool, 0600,
> +i915_param_named_unsafe(enable_hangcheck, bool, 0400,
>   	"Periodically check GPU activity for detecting hangs. "
>   	"WARNING: Disabling this can cause system wide hangs. "
>   	"(default: true)");
>   
> -i915_param_named_unsafe(enable_psr, int, 0600,
> +i915_param_named_unsafe(enable_psr, int, 0400,
>   	"Enable PSR "
>   	"(0=disabled, 1=enabled) "
>   	"Default: -1 (use per-chip default)");
> @@ -96,22 +105,22 @@ i915_param_named_unsafe(disable_power_well, int, 0400,
>   	"Disable display power wells when possible "
>   	"(-1=auto [default], 0=power wells always on, 1=power wells disabled when possible)");
>   
> -i915_param_named_unsafe(enable_ips, int, 0600, "Enable IPS (default: true)");
> +i915_param_named_unsafe(enable_ips, int, 0400, "Enable IPS (default: true)");
>   
> -i915_param_named(fastboot, int, 0600,
> +i915_param_named(fastboot, int, 0400,
>   	"Try to skip unnecessary mode sets at boot time "
>   	"(0=disabled, 1=enabled) "
>   	"Default: -1 (use per-chip default)");
>   
> -i915_param_named_unsafe(load_detect_test, bool, 0600,
> +i915_param_named_unsafe(load_detect_test, bool, 0400,
>   	"Force-enable the VGA load detect code for testing (default:false). "
>   	"For developers only.");
>   
> -i915_param_named_unsafe(force_reset_modeset_test, bool, 0600,
> +i915_param_named_unsafe(force_reset_modeset_test, bool, 0400,
>   	"Force a modeset during gpu reset for testing (default:false). "
>   	"For developers only.");
>   
> -i915_param_named_unsafe(invert_brightness, int, 0600,
> +i915_param_named_unsafe(invert_brightness, int, 0400,
>   	"Invert backlight brightness "
>   	"(-1 force normal, 0 machine defaults, 1 force inversion), please "
>   	"report PCI device ID, subsystem vendor and subsystem device ID "
> @@ -121,10 +130,11 @@ i915_param_named_unsafe(invert_brightness, int, 0600,
>   i915_param_named(disable_display, bool, 0400,
>   	"Disable display (default: false)");
>   
> -i915_param_named(mmio_debug, int, 0600,
> +i915_param_named(mmio_debug, int, 0400,
>   	"Enable the MMIO debug code for the first N failures (default: off). "
>   	"This may negatively affect performance.");
>   
> +/* Special case writable file */
>   i915_param_named(verbose_state_checks, bool, 0600,
>   	"Enable verbose logs (ie. WARN_ON()) in case of unexpected hw state conditions.");
>   
> @@ -155,7 +165,7 @@ i915_param_named_unsafe(huc_firmware_path, charp, 0400,
>   i915_param_named_unsafe(dmc_firmware_path, charp, 0400,
>   	"DMC firmware path to use instead of the default one");
>   
> -i915_param_named_unsafe(enable_dp_mst, bool, 0600,
> +i915_param_named_unsafe(enable_dp_mst, bool, 0400,
>   	"Enable multi-stream transport (MST) for new DisplayPort sinks. (default: true)");
>   
>   #if IS_ENABLED(CONFIG_DRM_I915_DEBUG)
> @@ -163,7 +173,7 @@ i915_param_named_unsafe(inject_probe_failure, uint, 0400,
>   	"Force an error after a number of failure check points (0:disabled (default), N:force failure at the Nth failure check point)");
>   #endif
>   
> -i915_param_named(enable_dpcd_backlight, int, 0600,
> +i915_param_named(enable_dpcd_backlight, int, 0400,
>   	"Enable support for DPCD backlight control"
>   	"(-1=use per-VBT LFP backlight type setting [default], 0=disabled, 1=enabled)");
>   
> 

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PATCH 4/4] drm/i915/params: switch to device specific parameters

[Jani Nikula]

Start using device specific parameters instead of module parameters for
most things. The module parameters become the immutable initial values
for i915 parameters. The device specific parameters in i915->params
start life as a copy of i915_modparams. Any later changes are only
reflected in the debugfs.

The stragglers are:

* i915.force_probe and i915.modeset. Needed before dev_priv is
  available. This is fine because the parameters are read-only and never
  modified.

* i915.verbose_state_checks. Passing dev_priv to I915_STATE_WARN and
  I915_STATE_WARN_ON would result in massive and ugly churn. This is
  handled by not exposing the parameter via debugfs, and leaving the
  parameter writable in sysfs. This may be fixed up in follow-up work.

* i915.inject_probe_failure. Only makes sense in terms of the module,
  not the device. This is handled by not exposing the parameter via
  debugfs.

Cc: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com>
Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_bios.c     |  6 ++---
 drivers/gpu/drm/i915/display/intel_crt.c      |  4 ++--
 drivers/gpu/drm/i915/display/intel_csr.c      |  6 ++---
 drivers/gpu/drm/i915/display/intel_display.c  | 12 +++++-----
 .../drm/i915/display/intel_display_debugfs.c  |  2 +-
 .../drm/i915/display/intel_display_power.c    | 14 ++++++------
 drivers/gpu/drm/i915/display/intel_dp.c       |  8 ++++---
 .../drm/i915/display/intel_dp_aux_backlight.c |  4 ++--
 drivers/gpu/drm/i915/display/intel_fbc.c      | 12 +++++-----
 drivers/gpu/drm/i915/display/intel_lvds.c     |  4 ++--
 drivers/gpu/drm/i915/display/intel_opregion.c |  2 +-
 drivers/gpu/drm/i915/display/intel_panel.c    |  4 ++--
 drivers/gpu/drm/i915/display/intel_psr.c      |  6 ++---
 drivers/gpu/drm/i915/gem/i915_gem_context.c   |  4 ++--
 .../gpu/drm/i915/gt/intel_engine_heartbeat.c  |  3 ++-
 drivers/gpu/drm/i915/gt/intel_reset.c         |  6 ++---
 .../drm/i915/gt/selftest_engine_heartbeat.c   |  6 ++---
 drivers/gpu/drm/i915/gt/uc/intel_guc_log.c    | 15 ++++++++-----
 .../gpu/drm/i915/gt/uc/intel_guc_submission.c |  4 +++-
 drivers/gpu/drm/i915/gt/uc/intel_uc.c         | 20 ++++++++---------
 drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c      | 22 ++++++++++---------
 drivers/gpu/drm/i915/i915_debugfs.c           |  2 +-
 drivers/gpu/drm/i915/i915_debugfs_params.c    |  7 +-----
 drivers/gpu/drm/i915/i915_drv.c               |  9 ++++++--
 drivers/gpu/drm/i915/i915_drv.h               |  5 ++++-
 drivers/gpu/drm/i915/i915_getparam.c          |  2 +-
 drivers/gpu/drm/i915/i915_gpu_error.c         |  4 ++--
 drivers/gpu/drm/i915/intel_gvt.c              |  8 +++----
 drivers/gpu/drm/i915/intel_region_lmem.c      |  6 ++---
 drivers/gpu/drm/i915/intel_uncore.c           |  8 +++----
 30 files changed, 114 insertions(+), 101 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
index 839124647202..ec8af2b7bf01 100644
--- a/drivers/gpu/drm/i915/display/intel_bios.c
+++ b/drivers/gpu/drm/i915/display/intel_bios.c
@@ -479,7 +479,7 @@ parse_sdvo_panel_data(struct drm_i915_private *dev_priv,
 	struct drm_display_mode *panel_fixed_mode;
 	int index;
 
-	index = i915_modparams.vbt_sdvo_panel_type;
+	index = dev_priv->params.vbt_sdvo_panel_type;
 	if (index == -2) {
 		drm_dbg_kms(&dev_priv->drm,
 			    "Ignore SDVO panel mode from BIOS VBT tables.\n");
@@ -829,9 +829,9 @@ parse_edp(struct drm_i915_private *dev_priv, const struct bdb_header *bdb)
 		u8 vswing;
 
 		/* Don't read from VBT if module parameter has valid value*/
-		if (i915_modparams.edp_vswing) {
+		if (dev_priv->params.edp_vswing) {
 			dev_priv->vbt.edp.low_vswing =
-				i915_modparams.edp_vswing == 1;
+				dev_priv->params.edp_vswing == 1;
 		} else {
 			vswing = (edp->edp_vswing_preemph >> (panel_type * 4)) & 0xF;
 			dev_priv->vbt.edp.low_vswing = vswing == 0;
diff --git a/drivers/gpu/drm/i915/display/intel_crt.c b/drivers/gpu/drm/i915/display/intel_crt.c
index 2f5b9a4baafd..5b4510ce5693 100644
--- a/drivers/gpu/drm/i915/display/intel_crt.c
+++ b/drivers/gpu/drm/i915/display/intel_crt.c
@@ -833,7 +833,7 @@ intel_crt_detect(struct drm_connector *connector,
 		    connector->base.id, connector->name,
 		    force);
 
-	if (i915_modparams.load_detect_test) {
+	if (dev_priv->params.load_detect_test) {
 		wakeref = intel_display_power_get(dev_priv,
 						  intel_encoder->power_domain);
 		goto load_detect;
@@ -889,7 +889,7 @@ intel_crt_detect(struct drm_connector *connector,
 		else if (INTEL_GEN(dev_priv) < 4)
 			status = intel_crt_load_detect(crt,
 				to_intel_crtc(connector->state->crtc)->pipe);
-		else if (i915_modparams.load_detect_test)
+		else if (dev_priv->params.load_detect_test)
 			status = connector_status_disconnected;
 		else
 			status = connector_status_unknown;
diff --git a/drivers/gpu/drm/i915/display/intel_csr.c b/drivers/gpu/drm/i915/display/intel_csr.c
index 3112572cfb7d..8c481270a160 100644
--- a/drivers/gpu/drm/i915/display/intel_csr.c
+++ b/drivers/gpu/drm/i915/display/intel_csr.c
@@ -713,15 +713,15 @@ void intel_csr_ucode_init(struct drm_i915_private *dev_priv)
 		csr->max_fw_size = BXT_CSR_MAX_FW_SIZE;
 	}
 
-	if (i915_modparams.dmc_firmware_path) {
-		if (strlen(i915_modparams.dmc_firmware_path) == 0) {
+	if (dev_priv->params.dmc_firmware_path) {
+		if (strlen(dev_priv->params.dmc_firmware_path) == 0) {
 			csr->fw_path = NULL;
 			drm_info(&dev_priv->drm,
 				 "Disabling CSR firmware and runtime PM\n");
 			return;
 		}
 
-		csr->fw_path = i915_modparams.dmc_firmware_path;
+		csr->fw_path = dev_priv->params.dmc_firmware_path;
 		/* Bypass version check for firmware override. */
 		csr->required_version = 0;
 	}
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index ab621926def1..507ab78b9f3b 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -4879,7 +4879,7 @@ void intel_prepare_reset(struct drm_i915_private *dev_priv)
 	int ret;
 
 	/* reset doesn't touch the display */
-	if (!i915_modparams.force_reset_modeset_test &&
+	if (!dev_priv->params.force_reset_modeset_test &&
 	    !gpu_reset_clobbers_display(dev_priv))
 		return;
 
@@ -7869,7 +7869,7 @@ bool hsw_crtc_state_ips_capable(const struct intel_crtc_state *crtc_state)
 	if (!hsw_crtc_supports_ips(crtc))
 		return false;
 
-	if (!i915_modparams.enable_ips)
+	if (!dev_priv->params.enable_ips)
 		return false;
 
 	if (crtc_state->pipe_bpp > 24)
@@ -8140,8 +8140,8 @@ static void intel_panel_sanitize_ssc(struct drm_i915_private *dev_priv)
 
 static bool intel_panel_use_ssc(struct drm_i915_private *dev_priv)
 {
-	if (i915_modparams.panel_use_ssc >= 0)
-		return i915_modparams.panel_use_ssc != 0;
+	if (dev_priv->params.panel_use_ssc >= 0)
+		return dev_priv->params.panel_use_ssc != 0;
 	return dev_priv->vbt.lvds_use_ssc
 		&& !(dev_priv->quirks & QUIRK_LVDS_SSC_DISABLE);
 }
@@ -13571,8 +13571,8 @@ pipe_config_mismatch(bool fastset, const struct intel_crtc *crtc,
 
 static bool fastboot_enabled(struct drm_i915_private *dev_priv)
 {
-	if (i915_modparams.fastboot != -1)
-		return i915_modparams.fastboot;
+	if (dev_priv->params.fastboot != -1)
+		return dev_priv->params.fastboot;
 
 	/* Enable fastboot by default on Skylake and newer */
 	if (INTEL_GEN(dev_priv) >= 9)
diff --git a/drivers/gpu/drm/i915/display/intel_display_debugfs.c b/drivers/gpu/drm/i915/display/intel_display_debugfs.c
index 70525623bcdf..87db7e906621 100644
--- a/drivers/gpu/drm/i915/display/intel_display_debugfs.c
+++ b/drivers/gpu/drm/i915/display/intel_display_debugfs.c
@@ -125,7 +125,7 @@ static int i915_ips_status(struct seq_file *m, void *unused)
 	wakeref = intel_runtime_pm_get(&dev_priv->runtime_pm);
 
 	seq_printf(m, "Enabled by kernel parameter: %s\n",
-		   yesno(i915_modparams.enable_ips));
+		   yesno(dev_priv->params.enable_ips));
 
 	if (INTEL_GEN(dev_priv) >= 8) {
 		seq_puts(m, "Currently: unknown\n");
diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c
index 49998906cc61..ad6729ba90ab 100644
--- a/drivers/gpu/drm/i915/display/intel_display_power.c
+++ b/drivers/gpu/drm/i915/display/intel_display_power.c
@@ -4322,7 +4322,7 @@ static u32 get_allowed_dc_mask(const struct drm_i915_private *dev_priv,
 		mask = 0;
 	}
 
-	if (!i915_modparams.disable_power_well)
+	if (!dev_priv->params.disable_power_well)
 		max_dc = 0;
 
 	if (enable_dc >= 0 && enable_dc <= max_dc) {
@@ -4408,11 +4408,11 @@ int intel_power_domains_init(struct drm_i915_private *dev_priv)
 	struct i915_power_domains *power_domains = &dev_priv->power_domains;
 	int err;
 
-	i915_modparams.disable_power_well =
+	dev_priv->params.disable_power_well =
 		sanitize_disable_power_well_option(dev_priv,
-						   i915_modparams.disable_power_well);
+						   dev_priv->params.disable_power_well);
 	dev_priv->csr.allowed_dc_mask =
-		get_allowed_dc_mask(dev_priv, i915_modparams.enable_dc);
+		get_allowed_dc_mask(dev_priv, dev_priv->params.enable_dc);
 
 	dev_priv->csr.target_dc_state =
 		sanitize_target_dc_state(dev_priv, DC_STATE_EN_UPTO_DC6);
@@ -5375,7 +5375,7 @@ void intel_power_domains_init_hw(struct drm_i915_private *i915, bool resume)
 		intel_display_power_get(i915, POWER_DOMAIN_INIT);
 
 	/* Disable power support if the user asked so. */
-	if (!i915_modparams.disable_power_well)
+	if (!i915->params.disable_power_well)
 		intel_display_power_get(i915, POWER_DOMAIN_INIT);
 	intel_power_domains_sync_hw(i915);
 
@@ -5399,7 +5399,7 @@ void intel_power_domains_driver_remove(struct drm_i915_private *i915)
 		fetch_and_zero(&i915->power_domains.wakeref);
 
 	/* Remove the refcount we took to keep power well support disabled. */
-	if (!i915_modparams.disable_power_well)
+	if (!i915->params.disable_power_well)
 		intel_display_power_put_unchecked(i915, POWER_DOMAIN_INIT);
 
 	intel_display_power_flush_work_sync(i915);
@@ -5488,7 +5488,7 @@ void intel_power_domains_suspend(struct drm_i915_private *i915,
 	 * Even if power well support was disabled we still want to disable
 	 * power wells if power domains must be deinitialized for suspend.
 	 */
-	if (!i915_modparams.disable_power_well)
+	if (!i915->params.disable_power_well)
 		intel_display_power_put_unchecked(i915, POWER_DOMAIN_INIT);
 
 	intel_display_power_flush_work(i915);
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 408c3c1c5e81..dec0cdce0e68 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -4745,7 +4745,9 @@ intel_dp_sink_can_mst(struct intel_dp *intel_dp)
 static bool
 intel_dp_can_mst(struct intel_dp *intel_dp)
 {
-	return i915_modparams.enable_dp_mst &&
+	struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+
+	return i915->params.enable_dp_mst &&
 		intel_dp->can_mst &&
 		intel_dp_sink_can_mst(intel_dp);
 }
@@ -4762,13 +4764,13 @@ intel_dp_configure_mst(struct intel_dp *intel_dp)
 		    "[ENCODER:%d:%s] MST support: port: %s, sink: %s, modparam: %s\n",
 		    encoder->base.base.id, encoder->base.name,
 		    yesno(intel_dp->can_mst), yesno(sink_can_mst),
-		    yesno(i915_modparams.enable_dp_mst));
+		    yesno(i915->params.enable_dp_mst));
 
 	if (!intel_dp->can_mst)
 		return;
 
 	intel_dp->is_mst = sink_can_mst &&
-		i915_modparams.enable_dp_mst;
+		i915->params.enable_dp_mst;
 
 	drm_dp_mst_topology_mgr_set_mst(&intel_dp->mst_mgr,
 					intel_dp->is_mst);
diff --git a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
index 0722540d64ad..acbd7eb66cbe 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
@@ -348,7 +348,7 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *intel_connector)
 	struct intel_dp *intel_dp = enc_to_intel_dp(intel_connector->encoder);
 	struct drm_i915_private *i915 = dp_to_i915(intel_dp);
 
-	if (i915_modparams.enable_dpcd_backlight == 0 ||
+	if (i915->params.enable_dpcd_backlight == 0 ||
 	    !intel_dp_aux_display_control_capable(intel_connector))
 		return -ENODEV;
 
@@ -358,7 +358,7 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *intel_connector)
 	 */
 	if (i915->vbt.backlight.type !=
 	    INTEL_BACKLIGHT_VESA_EDP_AUX_INTERFACE &&
-	    i915_modparams.enable_dpcd_backlight != 1 &&
+	    i915->params.enable_dpcd_backlight != 1 &&
 	    !drm_dp_has_quirk(&intel_dp->desc, intel_dp->edid_quirks,
 			      DP_QUIRK_FORCE_DPCD_BACKLIGHT)) {
 		drm_info(&i915->drm,
diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
index 1c26673acb2d..30649e17cfb7 100644
--- a/drivers/gpu/drm/i915/display/intel_fbc.c
+++ b/drivers/gpu/drm/i915/display/intel_fbc.c
@@ -740,7 +740,7 @@ static bool intel_fbc_can_enable(struct drm_i915_private *dev_priv)
 		return false;
 	}
 
-	if (!i915_modparams.enable_fbc) {
+	if (!dev_priv->params.enable_fbc) {
 		fbc->no_fbc_reason = "disabled per module param or by default";
 		return false;
 	}
@@ -1017,7 +1017,7 @@ static void __intel_fbc_post_update(struct intel_crtc *crtc)
 
 	fbc->flip_pending = false;
 
-	if (!i915_modparams.enable_fbc) {
+	if (!dev_priv->params.enable_fbc) {
 		intel_fbc_deactivate(dev_priv, "disabled at runtime per module param");
 		__intel_fbc_disable(dev_priv);
 
@@ -1370,8 +1370,8 @@ void intel_fbc_handle_fifo_underrun_irq(struct drm_i915_private *dev_priv)
  */
 static int intel_sanitize_fbc_option(struct drm_i915_private *dev_priv)
 {
-	if (i915_modparams.enable_fbc >= 0)
-		return !!i915_modparams.enable_fbc;
+	if (dev_priv->params.enable_fbc >= 0)
+		return !!dev_priv->params.enable_fbc;
 
 	if (!HAS_FBC(dev_priv))
 		return 0;
@@ -1415,9 +1415,9 @@ void intel_fbc_init(struct drm_i915_private *dev_priv)
 	if (need_fbc_vtd_wa(dev_priv))
 		mkwrite_device_info(dev_priv)->display.has_fbc = false;
 
-	i915_modparams.enable_fbc = intel_sanitize_fbc_option(dev_priv);
+	dev_priv->params.enable_fbc = intel_sanitize_fbc_option(dev_priv);
 	drm_dbg_kms(&dev_priv->drm, "Sanitized enable_fbc value: %d\n",
-		    i915_modparams.enable_fbc);
+		    dev_priv->params.enable_fbc);
 
 	if (!HAS_FBC(dev_priv)) {
 		fbc->no_fbc_reason = "unsupported by this chipset";
diff --git a/drivers/gpu/drm/i915/display/intel_lvds.c b/drivers/gpu/drm/i915/display/intel_lvds.c
index 872f2a489339..1888611244db 100644
--- a/drivers/gpu/drm/i915/display/intel_lvds.c
+++ b/drivers/gpu/drm/i915/display/intel_lvds.c
@@ -784,8 +784,8 @@ static bool compute_is_dual_link_lvds(struct intel_lvds_encoder *lvds_encoder)
 	struct drm_i915_private *dev_priv = to_i915(dev);
 
 	/* use the module option value if specified */
-	if (i915_modparams.lvds_channel_mode > 0)
-		return i915_modparams.lvds_channel_mode == 2;
+	if (dev_priv->params.lvds_channel_mode > 0)
+		return dev_priv->params.lvds_channel_mode == 2;
 
 	/* single channel LVDS is limited to 112 MHz */
 	if (lvds_encoder->attached_connector->panel.fixed_mode->clock > 112999)
diff --git a/drivers/gpu/drm/i915/display/intel_opregion.c b/drivers/gpu/drm/i915/display/intel_opregion.c
index cc6b00959586..de995362f428 100644
--- a/drivers/gpu/drm/i915/display/intel_opregion.c
+++ b/drivers/gpu/drm/i915/display/intel_opregion.c
@@ -801,7 +801,7 @@ static int intel_load_vbt_firmware(struct drm_i915_private *dev_priv)
 {
 	struct intel_opregion *opregion = &dev_priv->opregion;
 	const struct firmware *fw = NULL;
-	const char *name = i915_modparams.vbt_firmware;
+	const char *name = dev_priv->params.vbt_firmware;
 	int ret;
 
 	if (!name || !*name)
diff --git a/drivers/gpu/drm/i915/display/intel_panel.c b/drivers/gpu/drm/i915/display/intel_panel.c
index aa931f9f0d6a..c6d1876b3e8d 100644
--- a/drivers/gpu/drm/i915/display/intel_panel.c
+++ b/drivers/gpu/drm/i915/display/intel_panel.c
@@ -531,10 +531,10 @@ static u32 intel_panel_compute_brightness(struct intel_connector *connector,
 
 	drm_WARN_ON(&dev_priv->drm, panel->backlight.max == 0);
 
-	if (i915_modparams.invert_brightness < 0)
+	if (dev_priv->params.invert_brightness < 0)
 		return val;
 
-	if (i915_modparams.invert_brightness > 0 ||
+	if (dev_priv->params.invert_brightness > 0 ||
 	    dev_priv->quirks & QUIRK_INVERT_BRIGHTNESS) {
 		return panel->backlight.max - val + panel->backlight.min;
 	}
diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c
index b7a2c102648a..c84a2ed19f7b 100644
--- a/drivers/gpu/drm/i915/display/intel_psr.c
+++ b/drivers/gpu/drm/i915/display/intel_psr.c
@@ -83,7 +83,7 @@ static bool psr_global_enabled(struct drm_i915_private *i915)
 {
 	switch (i915->psr.debug & I915_PSR_DEBUG_MODE_MASK) {
 	case I915_PSR_DEBUG_DEFAULT:
-		return i915_modparams.enable_psr;
+		return i915->params.enable_psr;
 	case I915_PSR_DEBUG_DISABLE:
 		return false;
 	default:
@@ -1450,9 +1450,9 @@ void intel_psr_init(struct drm_i915_private *dev_priv)
 		 */
 		dev_priv->hsw_psr_mmio_adjust = _SRD_CTL_EDP - _HSW_EDP_PSR_BASE;
 
-	if (i915_modparams.enable_psr == -1)
+	if (dev_priv->params.enable_psr == -1)
 		if (INTEL_GEN(dev_priv) < 9 || !dev_priv->vbt.psr.enable)
-			i915_modparams.enable_psr = 0;
+			dev_priv->params.enable_psr = 0;
 
 	/* Set link_standby x link_off defaults */
 	if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_context.c b/drivers/gpu/drm/i915/gem/i915_gem_context.c
index 900ea8b7fc8f..1d980dd6b186 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_context.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_context.c
@@ -650,7 +650,7 @@ static void context_close(struct i915_gem_context *ctx)
 	 * context close.
 	 */
 	if (!i915_gem_context_is_persistent(ctx) ||
-	    !i915_modparams.enable_hangcheck)
+	    !ctx->i915->params.enable_hangcheck)
 		kill_context(ctx);
 
 	i915_gem_context_put(ctx);
@@ -667,7 +667,7 @@ static int __context_set_persistence(struct i915_gem_context *ctx, bool state)
 		 * reset] are allowed to survive past termination. We require
 		 * hangcheck to ensure that the persistent requests are healthy.
 		 */
-		if (!i915_modparams.enable_hangcheck)
+		if (!ctx->i915->params.enable_hangcheck)
 			return -EINVAL;
 
 		i915_gem_context_set_persistence(ctx);
diff --git a/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c b/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c
index 5136c8bf112d..1ed4c95b2ea6 100644
--- a/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c
+++ b/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c
@@ -4,6 +4,7 @@
  * Copyright © 2019 Intel Corporation
  */
 
+#include "i915_drv.h"
 #include "i915_request.h"
 
 #include "intel_context.h"
@@ -117,7 +118,7 @@ static void heartbeat(struct work_struct *wrk)
 		goto unlock;
 
 	idle_pulse(engine, rq);
-	if (i915_modparams.enable_hangcheck)
+	if (engine->i915->params.enable_hangcheck)
 		engine->heartbeat.systole = i915_request_get(rq);
 
 	__i915_request_commit(rq);
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
index 39070b514e65..0156f1f5c736 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.c
+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
@@ -638,7 +638,7 @@ int __intel_gt_reset(struct intel_gt *gt, intel_engine_mask_t engine_mask)
 
 bool intel_has_gpu_reset(const struct intel_gt *gt)
 {
-	if (!i915_modparams.reset)
+	if (!gt->i915->params.reset)
 		return NULL;
 
 	return intel_get_gpu_reset(gt);
@@ -646,7 +646,7 @@ bool intel_has_gpu_reset(const struct intel_gt *gt)
 
 bool intel_has_reset_engine(const struct intel_gt *gt)
 {
-	if (i915_modparams.reset < 2)
+	if (gt->i915->params.reset < 2)
 		return false;
 
 	return INTEL_INFO(gt->i915)->has_reset_engine;
@@ -1038,7 +1038,7 @@ void intel_gt_reset(struct intel_gt *gt,
 	awake = reset_prepare(gt);
 
 	if (!intel_has_gpu_reset(gt)) {
-		if (i915_modparams.reset)
+		if (gt->i915->params.reset)
 			drm_err(&gt->i915->drm, "GPU reset not supported\n");
 		else
 			drm_dbg(&gt->i915->drm, "GPU reset disabled\n");
diff --git a/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c b/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c
index 697114dd1f47..9f04c54c4d0f 100644
--- a/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c
+++ b/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c
@@ -386,11 +386,11 @@ int intel_heartbeat_live_selftests(struct drm_i915_private *i915)
 	if (intel_gt_is_wedged(&i915->gt))
 		return 0;
 
-	saved_hangcheck = i915_modparams.enable_hangcheck;
-	i915_modparams.enable_hangcheck = INT_MAX;
+	saved_hangcheck = i915->params.enable_hangcheck;
+	i915->params.enable_hangcheck = INT_MAX;
 
 	err = intel_gt_live_subtests(tests, &i915->gt);
 
-	i915_modparams.enable_hangcheck = saved_hangcheck;
+	i915->params.enable_hangcheck = saved_hangcheck;
 	return err;
 }
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
index fb10f3597ea5..9bbe8a795cb8 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
@@ -424,25 +424,28 @@ static void guc_log_capture_logs(struct intel_guc_log *log)
 
 static u32 __get_default_log_level(struct intel_guc_log *log)
 {
+	struct intel_guc *guc = log_to_guc(log);
+	struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
+
 	/* A negative value means "use platform/config default" */
-	if (i915_modparams.guc_log_level < 0) {
+	if (i915->params.guc_log_level < 0) {
 		return (IS_ENABLED(CONFIG_DRM_I915_DEBUG) ||
 			IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM)) ?
 			GUC_LOG_LEVEL_MAX : GUC_LOG_LEVEL_NON_VERBOSE;
 	}
 
-	if (i915_modparams.guc_log_level > GUC_LOG_LEVEL_MAX) {
+	if (i915->params.guc_log_level > GUC_LOG_LEVEL_MAX) {
 		DRM_WARN("Incompatible option detected: %s=%d, %s!\n",
-			 "guc_log_level", i915_modparams.guc_log_level,
+			 "guc_log_level", i915->params.guc_log_level,
 			 "verbosity too high");
 		return (IS_ENABLED(CONFIG_DRM_I915_DEBUG) ||
 			IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM)) ?
 			GUC_LOG_LEVEL_MAX : GUC_LOG_LEVEL_DISABLED;
 	}
 
-	GEM_BUG_ON(i915_modparams.guc_log_level < GUC_LOG_LEVEL_DISABLED);
-	GEM_BUG_ON(i915_modparams.guc_log_level > GUC_LOG_LEVEL_MAX);
-	return i915_modparams.guc_log_level;
+	GEM_BUG_ON(i915->params.guc_log_level < GUC_LOG_LEVEL_DISABLED);
+	GEM_BUG_ON(i915->params.guc_log_level > GUC_LOG_LEVEL_MAX);
+	return i915->params.guc_log_level;
 }
 
 int intel_guc_log_create(struct intel_guc_log *log)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
index 94eb63f309ce..fdfeb4b9b0f5 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -660,10 +660,12 @@ void intel_guc_submission_disable(struct intel_guc *guc)
 
 static bool __guc_submission_selected(struct intel_guc *guc)
 {
+	struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
+
 	if (!intel_guc_submission_is_supported(guc))
 		return false;
 
-	return i915_modparams.enable_guc & ENABLE_GUC_SUBMISSION;
+	return i915->params.enable_guc & ENABLE_GUC_SUBMISSION;
 }
 
 void intel_guc_submission_init_early(struct intel_guc *guc)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc.c b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
index f518fe05c6f9..1c2d6358826c 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_uc.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
@@ -47,15 +47,15 @@ static void __confirm_options(struct intel_uc *uc)
 
 	drm_dbg(&i915->drm,
 		"enable_guc=%d (guc:%s submission:%s huc:%s)\n",
-		i915_modparams.enable_guc,
+		i915->params.enable_guc,
 		yesno(intel_uc_wants_guc(uc)),
 		yesno(intel_uc_wants_guc_submission(uc)),
 		yesno(intel_uc_wants_huc(uc)));
 
-	if (i915_modparams.enable_guc == -1)
+	if (i915->params.enable_guc == -1)
 		return;
 
-	if (i915_modparams.enable_guc == 0) {
+	if (i915->params.enable_guc == 0) {
 		GEM_BUG_ON(intel_uc_wants_guc(uc));
 		GEM_BUG_ON(intel_uc_wants_guc_submission(uc));
 		GEM_BUG_ON(intel_uc_wants_huc(uc));
@@ -65,25 +65,25 @@ static void __confirm_options(struct intel_uc *uc)
 	if (!intel_uc_supports_guc(uc))
 		drm_info(&i915->drm,
 			 "Incompatible option enable_guc=%d - %s\n",
-			 i915_modparams.enable_guc, "GuC is not supported!");
+			 i915->params.enable_guc, "GuC is not supported!");
 
-	if (i915_modparams.enable_guc & ENABLE_GUC_LOAD_HUC &&
+	if (i915->params.enable_guc & ENABLE_GUC_LOAD_HUC &&
 	    !intel_uc_supports_huc(uc))
 		drm_info(&i915->drm,
 			 "Incompatible option enable_guc=%d - %s\n",
-			 i915_modparams.enable_guc, "HuC is not supported!");
+			 i915->params.enable_guc, "HuC is not supported!");
 
-	if (i915_modparams.enable_guc & ENABLE_GUC_SUBMISSION &&
+	if (i915->params.enable_guc & ENABLE_GUC_SUBMISSION &&
 	    !intel_uc_supports_guc_submission(uc))
 		drm_info(&i915->drm,
 			 "Incompatible option enable_guc=%d - %s\n",
-			 i915_modparams.enable_guc, "GuC submission is N/A");
+			 i915->params.enable_guc, "GuC submission is N/A");
 
-	if (i915_modparams.enable_guc & ~(ENABLE_GUC_SUBMISSION |
+	if (i915->params.enable_guc & ~(ENABLE_GUC_SUBMISSION |
 					  ENABLE_GUC_LOAD_HUC))
 		drm_info(&i915->drm,
 			 "Incompatible option enable_guc=%d - %s\n",
-			 i915_modparams.enable_guc, "undocumented flag");
+			 i915->params.enable_guc, "undocumented flag");
 }
 
 void intel_uc_init_early(struct intel_uc *uc)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c
index e1caae93996d..ef777e207f5c 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c
@@ -114,6 +114,7 @@ struct __packed uc_fw_platform_requirement {
 static void
 __uc_fw_auto_select(struct intel_uc_fw *uc_fw, enum intel_platform p, u8 rev)
 {
+	struct drm_i915_private *i915 = __uc_fw_to_gt(uc_fw)->i915;
 	static const struct uc_fw_platform_requirement fw_blobs[] = {
 		INTEL_UC_FIRMWARE_DEFS(MAKE_FW_LIST, GUC_FW_BLOB, HUC_FW_BLOB)
 	};
@@ -151,35 +152,36 @@ __uc_fw_auto_select(struct intel_uc_fw *uc_fw, enum intel_platform p, u8 rev)
 	}
 
 	/* We don't want to enable GuC/HuC on pre-Gen11 by default */
-	if (i915_modparams.enable_guc == -1 && p < INTEL_ICELAKE)
+	if (i915->params.enable_guc == -1 && p < INTEL_ICELAKE)
 		uc_fw->path = NULL;
 }
 
-static const char *__override_guc_firmware_path(void)
+static const char *__override_guc_firmware_path(struct drm_i915_private *i915)
 {
-	if (i915_modparams.enable_guc & (ENABLE_GUC_SUBMISSION |
-					 ENABLE_GUC_LOAD_HUC))
-		return i915_modparams.guc_firmware_path;
+	if (i915->params.enable_guc & (ENABLE_GUC_SUBMISSION |
+				       ENABLE_GUC_LOAD_HUC))
+		return i915->params.guc_firmware_path;
 	return "";
 }
 
-static const char *__override_huc_firmware_path(void)
+static const char *__override_huc_firmware_path(struct drm_i915_private *i915)
 {
-	if (i915_modparams.enable_guc & ENABLE_GUC_LOAD_HUC)
-		return i915_modparams.huc_firmware_path;
+	if (i915->params.enable_guc & ENABLE_GUC_LOAD_HUC)
+		return i915->params.huc_firmware_path;
 	return "";
 }
 
 static void __uc_fw_user_override(struct intel_uc_fw *uc_fw)
 {
+	struct drm_i915_private *i915 = __uc_fw_to_gt(uc_fw)->i915;
 	const char *path = NULL;
 
 	switch (uc_fw->type) {
 	case INTEL_UC_FW_TYPE_GUC:
-		path = __override_guc_firmware_path();
+		path = __override_guc_firmware_path(i915);
 		break;
 	case INTEL_UC_FW_TYPE_HUC:
-		path = __override_huc_firmware_path();
+		path = __override_huc_firmware_path(i915);
 		break;
 	}
 
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index 8e98df6a3045..9c190c7d8f74 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -64,7 +64,7 @@ static int i915_capabilities(struct seq_file *m, void *data)
 	intel_driver_caps_print(&i915->caps, &p);
 
 	kernel_param_lock(THIS_MODULE);
-	i915_params_dump(&i915_modparams, &p);
+	i915_params_dump(&i915->params, &p);
 	kernel_param_unlock(THIS_MODULE);
 
 	return 0;
diff --git a/drivers/gpu/drm/i915/i915_debugfs_params.c b/drivers/gpu/drm/i915/i915_debugfs_params.c
index 62b2c5f0495d..4e2b077692cb 100644
--- a/drivers/gpu/drm/i915/i915_debugfs_params.c
+++ b/drivers/gpu/drm/i915/i915_debugfs_params.c
@@ -138,9 +138,6 @@ static ssize_t i915_param_charp_write(struct file *file,
 	char **s = m->private;
 	char *new, *old;
 
-	/* FIXME: remove locking after params aren't the module params */
-	kernel_param_lock(THIS_MODULE);
-
 	old = *s;
 	new = strndup_user(ubuf, PAGE_SIZE);
 	if (IS_ERR(new)) {
@@ -152,8 +149,6 @@ static ssize_t i915_param_charp_write(struct file *file,
 
 	kfree(old);
 out:
-	kernel_param_unlock(THIS_MODULE);
-
 	return len;
 }
 
@@ -229,7 +224,7 @@ _i915_param_create_file(struct dentry *parent, const char *name,
 struct dentry *i915_debugfs_params(struct drm_i915_private *i915)
 {
 	struct drm_minor *minor = i915->drm.primary;
-	struct i915_params *params = &i915_modparams;
+	struct i915_params *params = &i915->params;
 	struct dentry *dir;
 
 	dir = debugfs_create_dir("i915_params", minor->debugfs_root);
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index 34ee12f3f02d..67102dc26fce 100644
--- a/drivers/gpu/drm/i915/i915_drv.c
+++ b/drivers/gpu/drm/i915/i915_drv.c
@@ -501,6 +501,8 @@ static void i915_driver_late_release(struct drm_i915_private *dev_priv)
 
 	cpu_latency_qos_remove_request(&dev_priv->sb_qos);
 	mutex_destroy(&dev_priv->sb_lock);
+
+	i915_params_free(&dev_priv->params);
 }
 
 /**
@@ -915,6 +917,9 @@ i915_driver_create(struct pci_dev *pdev, const struct pci_device_id *ent)
 	i915->drm.pdev = pdev;
 	pci_set_drvdata(pdev, i915);
 
+	/* Device parameters start as a copy of module parameters. */
+	i915_params_copy(&i915->params, &i915_modparams);
+
 	/* Setup the write-once "constant" device info */
 	device_info = mkwrite_device_info(i915);
 	memcpy(device_info, match_info, sizeof(*device_info));
@@ -948,7 +953,7 @@ int i915_driver_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 		return PTR_ERR(i915);
 
 	/* Disable nuclear pageflip by default on pre-ILK */
-	if (!i915_modparams.nuclear_pageflip && match_info->gen < 5)
+	if (!i915->params.nuclear_pageflip && match_info->gen < 5)
 		i915->drm.driver_features &= ~DRIVER_ATOMIC;
 
 	/*
@@ -958,7 +963,7 @@ int i915_driver_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 #if IS_ENABLED(CONFIG_DRM_I915_SELFTEST)
 	if (IS_ENABLED(CONFIG_DRM_I915_UNSTABLE_FAKE_LMEM)) {
 		if (INTEL_GEN(i915) >= 9 && i915_selftest.live < 0 &&
-		    i915_modparams.fake_lmem_start) {
+		    i915->params.fake_lmem_start) {
 			mkwrite_device_info(i915)->memory_regions =
 				REGION_SMEM | REGION_LMEM | REGION_STOLEN;
 			mkwrite_device_info(i915)->is_dgfx = true;
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index a378d52341e6..32e899c52ed9 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -829,6 +829,9 @@ struct drm_i915_private {
 	/* FIXME: Device release actions should all be moved to drmm_ */
 	bool do_release;
 
+	/* i915 device parameters */
+	struct i915_params params;
+
 	const struct intel_device_info __info; /* Use INTEL_INFO() to access. */
 	struct intel_runtime_info __runtime; /* Use RUNTIME_INFO() to access. */
 	struct intel_driver_caps caps;
@@ -1658,7 +1661,7 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915,
 #define HAS_DISPLAY(dev_priv) (INTEL_INFO(dev_priv)->pipe_mask != 0)
 
 /* Only valid when HAS_DISPLAY() is true */
-#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !i915_modparams.disable_display)
+#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !dev_priv->params.disable_display)
 
 static inline bool intel_vtd_active(void)
 {
diff --git a/drivers/gpu/drm/i915/i915_getparam.c b/drivers/gpu/drm/i915/i915_getparam.c
index 54fce81d5724..c4a1fac9eb93 100644
--- a/drivers/gpu/drm/i915/i915_getparam.c
+++ b/drivers/gpu/drm/i915/i915_getparam.c
@@ -80,7 +80,7 @@ int i915_getparam_ioctl(struct drm_device *dev, void *data,
 			return -ENODEV;
 		break;
 	case I915_PARAM_HAS_GPU_RESET:
-		value = i915_modparams.enable_hangcheck &&
+		value = i915->params.enable_hangcheck &&
 			intel_has_gpu_reset(&i915->gt);
 		if (value && intel_has_reset_engine(&i915->gt))
 			value = 2;
diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c
index eec292d06f11..866166ada10e 100644
--- a/drivers/gpu/drm/i915/i915_gpu_error.c
+++ b/drivers/gpu/drm/i915/i915_gpu_error.c
@@ -1698,7 +1698,7 @@ static void capture_gen(struct i915_gpu_coredump *error)
 	error->reset_count = i915_reset_count(&i915->gpu_error);
 	error->suspend_count = i915->suspend_count;
 
-	i915_params_copy(&error->params, &i915_modparams);
+	i915_params_copy(&error->params, &i915->params);
 	memcpy(&error->device_info,
 	       INTEL_INFO(i915),
 	       sizeof(error->device_info));
@@ -1713,7 +1713,7 @@ i915_gpu_coredump_alloc(struct drm_i915_private *i915, gfp_t gfp)
 {
 	struct i915_gpu_coredump *error;
 
-	if (!i915_modparams.error_capture)
+	if (!i915->params.error_capture)
 		return NULL;
 
 	error = kzalloc(sizeof(*error), gfp);
diff --git a/drivers/gpu/drm/i915/intel_gvt.c b/drivers/gpu/drm/i915/intel_gvt.c
index 21b91313cc5d..d9bb9a759740 100644
--- a/drivers/gpu/drm/i915/intel_gvt.c
+++ b/drivers/gpu/drm/i915/intel_gvt.c
@@ -64,7 +64,7 @@ static bool is_supported_device(struct drm_i915_private *dev_priv)
  */
 void intel_gvt_sanitize_options(struct drm_i915_private *dev_priv)
 {
-	if (!i915_modparams.enable_gvt)
+	if (!dev_priv->params.enable_gvt)
 		return;
 
 	if (intel_vgpu_active(dev_priv)) {
@@ -80,7 +80,7 @@ void intel_gvt_sanitize_options(struct drm_i915_private *dev_priv)
 
 	return;
 bail:
-	i915_modparams.enable_gvt = 0;
+	dev_priv->params.enable_gvt = 0;
 }
 
 /**
@@ -100,7 +100,7 @@ int intel_gvt_init(struct drm_i915_private *dev_priv)
 	if (i915_inject_probe_failure(dev_priv))
 		return -ENODEV;
 
-	if (!i915_modparams.enable_gvt) {
+	if (!dev_priv->params.enable_gvt) {
 		drm_dbg(&dev_priv->drm,
 			"GVT-g is disabled by kernel params\n");
 		return 0;
@@ -121,7 +121,7 @@ int intel_gvt_init(struct drm_i915_private *dev_priv)
 	return 0;
 
 bail:
-	i915_modparams.enable_gvt = 0;
+	dev_priv->params.enable_gvt = 0;
 	return 0;
 }
 
diff --git a/drivers/gpu/drm/i915/intel_region_lmem.c b/drivers/gpu/drm/i915/intel_region_lmem.c
index 14b59b899c9b..40d8f1a95df6 100644
--- a/drivers/gpu/drm/i915/intel_region_lmem.c
+++ b/drivers/gpu/drm/i915/intel_region_lmem.c
@@ -76,7 +76,7 @@ region_lmem_init(struct intel_memory_region *mem)
 {
 	int ret;
 
-	if (i915_modparams.fake_lmem_start) {
+	if (mem->i915->params.fake_lmem_start) {
 		ret = init_fake_lmem_bar(mem);
 		GEM_BUG_ON(ret);
 	}
@@ -111,12 +111,12 @@ intel_setup_fake_lmem(struct drm_i915_private *i915)
 	resource_size_t start;
 
 	GEM_BUG_ON(i915_ggtt_has_aperture(&i915->ggtt));
-	GEM_BUG_ON(!i915_modparams.fake_lmem_start);
+	GEM_BUG_ON(!i915->params.fake_lmem_start);
 
 	/* Your mappable aperture belongs to me now! */
 	mappable_end = pci_resource_len(pdev, 2);
 	io_start = pci_resource_start(pdev, 2),
-	start = i915_modparams.fake_lmem_start;
+	start = i915->params.fake_lmem_start;
 
 	mem = intel_memory_region_create(i915,
 					 start,
diff --git a/drivers/gpu/drm/i915/intel_uncore.c b/drivers/gpu/drm/i915/intel_uncore.c
index a61cb8ca4d50..fb08b24e92c8 100644
--- a/drivers/gpu/drm/i915/intel_uncore.c
+++ b/drivers/gpu/drm/i915/intel_uncore.c
@@ -1185,7 +1185,7 @@ __unclaimed_reg_debug(struct intel_uncore *uncore,
 		     read ? "read from" : "write to",
 		     i915_mmio_reg_offset(reg)))
 		/* Only report the first N failures */
-		i915_modparams.mmio_debug--;
+		uncore->i915->params.mmio_debug--;
 }
 
 static inline void
@@ -1194,7 +1194,7 @@ unclaimed_reg_debug(struct intel_uncore *uncore,
 		    const bool read,
 		    const bool before)
 {
-	if (likely(!i915_modparams.mmio_debug))
+	if (likely(!uncore->i915->params.mmio_debug))
 		return;
 
 	/* interrupts are disabled and re-enabled around uncore->lock usage */
@@ -2093,12 +2093,12 @@ intel_uncore_arm_unclaimed_mmio_detection(struct intel_uncore *uncore)
 		goto out;
 
 	if (unlikely(check_for_unclaimed_mmio(uncore))) {
-		if (!i915_modparams.mmio_debug) {
+		if (!uncore->i915->params.mmio_debug) {
 			drm_dbg(&uncore->i915->drm,
 				"Unclaimed register detected, "
 				"enabling oneshot unclaimed register reporting. "
 				"Please use i915.mmio_debug=N for more information.\n");
-			i915_modparams.mmio_debug++;
+			uncore->i915->params.mmio_debug++;
 		}
 		uncore->debug->unclaimed_mmio_check--;
 		ret = true;
-- 
2.20.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH 4/4] drm/i915/params: switch to device specific parameters

[Rodrigo Vivi]

On Thu, May 14, 2020 at 06:40:06PM +0300, Jani Nikula wrote:
> Start using device specific parameters instead of module parameters for
> most things. The module parameters become the immutable initial values
> for i915 parameters. The device specific parameters in i915->params
> start life as a copy of i915_modparams. Any later changes are only
> reflected in the debugfs.
> 
> The stragglers are:
> 
> * i915.force_probe and i915.modeset. Needed before dev_priv is
>   available. This is fine because the parameters are read-only and never
>   modified.
> 
> * i915.verbose_state_checks. Passing dev_priv to I915_STATE_WARN and
>   I915_STATE_WARN_ON would result in massive and ugly churn. This is
>   handled by not exposing the parameter via debugfs, and leaving the
>   parameter writable in sysfs. This may be fixed up in follow-up work.
> 
> * i915.inject_probe_failure. Only makes sense in terms of the module,
>   not the device. This is handled by not exposing the parameter via
>   debugfs.
> 
> Cc: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com>
> Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  drivers/gpu/drm/i915/display/intel_bios.c     |  6 ++---
>  drivers/gpu/drm/i915/display/intel_crt.c      |  4 ++--
>  drivers/gpu/drm/i915/display/intel_csr.c      |  6 ++---
>  drivers/gpu/drm/i915/display/intel_display.c  | 12 +++++-----
>  .../drm/i915/display/intel_display_debugfs.c  |  2 +-
>  .../drm/i915/display/intel_display_power.c    | 14 ++++++------
>  drivers/gpu/drm/i915/display/intel_dp.c       |  8 ++++---
>  .../drm/i915/display/intel_dp_aux_backlight.c |  4 ++--
>  drivers/gpu/drm/i915/display/intel_fbc.c      | 12 +++++-----
>  drivers/gpu/drm/i915/display/intel_lvds.c     |  4 ++--
>  drivers/gpu/drm/i915/display/intel_opregion.c |  2 +-
>  drivers/gpu/drm/i915/display/intel_panel.c    |  4 ++--
>  drivers/gpu/drm/i915/display/intel_psr.c      |  6 ++---
>  drivers/gpu/drm/i915/gem/i915_gem_context.c   |  4 ++--
>  .../gpu/drm/i915/gt/intel_engine_heartbeat.c  |  3 ++-
>  drivers/gpu/drm/i915/gt/intel_reset.c         |  6 ++---
>  .../drm/i915/gt/selftest_engine_heartbeat.c   |  6 ++---
>  drivers/gpu/drm/i915/gt/uc/intel_guc_log.c    | 15 ++++++++-----
>  .../gpu/drm/i915/gt/uc/intel_guc_submission.c |  4 +++-
>  drivers/gpu/drm/i915/gt/uc/intel_uc.c         | 20 ++++++++---------
>  drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c      | 22 ++++++++++---------
>  drivers/gpu/drm/i915/i915_debugfs.c           |  2 +-
>  drivers/gpu/drm/i915/i915_debugfs_params.c    |  7 +-----
>  drivers/gpu/drm/i915/i915_drv.c               |  9 ++++++--
>  drivers/gpu/drm/i915/i915_drv.h               |  5 ++++-
>  drivers/gpu/drm/i915/i915_getparam.c          |  2 +-
>  drivers/gpu/drm/i915/i915_gpu_error.c         |  4 ++--
>  drivers/gpu/drm/i915/intel_gvt.c              |  8 +++----
>  drivers/gpu/drm/i915/intel_region_lmem.c      |  6 ++---
>  drivers/gpu/drm/i915/intel_uncore.c           |  8 +++----
>  30 files changed, 114 insertions(+), 101 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
> index 839124647202..ec8af2b7bf01 100644
> --- a/drivers/gpu/drm/i915/display/intel_bios.c
> +++ b/drivers/gpu/drm/i915/display/intel_bios.c
> @@ -479,7 +479,7 @@ parse_sdvo_panel_data(struct drm_i915_private *dev_priv,
>  	struct drm_display_mode *panel_fixed_mode;
>  	int index;
>  
> -	index = i915_modparams.vbt_sdvo_panel_type;
> +	index = dev_priv->params.vbt_sdvo_panel_type;
>  	if (index == -2) {
>  		drm_dbg_kms(&dev_priv->drm,
>  			    "Ignore SDVO panel mode from BIOS VBT tables.\n");
> @@ -829,9 +829,9 @@ parse_edp(struct drm_i915_private *dev_priv, const struct bdb_header *bdb)
>  		u8 vswing;
>  
>  		/* Don't read from VBT if module parameter has valid value*/
> -		if (i915_modparams.edp_vswing) {
> +		if (dev_priv->params.edp_vswing) {
>  			dev_priv->vbt.edp.low_vswing =
> -				i915_modparams.edp_vswing == 1;
> +				dev_priv->params.edp_vswing == 1;
>  		} else {
>  			vswing = (edp->edp_vswing_preemph >> (panel_type * 4)) & 0xF;
>  			dev_priv->vbt.edp.low_vswing = vswing == 0;
> diff --git a/drivers/gpu/drm/i915/display/intel_crt.c b/drivers/gpu/drm/i915/display/intel_crt.c
> index 2f5b9a4baafd..5b4510ce5693 100644
> --- a/drivers/gpu/drm/i915/display/intel_crt.c
> +++ b/drivers/gpu/drm/i915/display/intel_crt.c
> @@ -833,7 +833,7 @@ intel_crt_detect(struct drm_connector *connector,
>  		    connector->base.id, connector->name,
>  		    force);
>  
> -	if (i915_modparams.load_detect_test) {
> +	if (dev_priv->params.load_detect_test) {
>  		wakeref = intel_display_power_get(dev_priv,
>  						  intel_encoder->power_domain);
>  		goto load_detect;
> @@ -889,7 +889,7 @@ intel_crt_detect(struct drm_connector *connector,
>  		else if (INTEL_GEN(dev_priv) < 4)
>  			status = intel_crt_load_detect(crt,
>  				to_intel_crtc(connector->state->crtc)->pipe);
> -		else if (i915_modparams.load_detect_test)
> +		else if (dev_priv->params.load_detect_test)
>  			status = connector_status_disconnected;
>  		else
>  			status = connector_status_unknown;
> diff --git a/drivers/gpu/drm/i915/display/intel_csr.c b/drivers/gpu/drm/i915/display/intel_csr.c
> index 3112572cfb7d..8c481270a160 100644
> --- a/drivers/gpu/drm/i915/display/intel_csr.c
> +++ b/drivers/gpu/drm/i915/display/intel_csr.c
> @@ -713,15 +713,15 @@ void intel_csr_ucode_init(struct drm_i915_private *dev_priv)
>  		csr->max_fw_size = BXT_CSR_MAX_FW_SIZE;
>  	}
>  
> -	if (i915_modparams.dmc_firmware_path) {
> -		if (strlen(i915_modparams.dmc_firmware_path) == 0) {
> +	if (dev_priv->params.dmc_firmware_path) {
> +		if (strlen(dev_priv->params.dmc_firmware_path) == 0) {
>  			csr->fw_path = NULL;
>  			drm_info(&dev_priv->drm,
>  				 "Disabling CSR firmware and runtime PM\n");
>  			return;
>  		}
>  
> -		csr->fw_path = i915_modparams.dmc_firmware_path;
> +		csr->fw_path = dev_priv->params.dmc_firmware_path;
>  		/* Bypass version check for firmware override. */
>  		csr->required_version = 0;
>  	}
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index ab621926def1..507ab78b9f3b 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -4879,7 +4879,7 @@ void intel_prepare_reset(struct drm_i915_private *dev_priv)
>  	int ret;
>  
>  	/* reset doesn't touch the display */
> -	if (!i915_modparams.force_reset_modeset_test &&
> +	if (!dev_priv->params.force_reset_modeset_test &&
>  	    !gpu_reset_clobbers_display(dev_priv))
>  		return;
>  
> @@ -7869,7 +7869,7 @@ bool hsw_crtc_state_ips_capable(const struct intel_crtc_state *crtc_state)
>  	if (!hsw_crtc_supports_ips(crtc))
>  		return false;
>  
> -	if (!i915_modparams.enable_ips)
> +	if (!dev_priv->params.enable_ips)
>  		return false;
>  
>  	if (crtc_state->pipe_bpp > 24)
> @@ -8140,8 +8140,8 @@ static void intel_panel_sanitize_ssc(struct drm_i915_private *dev_priv)
>  
>  static bool intel_panel_use_ssc(struct drm_i915_private *dev_priv)
>  {
> -	if (i915_modparams.panel_use_ssc >= 0)
> -		return i915_modparams.panel_use_ssc != 0;
> +	if (dev_priv->params.panel_use_ssc >= 0)
> +		return dev_priv->params.panel_use_ssc != 0;
>  	return dev_priv->vbt.lvds_use_ssc
>  		&& !(dev_priv->quirks & QUIRK_LVDS_SSC_DISABLE);
>  }
> @@ -13571,8 +13571,8 @@ pipe_config_mismatch(bool fastset, const struct intel_crtc *crtc,
>  
>  static bool fastboot_enabled(struct drm_i915_private *dev_priv)
>  {
> -	if (i915_modparams.fastboot != -1)
> -		return i915_modparams.fastboot;
> +	if (dev_priv->params.fastboot != -1)
> +		return dev_priv->params.fastboot;
>  
>  	/* Enable fastboot by default on Skylake and newer */
>  	if (INTEL_GEN(dev_priv) >= 9)
> diff --git a/drivers/gpu/drm/i915/display/intel_display_debugfs.c b/drivers/gpu/drm/i915/display/intel_display_debugfs.c
> index 70525623bcdf..87db7e906621 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_debugfs.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_debugfs.c
> @@ -125,7 +125,7 @@ static int i915_ips_status(struct seq_file *m, void *unused)
>  	wakeref = intel_runtime_pm_get(&dev_priv->runtime_pm);
>  
>  	seq_printf(m, "Enabled by kernel parameter: %s\n",
> -		   yesno(i915_modparams.enable_ips));
> +		   yesno(dev_priv->params.enable_ips));
>  
>  	if (INTEL_GEN(dev_priv) >= 8) {
>  		seq_puts(m, "Currently: unknown\n");
> diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c
> index 49998906cc61..ad6729ba90ab 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_power.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_power.c
> @@ -4322,7 +4322,7 @@ static u32 get_allowed_dc_mask(const struct drm_i915_private *dev_priv,
>  		mask = 0;
>  	}
>  
> -	if (!i915_modparams.disable_power_well)
> +	if (!dev_priv->params.disable_power_well)
>  		max_dc = 0;
>  
>  	if (enable_dc >= 0 && enable_dc <= max_dc) {
> @@ -4408,11 +4408,11 @@ int intel_power_domains_init(struct drm_i915_private *dev_priv)
>  	struct i915_power_domains *power_domains = &dev_priv->power_domains;
>  	int err;
>  
> -	i915_modparams.disable_power_well =
> +	dev_priv->params.disable_power_well =
>  		sanitize_disable_power_well_option(dev_priv,
> -						   i915_modparams.disable_power_well);
> +						   dev_priv->params.disable_power_well);
>  	dev_priv->csr.allowed_dc_mask =
> -		get_allowed_dc_mask(dev_priv, i915_modparams.enable_dc);
> +		get_allowed_dc_mask(dev_priv, dev_priv->params.enable_dc);
>  
>  	dev_priv->csr.target_dc_state =
>  		sanitize_target_dc_state(dev_priv, DC_STATE_EN_UPTO_DC6);
> @@ -5375,7 +5375,7 @@ void intel_power_domains_init_hw(struct drm_i915_private *i915, bool resume)
>  		intel_display_power_get(i915, POWER_DOMAIN_INIT);
>  
>  	/* Disable power support if the user asked so. */
> -	if (!i915_modparams.disable_power_well)
> +	if (!i915->params.disable_power_well)
>  		intel_display_power_get(i915, POWER_DOMAIN_INIT);
>  	intel_power_domains_sync_hw(i915);
>  
> @@ -5399,7 +5399,7 @@ void intel_power_domains_driver_remove(struct drm_i915_private *i915)
>  		fetch_and_zero(&i915->power_domains.wakeref);
>  
>  	/* Remove the refcount we took to keep power well support disabled. */
> -	if (!i915_modparams.disable_power_well)
> +	if (!i915->params.disable_power_well)
>  		intel_display_power_put_unchecked(i915, POWER_DOMAIN_INIT);
>  
>  	intel_display_power_flush_work_sync(i915);
> @@ -5488,7 +5488,7 @@ void intel_power_domains_suspend(struct drm_i915_private *i915,
>  	 * Even if power well support was disabled we still want to disable
>  	 * power wells if power domains must be deinitialized for suspend.
>  	 */
> -	if (!i915_modparams.disable_power_well)
> +	if (!i915->params.disable_power_well)
>  		intel_display_power_put_unchecked(i915, POWER_DOMAIN_INIT);
>  
>  	intel_display_power_flush_work(i915);
> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
> index 408c3c1c5e81..dec0cdce0e68 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> @@ -4745,7 +4745,9 @@ intel_dp_sink_can_mst(struct intel_dp *intel_dp)
>  static bool
>  intel_dp_can_mst(struct intel_dp *intel_dp)
>  {
> -	return i915_modparams.enable_dp_mst &&
> +	struct drm_i915_private *i915 = dp_to_i915(intel_dp);
> +
> +	return i915->params.enable_dp_mst &&
>  		intel_dp->can_mst &&
>  		intel_dp_sink_can_mst(intel_dp);
>  }
> @@ -4762,13 +4764,13 @@ intel_dp_configure_mst(struct intel_dp *intel_dp)
>  		    "[ENCODER:%d:%s] MST support: port: %s, sink: %s, modparam: %s\n",
>  		    encoder->base.base.id, encoder->base.name,
>  		    yesno(intel_dp->can_mst), yesno(sink_can_mst),
> -		    yesno(i915_modparams.enable_dp_mst));
> +		    yesno(i915->params.enable_dp_mst));
>  
>  	if (!intel_dp->can_mst)
>  		return;
>  
>  	intel_dp->is_mst = sink_can_mst &&
> -		i915_modparams.enable_dp_mst;
> +		i915->params.enable_dp_mst;
>  
>  	drm_dp_mst_topology_mgr_set_mst(&intel_dp->mst_mgr,
>  					intel_dp->is_mst);
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
> index 0722540d64ad..acbd7eb66cbe 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
> @@ -348,7 +348,7 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *intel_connector)
>  	struct intel_dp *intel_dp = enc_to_intel_dp(intel_connector->encoder);
>  	struct drm_i915_private *i915 = dp_to_i915(intel_dp);
>  
> -	if (i915_modparams.enable_dpcd_backlight == 0 ||
> +	if (i915->params.enable_dpcd_backlight == 0 ||
>  	    !intel_dp_aux_display_control_capable(intel_connector))
>  		return -ENODEV;
>  
> @@ -358,7 +358,7 @@ int intel_dp_aux_init_backlight_funcs(struct intel_connector *intel_connector)
>  	 */
>  	if (i915->vbt.backlight.type !=
>  	    INTEL_BACKLIGHT_VESA_EDP_AUX_INTERFACE &&
> -	    i915_modparams.enable_dpcd_backlight != 1 &&
> +	    i915->params.enable_dpcd_backlight != 1 &&
>  	    !drm_dp_has_quirk(&intel_dp->desc, intel_dp->edid_quirks,
>  			      DP_QUIRK_FORCE_DPCD_BACKLIGHT)) {
>  		drm_info(&i915->drm,
> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
> index 1c26673acb2d..30649e17cfb7 100644
> --- a/drivers/gpu/drm/i915/display/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/display/intel_fbc.c
> @@ -740,7 +740,7 @@ static bool intel_fbc_can_enable(struct drm_i915_private *dev_priv)
>  		return false;
>  	}
>  
> -	if (!i915_modparams.enable_fbc) {
> +	if (!dev_priv->params.enable_fbc) {
>  		fbc->no_fbc_reason = "disabled per module param or by default";
>  		return false;
>  	}
> @@ -1017,7 +1017,7 @@ static void __intel_fbc_post_update(struct intel_crtc *crtc)
>  
>  	fbc->flip_pending = false;
>  
> -	if (!i915_modparams.enable_fbc) {
> +	if (!dev_priv->params.enable_fbc) {
>  		intel_fbc_deactivate(dev_priv, "disabled at runtime per module param");
>  		__intel_fbc_disable(dev_priv);
>  
> @@ -1370,8 +1370,8 @@ void intel_fbc_handle_fifo_underrun_irq(struct drm_i915_private *dev_priv)
>   */
>  static int intel_sanitize_fbc_option(struct drm_i915_private *dev_priv)
>  {
> -	if (i915_modparams.enable_fbc >= 0)
> -		return !!i915_modparams.enable_fbc;
> +	if (dev_priv->params.enable_fbc >= 0)
> +		return !!dev_priv->params.enable_fbc;
>  
>  	if (!HAS_FBC(dev_priv))
>  		return 0;
> @@ -1415,9 +1415,9 @@ void intel_fbc_init(struct drm_i915_private *dev_priv)
>  	if (need_fbc_vtd_wa(dev_priv))
>  		mkwrite_device_info(dev_priv)->display.has_fbc = false;
>  
> -	i915_modparams.enable_fbc = intel_sanitize_fbc_option(dev_priv);
> +	dev_priv->params.enable_fbc = intel_sanitize_fbc_option(dev_priv);
>  	drm_dbg_kms(&dev_priv->drm, "Sanitized enable_fbc value: %d\n",
> -		    i915_modparams.enable_fbc);
> +		    dev_priv->params.enable_fbc);
>  
>  	if (!HAS_FBC(dev_priv)) {
>  		fbc->no_fbc_reason = "unsupported by this chipset";
> diff --git a/drivers/gpu/drm/i915/display/intel_lvds.c b/drivers/gpu/drm/i915/display/intel_lvds.c
> index 872f2a489339..1888611244db 100644
> --- a/drivers/gpu/drm/i915/display/intel_lvds.c
> +++ b/drivers/gpu/drm/i915/display/intel_lvds.c
> @@ -784,8 +784,8 @@ static bool compute_is_dual_link_lvds(struct intel_lvds_encoder *lvds_encoder)
>  	struct drm_i915_private *dev_priv = to_i915(dev);
>  
>  	/* use the module option value if specified */
> -	if (i915_modparams.lvds_channel_mode > 0)
> -		return i915_modparams.lvds_channel_mode == 2;
> +	if (dev_priv->params.lvds_channel_mode > 0)
> +		return dev_priv->params.lvds_channel_mode == 2;
>  
>  	/* single channel LVDS is limited to 112 MHz */
>  	if (lvds_encoder->attached_connector->panel.fixed_mode->clock > 112999)
> diff --git a/drivers/gpu/drm/i915/display/intel_opregion.c b/drivers/gpu/drm/i915/display/intel_opregion.c
> index cc6b00959586..de995362f428 100644
> --- a/drivers/gpu/drm/i915/display/intel_opregion.c
> +++ b/drivers/gpu/drm/i915/display/intel_opregion.c
> @@ -801,7 +801,7 @@ static int intel_load_vbt_firmware(struct drm_i915_private *dev_priv)
>  {
>  	struct intel_opregion *opregion = &dev_priv->opregion;
>  	const struct firmware *fw = NULL;
> -	const char *name = i915_modparams.vbt_firmware;
> +	const char *name = dev_priv->params.vbt_firmware;
>  	int ret;
>  
>  	if (!name || !*name)
> diff --git a/drivers/gpu/drm/i915/display/intel_panel.c b/drivers/gpu/drm/i915/display/intel_panel.c
> index aa931f9f0d6a..c6d1876b3e8d 100644
> --- a/drivers/gpu/drm/i915/display/intel_panel.c
> +++ b/drivers/gpu/drm/i915/display/intel_panel.c
> @@ -531,10 +531,10 @@ static u32 intel_panel_compute_brightness(struct intel_connector *connector,
>  
>  	drm_WARN_ON(&dev_priv->drm, panel->backlight.max == 0);
>  
> -	if (i915_modparams.invert_brightness < 0)
> +	if (dev_priv->params.invert_brightness < 0)
>  		return val;
>  
> -	if (i915_modparams.invert_brightness > 0 ||
> +	if (dev_priv->params.invert_brightness > 0 ||
>  	    dev_priv->quirks & QUIRK_INVERT_BRIGHTNESS) {
>  		return panel->backlight.max - val + panel->backlight.min;
>  	}
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c
> index b7a2c102648a..c84a2ed19f7b 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.c
> +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> @@ -83,7 +83,7 @@ static bool psr_global_enabled(struct drm_i915_private *i915)
>  {
>  	switch (i915->psr.debug & I915_PSR_DEBUG_MODE_MASK) {
>  	case I915_PSR_DEBUG_DEFAULT:
> -		return i915_modparams.enable_psr;
> +		return i915->params.enable_psr;
>  	case I915_PSR_DEBUG_DISABLE:
>  		return false;
>  	default:
> @@ -1450,9 +1450,9 @@ void intel_psr_init(struct drm_i915_private *dev_priv)
>  		 */
>  		dev_priv->hsw_psr_mmio_adjust = _SRD_CTL_EDP - _HSW_EDP_PSR_BASE;
>  
> -	if (i915_modparams.enable_psr == -1)
> +	if (dev_priv->params.enable_psr == -1)
>  		if (INTEL_GEN(dev_priv) < 9 || !dev_priv->vbt.psr.enable)
> -			i915_modparams.enable_psr = 0;
> +			dev_priv->params.enable_psr = 0;
>  
>  	/* Set link_standby x link_off defaults */
>  	if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_context.c b/drivers/gpu/drm/i915/gem/i915_gem_context.c
> index 900ea8b7fc8f..1d980dd6b186 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_context.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_context.c
> @@ -650,7 +650,7 @@ static void context_close(struct i915_gem_context *ctx)
>  	 * context close.
>  	 */
>  	if (!i915_gem_context_is_persistent(ctx) ||
> -	    !i915_modparams.enable_hangcheck)
> +	    !ctx->i915->params.enable_hangcheck)
>  		kill_context(ctx);
>  
>  	i915_gem_context_put(ctx);
> @@ -667,7 +667,7 @@ static int __context_set_persistence(struct i915_gem_context *ctx, bool state)
>  		 * reset] are allowed to survive past termination. We require
>  		 * hangcheck to ensure that the persistent requests are healthy.
>  		 */
> -		if (!i915_modparams.enable_hangcheck)
> +		if (!ctx->i915->params.enable_hangcheck)
>  			return -EINVAL;
>  
>  		i915_gem_context_set_persistence(ctx);
> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c b/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c
> index 5136c8bf112d..1ed4c95b2ea6 100644
> --- a/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c
> +++ b/drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c
> @@ -4,6 +4,7 @@
>   * Copyright © 2019 Intel Corporation
>   */
>  
> +#include "i915_drv.h"
>  #include "i915_request.h"
>  
>  #include "intel_context.h"
> @@ -117,7 +118,7 @@ static void heartbeat(struct work_struct *wrk)
>  		goto unlock;
>  
>  	idle_pulse(engine, rq);
> -	if (i915_modparams.enable_hangcheck)
> +	if (engine->i915->params.enable_hangcheck)
>  		engine->heartbeat.systole = i915_request_get(rq);
>  
>  	__i915_request_commit(rq);
> diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
> index 39070b514e65..0156f1f5c736 100644
> --- a/drivers/gpu/drm/i915/gt/intel_reset.c
> +++ b/drivers/gpu/drm/i915/gt/intel_reset.c
> @@ -638,7 +638,7 @@ int __intel_gt_reset(struct intel_gt *gt, intel_engine_mask_t engine_mask)
>  
>  bool intel_has_gpu_reset(const struct intel_gt *gt)
>  {
> -	if (!i915_modparams.reset)
> +	if (!gt->i915->params.reset)
>  		return NULL;
>  
>  	return intel_get_gpu_reset(gt);
> @@ -646,7 +646,7 @@ bool intel_has_gpu_reset(const struct intel_gt *gt)
>  
>  bool intel_has_reset_engine(const struct intel_gt *gt)
>  {
> -	if (i915_modparams.reset < 2)
> +	if (gt->i915->params.reset < 2)
>  		return false;
>  
>  	return INTEL_INFO(gt->i915)->has_reset_engine;
> @@ -1038,7 +1038,7 @@ void intel_gt_reset(struct intel_gt *gt,
>  	awake = reset_prepare(gt);
>  
>  	if (!intel_has_gpu_reset(gt)) {
> -		if (i915_modparams.reset)
> +		if (gt->i915->params.reset)
>  			drm_err(&gt->i915->drm, "GPU reset not supported\n");
>  		else
>  			drm_dbg(&gt->i915->drm, "GPU reset disabled\n");
> diff --git a/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c b/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c
> index 697114dd1f47..9f04c54c4d0f 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_engine_heartbeat.c
> @@ -386,11 +386,11 @@ int intel_heartbeat_live_selftests(struct drm_i915_private *i915)
>  	if (intel_gt_is_wedged(&i915->gt))
>  		return 0;
>  
> -	saved_hangcheck = i915_modparams.enable_hangcheck;
> -	i915_modparams.enable_hangcheck = INT_MAX;
> +	saved_hangcheck = i915->params.enable_hangcheck;
> +	i915->params.enable_hangcheck = INT_MAX;
>  
>  	err = intel_gt_live_subtests(tests, &i915->gt);
>  
> -	i915_modparams.enable_hangcheck = saved_hangcheck;
> +	i915->params.enable_hangcheck = saved_hangcheck;
>  	return err;
>  }
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
> index fb10f3597ea5..9bbe8a795cb8 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
> @@ -424,25 +424,28 @@ static void guc_log_capture_logs(struct intel_guc_log *log)
>  
>  static u32 __get_default_log_level(struct intel_guc_log *log)
>  {
> +	struct intel_guc *guc = log_to_guc(log);
> +	struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
> +
>  	/* A negative value means "use platform/config default" */
> -	if (i915_modparams.guc_log_level < 0) {
> +	if (i915->params.guc_log_level < 0) {
>  		return (IS_ENABLED(CONFIG_DRM_I915_DEBUG) ||
>  			IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM)) ?
>  			GUC_LOG_LEVEL_MAX : GUC_LOG_LEVEL_NON_VERBOSE;
>  	}
>  
> -	if (i915_modparams.guc_log_level > GUC_LOG_LEVEL_MAX) {
> +	if (i915->params.guc_log_level > GUC_LOG_LEVEL_MAX) {
>  		DRM_WARN("Incompatible option detected: %s=%d, %s!\n",
> -			 "guc_log_level", i915_modparams.guc_log_level,
> +			 "guc_log_level", i915->params.guc_log_level,
>  			 "verbosity too high");
>  		return (IS_ENABLED(CONFIG_DRM_I915_DEBUG) ||
>  			IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM)) ?
>  			GUC_LOG_LEVEL_MAX : GUC_LOG_LEVEL_DISABLED;
>  	}
>  
> -	GEM_BUG_ON(i915_modparams.guc_log_level < GUC_LOG_LEVEL_DISABLED);
> -	GEM_BUG_ON(i915_modparams.guc_log_level > GUC_LOG_LEVEL_MAX);
> -	return i915_modparams.guc_log_level;
> +	GEM_BUG_ON(i915->params.guc_log_level < GUC_LOG_LEVEL_DISABLED);
> +	GEM_BUG_ON(i915->params.guc_log_level > GUC_LOG_LEVEL_MAX);
> +	return i915->params.guc_log_level;
>  }
>  
>  int intel_guc_log_create(struct intel_guc_log *log)
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> index 94eb63f309ce..fdfeb4b9b0f5 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> @@ -660,10 +660,12 @@ void intel_guc_submission_disable(struct intel_guc *guc)
>  
>  static bool __guc_submission_selected(struct intel_guc *guc)
>  {
> +	struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
> +
>  	if (!intel_guc_submission_is_supported(guc))
>  		return false;
>  
> -	return i915_modparams.enable_guc & ENABLE_GUC_SUBMISSION;
> +	return i915->params.enable_guc & ENABLE_GUC_SUBMISSION;
>  }
>  
>  void intel_guc_submission_init_early(struct intel_guc *guc)
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc.c b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
> index f518fe05c6f9..1c2d6358826c 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_uc.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
> @@ -47,15 +47,15 @@ static void __confirm_options(struct intel_uc *uc)
>  
>  	drm_dbg(&i915->drm,
>  		"enable_guc=%d (guc:%s submission:%s huc:%s)\n",
> -		i915_modparams.enable_guc,
> +		i915->params.enable_guc,
>  		yesno(intel_uc_wants_guc(uc)),
>  		yesno(intel_uc_wants_guc_submission(uc)),
>  		yesno(intel_uc_wants_huc(uc)));
>  
> -	if (i915_modparams.enable_guc == -1)
> +	if (i915->params.enable_guc == -1)
>  		return;
>  
> -	if (i915_modparams.enable_guc == 0) {
> +	if (i915->params.enable_guc == 0) {
>  		GEM_BUG_ON(intel_uc_wants_guc(uc));
>  		GEM_BUG_ON(intel_uc_wants_guc_submission(uc));
>  		GEM_BUG_ON(intel_uc_wants_huc(uc));
> @@ -65,25 +65,25 @@ static void __confirm_options(struct intel_uc *uc)
>  	if (!intel_uc_supports_guc(uc))
>  		drm_info(&i915->drm,
>  			 "Incompatible option enable_guc=%d - %s\n",
> -			 i915_modparams.enable_guc, "GuC is not supported!");
> +			 i915->params.enable_guc, "GuC is not supported!");
>  
> -	if (i915_modparams.enable_guc & ENABLE_GUC_LOAD_HUC &&
> +	if (i915->params.enable_guc & ENABLE_GUC_LOAD_HUC &&
>  	    !intel_uc_supports_huc(uc))
>  		drm_info(&i915->drm,
>  			 "Incompatible option enable_guc=%d - %s\n",
> -			 i915_modparams.enable_guc, "HuC is not supported!");
> +			 i915->params.enable_guc, "HuC is not supported!");
>  
> -	if (i915_modparams.enable_guc & ENABLE_GUC_SUBMISSION &&
> +	if (i915->params.enable_guc & ENABLE_GUC_SUBMISSION &&
>  	    !intel_uc_supports_guc_submission(uc))
>  		drm_info(&i915->drm,
>  			 "Incompatible option enable_guc=%d - %s\n",
> -			 i915_modparams.enable_guc, "GuC submission is N/A");
> +			 i915->params.enable_guc, "GuC submission is N/A");
>  
> -	if (i915_modparams.enable_guc & ~(ENABLE_GUC_SUBMISSION |
> +	if (i915->params.enable_guc & ~(ENABLE_GUC_SUBMISSION |
>  					  ENABLE_GUC_LOAD_HUC))
>  		drm_info(&i915->drm,
>  			 "Incompatible option enable_guc=%d - %s\n",
> -			 i915_modparams.enable_guc, "undocumented flag");
> +			 i915->params.enable_guc, "undocumented flag");
>  }
>  
>  void intel_uc_init_early(struct intel_uc *uc)
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c
> index e1caae93996d..ef777e207f5c 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c
> @@ -114,6 +114,7 @@ struct __packed uc_fw_platform_requirement {
>  static void
>  __uc_fw_auto_select(struct intel_uc_fw *uc_fw, enum intel_platform p, u8 rev)
>  {
> +	struct drm_i915_private *i915 = __uc_fw_to_gt(uc_fw)->i915;
>  	static const struct uc_fw_platform_requirement fw_blobs[] = {
>  		INTEL_UC_FIRMWARE_DEFS(MAKE_FW_LIST, GUC_FW_BLOB, HUC_FW_BLOB)
>  	};
> @@ -151,35 +152,36 @@ __uc_fw_auto_select(struct intel_uc_fw *uc_fw, enum intel_platform p, u8 rev)
>  	}
>  
>  	/* We don't want to enable GuC/HuC on pre-Gen11 by default */
> -	if (i915_modparams.enable_guc == -1 && p < INTEL_ICELAKE)
> +	if (i915->params.enable_guc == -1 && p < INTEL_ICELAKE)
>  		uc_fw->path = NULL;
>  }
>  
> -static const char *__override_guc_firmware_path(void)
> +static const char *__override_guc_firmware_path(struct drm_i915_private *i915)
>  {
> -	if (i915_modparams.enable_guc & (ENABLE_GUC_SUBMISSION |
> -					 ENABLE_GUC_LOAD_HUC))
> -		return i915_modparams.guc_firmware_path;
> +	if (i915->params.enable_guc & (ENABLE_GUC_SUBMISSION |
> +				       ENABLE_GUC_LOAD_HUC))
> +		return i915->params.guc_firmware_path;
>  	return "";
>  }
>  
> -static const char *__override_huc_firmware_path(void)
> +static const char *__override_huc_firmware_path(struct drm_i915_private *i915)
>  {
> -	if (i915_modparams.enable_guc & ENABLE_GUC_LOAD_HUC)
> -		return i915_modparams.huc_firmware_path;
> +	if (i915->params.enable_guc & ENABLE_GUC_LOAD_HUC)
> +		return i915->params.huc_firmware_path;
>  	return "";
>  }
>  
>  static void __uc_fw_user_override(struct intel_uc_fw *uc_fw)
>  {
> +	struct drm_i915_private *i915 = __uc_fw_to_gt(uc_fw)->i915;
>  	const char *path = NULL;
>  
>  	switch (uc_fw->type) {
>  	case INTEL_UC_FW_TYPE_GUC:
> -		path = __override_guc_firmware_path();
> +		path = __override_guc_firmware_path(i915);
>  		break;
>  	case INTEL_UC_FW_TYPE_HUC:
> -		path = __override_huc_firmware_path();
> +		path = __override_huc_firmware_path(i915);
>  		break;
>  	}
>  
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 8e98df6a3045..9c190c7d8f74 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -64,7 +64,7 @@ static int i915_capabilities(struct seq_file *m, void *data)
>  	intel_driver_caps_print(&i915->caps, &p);
>  
>  	kernel_param_lock(THIS_MODULE);
> -	i915_params_dump(&i915_modparams, &p);
> +	i915_params_dump(&i915->params, &p);
>  	kernel_param_unlock(THIS_MODULE);
>  
>  	return 0;
> diff --git a/drivers/gpu/drm/i915/i915_debugfs_params.c b/drivers/gpu/drm/i915/i915_debugfs_params.c
> index 62b2c5f0495d..4e2b077692cb 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs_params.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs_params.c
> @@ -138,9 +138,6 @@ static ssize_t i915_param_charp_write(struct file *file,
>  	char **s = m->private;
>  	char *new, *old;
>  
> -	/* FIXME: remove locking after params aren't the module params */
> -	kernel_param_lock(THIS_MODULE);
> -
>  	old = *s;
>  	new = strndup_user(ubuf, PAGE_SIZE);
>  	if (IS_ERR(new)) {
> @@ -152,8 +149,6 @@ static ssize_t i915_param_charp_write(struct file *file,
>  
>  	kfree(old);
>  out:
> -	kernel_param_unlock(THIS_MODULE);
> -
>  	return len;
>  }
>  
> @@ -229,7 +224,7 @@ _i915_param_create_file(struct dentry *parent, const char *name,
>  struct dentry *i915_debugfs_params(struct drm_i915_private *i915)
>  {
>  	struct drm_minor *minor = i915->drm.primary;
> -	struct i915_params *params = &i915_modparams;
> +	struct i915_params *params = &i915->params;
>  	struct dentry *dir;
>  
>  	dir = debugfs_create_dir("i915_params", minor->debugfs_root);
> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> index 34ee12f3f02d..67102dc26fce 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -501,6 +501,8 @@ static void i915_driver_late_release(struct drm_i915_private *dev_priv)
>  
>  	cpu_latency_qos_remove_request(&dev_priv->sb_qos);
>  	mutex_destroy(&dev_priv->sb_lock);
> +
> +	i915_params_free(&dev_priv->params);
>  }
>  
>  /**
> @@ -915,6 +917,9 @@ i915_driver_create(struct pci_dev *pdev, const struct pci_device_id *ent)
>  	i915->drm.pdev = pdev;
>  	pci_set_drvdata(pdev, i915);
>  
> +	/* Device parameters start as a copy of module parameters. */
> +	i915_params_copy(&i915->params, &i915_modparams);
> +
>  	/* Setup the write-once "constant" device info */
>  	device_info = mkwrite_device_info(i915);
>  	memcpy(device_info, match_info, sizeof(*device_info));
> @@ -948,7 +953,7 @@ int i915_driver_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>  		return PTR_ERR(i915);
>  
>  	/* Disable nuclear pageflip by default on pre-ILK */
> -	if (!i915_modparams.nuclear_pageflip && match_info->gen < 5)
> +	if (!i915->params.nuclear_pageflip && match_info->gen < 5)
>  		i915->drm.driver_features &= ~DRIVER_ATOMIC;
>  
>  	/*
> @@ -958,7 +963,7 @@ int i915_driver_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>  #if IS_ENABLED(CONFIG_DRM_I915_SELFTEST)
>  	if (IS_ENABLED(CONFIG_DRM_I915_UNSTABLE_FAKE_LMEM)) {
>  		if (INTEL_GEN(i915) >= 9 && i915_selftest.live < 0 &&
> -		    i915_modparams.fake_lmem_start) {
> +		    i915->params.fake_lmem_start) {
>  			mkwrite_device_info(i915)->memory_regions =
>  				REGION_SMEM | REGION_LMEM | REGION_STOLEN;
>  			mkwrite_device_info(i915)->is_dgfx = true;
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index a378d52341e6..32e899c52ed9 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -829,6 +829,9 @@ struct drm_i915_private {
>  	/* FIXME: Device release actions should all be moved to drmm_ */
>  	bool do_release;
>  
> +	/* i915 device parameters */
> +	struct i915_params params;
> +
>  	const struct intel_device_info __info; /* Use INTEL_INFO() to access. */
>  	struct intel_runtime_info __runtime; /* Use RUNTIME_INFO() to access. */
>  	struct intel_driver_caps caps;
> @@ -1658,7 +1661,7 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915,
>  #define HAS_DISPLAY(dev_priv) (INTEL_INFO(dev_priv)->pipe_mask != 0)
>  
>  /* Only valid when HAS_DISPLAY() is true */
> -#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !i915_modparams.disable_display)
> +#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !dev_priv->params.disable_display)
>  
>  static inline bool intel_vtd_active(void)
>  {
> diff --git a/drivers/gpu/drm/i915/i915_getparam.c b/drivers/gpu/drm/i915/i915_getparam.c
> index 54fce81d5724..c4a1fac9eb93 100644
> --- a/drivers/gpu/drm/i915/i915_getparam.c
> +++ b/drivers/gpu/drm/i915/i915_getparam.c
> @@ -80,7 +80,7 @@ int i915_getparam_ioctl(struct drm_device *dev, void *data,
>  			return -ENODEV;
>  		break;
>  	case I915_PARAM_HAS_GPU_RESET:
> -		value = i915_modparams.enable_hangcheck &&
> +		value = i915->params.enable_hangcheck &&
>  			intel_has_gpu_reset(&i915->gt);
>  		if (value && intel_has_reset_engine(&i915->gt))
>  			value = 2;
> diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c
> index eec292d06f11..866166ada10e 100644
> --- a/drivers/gpu/drm/i915/i915_gpu_error.c
> +++ b/drivers/gpu/drm/i915/i915_gpu_error.c
> @@ -1698,7 +1698,7 @@ static void capture_gen(struct i915_gpu_coredump *error)
>  	error->reset_count = i915_reset_count(&i915->gpu_error);
>  	error->suspend_count = i915->suspend_count;
>  
> -	i915_params_copy(&error->params, &i915_modparams);
> +	i915_params_copy(&error->params, &i915->params);
>  	memcpy(&error->device_info,
>  	       INTEL_INFO(i915),
>  	       sizeof(error->device_info));
> @@ -1713,7 +1713,7 @@ i915_gpu_coredump_alloc(struct drm_i915_private *i915, gfp_t gfp)
>  {
>  	struct i915_gpu_coredump *error;
>  
> -	if (!i915_modparams.error_capture)
> +	if (!i915->params.error_capture)
>  		return NULL;
>  
>  	error = kzalloc(sizeof(*error), gfp);
> diff --git a/drivers/gpu/drm/i915/intel_gvt.c b/drivers/gpu/drm/i915/intel_gvt.c
> index 21b91313cc5d..d9bb9a759740 100644
> --- a/drivers/gpu/drm/i915/intel_gvt.c
> +++ b/drivers/gpu/drm/i915/intel_gvt.c
> @@ -64,7 +64,7 @@ static bool is_supported_device(struct drm_i915_private *dev_priv)
>   */
>  void intel_gvt_sanitize_options(struct drm_i915_private *dev_priv)
>  {
> -	if (!i915_modparams.enable_gvt)
> +	if (!dev_priv->params.enable_gvt)
>  		return;
>  
>  	if (intel_vgpu_active(dev_priv)) {
> @@ -80,7 +80,7 @@ void intel_gvt_sanitize_options(struct drm_i915_private *dev_priv)
>  
>  	return;
>  bail:
> -	i915_modparams.enable_gvt = 0;
> +	dev_priv->params.enable_gvt = 0;
>  }
>  
>  /**
> @@ -100,7 +100,7 @@ int intel_gvt_init(struct drm_i915_private *dev_priv)
>  	if (i915_inject_probe_failure(dev_priv))
>  		return -ENODEV;
>  
> -	if (!i915_modparams.enable_gvt) {
> +	if (!dev_priv->params.enable_gvt) {
>  		drm_dbg(&dev_priv->drm,
>  			"GVT-g is disabled by kernel params\n");
>  		return 0;
> @@ -121,7 +121,7 @@ int intel_gvt_init(struct drm_i915_private *dev_priv)
>  	return 0;
>  
>  bail:
> -	i915_modparams.enable_gvt = 0;
> +	dev_priv->params.enable_gvt = 0;
>  	return 0;
>  }
>  
> diff --git a/drivers/gpu/drm/i915/intel_region_lmem.c b/drivers/gpu/drm/i915/intel_region_lmem.c
> index 14b59b899c9b..40d8f1a95df6 100644
> --- a/drivers/gpu/drm/i915/intel_region_lmem.c
> +++ b/drivers/gpu/drm/i915/intel_region_lmem.c
> @@ -76,7 +76,7 @@ region_lmem_init(struct intel_memory_region *mem)
>  {
>  	int ret;
>  
> -	if (i915_modparams.fake_lmem_start) {
> +	if (mem->i915->params.fake_lmem_start) {
>  		ret = init_fake_lmem_bar(mem);
>  		GEM_BUG_ON(ret);
>  	}
> @@ -111,12 +111,12 @@ intel_setup_fake_lmem(struct drm_i915_private *i915)
>  	resource_size_t start;
>  
>  	GEM_BUG_ON(i915_ggtt_has_aperture(&i915->ggtt));
> -	GEM_BUG_ON(!i915_modparams.fake_lmem_start);
> +	GEM_BUG_ON(!i915->params.fake_lmem_start);
>  
>  	/* Your mappable aperture belongs to me now! */
>  	mappable_end = pci_resource_len(pdev, 2);
>  	io_start = pci_resource_start(pdev, 2),
> -	start = i915_modparams.fake_lmem_start;
> +	start = i915->params.fake_lmem_start;
>  
>  	mem = intel_memory_region_create(i915,
>  					 start,
> diff --git a/drivers/gpu/drm/i915/intel_uncore.c b/drivers/gpu/drm/i915/intel_uncore.c
> index a61cb8ca4d50..fb08b24e92c8 100644
> --- a/drivers/gpu/drm/i915/intel_uncore.c
> +++ b/drivers/gpu/drm/i915/intel_uncore.c
> @@ -1185,7 +1185,7 @@ __unclaimed_reg_debug(struct intel_uncore *uncore,
>  		     read ? "read from" : "write to",
>  		     i915_mmio_reg_offset(reg)))
>  		/* Only report the first N failures */
> -		i915_modparams.mmio_debug--;
> +		uncore->i915->params.mmio_debug--;
>  }
>  
>  static inline void
> @@ -1194,7 +1194,7 @@ unclaimed_reg_debug(struct intel_uncore *uncore,
>  		    const bool read,
>  		    const bool before)
>  {
> -	if (likely(!i915_modparams.mmio_debug))
> +	if (likely(!uncore->i915->params.mmio_debug))
>  		return;
>  
>  	/* interrupts are disabled and re-enabled around uncore->lock usage */
> @@ -2093,12 +2093,12 @@ intel_uncore_arm_unclaimed_mmio_detection(struct intel_uncore *uncore)
>  		goto out;
>  
>  	if (unlikely(check_for_unclaimed_mmio(uncore))) {
> -		if (!i915_modparams.mmio_debug) {
> +		if (!uncore->i915->params.mmio_debug) {
>  			drm_dbg(&uncore->i915->drm,
>  				"Unclaimed register detected, "
>  				"enabling oneshot unclaimed register reporting. "
>  				"Please use i915.mmio_debug=N for more information.\n");
> -			i915_modparams.mmio_debug++;
> +			uncore->i915->params.mmio_debug++;
>  		}
>  		uncore->debug->unclaimed_mmio_check--;
>  		ret = true;
> -- 
> 2.20.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH 1/4] drm/i915/params: don't expose inject_probe_failure in debugfs

[Juha-Pekka Heikkila]

Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>

On 14.5.2020 18.40, Jani Nikula wrote:
> The parameter only makes sense as a module parameter only.
> 
> Fixes: c43c5a8818d4 ("drm/i915/params: add i915 parameters to debugfs")
> Cc: Juha-Pekka Heikkilä <juha-pekka.heikkila@intel.com>
> Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> ---
>   drivers/gpu/drm/i915/i915_params.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_params.h b/drivers/gpu/drm/i915/i915_params.h
> index 45323732f099..4f21bfffbf0e 100644
> --- a/drivers/gpu/drm/i915/i915_params.h
> +++ b/drivers/gpu/drm/i915/i915_params.h
> @@ -64,7 +64,7 @@ struct drm_printer;
>   	param(int, mmio_debug, -IS_ENABLED(CONFIG_DRM_I915_DEBUG_MMIO), 0600) \
>   	param(int, edp_vswing, 0, 0400) \
>   	param(unsigned int, reset, 3, 0600) \
> -	param(unsigned int, inject_probe_failure, 0, 0600) \
> +	param(unsigned int, inject_probe_failure, 0, 0) \
>   	param(int, fastboot, -1, 0600) \
>   	param(int, enable_dpcd_backlight, -1, 0600) \
>   	param(char *, force_probe, CONFIG_DRM_I915_FORCE_PROBE, 0400) \
> 

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs

[Patchwork]

== Series Details ==

Series: series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs
URL   : https://patchwork.freedesktop.org/series/77272/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
e80307f07467 drm/i915/params: don't expose inject_probe_failure in debugfs
5bfd76f34ce0 drm/i915/params: fix i915.fake_lmem_start module param sysfs permissions
-:27: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#27: FILE: drivers/gpu/drm/i915/i915_params.c:177:
+i915_param_named_unsafe(fake_lmem_start, ulong, 0400,
 	"Fake LMEM start offset (default: 0)");

total: 0 errors, 0 warnings, 1 checks, 8 lines checked
ed04031bb3e1 drm/i915/params: prevent changing module params runtime
-:48: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#48: FILE: drivers/gpu/drm/i915/i915_params.c:62:
+i915_param_named_unsafe(enable_fbc, int, 0400,
 	"Enable frame buffer compression for power savings "

-:57: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#57: FILE: drivers/gpu/drm/i915/i915_params.c:70:
+i915_param_named_unsafe(panel_use_ssc, int, 0400,
 	"Use Spread Spectrum Clock with panels [LVDS/eDP] "

-:66: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#66: FILE: drivers/gpu/drm/i915/i915_params.c:78:
+i915_param_named_unsafe(reset, int, 0400,
 	"Attempt GPU resets (0=disabled, 1=full gpu reset, 2=engine reset [default])");

-:74: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#74: FILE: drivers/gpu/drm/i915/i915_params.c:85:
+i915_param_named(error_capture, bool, 0400,
 	"Record the GPU state following a hang. "

-:81: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#81: FILE: drivers/gpu/drm/i915/i915_params.c:91:
+i915_param_named_unsafe(enable_hangcheck, bool, 0400,
 	"Periodically check GPU activity for detecting hangs. "

-:87: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#87: FILE: drivers/gpu/drm/i915/i915_params.c:96:
+i915_param_named_unsafe(enable_psr, int, 0400,
 	"Enable PSR "

-:99: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#99: FILE: drivers/gpu/drm/i915/i915_params.c:111:
+i915_param_named(fastboot, int, 0400,
 	"Try to skip unnecessary mode sets at boot time "

-:105: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#105: FILE: drivers/gpu/drm/i915/i915_params.c:116:
+i915_param_named_unsafe(load_detect_test, bool, 0400,
 	"Force-enable the VGA load detect code for testing (default:false). "

-:110: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#110: FILE: drivers/gpu/drm/i915/i915_params.c:120:
+i915_param_named_unsafe(force_reset_modeset_test, bool, 0400,
 	"Force a modeset during gpu reset for testing (default:false). "

-:115: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#115: FILE: drivers/gpu/drm/i915/i915_params.c:124:
+i915_param_named_unsafe(invert_brightness, int, 0400,
 	"Invert backlight brightness "

-:124: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#124: FILE: drivers/gpu/drm/i915/i915_params.c:134:
+i915_param_named(mmio_debug, int, 0400,
 	"Enable the MMIO debug code for the first N failures (default: off). "

-:137: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#137: FILE: drivers/gpu/drm/i915/i915_params.c:169:
+i915_param_named_unsafe(enable_dp_mst, bool, 0400,
 	"Enable multi-stream transport (MST) for new DisplayPort sinks. (default: true)");

-:146: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#146: FILE: drivers/gpu/drm/i915/i915_params.c:177:
+i915_param_named(enable_dpcd_backlight, int, 0400,
 	"Enable support for DPCD backlight control"

total: 0 errors, 0 warnings, 13 checks, 115 lines checked
581762905710 drm/i915/params: switch to device specific parameters
-:755: WARNING:LONG_LINE: line over 100 characters
#755: FILE: drivers/gpu/drm/i915/i915_drv.h:1664:
+#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !dev_priv->params.disable_display)

-:755: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dev_priv' - possible side-effects?
#755: FILE: drivers/gpu/drm/i915/i915_drv.h:1664:
+#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !dev_priv->params.disable_display)

total: 0 errors, 1 warnings, 1 checks, 680 lines checked

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs

[Patchwork]

== Series Details ==

Series: series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs
URL   : https://patchwork.freedesktop.org/series/77272/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_8488 -> Patchwork_17663
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with Patchwork_17663 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_17663, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/index.html

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_17663:

### CI changes ###

#### Possible regressions ####

  * boot:
    - fi-kbl-8809g:       [PASS][1] -> [FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-kbl-8809g/boot.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-kbl-8809g/boot.html
    - fi-cfl-8109u:       [PASS][3] -> [FAIL][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-cfl-8109u/boot.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-cfl-8109u/boot.html
    - fi-skl-6600u:       [PASS][5] -> [FAIL][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-skl-6600u/boot.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-skl-6600u/boot.html
    - fi-cfl-8700k:       [PASS][7] -> [FAIL][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-cfl-8700k/boot.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-cfl-8700k/boot.html
    - fi-cml-u2:          [PASS][9] -> [FAIL][10]
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-cml-u2/boot.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-cml-u2/boot.html
    - fi-skl-6700k2:      [PASS][11] -> [FAIL][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-skl-6700k2/boot.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-skl-6700k2/boot.html
    - fi-cfl-guc:         [PASS][13] -> [FAIL][14]
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-cfl-guc/boot.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-cfl-guc/boot.html
    - fi-kbl-soraka:      [PASS][15] -> [FAIL][16]
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-kbl-soraka/boot.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-kbl-soraka/boot.html
    - fi-cml-s:           [PASS][17] -> [FAIL][18]
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-cml-s/boot.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-cml-s/boot.html
    - fi-skl-lmem:        [PASS][19] -> [FAIL][20]
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-skl-lmem/boot.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-skl-lmem/boot.html
    - fi-kbl-guc:         [PASS][21] -> [FAIL][22]
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-kbl-guc/boot.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-kbl-guc/boot.html
    - fi-kbl-x1275:       [PASS][23] -> [FAIL][24]
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-kbl-x1275/boot.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-kbl-x1275/boot.html
    - fi-kbl-7500u:       [PASS][25] -> [FAIL][26]
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-kbl-7500u/boot.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-kbl-7500u/boot.html
    - fi-skl-guc:         [PASS][27] -> [FAIL][28]
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-skl-guc/boot.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-skl-guc/boot.html
    - fi-kbl-r:           [PASS][29] -> [FAIL][30]
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-kbl-r/boot.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-kbl-r/boot.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * boot:
    - {fi-kbl-7560u}:     NOTRUN -> [FAIL][31]
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-kbl-7560u/boot.html
    - {fi-ehl-1}:         [PASS][32] -> [FAIL][33]
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-ehl-1/boot.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ehl-1/boot.html

  

### IGT changes ###

#### Possible regressions ####

  * igt@gem_close_race@basic-process:
    - fi-ivb-3770:        [PASS][34] -> [FAIL][35] +11 similar issues
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-ivb-3770/igt@gem_close_race@basic-process.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ivb-3770/igt@gem_close_race@basic-process.html

  * igt@gem_close_race@basic-threads:
    - fi-bwr-2160:        [PASS][36] -> [FAIL][37] +8 similar issues
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bwr-2160/igt@gem_close_race@basic-threads.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bwr-2160/igt@gem_close_race@basic-threads.html

  * igt@gem_ctx_create@basic:
    - fi-elk-e7500:       [PASS][38] -> [FAIL][39] +11 similar issues
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-elk-e7500/igt@gem_ctx_create@basic.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-elk-e7500/igt@gem_ctx_create@basic.html
    - fi-bsw-n3050:       [PASS][40] -> [FAIL][41] +9 similar issues
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-n3050/igt@gem_ctx_create@basic.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-n3050/igt@gem_ctx_create@basic.html

  * igt@gem_exec_basic@basic:
    - fi-snb-2520m:       NOTRUN -> [FAIL][42]
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2520m/igt@gem_exec_basic@basic.html
    - fi-pnv-d510:        NOTRUN -> [FAIL][43] +4 similar issues
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-pnv-d510/igt@gem_exec_basic@basic.html
    - fi-snb-2600:        NOTRUN -> [FAIL][44]
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2600/igt@gem_exec_basic@basic.html
    - fi-bwr-2160:        NOTRUN -> [FAIL][45]
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bwr-2160/igt@gem_exec_basic@basic.html
    - fi-gdg-551:         NOTRUN -> [FAIL][46]
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-gdg-551/igt@gem_exec_basic@basic.html

  * igt@gem_exec_create@basic:
    - fi-bsw-kefka:       [PASS][47] -> [FAIL][48] +9 similar issues
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-kefka/igt@gem_exec_create@basic.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-kefka/igt@gem_exec_create@basic.html
    - fi-gdg-551:         [PASS][49] -> [FAIL][50] +6 similar issues
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-gdg-551/igt@gem_exec_create@basic.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-gdg-551/igt@gem_exec_create@basic.html

  * igt@gem_exec_fence@basic-await:
    - fi-bsw-nick:        NOTRUN -> [FAIL][51] +4 similar issues
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-nick/igt@gem_exec_fence@basic-await.html
    - fi-byt-j1900:       NOTRUN -> [FAIL][52] +4 similar issues
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-j1900/igt@gem_exec_fence@basic-await.html
    - fi-elk-e7500:       NOTRUN -> [FAIL][53] +4 similar issues
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-elk-e7500/igt@gem_exec_fence@basic-await.html
    - fi-blb-e6850:       NOTRUN -> [FAIL][54] +4 similar issues
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-blb-e6850/igt@gem_exec_fence@basic-await.html

  * igt@gem_exec_fence@basic-busy:
    - fi-bsw-kefka:       NOTRUN -> [FAIL][55] +4 similar issues
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-kefka/igt@gem_exec_fence@basic-busy.html
    - fi-bdw-5557u:       NOTRUN -> [FAIL][56] +4 similar issues
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bdw-5557u/igt@gem_exec_fence@basic-busy.html

  * igt@gem_exec_fence@basic-wait:
    - fi-ivb-3770:        NOTRUN -> [FAIL][57] +4 similar issues
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ivb-3770/igt@gem_exec_fence@basic-wait.html
    - fi-ilk-650:         NOTRUN -> [FAIL][58] +4 similar issues
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ilk-650/igt@gem_exec_fence@basic-wait.html
    - fi-bsw-n3050:       NOTRUN -> [FAIL][59] +4 similar issues
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-n3050/igt@gem_exec_fence@basic-wait.html

  * igt@gem_exec_fence@nb-await:
    - fi-byt-n2820:       NOTRUN -> [FAIL][60] +4 similar issues
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-n2820/igt@gem_exec_fence@nb-await.html

  * igt@gem_exec_gttfill@basic:
    - fi-snb-2600:        [PASS][61] -> [FAIL][62] +11 similar issues
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-snb-2600/igt@gem_exec_gttfill@basic.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2600/igt@gem_exec_gttfill@basic.html

  * igt@gem_exec_suspend@basic-s0:
    - fi-bsw-nick:        [PASS][63] -> [FAIL][64] +9 similar issues
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-nick/igt@gem_exec_suspend@basic-s0.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-nick/igt@gem_exec_suspend@basic-s0.html

  * igt@gem_exec_suspend@basic-s3:
    - fi-bdw-5557u:       [PASS][65] -> [FAIL][66] +10 similar issues
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html

  * igt@gem_sync@basic-all:
    - fi-byt-n2820:       [PASS][67] -> [FAIL][68] +10 similar issues
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-byt-n2820/igt@gem_sync@basic-all.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-n2820/igt@gem_sync@basic-all.html
    - fi-ilk-650:         [PASS][69] -> [FAIL][70] +11 similar issues
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-ilk-650/igt@gem_sync@basic-all.html
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ilk-650/igt@gem_sync@basic-all.html

  * igt@gem_sync@basic-each:
    - fi-pnv-d510:        [PASS][71] -> [FAIL][72] +9 similar issues
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-pnv-d510/igt@gem_sync@basic-each.html
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-pnv-d510/igt@gem_sync@basic-each.html

  * igt@i915_module_load@reload:
    - fi-blb-e6850:       [PASS][73] -> [FAIL][74] +9 similar issues
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-blb-e6850/igt@i915_module_load@reload.html
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-blb-e6850/igt@i915_module_load@reload.html
    - fi-byt-j1900:       [PASS][75] -> [FAIL][76] +11 similar issues
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-byt-j1900/igt@i915_module_load@reload.html
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-j1900/igt@i915_module_load@reload.html
    - fi-snb-2520m:       [PASS][77] -> [FAIL][78] +11 similar issues
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-snb-2520m/igt@i915_module_load@reload.html
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2520m/igt@i915_module_load@reload.html

  
#### Warnings ####

  * igt@gem_exec_gttfill@basic:
    - fi-bsw-kefka:       [SKIP][79] ([fdo#109271]) -> [FAIL][80]
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-kefka/igt@gem_exec_gttfill@basic.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-kefka/igt@gem_exec_gttfill@basic.html
    - fi-gdg-551:         [SKIP][81] ([fdo#109271]) -> [FAIL][82]
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-gdg-551/igt@gem_exec_gttfill@basic.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-gdg-551/igt@gem_exec_gttfill@basic.html
    - fi-bsw-nick:        [SKIP][83] ([fdo#109271]) -> [FAIL][84]
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-nick/igt@gem_exec_gttfill@basic.html
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-nick/igt@gem_exec_gttfill@basic.html
    - fi-bwr-2160:        [SKIP][85] ([fdo#109271]) -> [FAIL][86]
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bwr-2160/igt@gem_exec_gttfill@basic.html
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bwr-2160/igt@gem_exec_gttfill@basic.html
    - fi-bsw-n3050:       [SKIP][87] ([fdo#109271]) -> [FAIL][88]
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-n3050/igt@gem_exec_gttfill@basic.html
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-n3050/igt@gem_exec_gttfill@basic.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * {igt@gem_busy@busy}:
    - fi-snb-2520m:       NOTRUN -> [FAIL][89] +3 similar issues
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2520m/igt@gem_busy@busy.html
    - fi-bdw-5557u:       NOTRUN -> [FAIL][90] +3 similar issues
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bdw-5557u/igt@gem_busy@busy.html
    - fi-pnv-d510:        NOTRUN -> [FAIL][91] +3 similar issues
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-pnv-d510/igt@gem_busy@busy.html

  * {igt@gem_exec_parallel@engines}:
    - fi-bsw-kefka:       NOTRUN -> [FAIL][92] +3 similar issues
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-kefka/igt@gem_exec_parallel@engines.html
    - fi-ilk-650:         NOTRUN -> [FAIL][93] +3 similar issues
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ilk-650/igt@gem_exec_parallel@engines.html
    - fi-bsw-n3050:       NOTRUN -> [FAIL][94] +3 similar issues
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-n3050/igt@gem_exec_parallel@engines.html
    - fi-byt-n2820:       NOTRUN -> [FAIL][95] +3 similar issues
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-n2820/igt@gem_exec_parallel@engines.html
    - fi-elk-e7500:       NOTRUN -> [FAIL][96] +3 similar issues
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-elk-e7500/igt@gem_exec_parallel@engines.html
    - fi-ivb-3770:        NOTRUN -> [FAIL][97] +3 similar issues
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ivb-3770/igt@gem_exec_parallel@engines.html
    - fi-byt-j1900:       NOTRUN -> [FAIL][98] +3 similar issues
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-j1900/igt@gem_exec_parallel@engines.html

  * {igt@gem_exec_store@basic}:
    - fi-snb-2600:        [PASS][99] -> [FAIL][100]
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-snb-2600/igt@gem_exec_store@basic.html
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2600/igt@gem_exec_store@basic.html
    - fi-bsw-kefka:       [PASS][101] -> [FAIL][102]
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-kefka/igt@gem_exec_store@basic.html
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-kefka/igt@gem_exec_store@basic.html
    - fi-bsw-n3050:       [PASS][103] -> [FAIL][104]
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-n3050/igt@gem_exec_store@basic.html
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-n3050/igt@gem_exec_store@basic.html
    - fi-ilk-650:         [PASS][105] -> [FAIL][106]
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-ilk-650/igt@gem_exec_store@basic.html
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ilk-650/igt@gem_exec_store@basic.html
    - fi-elk-e7500:       [PASS][107] -> [FAIL][108]
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-elk-e7500/igt@gem_exec_store@basic.html
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-elk-e7500/igt@gem_exec_store@basic.html
    - fi-byt-n2820:       [PASS][109] -> [FAIL][110]
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-byt-n2820/igt@gem_exec_store@basic.html
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-n2820/igt@gem_exec_store@basic.html
    - fi-ivb-3770:        [PASS][111] -> [FAIL][112]
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-ivb-3770/igt@gem_exec_store@basic.html
   [112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ivb-3770/igt@gem_exec_store@basic.html
    - fi-snb-2520m:       [PASS][113] -> [FAIL][114]
   [113]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-snb-2520m/igt@gem_exec_store@basic.html
   [114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2520m/igt@gem_exec_store@basic.html
    - fi-byt-j1900:       [PASS][115] -> [FAIL][116]
   [115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-byt-j1900/igt@gem_exec_store@basic.html
   [116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-j1900/igt@gem_exec_store@basic.html
    - fi-pnv-d510:        [PASS][117] -> [FAIL][118]
   [117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-pnv-d510/igt@gem_exec_store@basic.html
   [118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-pnv-d510/igt@gem_exec_store@basic.html
    - fi-bdw-5557u:       [PASS][119] -> [FAIL][120]
   [119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bdw-5557u/igt@gem_exec_store@basic.html
   [120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bdw-5557u/igt@gem_exec_store@basic.html
    - fi-blb-e6850:       [PASS][121] -> [FAIL][122]
   [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-blb-e6850/igt@gem_exec_store@basic.html
   [122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-blb-e6850/igt@gem_exec_store@basic.html
    - fi-bwr-2160:        [SKIP][123] ([fdo#109271]) -> [FAIL][124]
   [123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bwr-2160/igt@gem_exec_store@basic.html
   [124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bwr-2160/igt@gem_exec_store@basic.html
    - fi-gdg-551:         [SKIP][125] ([fdo#109271]) -> [FAIL][126]
   [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-gdg-551/igt@gem_exec_store@basic.html
   [126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-gdg-551/igt@gem_exec_store@basic.html
    - fi-bsw-nick:        [PASS][127] -> [FAIL][128]
   [127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-nick/igt@gem_exec_store@basic.html
   [128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-nick/igt@gem_exec_store@basic.html

  * {igt@gem_wait@wait}:
    - fi-gdg-551:         NOTRUN -> [FAIL][129] +3 similar issues
   [129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-gdg-551/igt@gem_wait@wait.html
    - fi-bsw-nick:        NOTRUN -> [FAIL][130] +3 similar issues
   [130]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-nick/igt@gem_wait@wait.html
    - fi-bwr-2160:        NOTRUN -> [FAIL][131] +3 similar issues
   [131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bwr-2160/igt@gem_wait@wait.html
    - fi-blb-e6850:       NOTRUN -> [FAIL][132] +3 similar issues
   [132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-blb-e6850/igt@gem_wait@wait.html
    - fi-snb-2600:        NOTRUN -> [FAIL][133] +3 similar issues
   [133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2600/igt@gem_wait@wait.html

  
Known issues
------------

  Here are the changes found in Patchwork_17663 that come from known issues:

### CI changes ###

#### Issues hit ####

  * boot:
    - fi-apl-guc:         [PASS][134] -> [FAIL][135] ([i915#348])
   [134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-apl-guc/boot.html
   [135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-apl-guc/boot.html
    - fi-icl-y:           [PASS][136] -> [FAIL][137] ([CI#80])
   [136]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-icl-y/boot.html
   [137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-icl-y/boot.html
    - fi-icl-u2:          [PASS][138] -> [FAIL][139] ([CI#80])
   [138]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-icl-u2/boot.html
   [139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-icl-u2/boot.html
    - fi-bxt-dsi:         [PASS][140] -> [FAIL][141] ([i915#1529])
   [140]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bxt-dsi/boot.html
   [141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bxt-dsi/boot.html
    - fi-icl-dsi:         [PASS][142] -> [FAIL][143] ([CI#80])
   [142]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-icl-dsi/boot.html
   [143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-icl-dsi/boot.html
    - fi-whl-u:           [PASS][144] -> [FAIL][145] ([CI#80])
   [144]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-whl-u/boot.html
   [145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-whl-u/boot.html
    - fi-icl-guc:         [PASS][146] -> [FAIL][147] ([CI#80])
   [146]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-icl-guc/boot.html
   [147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-icl-guc/boot.html
    - fi-glk-dsi:         [PASS][148] -> [FAIL][149] ([CI#80])
   [148]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-glk-dsi/boot.html
   [149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-glk-dsi/boot.html
    - fi-tgl-y:           [PASS][150] -> [FAIL][151] ([CI#80])
   [150]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-tgl-y/boot.html
   [151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-tgl-y/boot.html

  

### IGT changes ###

#### Issues hit ####

  * igt@i915_hangman@error-state-basic:
    - fi-byt-n2820:       [PASS][152] -> [SKIP][153] ([fdo#109271]) +1 similar issue
   [152]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-byt-n2820/igt@i915_hangman@error-state-basic.html
   [153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-n2820/igt@i915_hangman@error-state-basic.html
    - fi-bwr-2160:        [PASS][154] -> [SKIP][155] ([fdo#109271]) +1 similar issue
   [154]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bwr-2160/igt@i915_hangman@error-state-basic.html
   [155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bwr-2160/igt@i915_hangman@error-state-basic.html
    - fi-bdw-5557u:       [PASS][156] -> [SKIP][157] ([fdo#109271]) +1 similar issue
   [156]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bdw-5557u/igt@i915_hangman@error-state-basic.html
   [157]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bdw-5557u/igt@i915_hangman@error-state-basic.html
    - fi-snb-2520m:       [PASS][158] -> [SKIP][159] ([fdo#109271]) +1 similar issue
   [158]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-snb-2520m/igt@i915_hangman@error-state-basic.html
   [159]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2520m/igt@i915_hangman@error-state-basic.html
    - fi-bsw-nick:        [PASS][160] -> [SKIP][161] ([fdo#109271])
   [160]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-nick/igt@i915_hangman@error-state-basic.html
   [161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-nick/igt@i915_hangman@error-state-basic.html
    - fi-gdg-551:         [PASS][162] -> [SKIP][163] ([fdo#109271]) +1 similar issue
   [162]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-gdg-551/igt@i915_hangman@error-state-basic.html
   [163]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-gdg-551/igt@i915_hangman@error-state-basic.html
    - fi-bsw-kefka:       [PASS][164] -> [SKIP][165] ([fdo#109271]) +1 similar issue
   [164]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-kefka/igt@i915_hangman@error-state-basic.html
   [165]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-kefka/igt@i915_hangman@error-state-basic.html
    - fi-blb-e6850:       [PASS][166] -> [SKIP][167] ([fdo#109271]) +1 similar issue
   [166]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-blb-e6850/igt@i915_hangman@error-state-basic.html
   [167]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-blb-e6850/igt@i915_hangman@error-state-basic.html
    - fi-bsw-n3050:       [PASS][168] -> [SKIP][169] ([fdo#109271])
   [168]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-bsw-n3050/igt@i915_hangman@error-state-basic.html
   [169]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-bsw-n3050/igt@i915_hangman@error-state-basic.html
    - fi-snb-2600:        [PASS][170] -> [SKIP][171] ([fdo#109271]) +1 similar issue
   [170]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-snb-2600/igt@i915_hangman@error-state-basic.html
   [171]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-snb-2600/igt@i915_hangman@error-state-basic.html

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
    - fi-ilk-650:         [PASS][172] -> [SKIP][173] ([fdo#109271]) +1 similar issue
   [172]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-ilk-650/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [173]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ilk-650/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-ivb-3770:        [PASS][174] -> [SKIP][175] ([fdo#109271]) +1 similar issue
   [174]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-ivb-3770/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [175]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-ivb-3770/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-byt-j1900:       [PASS][176] -> [SKIP][177] ([fdo#109271]) +1 similar issue
   [176]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-byt-j1900/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [177]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-byt-j1900/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-elk-e7500:       [PASS][178] -> [SKIP][179] ([fdo#109271]) +1 similar issue
   [178]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-elk-e7500/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [179]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-elk-e7500/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-pnv-d510:        [PASS][180] -> [SKIP][181] ([fdo#109271]) +1 similar issue
   [180]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8488/fi-pnv-d510/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [181]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/fi-pnv-d510/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [CI#80]: https://gitlab.freedesktop.org/gfx-ci/i915-infra/issues/80
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [i915#1529]: https://gitlab.freedesktop.org/drm/intel/issues/1529
  [i915#348]: https://gitlab.freedesktop.org/drm/intel/issues/348


Participating hosts (51 -> 45)
------------------------------

  Additional (1): fi-kbl-7560u 
  Missing    (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * Linux: CI_DRM_8488 -> Patchwork_17663

  CI-20190529: 20190529
  CI_DRM_8488: d40ec60813532e485e9c63623c91babf556cfbe3 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5655: 2cc4c1edc3065590f9917930b6d049a90c4a38fd @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_17663: 58176290571070ce48b311da1ba7c2b1e7a1eead @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

581762905710 drm/i915/params: switch to device specific parameters
ed04031bb3e1 drm/i915/params: prevent changing module params runtime
5bfd76f34ce0 drm/i915/params: fix i915.fake_lmem_start module param sysfs permissions
e80307f07467 drm/i915/params: don't expose inject_probe_failure in debugfs

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17663/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs (rev2)

[Patchwork]

== Series Details ==

Series: series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs (rev2)
URL   : https://patchwork.freedesktop.org/series/77272/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
0df592070475 drm/i915/params: don't expose inject_probe_failure in debugfs
f371da7198c1 drm/i915/params: fix i915.fake_lmem_start module param sysfs permissions
-:27: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#27: FILE: drivers/gpu/drm/i915/i915_params.c:177:
+i915_param_named_unsafe(fake_lmem_start, ulong, 0400,
 	"Fake LMEM start offset (default: 0)");

total: 0 errors, 0 warnings, 1 checks, 8 lines checked
0e5029451f95 drm/i915/params: prevent changing module params runtime
-:48: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#48: FILE: drivers/gpu/drm/i915/i915_params.c:62:
+i915_param_named_unsafe(enable_fbc, int, 0400,
 	"Enable frame buffer compression for power savings "

-:57: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#57: FILE: drivers/gpu/drm/i915/i915_params.c:70:
+i915_param_named_unsafe(panel_use_ssc, int, 0400,
 	"Use Spread Spectrum Clock with panels [LVDS/eDP] "

-:66: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#66: FILE: drivers/gpu/drm/i915/i915_params.c:78:
+i915_param_named_unsafe(reset, int, 0400,
 	"Attempt GPU resets (0=disabled, 1=full gpu reset, 2=engine reset [default])");

-:74: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#74: FILE: drivers/gpu/drm/i915/i915_params.c:85:
+i915_param_named(error_capture, bool, 0400,
 	"Record the GPU state following a hang. "

-:81: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#81: FILE: drivers/gpu/drm/i915/i915_params.c:91:
+i915_param_named_unsafe(enable_hangcheck, bool, 0400,
 	"Periodically check GPU activity for detecting hangs. "

-:87: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#87: FILE: drivers/gpu/drm/i915/i915_params.c:96:
+i915_param_named_unsafe(enable_psr, int, 0400,
 	"Enable PSR "

-:99: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#99: FILE: drivers/gpu/drm/i915/i915_params.c:111:
+i915_param_named(fastboot, int, 0400,
 	"Try to skip unnecessary mode sets at boot time "

-:105: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#105: FILE: drivers/gpu/drm/i915/i915_params.c:116:
+i915_param_named_unsafe(load_detect_test, bool, 0400,
 	"Force-enable the VGA load detect code for testing (default:false). "

-:110: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#110: FILE: drivers/gpu/drm/i915/i915_params.c:120:
+i915_param_named_unsafe(force_reset_modeset_test, bool, 0400,
 	"Force a modeset during gpu reset for testing (default:false). "

-:115: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#115: FILE: drivers/gpu/drm/i915/i915_params.c:124:
+i915_param_named_unsafe(invert_brightness, int, 0400,
 	"Invert backlight brightness "

-:124: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#124: FILE: drivers/gpu/drm/i915/i915_params.c:134:
+i915_param_named(mmio_debug, int, 0400,
 	"Enable the MMIO debug code for the first N failures (default: off). "

-:137: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#137: FILE: drivers/gpu/drm/i915/i915_params.c:169:
+i915_param_named_unsafe(enable_dp_mst, bool, 0400,
 	"Enable multi-stream transport (MST) for new DisplayPort sinks. (default: true)");

-:146: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#146: FILE: drivers/gpu/drm/i915/i915_params.c:177:
+i915_param_named(enable_dpcd_backlight, int, 0400,
 	"Enable support for DPCD backlight control"

total: 0 errors, 0 warnings, 13 checks, 115 lines checked
7918634c08d4 drm/i915/params: switch to device specific parameters
-:755: WARNING:LONG_LINE: line over 100 characters
#755: FILE: drivers/gpu/drm/i915/i915_drv.h:1671:
+#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !dev_priv->params.disable_display)

-:755: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dev_priv' - possible side-effects?
#755: FILE: drivers/gpu/drm/i915/i915_drv.h:1671:
+#define INTEL_DISPLAY_ENABLED(dev_priv) (WARN_ON(!HAS_DISPLAY(dev_priv)), !dev_priv->params.disable_display)

total: 0 errors, 1 warnings, 1 checks, 680 lines checked

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs (rev2)

[Patchwork]

== Series Details ==

Series: series starting with [1/4] drm/i915/params: don't expose inject_probe_failure in debugfs (rev2)
URL   : https://patchwork.freedesktop.org/series/77272/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_8494 -> Patchwork_17689
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with Patchwork_17689 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_17689, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/index.html

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_17689:

### CI changes ###

#### Possible regressions ####

  * boot:
    - fi-kbl-8809g:       [PASS][1] -> [FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-kbl-8809g/boot.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-kbl-8809g/boot.html
    - fi-icl-y:           [PASS][3] -> [FAIL][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-icl-y/boot.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-icl-y/boot.html
    - fi-icl-u2:          [PASS][5] -> [FAIL][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-icl-u2/boot.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-icl-u2/boot.html
    - fi-cfl-8109u:       [PASS][7] -> [FAIL][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-cfl-8109u/boot.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-cfl-8109u/boot.html
    - fi-skl-6600u:       [PASS][9] -> [FAIL][10]
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-skl-6600u/boot.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-skl-6600u/boot.html
    - fi-cfl-8700k:       [PASS][11] -> [FAIL][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-cfl-8700k/boot.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-cfl-8700k/boot.html
    - fi-icl-dsi:         [PASS][13] -> [FAIL][14]
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-icl-dsi/boot.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-icl-dsi/boot.html
    - fi-whl-u:           [PASS][15] -> [FAIL][16]
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-whl-u/boot.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-whl-u/boot.html
    - fi-cml-u2:          [PASS][17] -> [FAIL][18]
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-cml-u2/boot.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-cml-u2/boot.html
    - fi-skl-6700k2:      [PASS][19] -> [FAIL][20]
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-skl-6700k2/boot.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-skl-6700k2/boot.html
    - fi-cfl-guc:         [PASS][21] -> [FAIL][22]
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-cfl-guc/boot.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-cfl-guc/boot.html
    - fi-kbl-soraka:      [PASS][23] -> [FAIL][24]
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-kbl-soraka/boot.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-kbl-soraka/boot.html
    - fi-icl-guc:         [PASS][25] -> [FAIL][26]
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-icl-guc/boot.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-icl-guc/boot.html
    - fi-cml-s:           [PASS][27] -> [FAIL][28]
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-cml-s/boot.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-cml-s/boot.html
    - fi-skl-lmem:        [PASS][29] -> [FAIL][30]
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-skl-lmem/boot.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-skl-lmem/boot.html
    - fi-glk-dsi:         [PASS][31] -> [FAIL][32]
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-glk-dsi/boot.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-glk-dsi/boot.html
    - fi-tgl-y:           [PASS][33] -> [FAIL][34]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-tgl-y/boot.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-tgl-y/boot.html
    - fi-kbl-guc:         [PASS][35] -> [FAIL][36]
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-kbl-guc/boot.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-kbl-guc/boot.html
    - fi-kbl-x1275:       [PASS][37] -> [FAIL][38]
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-kbl-x1275/boot.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-kbl-x1275/boot.html
    - fi-kbl-7500u:       [PASS][39] -> [FAIL][40]
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-kbl-7500u/boot.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-kbl-7500u/boot.html
    - fi-skl-guc:         [PASS][41] -> [FAIL][42]
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-skl-guc/boot.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-skl-guc/boot.html
    - fi-kbl-r:           [PASS][43] -> [FAIL][44]
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-kbl-r/boot.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-kbl-r/boot.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * boot:
    - {fi-tgl-dsi}:       [PASS][45] -> [FAIL][46]
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-tgl-dsi/boot.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-tgl-dsi/boot.html
    - {fi-tgl-u}:         [PASS][47] -> [FAIL][48]
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-tgl-u/boot.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-tgl-u/boot.html
    - {fi-kbl-7560u}:     NOTRUN -> [FAIL][49]
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-kbl-7560u/boot.html
    - {fi-ehl-1}:         [PASS][50] -> [FAIL][51]
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-ehl-1/boot.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ehl-1/boot.html

  

### IGT changes ###

#### Possible regressions ####

  * igt@gem_close_race@basic-process:
    - fi-ivb-3770:        [PASS][52] -> [FAIL][53] +11 similar issues
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-ivb-3770/igt@gem_close_race@basic-process.html
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ivb-3770/igt@gem_close_race@basic-process.html

  * igt@gem_close_race@basic-threads:
    - fi-bwr-2160:        [PASS][54] -> [FAIL][55] +8 similar issues
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bwr-2160/igt@gem_close_race@basic-threads.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bwr-2160/igt@gem_close_race@basic-threads.html

  * igt@gem_ctx_create@basic:
    - fi-elk-e7500:       [PASS][56] -> [FAIL][57] +11 similar issues
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-elk-e7500/igt@gem_ctx_create@basic.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-elk-e7500/igt@gem_ctx_create@basic.html
    - fi-bsw-n3050:       [PASS][58] -> [FAIL][59] +9 similar issues
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-n3050/igt@gem_ctx_create@basic.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-n3050/igt@gem_ctx_create@basic.html

  * igt@gem_exec_basic@basic:
    - fi-snb-2520m:       NOTRUN -> [FAIL][60]
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2520m/igt@gem_exec_basic@basic.html
    - fi-pnv-d510:        NOTRUN -> [FAIL][61] +4 similar issues
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-pnv-d510/igt@gem_exec_basic@basic.html
    - fi-snb-2600:        NOTRUN -> [FAIL][62]
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2600/igt@gem_exec_basic@basic.html
    - fi-bwr-2160:        NOTRUN -> [FAIL][63]
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bwr-2160/igt@gem_exec_basic@basic.html
    - fi-gdg-551:         NOTRUN -> [FAIL][64]
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-gdg-551/igt@gem_exec_basic@basic.html

  * igt@gem_exec_create@basic:
    - fi-bsw-kefka:       [PASS][65] -> [FAIL][66] +9 similar issues
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-kefka/igt@gem_exec_create@basic.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-kefka/igt@gem_exec_create@basic.html
    - fi-gdg-551:         [PASS][67] -> [FAIL][68] +6 similar issues
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-gdg-551/igt@gem_exec_create@basic.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-gdg-551/igt@gem_exec_create@basic.html

  * igt@gem_exec_fence@basic-await:
    - fi-bsw-nick:        NOTRUN -> [FAIL][69] +4 similar issues
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-nick/igt@gem_exec_fence@basic-await.html
    - fi-byt-j1900:       NOTRUN -> [FAIL][70] +4 similar issues
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-j1900/igt@gem_exec_fence@basic-await.html
    - fi-elk-e7500:       NOTRUN -> [FAIL][71] +4 similar issues
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-elk-e7500/igt@gem_exec_fence@basic-await.html
    - fi-blb-e6850:       NOTRUN -> [FAIL][72] +4 similar issues
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-blb-e6850/igt@gem_exec_fence@basic-await.html

  * igt@gem_exec_fence@basic-busy:
    - fi-bsw-kefka:       NOTRUN -> [FAIL][73] +4 similar issues
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-kefka/igt@gem_exec_fence@basic-busy.html
    - fi-bdw-5557u:       NOTRUN -> [FAIL][74] +4 similar issues
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bdw-5557u/igt@gem_exec_fence@basic-busy.html

  * igt@gem_exec_fence@basic-wait:
    - fi-ivb-3770:        NOTRUN -> [FAIL][75] +4 similar issues
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ivb-3770/igt@gem_exec_fence@basic-wait.html
    - fi-ilk-650:         NOTRUN -> [FAIL][76] +4 similar issues
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ilk-650/igt@gem_exec_fence@basic-wait.html
    - fi-bsw-n3050:       NOTRUN -> [FAIL][77] +4 similar issues
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-n3050/igt@gem_exec_fence@basic-wait.html

  * igt@gem_exec_fence@nb-await:
    - fi-byt-n2820:       NOTRUN -> [FAIL][78] +4 similar issues
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-n2820/igt@gem_exec_fence@nb-await.html

  * igt@gem_exec_gttfill@basic:
    - fi-snb-2600:        [PASS][79] -> [FAIL][80] +11 similar issues
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-snb-2600/igt@gem_exec_gttfill@basic.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2600/igt@gem_exec_gttfill@basic.html

  * igt@gem_exec_suspend@basic-s0:
    - fi-bsw-nick:        [PASS][81] -> [FAIL][82] +9 similar issues
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-nick/igt@gem_exec_suspend@basic-s0.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-nick/igt@gem_exec_suspend@basic-s0.html

  * igt@gem_exec_suspend@basic-s3:
    - fi-bdw-5557u:       [PASS][83] -> [FAIL][84] +10 similar issues
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3.html

  * igt@gem_sync@basic-all:
    - fi-byt-n2820:       [PASS][85] -> [FAIL][86] +10 similar issues
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-byt-n2820/igt@gem_sync@basic-all.html
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-n2820/igt@gem_sync@basic-all.html
    - fi-ilk-650:         [PASS][87] -> [FAIL][88] +11 similar issues
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-ilk-650/igt@gem_sync@basic-all.html
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ilk-650/igt@gem_sync@basic-all.html

  * igt@gem_sync@basic-each:
    - fi-pnv-d510:        [PASS][89] -> [FAIL][90] +9 similar issues
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-pnv-d510/igt@gem_sync@basic-each.html
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-pnv-d510/igt@gem_sync@basic-each.html

  * igt@i915_module_load@reload:
    - fi-blb-e6850:       [PASS][91] -> [FAIL][92] +9 similar issues
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-blb-e6850/igt@i915_module_load@reload.html
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-blb-e6850/igt@i915_module_load@reload.html
    - fi-byt-j1900:       [PASS][93] -> [FAIL][94] +11 similar issues
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-byt-j1900/igt@i915_module_load@reload.html
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-j1900/igt@i915_module_load@reload.html
    - fi-snb-2520m:       [PASS][95] -> [FAIL][96] +11 similar issues
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-snb-2520m/igt@i915_module_load@reload.html
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2520m/igt@i915_module_load@reload.html

  
#### Warnings ####

  * igt@gem_exec_gttfill@basic:
    - fi-bsw-kefka:       [SKIP][97] ([fdo#109271]) -> [FAIL][98]
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-kefka/igt@gem_exec_gttfill@basic.html
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-kefka/igt@gem_exec_gttfill@basic.html
    - fi-gdg-551:         [SKIP][99] ([fdo#109271]) -> [FAIL][100]
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-gdg-551/igt@gem_exec_gttfill@basic.html
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-gdg-551/igt@gem_exec_gttfill@basic.html
    - fi-bsw-nick:        [SKIP][101] ([fdo#109271]) -> [FAIL][102]
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-nick/igt@gem_exec_gttfill@basic.html
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-nick/igt@gem_exec_gttfill@basic.html
    - fi-bwr-2160:        [SKIP][103] ([fdo#109271]) -> [FAIL][104]
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bwr-2160/igt@gem_exec_gttfill@basic.html
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bwr-2160/igt@gem_exec_gttfill@basic.html
    - fi-bsw-n3050:       [SKIP][105] ([fdo#109271]) -> [FAIL][106]
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-n3050/igt@gem_exec_gttfill@basic.html
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-n3050/igt@gem_exec_gttfill@basic.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * {igt@gem_busy@busy}:
    - fi-snb-2520m:       NOTRUN -> [FAIL][107] +3 similar issues
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2520m/igt@gem_busy@busy.html
    - fi-bdw-5557u:       NOTRUN -> [FAIL][108] +3 similar issues
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bdw-5557u/igt@gem_busy@busy.html
    - fi-pnv-d510:        NOTRUN -> [FAIL][109] +3 similar issues
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-pnv-d510/igt@gem_busy@busy.html

  * {igt@gem_exec_parallel@engines}:
    - fi-bsw-kefka:       NOTRUN -> [FAIL][110] +3 similar issues
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-kefka/igt@gem_exec_parallel@engines.html
    - fi-ilk-650:         NOTRUN -> [FAIL][111] +3 similar issues
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ilk-650/igt@gem_exec_parallel@engines.html
    - fi-bsw-n3050:       NOTRUN -> [FAIL][112] +3 similar issues
   [112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-n3050/igt@gem_exec_parallel@engines.html
    - fi-byt-n2820:       NOTRUN -> [FAIL][113] +3 similar issues
   [113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-n2820/igt@gem_exec_parallel@engines.html
    - fi-elk-e7500:       NOTRUN -> [FAIL][114] +3 similar issues
   [114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-elk-e7500/igt@gem_exec_parallel@engines.html
    - fi-ivb-3770:        NOTRUN -> [FAIL][115] +3 similar issues
   [115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ivb-3770/igt@gem_exec_parallel@engines.html
    - fi-byt-j1900:       NOTRUN -> [FAIL][116] +3 similar issues
   [116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-j1900/igt@gem_exec_parallel@engines.html

  * {igt@gem_exec_store@basic}:
    - fi-snb-2600:        [PASS][117] -> [FAIL][118]
   [117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-snb-2600/igt@gem_exec_store@basic.html
   [118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2600/igt@gem_exec_store@basic.html
    - fi-bsw-kefka:       [PASS][119] -> [FAIL][120]
   [119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-kefka/igt@gem_exec_store@basic.html
   [120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-kefka/igt@gem_exec_store@basic.html
    - fi-bsw-n3050:       [PASS][121] -> [FAIL][122]
   [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-n3050/igt@gem_exec_store@basic.html
   [122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-n3050/igt@gem_exec_store@basic.html
    - fi-ilk-650:         [PASS][123] -> [FAIL][124]
   [123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-ilk-650/igt@gem_exec_store@basic.html
   [124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ilk-650/igt@gem_exec_store@basic.html
    - fi-elk-e7500:       [PASS][125] -> [FAIL][126]
   [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-elk-e7500/igt@gem_exec_store@basic.html
   [126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-elk-e7500/igt@gem_exec_store@basic.html
    - fi-byt-n2820:       [PASS][127] -> [FAIL][128]
   [127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-byt-n2820/igt@gem_exec_store@basic.html
   [128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-n2820/igt@gem_exec_store@basic.html
    - fi-ivb-3770:        [PASS][129] -> [FAIL][130]
   [129]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-ivb-3770/igt@gem_exec_store@basic.html
   [130]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ivb-3770/igt@gem_exec_store@basic.html
    - fi-snb-2520m:       [PASS][131] -> [FAIL][132]
   [131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-snb-2520m/igt@gem_exec_store@basic.html
   [132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2520m/igt@gem_exec_store@basic.html
    - fi-byt-j1900:       [PASS][133] -> [FAIL][134]
   [133]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-byt-j1900/igt@gem_exec_store@basic.html
   [134]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-j1900/igt@gem_exec_store@basic.html
    - fi-pnv-d510:        [PASS][135] -> [FAIL][136]
   [135]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-pnv-d510/igt@gem_exec_store@basic.html
   [136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-pnv-d510/igt@gem_exec_store@basic.html
    - fi-bdw-5557u:       [PASS][137] -> [FAIL][138]
   [137]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bdw-5557u/igt@gem_exec_store@basic.html
   [138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bdw-5557u/igt@gem_exec_store@basic.html
    - fi-blb-e6850:       [PASS][139] -> [FAIL][140]
   [139]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-blb-e6850/igt@gem_exec_store@basic.html
   [140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-blb-e6850/igt@gem_exec_store@basic.html
    - fi-bwr-2160:        [SKIP][141] ([fdo#109271]) -> [FAIL][142]
   [141]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bwr-2160/igt@gem_exec_store@basic.html
   [142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bwr-2160/igt@gem_exec_store@basic.html
    - fi-gdg-551:         [SKIP][143] ([fdo#109271]) -> [FAIL][144]
   [143]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-gdg-551/igt@gem_exec_store@basic.html
   [144]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-gdg-551/igt@gem_exec_store@basic.html
    - fi-bsw-nick:        [PASS][145] -> [FAIL][146]
   [145]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-nick/igt@gem_exec_store@basic.html
   [146]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-nick/igt@gem_exec_store@basic.html

  * {igt@gem_wait@wait}:
    - fi-gdg-551:         NOTRUN -> [FAIL][147] +3 similar issues
   [147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-gdg-551/igt@gem_wait@wait.html
    - fi-bsw-nick:        NOTRUN -> [FAIL][148] +3 similar issues
   [148]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-nick/igt@gem_wait@wait.html
    - fi-bwr-2160:        NOTRUN -> [FAIL][149] +3 similar issues
   [149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bwr-2160/igt@gem_wait@wait.html
    - fi-blb-e6850:       NOTRUN -> [FAIL][150] +3 similar issues
   [150]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-blb-e6850/igt@gem_wait@wait.html
    - fi-snb-2600:        NOTRUN -> [FAIL][151] +3 similar issues
   [151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2600/igt@gem_wait@wait.html

  
Known issues
------------

  Here are the changes found in Patchwork_17689 that come from known issues:

### CI changes ###

#### Issues hit ####

  * boot:
    - fi-apl-guc:         [PASS][152] -> [FAIL][153] ([i915#348])
   [152]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-apl-guc/boot.html
   [153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-apl-guc/boot.html
    - fi-bxt-dsi:         [PASS][154] -> [FAIL][155] ([i915#1529])
   [154]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bxt-dsi/boot.html
   [155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bxt-dsi/boot.html

  

### IGT changes ###

#### Issues hit ####

  * igt@i915_hangman@error-state-basic:
    - fi-byt-n2820:       [PASS][156] -> [SKIP][157] ([fdo#109271]) +1 similar issue
   [156]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-byt-n2820/igt@i915_hangman@error-state-basic.html
   [157]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-n2820/igt@i915_hangman@error-state-basic.html
    - fi-bwr-2160:        [PASS][158] -> [SKIP][159] ([fdo#109271]) +1 similar issue
   [158]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bwr-2160/igt@i915_hangman@error-state-basic.html
   [159]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bwr-2160/igt@i915_hangman@error-state-basic.html
    - fi-bdw-5557u:       [PASS][160] -> [SKIP][161] ([fdo#109271]) +1 similar issue
   [160]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bdw-5557u/igt@i915_hangman@error-state-basic.html
   [161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bdw-5557u/igt@i915_hangman@error-state-basic.html
    - fi-snb-2520m:       [PASS][162] -> [SKIP][163] ([fdo#109271]) +1 similar issue
   [162]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-snb-2520m/igt@i915_hangman@error-state-basic.html
   [163]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2520m/igt@i915_hangman@error-state-basic.html
    - fi-bsw-nick:        [PASS][164] -> [SKIP][165] ([fdo#109271])
   [164]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-nick/igt@i915_hangman@error-state-basic.html
   [165]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-nick/igt@i915_hangman@error-state-basic.html
    - fi-gdg-551:         [PASS][166] -> [SKIP][167] ([fdo#109271]) +1 similar issue
   [166]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-gdg-551/igt@i915_hangman@error-state-basic.html
   [167]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-gdg-551/igt@i915_hangman@error-state-basic.html
    - fi-bsw-kefka:       [PASS][168] -> [SKIP][169] ([fdo#109271]) +1 similar issue
   [168]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-kefka/igt@i915_hangman@error-state-basic.html
   [169]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-kefka/igt@i915_hangman@error-state-basic.html
    - fi-blb-e6850:       [PASS][170] -> [SKIP][171] ([fdo#109271]) +1 similar issue
   [170]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-blb-e6850/igt@i915_hangman@error-state-basic.html
   [171]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-blb-e6850/igt@i915_hangman@error-state-basic.html
    - fi-bsw-n3050:       [PASS][172] -> [SKIP][173] ([fdo#109271])
   [172]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-bsw-n3050/igt@i915_hangman@error-state-basic.html
   [173]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-bsw-n3050/igt@i915_hangman@error-state-basic.html
    - fi-snb-2600:        [PASS][174] -> [SKIP][175] ([fdo#109271]) +1 similar issue
   [174]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-snb-2600/igt@i915_hangman@error-state-basic.html
   [175]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-snb-2600/igt@i915_hangman@error-state-basic.html

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
    - fi-ilk-650:         [PASS][176] -> [SKIP][177] ([fdo#109271]) +1 similar issue
   [176]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-ilk-650/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [177]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ilk-650/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-ivb-3770:        [PASS][178] -> [SKIP][179] ([fdo#109271]) +1 similar issue
   [178]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-ivb-3770/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [179]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-ivb-3770/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-byt-j1900:       [PASS][180] -> [SKIP][181] ([fdo#109271]) +1 similar issue
   [180]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-byt-j1900/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [181]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-byt-j1900/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-elk-e7500:       [PASS][182] -> [SKIP][183] ([fdo#109271]) +1 similar issue
   [182]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-elk-e7500/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [183]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-elk-e7500/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
    - fi-pnv-d510:        [PASS][184] -> [SKIP][185] ([fdo#109271]) +1 similar issue
   [184]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8494/fi-pnv-d510/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
   [185]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/fi-pnv-d510/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [i915#1529]: https://gitlab.freedesktop.org/drm/intel/issues/1529
  [i915#348]: https://gitlab.freedesktop.org/drm/intel/issues/348


Participating hosts (51 -> 45)
------------------------------

  Additional (1): fi-kbl-7560u 
  Missing    (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * Linux: CI_DRM_8494 -> Patchwork_17689

  CI-20190529: 20190529
  CI_DRM_8494: 3d15348fde9b998e754da0b0655baf02b98e7f17 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5657: 649eae5c905a7460b44305800f95db83a6dd47cb @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_17689: 7918634c08d48d721c119df99c45f57217c86ea4 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

7918634c08d4 drm/i915/params: switch to device specific parameters
0e5029451f95 drm/i915/params: prevent changing module params runtime
f371da7198c1 drm/i915/params: fix i915.fake_lmem_start module param sysfs permissions
0df592070475 drm/i915/params: don't expose inject_probe_failure in debugfs

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17689/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx