Yep, sorry, my mistake. I suggest make it a relative "struct timespec
*" (more futureproof that timeval). It would make sense to split the
interface into futex_down and futex_up syuscalls, since futex_up
doesn't need a timeout arg, but I haven't for the moment.
(Untested) patch below,
Rusty.
-- Anyone who quotes me in their sig is an idiot. -- Rusty Russell.diff -urN -I \$.*\$ --exclude TAGS -X /home/rusty/devel/kernel/kernel-patches/current-dontdiff --minimal linux-2.5.7-pre1/kernel/futex.c working-2.5.7-pre1-futex-timeout/kernel/futex.c --- linux-2.5.7-pre1/kernel/futex.c Wed Mar 13 13:30:39 2002 +++ working-2.5.7-pre1-futex-timeout/kernel/futex.c Fri Mar 15 16:37:14 2002 @@ -32,6 +32,7 @@ #include <linux/fs.h> #include <linux/futex.h> #include <linux/highmem.h> +#include <linux/time.h> #include <asm/atomic.h> /* These mutexes are a very simple counter: the winner is the one who @@ -146,10 +147,19 @@ } /* Simplified from arch/ppc/kernel/semaphore.c: Paul M. is a genius. */ -static int futex_down(struct list_head *head, struct page *page, int offset) +static int futex_down(struct list_head *head, struct page *page, int offset, + struct timespec *utime) { int retval = 0; struct futex_q q; + unsigned long time = MAX_SCHEDULE_TIMEOUT; + + if (utime) { + struct timespec t; + if (copy_from_user(&t, utime, sizeof(t)) != 0) + return -EFAULT; + time = timespec_to_jiffies(&t) + 1; + } current->state = TASK_INTERRUPTIBLE; queue_me(head, &q, page, offset); @@ -159,13 +169,17 @@ retval = -EINTR; break; } - schedule(); + time = schedule_timeout(time); + if (time == 0) { + retval = -ETIMEDOUT; + break; + } current->state = TASK_INTERRUPTIBLE; } current->state = TASK_RUNNING; unqueue_me(&q); - /* If we were signalled, we might have just been woken: we - must wake another one. Otherwise we need to wake someone + /* If we were signalled, or timed out, we might have just been woken: + we must wake another one. Otherwise we need to wake someone else (if they are waiting) so they drop the count below 0, and when we "up" in userspace, we know there is a waiter. */ @@ -185,7 +199,7 @@ return 0; } -asmlinkage int sys_futex(void *uaddr, int op) +asmlinkage int sys_futex_down(void *uaddr, int op, struct timespec *utime) { int ret; unsigned long pos_in_page; @@ -210,7 +224,7 @@ ret = futex_up(head, page, pos_in_page); break; case FUTEX_DOWN: - ret = futex_down(head, page, pos_in_page); + ret = futex_down(head, page, pos_in_page, utime); break; /* Add other lock types here... */ default: - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/