2008-06-09 09:51:44

by ciaby

[permalink] [raw]
Subject: [PATCH] 2.6.25.4-rt6: missing global_rt_runtime() in sched.c


I already sent this patch, but seems that nobody cared... the last 2
revisions of the -rt kernel don't compile properly.
Second patch to fix this.
Regards

Giovanni "ciaby" Civardi

diff -uNr linux-2.6.25.4-rt6.orig/kernel/sched.c
linux-2.6.25.4-rt6/kernel/sched.c
--- linux-2.6.25.4-rt6.orig/kernel/sched.c 2008-06-09
11:35:57.000000000 +0200
+++ linux-2.6.25.4-rt6/kernel/sched.c 2008-06-09 11:40:45.000000000 +0200
@@ -672,6 +672,13 @@
*/
#define RUNTIME_INF ((u64)~0ULL)

+static u64 global_rt_runtime(void)
+{
+ if (sysctl_sched_rt_period < 0)
+ return RUNTIME_INF;
+ return (u64)sysctl_sched_rt_runtime * NSEC_PER_USEC;
+}
+
/*
* We really dont want to do anything complex within switch_to()
* on PREEMPT_RT - this check enforces this.


2008-06-09 12:55:18

by Tarkan Erimer

[permalink] [raw]
Subject: Re: [PATCH] 2.6.25.4-rt6: missing global_rt_runtime() in sched.c

ciaby wrote:
> I already sent this patch, but seems that nobody cared... the last 2
> revisions of the -rt kernel don't compile properly.
> Second patch to fix this.
> Regards
>
> Giovanni "ciaby" Civardi
>
Thanks for the patch. It worked perfectly for me.

2008-06-09 18:02:33

by Daniel Walker

[permalink] [raw]
Subject: Re: [PATCH] 2.6.25.4-rt6: missing global_rt_runtime() in sched.c


On Mon, 2008-06-09 at 09:51 +0000, ciaby wrote:
> I already sent this patch, but seems that nobody cared... the last 2
> revisions of the -rt kernel don't compile properly.
> Second patch to fix this.
> Regards
>
> Giovanni "ciaby" Civardi
>
> diff -uNr linux-2.6.25.4-rt6.orig/kernel/sched.c
> linux-2.6.25.4-rt6/kernel/sched.c
> --- linux-2.6.25.4-rt6.orig/kernel/sched.c 2008-06-09
> 11:35:57.000000000 +0200
> +++ linux-2.6.25.4-rt6/kernel/sched.c 2008-06-09 11:40:45.000000000 +0200
> @@ -672,6 +672,13 @@
> */
> #define RUNTIME_INF ((u64)~0ULL)
>
> +static u64 global_rt_runtime(void)
> +{
> + if (sysctl_sched_rt_period < 0)
> + return RUNTIME_INF;
> + return (u64)sysctl_sched_rt_runtime * NSEC_PER_USEC;

Your patch has a couple of problems .. It's line wrapped, it's missing
proper tabbing (got stripped maybe), and no Signed-off-by: ..

Daniel

2008-06-13 10:07:45

by ciaby

[permalink] [raw]
Subject: [PATCH] 2.6.25.4-rt6: missing global_rt_runtime() in sched.c


Not-line-wrapped, with proper tabs :)
Cheers

Giovanni 'ciaby' Civardi

2008-06-13 19:24:00

by Daniel Walker

[permalink] [raw]
Subject: Re: [PATCH] 2.6.25.4-rt6: missing global_rt_runtime() in sched.c


On Fri, 2008-06-13 at 10:07 +0000, ciaby wrote:
> Not-line-wrapped, with proper tabs :)
> Cheers
>
> Giovanni 'ciaby' Civardi

No patch tho included here tho ..

Daniel

2008-06-23 22:43:57

by Steven Rostedt

[permalink] [raw]
Subject: Re: [PATCH] 2.6.25.4-rt6: missing global_rt_runtime() in sched.c


On Mon, 9 Jun 2008, ciaby wrote:

>
>
> I already sent this patch, but seems that nobody cared... the last 2
> revisions of the -rt kernel don't compile properly.
> Second patch to fix this.
> Regards

I still don't have a "Signed-off-by:" for this patch. Please send it to me
ASAP, otherwise it will miss the next release.

-- Steve

>
> Giovanni "ciaby" Civardi
>
> diff -uNr linux-2.6.25.4-rt6.orig/kernel/sched.c
> linux-2.6.25.4-rt6/kernel/sched.c
> --- linux-2.6.25.4-rt6.orig/kernel/sched.c 2008-06-09
> 11:35:57.000000000 +0200
> +++ linux-2.6.25.4-rt6/kernel/sched.c 2008-06-09 11:40:45.000000000 +0200
> @@ -672,6 +672,13 @@
> */
> #define RUNTIME_INF ((u64)~0ULL)
>
> +static u64 global_rt_runtime(void)
> +{
> + if (sysctl_sched_rt_period < 0)
> + return RUNTIME_INF;
> + return (u64)sysctl_sched_rt_runtime * NSEC_PER_USEC;
> +}
> +
> /*
> * We really dont want to do anything complex within switch_to()
> * on PREEMPT_RT - this check enforces this.
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>

2008-06-24 01:31:00

by Steven Rostedt

[permalink] [raw]
Subject: Re: [PATCH] 2.6.25.4-rt6: missing global_rt_runtime() in sched.c



On Mon, 23 Jun 2008, Steven Rostedt wrote:

>
> On Mon, 9 Jun 2008, ciaby wrote:
>
> >
> >
> > I already sent this patch, but seems that nobody cared... the last 2
> > revisions of the -rt kernel don't compile properly.
> > Second patch to fix this.
> > Regards
>
> I still don't have a "Signed-off-by:" for this patch. Please send it to me
> ASAP, otherwise it will miss the next release.
>

I'm posting out -rt7 now without this patch. I'm hoping to get your
Signed-off-by for -rt8 which will come out soon (I want to get Gregory's
patches testest ASAP too).

-- Steve