linux-kernel - Re: [PATCH v2 1/2] ACPI: platform_profile: Add devm_platform_profile

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <925e1bb3-f21a-553e-24ce-6ef09ef7b4e7@linux.intel.com>
Date: Tue, 14 Jan 2025 16:33:25 +0200 (EET)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: "Rafael J. Wysocki" <rafael@...nel.org>
cc: Kurt Borja <kuurtb@...il.com>, platform-driver-x86@...r.kernel.org, 
    W_Armin@....de, Hans de Goede <hdegoede@...hat.com>, lenb@...nel.org, 
    linux-acpi@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>, 
    mario.limonciello@....com, Mark Pearson <mpearson-lenovo@...ebb.ca>, 
    soyer@....hu
Subject: Re: [PATCH v2 1/2] ACPI: platform_profile: Add
 devm_platform_profile_register()

On Tue, 14 Jan 2025, Rafael J. Wysocki wrote:

> On Tue, Dec 24, 2024 at 3:02 PM Kurt Borja <kuurtb@...il.com> wrote:
> >
> > Platform profile's lifetime is usually tied to a device's lifetime,
> > therefore add a device managed version of platform_profile_register().
> >
> > Signed-off-by: Kurt Borja <kuurtb@...il.com>
> 
> This is fine by me, so
> 
> Acked-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
> 
> and I think that it would be better to route it via platform/x86 along
> with the second patch.

That's were it already is among the other platform profile changes.

We still have one big platform profile series pending (though Kurt needs 
to do small tweaks still into it before it can be applied):

https://lore.kernel.org/all/20250109150731.110799-1-kuurtb@gmail.com/

-- 
 i.

> > ---
> >  drivers/acpi/platform_profile.c  | 29 +++++++++++++++++++++++++++++
> >  include/linux/platform_profile.h |  1 +
> >  2 files changed, 30 insertions(+)
> >
> > diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c
> > index 75a1415190ac..4c4200a0b1a6 100644
> > --- a/drivers/acpi/platform_profile.c
> > +++ b/drivers/acpi/platform_profile.c
> > @@ -519,6 +519,35 @@ int platform_profile_remove(struct platform_profile_handler *pprof)
> >  }
> >  EXPORT_SYMBOL_GPL(platform_profile_remove);
> >
> > +static void devm_platform_profile_release(struct device *dev, void *res)
> > +{
> > +       struct platform_profile_handler **pprof = res;
> > +
> > +       platform_profile_remove(*pprof);
> > +}
> > +
> > +int devm_platform_profile_register(struct platform_profile_handler *pprof)
> > +{
> > +       struct platform_profile_handler **dr;
> > +       int ret;
> > +
> > +       dr = devres_alloc(devm_platform_profile_release, sizeof(*dr), GFP_KERNEL);
> > +       if (!dr)
> > +               return -ENOMEM;
> > +
> > +       ret = platform_profile_register(pprof);
> > +       if (ret) {
> > +               devres_free(dr);
> > +               return ret;
> > +       }
> > +
> > +       *dr = pprof;
> > +       devres_add(pprof->dev, dr);
> > +
> > +       return 0;
> > +}
> > +EXPORT_SYMBOL_GPL(devm_platform_profile_register);
> > +
> >  static int __init platform_profile_init(void)
> >  {
> >         int err;
> > diff --git a/include/linux/platform_profile.h b/include/linux/platform_profile.h
> > index 0682bb4c57e5..f1cd4b65e351 100644
> > --- a/include/linux/platform_profile.h
> > +++ b/include/linux/platform_profile.h
> > @@ -41,6 +41,7 @@ struct platform_profile_handler {
> >
> >  int platform_profile_register(struct platform_profile_handler *pprof);
> >  int platform_profile_remove(struct platform_profile_handler *pprof);
> > +int devm_platform_profile_register(struct platform_profile_handler *pprof);
> >  int platform_profile_cycle(void);
> >  void platform_profile_notify(struct platform_profile_handler *pprof);
> >
> > --
> > 2.47.1
> >
>