2013-07-17 10:05:56

by Baruch Siach

Subject: [PATCH] MAINTAINERS: add the generic sched_clock under timekeeping

Signed-off-by: Baruch Siach <[email protected]>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index bf61e04..bd9616a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7129,6 +7129,7 @@ M: Thomas Gleixner <[email protected]>
T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/core
S: Supported
F: include/linux/clocksource.h
+F: include/linux/sched_clock.h
F: include/linux/time.h
F: include/linux/timex.h
F: include/uapi/linux/time.h
@@ -7136,6 +7137,7 @@ F: include/uapi/linux/timex.h
F: kernel/time/clocksource.c
F: kernel/time/time*.c
F: kernel/time/ntp.c
+F: kernel/time/sched_clock.c
F: drivers/clocksource

TLG2300 VIDEO4LINUX-2 DRIVER
--
1.8.3.2

2013-07-17 18:57:38

by John Stultz

Subject: Re: [PATCH] MAINTAINERS: add the generic sched_clock under timekeeping

On 07/17/2013 03:05 AM, Baruch Siach wrote:
> Signed-off-by: Baruch Siach <[email protected]>
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bf61e04..bd9616a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7129,6 +7129,7 @@ M: Thomas Gleixner <[email protected]>
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/core
> S: Supported
> F: include/linux/clocksource.h
> +F: include/linux/sched_clock.h
> F: include/linux/time.h
> F: include/linux/timex.h
> F: include/uapi/linux/time.h
> @@ -7136,6 +7137,7 @@ F: include/uapi/linux/timex.h
> F: kernel/time/clocksource.c
> F: kernel/time/time*.c
> F: kernel/time/ntp.c
> +F: kernel/time/sched_clock.c

It seems like we could probably simplify this bit to kernel/time/*, no?

thanks
-john

2013-07-17 19:12:29

by Baruch Siach

Subject: Re: [PATCH] MAINTAINERS: add the generic sched_clock under timekeeping

Hi John,

On Wed, Jul 17, 2013 at 11:57:32AM -0700, John Stultz wrote:
> On 07/17/2013 03:05 AM, Baruch Siach wrote:
> >Signed-off-by: Baruch Siach <[email protected]>
> >---
> > MAINTAINERS | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> >diff --git a/MAINTAINERS b/MAINTAINERS
> >index bf61e04..bd9616a 100644
> >--- a/MAINTAINERS
> >+++ b/MAINTAINERS
> >@@ -7129,6 +7129,7 @@ M: Thomas Gleixner <[email protected]>
> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/core
> > S: Supported
> > F: include/linux/clocksource.h
> >+F: include/linux/sched_clock.h
> > F: include/linux/time.h
> > F: include/linux/timex.h
> > F: include/uapi/linux/time.h
> >@@ -7136,6 +7137,7 @@ F: include/uapi/linux/timex.h
> > F: kernel/time/clocksource.c
> > F: kernel/time/time*.c
> > F: kernel/time/ntp.c
> >+F: kernel/time/sched_clock.c
>
> It seems like we could probably simplify this bit to kernel/time/*, no?

This would add jiffies.c and all the tick-* files that are currently covered
by the "HIGH-RESOLUTION TIMERS, CLOCKEVENTS, DYNTICKS" section.

baruch

--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- [email protected] - tel: +972.2.679.5364, http://www.tkos.co.il -

2013-07-17 19:18:34

by Joe Perches

Subject: Re: [PATCH] MAINTAINERS: add the generic sched_clock under timekeeping

On Wed, 2013-07-17 at 22:12 +0300, Baruch Siach wrote:
[]
> > >diff --git a/MAINTAINERS b/MAINTAINERS
[]
> > >@@ -7129,6 +7129,7 @@ M: Thomas Gleixner <[email protected]>
[]
> > >+F: include/linux/sched_clock.h
[]
> > >+F: kernel/time/sched_clock.c
> >
> > It seems like we could probably simplify this bit to kernel/time/*, no?
>
> This would add jiffies.c and all the tick-* files that are currently covered
> by the "HIGH-RESOLUTION TIMERS, CLOCKEVENTS, DYNTICKS" section.

Another option would be to move files into
a separate directory, but I think it's not
a big deal either way.

2013-07-17 19:21:16

by John Stultz

Subject: Re: [PATCH] MAINTAINERS: add the generic sched_clock under timekeeping

On 07/17/2013 12:12 PM, Baruch Siach wrote:
> Hi John,
>
> On Wed, Jul 17, 2013 at 11:57:32AM -0700, John Stultz wrote:
>> On 07/17/2013 03:05 AM, Baruch Siach wrote:
>>> Signed-off-by: Baruch Siach <[email protected]>
>>> ---
>>> MAINTAINERS | 2 ++
>>> 1 file changed, 2 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index bf61e04..bd9616a 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -7129,6 +7129,7 @@ M: Thomas Gleixner <[email protected]>
>>> T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/core
>>> S: Supported
>>> F: include/linux/clocksource.h
>>> +F: include/linux/sched_clock.h
>>> F: include/linux/time.h
>>> F: include/linux/timex.h
>>> F: include/uapi/linux/time.h
>>> @@ -7136,6 +7137,7 @@ F: include/uapi/linux/timex.h
>>> F: kernel/time/clocksource.c
>>> F: kernel/time/time*.c
>>> F: kernel/time/ntp.c
>>> +F: kernel/time/sched_clock.c
>> It seems like we could probably simplify this bit to kernel/time/*, no?
> This would add jiffies.c and all the tick-* files that are currently covered
> by the "HIGH-RESOLUTION TIMERS, CLOCKEVENTS, DYNTICKS" section.

Ok, fair enough. My thought was it all goes through Thomas anyway, but
for those sections there are different combinations of folks who
co-maintain.

thanks
-john

2013-08-19 03:28:51

by Baruch Siach

Subject: Re: [PATCH] MAINTAINERS: add the generic sched_clock under timekeeping

Hi John,

On Wed, Jul 17, 2013 at 12:21:12PM -0700, John Stultz wrote:
> On 07/17/2013 12:12 PM, Baruch Siach wrote:
> >On Wed, Jul 17, 2013 at 11:57:32AM -0700, John Stultz wrote:
> >>On 07/17/2013 03:05 AM, Baruch Siach wrote:
> >>>Signed-off-by: Baruch Siach <[email protected]>
> >>>---
> >>> MAINTAINERS | 2 ++
> >>> 1 file changed, 2 insertions(+)
> >>>
> >>>diff --git a/MAINTAINERS b/MAINTAINERS
> >>>index bf61e04..bd9616a 100644
> >>>--- a/MAINTAINERS
> >>>+++ b/MAINTAINERS
> >>>@@ -7129,6 +7129,7 @@ M: Thomas Gleixner <[email protected]>
> >>> T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/core
> >>> S: Supported
> >>> F: include/linux/clocksource.h
> >>>+F: include/linux/sched_clock.h
> >>> F: include/linux/time.h
> >>> F: include/linux/timex.h
> >>> F: include/uapi/linux/time.h
> >>>@@ -7136,6 +7137,7 @@ F: include/uapi/linux/timex.h
> >>> F: kernel/time/clocksource.c
> >>> F: kernel/time/time*.c
> >>> F: kernel/time/ntp.c
> >>>+F: kernel/time/sched_clock.c
> >>It seems like we could probably simplify this bit to kernel/time/*, no?
> >This would add jiffies.c and all the tick-* files that are currently covered
> >by the "HIGH-RESOLUTION TIMERS, CLOCKEVENTS, DYNTICKS" section.
>
> Ok, fair enough. My thought was it all goes through Thomas anyway,
> but for those sections there are different combinations of folks who
> co-maintain.

So would you take it for v3.12?

baruch

--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- [email protected] - tel: +972.2.679.5364, http://www.tkos.co.il -