projects / external / glibc.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 05f4d5c)
Update.
authorUlrich Drepper <drepper@redhat.com>
Fri, 10 Sep 2004 21:18:48 +0000 (21:18 +0000)
committerUlrich Drepper <drepper@redhat.com>
Fri, 10 Sep 2004 21:18:48 +0000 (21:18 +0000)
2004-09-10  Kaz Kojima  <kkojima@rr.iij4u.or.jp>

* sysdeps/unix/sysv/linux/sh/pthread_cond_wait.S: Decrement
__nwaiters.  If pthread_cond_destroy has been called and this is
the last waiter, signal pthread_cond_destroy caller and avoid
using the pthread_cond_t structure after unlock.
* sysdeps/unix/sysv/linux/sh/pthread_cond_timedwait.S: Likewise.

ChangeLog patch | blob | history
nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_timedwait.S patch | blob | history
nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_wait.S patch | blob | history

diff --git a/ChangeLog b/ChangeLog
index 30ae386..8d3e98e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2004-09-10  Kaz Kojima  <kkojima@rr.iij4u.or.jp>
+
+       * sysdeps/unix/sysv/linux/sh/pthread_cond_wait.S: Decrement
+       __nwaiters.  If pthread_cond_destroy has been called and this is
+       the last waiter, signal pthread_cond_destroy caller and avoid
+       using the pthread_cond_t structure after unlock.
+       * sysdeps/unix/sysv/linux/sh/pthread_cond_timedwait.S: Likewise.
+
 2004-09-10  Ulrich Drepper  <drepper@redhat.com>
 
        * sysdeps/unix/sysv/linux/kernel-features.h: Don't define
diff --git a/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_timedwait.S b/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_timedwait.S
index 7694f36..d0c55ec 100644 (file)
--- a/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_timedwait.S
+++ b/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_timedwait.S
@@ -65,9 +65,10 @@ __pthread_cond_timedwait:
        mov.l   @(4,r13), r0
        mov.l   .L1g, r1
        cmp/hs  r1, r0
-       bt/s    18f
+       bf      0f
+       bra     18f
         mov    #EINVAL, r0
-
+0:
        /* Get internal lock.  */
        mov     #0, r3
        mov     #1, r4
@@ -119,8 +120,11 @@ __pthread_cond_timedwait:
        mov.l   @(cond_futex,r8), r0
        add     r2, r0
        mov.l   r0, @(cond_futex,r8)
+       mov     #(1 << clock_bits), r2
+       mov.l   @(cond_nwaiters,r8), r0
+       add     r2, r0
+       mov.l   r0, @(cond_nwaiters,r8)
        
-
        /* Get and store current wakeup_seq value.  */
        mov.l   @(wakeup_seq,r8), r10
        mov.l   @(wakeup_seq+4,r8), r11
@@ -271,7 +275,36 @@ __pthread_cond_timedwait:
        mov.l   r0,@(woken_seq,r8)
        mov.l   r1,@(woken_seq+4,r8)
 
-24:    
+24:
+       mov     #(1 << clock_bits), r2
+       mov.l   @(cond_nwaiters,r8),r0
+       sub     r2, r0
+       mov.l   r0,@(cond_nwaiters,r8)
+
+       /* Wake up a thread which wants to destroy the condvar object.  */
+       mov.l   @(total_seq,r8),r0
+       mov.l   @(total_seq+4,r8),r1
+       and     r1, r0
+       not     r0, r0
+       cmp/eq  #0, r0
+       bf/s    25f
+        mov    #((1 << clock_bits) - 1), r1
+       not     r1, r1
+       mov.l   @(cond_nwaiters,r8),r0
+       tst     r1, r0
+       bf      25f
+
+       mov     r8, r4
+       add     #cond_nwaiters, r4
+       mov     #FUTEX_WAKE, r5
+       mov     #1, r6
+       mov     #0, r7
+       mov     #SYS_futex, r3
+       extu.b  r3, r3
+       trapa   #0x14
+       SYSCALL_INST_PAD
+       
+25:    
 #if cond_lock != 0
        DEC (@(cond_lock,r8), r2)
 #else
@@ -461,6 +494,37 @@ __condvar_tw_cleanup:
        mov.l   r1,@(woken_seq+4,r8)
 
 3:
+       mov     #(1 << clock_bits), r2
+       mov.l   @(cond_nwaiters,r8),r0
+       sub     r2, r0
+       mov.l   r0,@(cond_nwaiters,r8)
+
+       /* Wake up a thread which wants to destroy the condvar object.  */
+       mov     #0, r10
+       mov.l   @(total_seq,r8),r0
+       mov.l   @(total_seq+4,r8),r1
+       and     r1, r0
+       not     r0, r0
+       cmp/eq  #0, r0
+       bf/s    4f
+        mov    #((1 << clock_bits) - 1), r1
+       not     r1, r1
+       mov.l   @(cond_nwaiters,r8),r0
+       tst     r1, r0
+       bf      4f
+
+       mov     r8, r4
+       add     #cond_nwaiters, r4
+       mov     #FUTEX_WAKE, r5
+       mov     #1, r6
+       mov     #0, r7
+       mov     #SYS_futex, r3
+       extu.b  r3, r3
+       trapa   #0x14
+       SYSCALL_INST_PAD
+       mov     #1, r10
+
+4:
 #if cond_lock != 0
        DEC (@(cond_lock,r8), r2)
 #else
@@ -480,7 +544,9 @@ __condvar_tw_cleanup:
 
 2:
        /* Wake up all waiters to make sure no signal gets lost.  */
-       mov     r8, r4
+       tst     r10, r10
+       bf/s    5f
+        mov    r8, r4
        add     #cond_futex, r4
        mov     #FUTEX_WAKE, r5
        mov     #-1, r6
@@ -491,6 +557,7 @@ __condvar_tw_cleanup:
        trapa   #0x14
        SYSCALL_INST_PAD
 
+5:
         mov.l   .Lmlocki5, r1
         bsrf    r1
          mov     r9, r4
diff --git a/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_wait.S b/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_wait.S
index b9190ab..2d6b685 100644 (file)
--- a/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_wait.S
+++ b/nptl/sysdeps/unix/sysv/linux/sh/pthread_cond_wait.S
@@ -107,6 +107,10 @@ __pthread_cond_wait:
        mov.l   @(cond_futex,r8),r0
        add     r2, r0
        mov.l   r0,@(cond_futex,r8)
+       mov     #(1 << clock_bits), r2
+       mov.l   @(cond_nwaiters,r8), r0
+       add     r2, r0
+       mov.l   r0, @(cond_nwaiters,r8)
 
        /* Get and store current wakeup_seq value.  */
        mov.l   @(wakeup_seq,r8), r10
@@ -192,7 +196,36 @@ __pthread_cond_wait:
        mov.l   r0,@(woken_seq,r8)
        mov.l   r1,@(woken_seq+4,r8)
 
-16:    
+16:
+       mov     #(1 << clock_bits), r2
+       mov.l   @(cond_nwaiters,r8),r0
+       sub     r2, r0
+       mov.l   r0,@(cond_nwaiters,r8)
+
+       /* Wake up a thread which wants to destroy the condvar object.  */
+       mov.l   @(total_seq,r8),r0
+       mov.l   @(total_seq+4,r8),r1
+       and     r1, r0
+       not     r0, r0
+       cmp/eq  #0, r0
+       bf/s    17f
+        mov    #((1 << clock_bits) - 1), r1
+       not     r1, r1
+       mov.l   @(cond_nwaiters,r8),r0
+       tst     r1, r0
+       bf      17f
+
+       mov     r8, r4
+       add     #cond_nwaiters, r4
+       mov     #FUTEX_WAKE, r5
+       mov     #1, r6
+       mov     #0, r7
+       mov     #SYS_futex, r3
+       extu.b  r3, r3
+       trapa   #0x14
+       SYSCALL_INST_PAD
+
+17:
 #if cond_lock != 0
        DEC (@(cond_lock,r8), r2)
 #else
@@ -371,7 +404,38 @@ __condvar_w_cleanup:
        mov.l   r0,@(woken_seq,r8)
        mov.l   r1,@(woken_seq+4,r8)
 
-3:     
+3:
+       mov     #(1 << clock_bits), r2
+       mov.l   @(cond_nwaiters,r8),r0
+       sub     r2, r0
+       mov.l   r0,@(cond_nwaiters,r8)
+
+       /* Wake up a thread which wants to destroy the condvar object.  */
+       mov     #0, r10
+       mov.l   @(total_seq,r8),r0
+       mov.l   @(total_seq+4,r8),r1
+       and     r1, r0
+       not     r0, r0
+       cmp/eq  #0, r0
+       bf/s    4f
+        mov    #((1 << clock_bits) - 1), r1
+       not     r1, r1
+       mov.l   @(cond_nwaiters,r8),r0
+       tst     r1, r0
+       bf      4f
+
+       mov     r8, r4
+       add     #cond_nwaiters, r4
+       mov     #FUTEX_WAKE, r5
+       mov     #1, r6
+       mov     #0, r7
+       mov     #SYS_futex, r3
+       extu.b  r3, r3
+       trapa   #0x14
+       SYSCALL_INST_PAD
+       mov     #1, r10
+
+4:
 #if cond_lock != 0
        DEC (@(cond_lock,r8), r2)
 #else
@@ -391,7 +455,9 @@ __condvar_w_cleanup:
 
 2:
        /* Wake up all waiters to make sure no signal gets lost.  */
-       mov     r8, r4
+       tst     r10, r10
+       bf/s    5f
+        mov    r8, r4
        add     #cond_futex, r4
        mov     #FUTEX_WAKE, r5
        mov     #-1, r6
@@ -402,6 +468,7 @@ __condvar_w_cleanup:
        trapa   #0x14
        SYSCALL_INST_PAD
 
+5:
         mov.l   .Lmlocki3, r1
         bsrf    r1
          mov     r9, r4
Domain: System / Toolchain;