On Tue, Mar 11, 2014 at 05:20:23PM +0800, Dongsheng Yang wrote: > From: Joe Perches <j...@perches.com> > > Convert 20 - task_nice(p) to nice_to_rlimit(task_nice(p)). > > Reduce the indent the switch case labels while there. > > git diff -w shows 3 lines changed and a /* fall-through */ comment added > > $ git diff -w -U0 kernel/sys.c > @@ -253 +253 @@ SYSCALL_DEFINE2(getpriority, int, which, int, who) > - niceval = 20 - task_nice(p); > + niceval = nice_to_rlimit(task_nice(p)); > @@ -264 +264 @@ SYSCALL_DEFINE2(getpriority, int, which, int, who) > - niceval = 20 - task_nice(p); > + niceval = nice_to_rlimit(task_nice(p)); > @@ -280 +280 @@ SYSCALL_DEFINE2(getpriority, int, which, int, who) > - niceval = 20 - task_nice(p); > + niceval = nice_to_rlimit(task_nice(p)); > @@ -1558 +1558 @@ static void k_getrusage(struct task_struct *p, int who, > struct rusage *r) > - > + /* fall-through */ >
I would really like to see the whitespace changes separated. I can push a scheduler related change into other subsystems, but massive surgery like this should not come through the scheduler tree. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
