Re: [patch V3 01/22] timer: Make pinned a timer property

[Frederic Weisbecker]

On Fri, Jun 24, 2016 at 02:32:01PM -0000, Thomas Gleixner wrote:
> We want to move the timer migration from a push to a pull model. This requires
> to store the pinned attribute of a timer in the timer itself. This must happen
> at initialization time.
> 
> Add the helper macros for this.
> 
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Cc: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> Cc: Chris Mason <clm@xxxxxx>
> Cc: Eric Dumazet <edumazet@xxxxxxxxxx>
> Cc: rt@xxxxxxxxxxxxx
> Cc: "Paul E. McKenney" <paulmck@xxxxxxxxxxxxxxxxxx>
> Cc: Arjan van de Ven <arjan@xxxxxxxxxxxxx>
> 
> 
> ---
>  include/linux/timer.h |   25 ++++++++++++++++++++++---
>  kernel/time/timer.c   |   10 +++++-----
>  2 files changed, 27 insertions(+), 8 deletions(-)
> 
> --- a/include/linux/timer.h
> +++ b/include/linux/timer.h
> @@ -62,7 +62,8 @@ struct timer_list {
>  #define TIMER_MIGRATING		0x00080000
>  #define TIMER_BASEMASK		(TIMER_CPUMASK | TIMER_MIGRATING)
>  #define TIMER_DEFERRABLE	0x00100000
> -#define TIMER_IRQSAFE		0x00200000
> +#define TIMER_PINNED		0x00200000
> +#define TIMER_IRQSAFE		0x00400000
>  
>  #define __TIMER_INITIALIZER(_function, _expires, _data, _flags) { \
>  		.entry = { .next = TIMER_ENTRY_STATIC },	\
> @@ -78,9 +79,15 @@ struct timer_list {
>  #define TIMER_INITIALIZER(_function, _expires, _data)		\
>  	__TIMER_INITIALIZER((_function), (_expires), (_data), 0)
>  
> +#define TIMER_PINNED_INITIALIZER(_function, _expires, _data)	\
> +	__TIMER_INITIALIZER((_function), (_expires), (_data), TIMER_PINNED)
> +
>  #define TIMER_DEFERRED_INITIALIZER(_function, _expires, _data)	\
>  	__TIMER_INITIALIZER((_function), (_expires), (_data), TIMER_DEFERRABLE)
>  
> +#define TIMER_PINNED_DEFERRED_INITIALIZER(_function, _expires, _data)	\
> +	__TIMER_INITIALIZER((_function), (_expires), (_data), TIMER_DEFERRABLE | TIMER_PINNED)
> +

I guess it means that no timer took the freedom to be sometimes enqueued pinned and
sometimes unpinned. Which is a good news :-)