• Ashutosh Dixit's avatar
    drm/i915/hwmon: Use 0 to designate disabled PL1 power limit · 4ed22f1e
    Ashutosh Dixit authored
    On ATSM the PL1 limit is disabled at power up. The previous uapi assumed
    that the PL1 limit is always enabled and therefore did not have a notion of
    a disabled PL1 limit. This results in erroneous PL1 limit values when the
    PL1 limit is disabled. For example at power up, the disabled ATSM PL1 limit
    was previously shown as 0 which means a low PL1 limit whereas the limit
    being disabled actually implies a high effective PL1 limit value.
    
    To get round this problem, the PL1 limit uapi is expanded to include a
    special value 0 to designate a disabled PL1 limit. A read value of 0 means
    that the PL1 power limit is disabled, writing 0 disables the limit.
    
    The link between this patch and the bugs mentioned below is as follows:
    * Because on ATSM the PL1 power limit is disabled on power up and there
      were no means to enable it, we previously implemented the means to
      enable the limit when the PL1 hwmon entry (power1_max) was written to.
    * Now there is a IGT igt@i915_hwmon@hwmon_write which (a) reads orig value
      from all hwmon sysfs  (b) does a bunch of random writes and finally (c)
      restores the orig value read. On ATSM since the orig value is 0, when
      the IGT restores the 0 value, the PL1 limit is now enabled with a value
      of 0.
    * PL1 limit of 0 implies a low PL1 limit which causes GPU freq to fall to
      100 MHz. This causes GuC FW load and several IGT's to start timing out
      and gives rise to these Intel CI bugs. After this patch, writing 0 would
      disable the PL1 limit instead of enabling it, avoiding the freq drop
      issue.
    
    v2: Add explanation for bugs mentioned below (Rodrigo)
    v3: Eliminate race during PL1 disable and verify (Tvrtko)
        Change return to -ENODEV if verify fails (Tvrtko)
    
    Link: https://gitlab.freedesktop.org/drm/intel/-/issues/8062
    Link: https://gitlab.freedesktop.org/drm/intel/-/issues/8060Signed-off-by: default avatarAshutosh Dixit <ashutosh.dixit@intel.com>
    Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
    Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20230401024146.1826092-1-ashutosh.dixit@intel.com
    4ed22f1e
sysfs-driver-intel-i915-hwmon 2.75 KB