Re: [patch V2 17/38] posix-cpu-timers: Create a container struct

2019-08-23 Thread Thomas Gleixner
On Thu, 22 Aug 2019, Frederic Weisbecker wrote: > On Wed, Aug 21, 2019 at 09:09:04PM +0200, Thomas Gleixner wrote: > > --- a/include/linux/posix-timers.h > > +++ b/include/linux/posix-timers.h > > @@ -62,6 +62,40 @@ static inline int clockid_to_fd(const cl > > return ~(clk >> 3); > > } > > S

Re: [patch V2 17/38] posix-cpu-timers: Create a container struct

2019-08-22 Thread Frederic Weisbecker
On Wed, Aug 21, 2019 at 09:09:04PM +0200, Thomas Gleixner wrote: > --- a/include/linux/posix-timers.h > +++ b/include/linux/posix-timers.h > @@ -62,6 +62,40 @@ static inline int clockid_to_fd(const cl > return ~(clk >> 3); > } Shouldn't you start the #ifdef CONFIG_POSIX_TIMERS here? Because

[patch V2 17/38] posix-cpu-timers: Create a container struct

2019-08-21 Thread Thomas Gleixner
Per task/process data of posix CPU timers is all over the place which makes the code hard to follow and requires ifdeffery. Create a container to hold all this information in one place, so data is consolidated and the ifdeffery can be confined to the posix timer header file and removed from places