Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=c289b074b66e2e59c65aba73f40b99e797e92d2f
Commit:     c289b074b66e2e59c65aba73f40b99e797e92d2f
Parent:     416529374b4793ba2d2e97e736d108a2e0f3ef07
Author:     Oleg Nesterov <[EMAIL PROTECTED]>
AuthorDate: Fri Feb 1 20:41:30 2008 +0300
Committer:  Thomas Gleixner <[EMAIL PROTECTED]>
CommitDate: Sun Feb 10 10:48:03 2008 +0100

    hrtimer: don't modify restart_block->fn in restart functions
    
    hrtimer_nanosleep_restart() clears/restores restart_block->fn. This is
    pointless and complicates its usage. Note that if sys_restart_syscall()
    doesn't actually happen, we have a bogus "pending" restart->fn anyway,
    this is harmless.
    
    Signed-off-by: Oleg Nesterov <[EMAIL PROTECTED]>
    Cc: Alexey Dobriyan <[EMAIL PROTECTED]>
    Cc: Pavel Emelyanov <[EMAIL PROTECTED]>
    Cc: Peter Zijlstra <[EMAIL PROTECTED]>
    Cc: Toyo Abe <[EMAIL PROTECTED]>
    Cc: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Thomas Gleixner <[EMAIL PROTECTED]>
---
 kernel/compat.c  |    1 -
 kernel/hrtimer.c |    4 ----
 2 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/kernel/compat.c b/kernel/compat.c
index f2a2975..5f0e201 100644
--- a/kernel/compat.c
+++ b/kernel/compat.c
@@ -55,7 +55,6 @@ static long compat_nanosleep_restart(struct restart_block 
*restart)
        set_fs(oldfs);
 
        if (ret) {
-               restart->fn = compat_nanosleep_restart;
                restart->arg1 = (unsigned long)rmtp;
 
                if (rmtp && put_compat_timespec(&rmt, rmtp))
diff --git a/kernel/hrtimer.c b/kernel/hrtimer.c
index 355085f..3f4a57c 100644
--- a/kernel/hrtimer.c
+++ b/kernel/hrtimer.c
@@ -1340,8 +1340,6 @@ long __sched hrtimer_nanosleep_restart(struct 
restart_block *restart)
        struct hrtimer_sleeper t;
        struct timespec __user  *rmtp;
 
-       restart->fn = do_no_restart_syscall;
-
        hrtimer_init(&t.timer, restart->arg0, HRTIMER_MODE_ABS);
        t.timer.expires.tv64 = ((u64)restart->arg3 << 32) | (u64) restart->arg2;
 
@@ -1355,8 +1353,6 @@ long __sched hrtimer_nanosleep_restart(struct 
restart_block *restart)
                        return ret;
        }
 
-       restart->fn = hrtimer_nanosleep_restart;
-
        /* The other values in restart are already filled in */
        return -ERESTART_RESTARTBLOCK;
 }
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to