diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2004-10-01 11:43:19 +0000 |
|---|---|---|
| committer | Jakub Jelinek <jakub@redhat.com> | 2004-10-01 11:43:19 +0000 |
| commit | b7a09b9201b09b722209cb430813709505927be0 (patch) | |
| tree | d42f6b7a1beba0394311fc49c20b7f81f678cab4 /nptl | |
| parent | 6a87697239310dd529781d9db1ee705eeaa5405e (diff) | |
Updated to fedora-glibc-20041001T1134cvs/fedora-glibc-2_3_3-62
Diffstat (limited to 'nptl')
| -rw-r--r-- | nptl/ChangeLog | 10 | ||||
| -rw-r--r-- | nptl/sysdeps/unix/sysv/linux/i386/i486/lowlevellock.S | 9 | ||||
| -rw-r--r-- | nptl/sysdeps/unix/sysv/linux/x86_64/lowlevellock.S | 18 |
3 files changed, 28 insertions, 9 deletions
diff --git a/nptl/ChangeLog b/nptl/ChangeLog index aba6fc4661..da7d842b36 100644 --- a/nptl/ChangeLog +++ b/nptl/ChangeLog @@ -1,3 +1,13 @@ +2004-10-01 Ulrich Drepper <drepper@redhat.com> + + * sysdeps/unix/sysv/linux/i386/i486/lowlevellock.S + (__lll_mutex_timedlock_wait): I woken but cannot get the lock, + make sure 2 is stored in the futex and we looked at the old value. + * sysdeps/unix/sysv/linux/x86_64/lowlevellock.S + (__lll_mutex_timedlock_wait): Likewise. Fix a few other problems + which might very well made the code not working at all before. + [BZ #417] + 2004-09-28 Ulrich Drepper <drepper@redhat.com> * sysdeps/unix/sysv/linux/pthread_kill.c (__pthread_kill): Don't diff --git a/nptl/sysdeps/unix/sysv/linux/i386/i486/lowlevellock.S b/nptl/sysdeps/unix/sysv/linux/i386/i486/lowlevellock.S index 96ea35caac..dc65b709a1 100644 --- a/nptl/sysdeps/unix/sysv/linux/i386/i486/lowlevellock.S +++ b/nptl/sysdeps/unix/sysv/linux/i386/i486/lowlevellock.S @@ -132,9 +132,8 @@ __lll_mutex_timedlock_wait: ENTER_KERNEL movl %eax, %ecx -8: +8: /* NB: %edx == 2 */ xorl %eax, %eax - movl $2, %edx LOCK cmpxchgl %edx, (%ebx) @@ -150,6 +149,12 @@ __lll_mutex_timedlock_wait: /* Check whether the time expired. */ 7: cmpl $-ETIMEDOUT, %ecx je 5f + + /* Make sure the current holder knows we are going to sleep. */ + movl %edx, %eax + xchgl %eax, (%ebx) + testl %eax, %eax + jz 6b jmp 1b 3: movl $EINVAL, %eax diff --git a/nptl/sysdeps/unix/sysv/linux/x86_64/lowlevellock.S b/nptl/sysdeps/unix/sysv/linux/x86_64/lowlevellock.S index cf3dad18fb..6827804f7f 100644 --- a/nptl/sysdeps/unix/sysv/linux/x86_64/lowlevellock.S +++ b/nptl/sysdeps/unix/sysv/linux/x86_64/lowlevellock.S @@ -95,7 +95,7 @@ __lll_mutex_timedlock_wait: movq %rsp, %rdi xorq %rsi, %rsi movq $VSYSCALL_ADDR_vgettimeofday, %rax - /* This is a regular function call, all calleer-save registers + /* This is a regular function call, all caller-save registers might be clobbered. */ callq *%rax @@ -132,15 +132,13 @@ __lll_mutex_timedlock_wait: syscall movq %rax, %rcx - movl $1, %eax +8: /* NB: %edx == 2 */ + xorl %eax, %eax LOCK cmpxchgl %edx, (%rdi) jnz 7f - movl $2, (%rdi) - xorl %eax, %eax - -8: addq $16, %rsp +6: addq $16, %rsp popq %r14 popq %r13 popq %r12 @@ -151,13 +149,19 @@ __lll_mutex_timedlock_wait: /* Check whether the time expired. */ 7: cmpq $-ETIMEDOUT, %rcx je 5f + + /* Make sure the current holder knows we are going to sleep. */ + movl %edx, %eax + xchgl %eax, (%ebx) + testl %eax, %eax + jz 6b jmp 1b 3: movl $EINVAL, %eax retq 5: movl $ETIMEDOUT, %eax - jmp 8b + jmp 6b .size __lll_mutex_timedlock_wait,.-__lll_mutex_timedlock_wait #endif |