Re: [PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2018-01-17 Thread Thomas Gleixner
On Tue, 2 Jan 2018, David Howells wrote: > Thomas Gleixner wrote: > > > > #define INIT_CPU_TIMERS(s) > > > \ > > ... > > That macro is only used in init_task.c Why not moving it there and get rid > > of the whole macro maze in

Re: [PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2018-01-17 Thread Thomas Gleixner
On Tue, 2 Jan 2018, David Howells wrote: > Thomas Gleixner wrote: > > > > #define INIT_CPU_TIMERS(s) > > > \ > > ... > > That macro is only used in init_task.c Why not moving it there and get rid > > of the whole macro maze in the header file? >

Re: [PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2018-01-02 Thread David Howells
Thomas Gleixner wrote: > > #define INIT_CPU_TIMERS(s) \ > ... > That macro is only used in init_task.c Why not moving it there and get rid > of the whole macro maze in the header file? The reason I didn't get rid of it is that it's

Re: [PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2018-01-02 Thread David Howells
Thomas Gleixner wrote: > > #define INIT_CPU_TIMERS(s) \ > ... > That macro is only used in init_task.c Why not moving it there and get rid > of the whole macro maze in the header file? The reason I didn't get rid of it is that it's got more than one

Re: [PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2017-12-28 Thread Thomas Gleixner
On Fri, 8 Dec 2017, David Howells wrote: > #define INIT_CPU_TIMERS(s) \ > .cpu_timers = { \ > LIST_HEAD_INIT(s.cpu_timers[0]),\ >

Re: [PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2017-12-28 Thread Thomas Gleixner
On Fri, 8 Dec 2017, David Howells wrote: > #define INIT_CPU_TIMERS(s) \ > .cpu_timers = { \ > LIST_HEAD_INIT(s.cpu_timers[0]),\ >

[PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2017-12-08 Thread David Howells
There doesn't seem to be any need to have the INIT_SIGNALS and INIT_SIGHAND macros, so expand them in their single places of use and remove them. Signed-off-by: David Howells --- include/linux/init_task.h | 43 --- init/init_task.c

[PATCH 4/5] Expand the INIT_SIGNALS and INIT_SIGHAND macros and remove

2017-12-08 Thread David Howells
There doesn't seem to be any need to have the INIT_SIGNALS and INIT_SIGHAND macros, so expand them in their single places of use and remove them. Signed-off-by: David Howells --- include/linux/init_task.h | 43 --- init/init_task.c | 30