ACPI: processor: Drop duplicate setting of shared_cpu_map
authorPunit Agrawal <punitagrawal@gmail.com>
Mon, 23 Nov 2020 23:59:51 +0000 (08:59 +0900)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 25 Nov 2020 15:44:36 +0000 (16:44 +0100)
'shared_cpu_map', stored as part of the per-processor
acpi_processor_performance structre, is used to store CPUs that share
a performance domain. By definition it contains the owning CPU.

While building the 'shared_cpu_map' it is being set twice - once while
initialising the performance domains and again when matching CPUs
belonging to the same domain.

Drop the unnecessary initialisation.

Signed-off-by: Punit Agrawal <punitagrawal@gmail.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/processor_perflib.c

index b04a689..b0d320f 100644 (file)
@@ -616,7 +616,6 @@ int acpi_processor_preregister_performance(
                        continue;
 
                pr->performance = per_cpu_ptr(performance, i);
-               cpumask_set_cpu(i, pr->performance->shared_cpu_map);
                pdomain = &(pr->performance->domain_info);
                if (acpi_processor_get_psd(pr->handle, pdomain)) {
                        retval = -EINVAL;