OSDN Git Service

ACPI: processor: perflib: Rearrange unregistration routine
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 5 Dec 2022 19:05:55 +0000 (20:05 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 7 Dec 2022 17:09:39 +0000 (18:09 +0100)
Rearrange acpi_processor_unregister_performance() to follow a more
common error handling pattern and drop a redundant "return" statement
from the end of it.

No expected functional impact.

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

index 388f045..10d2999 100644 (file)
@@ -754,18 +754,15 @@ void acpi_processor_unregister_performance(unsigned int cpu)
        mutex_lock(&performance_mutex);
 
        pr = per_cpu(processors, cpu);
-       if (!pr) {
-               mutex_unlock(&performance_mutex);
-               return;
-       }
+       if (!pr)
+               goto unlock;
 
        if (pr->performance)
                kfree(pr->performance->states);
 
        pr->performance = NULL;
 
+unlock:
        mutex_unlock(&performance_mutex);
-
-       return;
 }
 EXPORT_SYMBOL(acpi_processor_unregister_performance);