2022-11-23 11:31:03

by Anna-Maria Behnsen

Subject: Re: [patch V2 13/17] timers: Split [try_to_]del_timer[_sync]() to prepare for shutdown mode

On Tue, 22 Nov 2022, Thomas Gleixner wrote:

> Tearing down timers which have circular dependencies to other
> functionality, e.g. workqueues, where the timer can schedule work and work
> can arm timers is not trivial.

Comma missing (same as in previous commit message)

> In those cases it is desired to shutdown the timer in a way which prevents
> rearming of the timer. The mechanism to do so it to set timer->function to

s/it/is (same as in previous commit message)


Thanks,

Anna-Maria

2022-11-23 11:32:59

by Anna-Maria Behnsen

Subject: Re: [patch V2 13/17] timers: Split [try_to_]del_timer[_sync]() to prepare for shutdown mode

On Wed, 23 Nov 2022, Anna-Maria Behnsen wrote:

> On Tue, 22 Nov 2022, Thomas Gleixner wrote:
>
> > Tearing down timers which have circular dependencies to other
> > functionality, e.g. workqueues, where the timer can schedule work and work
> > can arm timers is not trivial.
>
> Comma missing (same as in previous commit message)
>
> > In those cases it is desired to shutdown the timer in a way which prevents
> > rearming of the timer. The mechanism to do so it to set timer->function to
>
> s/it/is (same as in previous commit message)
>
>

Same for patch 14 and 15...