Messages in this thread | | | From | "Rafael J. Wysocki" <> | Date | Sat, 24 Sep 2022 19:34:17 +0200 | Subject | Re: [PATCH 1/3] powercap: intel_rapl: Use standard Energy Unit for SPR Dram RAPL domain |
| |
On Sat, Sep 24, 2022 at 7:45 AM Zhang Rui <rui.zhang@intel.com> wrote: > > Intel Xeon servers used to use a fixed energy resolution (15.3uj) for > Dram RAPL domain. But on SPR, Dram RAPL domain follows the standard > energy resolution as described in MSR_RAPL_POWER_UNIT. > > Remove the SPR dram_domain_energy_unit quirk. > > Fixes: 2d798d9f5967 ("powercap: intel_rapl: add support for Sapphire Rapids") > Signed-off-by: Zhang Rui <rui.zhang@intel.com> > Tested-by: Wang Wendy <wendy.wang@intel.com> > --- > drivers/powercap/intel_rapl_common.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/powercap/intel_rapl_common.c b/drivers/powercap/intel_rapl_common.c > index 949eb90f5733..d8eaa04bf2af 100644 > --- a/drivers/powercap/intel_rapl_common.c > +++ b/drivers/powercap/intel_rapl_common.c > @@ -1035,7 +1035,6 @@ static const struct rapl_defaults rapl_defaults_spr_server = { > .check_unit = rapl_check_unit_core, > .set_floor_freq = set_floor_freq_default, > .compute_time_window = rapl_compute_time_window_core, > - .dram_domain_energy_unit = 15300, > .psys_domain_energy_unit = 1000000000, > .spr_psys_bits = true, > }; > --
Applied as 6.1 material, thanks!
| |