|
yann@1
|
1 |
ML: http://sources.redhat.com/ml/libc-hacker/2005-09/msg00002.html
|
|
yann@1
|
2 |
|
|
yann@1
|
3 |
Date: Mon, 05 Sep 2005 21:07:15 +0900 (JST)
|
|
yann@1
|
4 |
Message-Id: <20050905.210715.15267870.kkojima@rr.iij4u.or.jp>
|
|
yann@1
|
5 |
To: libc-hacker at sources dot redhat dot com
|
|
yann@1
|
6 |
Subject: SH: A typo in lowlevellock.S
|
|
yann@1
|
7 |
From: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
|
|
yann@1
|
8 |
|
|
yann@1
|
9 |
Hi,
|
|
yann@1
|
10 |
|
|
yann@1
|
11 |
The appended patch fixes a typo in a low-level lock function. It
|
|
yann@1
|
12 |
set the correct 3rd argument for the futex syscall in loop. Sorry
|
|
yann@1
|
13 |
for missing such an embarrassing bug.
|
|
yann@1
|
14 |
|
|
yann@1
|
15 |
Regards,
|
|
yann@1
|
16 |
kaz
|
|
yann@1
|
17 |
|
|
yann@1
|
18 |
nptl/ChangeLog:
|
|
yann@1
|
19 |
2005-09-05 Kaz Kojima <kkojima@rr.iij4u.or.jp>
|
|
yann@1
|
20 |
|
|
yann@1
|
21 |
* sysdeps/unix/sysv/linux/sh/lowlevellock.S (__lll_mutex_lock_wait):
|
|
yann@1
|
22 |
Fix typo in register name.
|
|
yann@1
|
23 |
|
|
yann@1
|
24 |
--- glibc.old/nptl/sysdeps/unix/sysv/linux/sh/lowlevellock.S 2004-10-26 04:06:44.000000000 +0900
|
|
yann@1
|
25 |
+++ glibc/nptl/sysdeps/unix/sysv/linux/sh/lowlevellock.S 2005-09-05 19:18:25.000000000 +0900
|
|
yann@1
|
26 |
@@ -1,4 +1,4 @@
|
|
yann@1
|
27 |
-/* Copyright (C) 2003, 2004 Free Software Foundation, Inc.
|
|
yann@1
|
28 |
+/* Copyright (C) 2003, 2004, 2005 Free Software Foundation, Inc.
|
|
yann@1
|
29 |
This file is part of the GNU C Library.
|
|
yann@1
|
30 |
|
|
yann@1
|
31 |
The GNU C Library is free software; you can redistribute it and/or
|
|
yann@1
|
32 |
@@ -51,8 +51,8 @@ __lll_mutex_lock_wait:
|
|
yann@1
|
33 |
SYSCALL_INST_PAD
|
|
yann@1
|
34 |
|
|
yann@1
|
35 |
2:
|
|
yann@1
|
36 |
- mov #2, r4
|
|
yann@1
|
37 |
- XCHG (r4, @r8, r2)
|
|
yann@1
|
38 |
+ mov #2, r6
|
|
yann@1
|
39 |
+ XCHG (r6, @r8, r2)
|
|
yann@1
|
40 |
tst r2, r2
|
|
yann@1
|
41 |
bf 1b
|
|
yann@1
|
42 |
|
|
yann@1
|
43 |
Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
|