[v2] PM: s2idle: A couple of minor lock-simplifications

[PATCH v2 0/2] PM: s2idle: A couple of minor lock-simplifications

Posted by Ulf Hansson 9 months, 1 week ago

While working on s2idle a few things caught my attention when looking
closer in the code in the s2idle_enter(), in regards to locks. These two
patches tries to improve the situation a bit.

Kind regards
Ulf Hansson

Ulf Hansson (2):
  PM: s2idle: Drop redundant locks when entering s2idle
  PM: s2idle: Extend comment in s2idle_enter()

 kernel/power/suspend.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

-- 
2.43.0

Re: [PATCH v2 0/2] PM: s2idle: A couple of minor lock-simplifications

Posted by Rafael J. Wysocki 9 months, 1 week ago

On Tue, Mar 11, 2025 at 5:08 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
>
> While working on s2idle a few things caught my attention when looking
> closer in the code in the s2idle_enter(), in regards to locks. These two
> patches tries to improve the situation a bit.
>
> Kind regards
> Ulf Hansson
>
> Ulf Hansson (2):
>   PM: s2idle: Drop redundant locks when entering s2idle
>   PM: s2idle: Extend comment in s2idle_enter()
>
>  kernel/power/suspend.c | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
>
> --

Both patches applied as 6.15 material, but I rewrote the new comments.
Hopefully, this is not a problem.

Thanks!

Re: [PATCH v2 0/2] PM: s2idle: A couple of minor lock-simplifications

Posted by Ulf Hansson 9 months, 1 week ago

On Wed, 12 Mar 2025 at 21:20, Rafael J. Wysocki <rafael@kernel.org> wrote:
>
> On Tue, Mar 11, 2025 at 5:08 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> >
> > While working on s2idle a few things caught my attention when looking
> > closer in the code in the s2idle_enter(), in regards to locks. These two
> > patches tries to improve the situation a bit.
> >
> > Kind regards
> > Ulf Hansson
> >
> > Ulf Hansson (2):
> >   PM: s2idle: Drop redundant locks when entering s2idle
> >   PM: s2idle: Extend comment in s2idle_enter()
> >
> >  kernel/power/suspend.c | 10 ++++++----
> >  1 file changed, 6 insertions(+), 4 deletions(-)
> >
> > --
>
> Both patches applied as 6.15 material, but I rewrote the new comments.

Thanks!

> Hopefully, this is not a problem.

Looks good, thanks!

Kind regards
Uffe