diff mbox

[roland/nptl-sparc] Move SPARC code out of nptl/sysdeps/sparc/.

Message ID 20140612213633.3E65B2C39BF@topped-with-meat.com
State New
Headers show

Commit Message

Roland McGrath June 12, 2014, 9:36 p.m. UTC
Not tested at all, but this should not change anything.
Note it presumes the previously-posted patches on the same branch.


Thanks,
Roland


	* nptl/sysdeps/sparc/Makefile: Moved ...
	* sysdeps/sparc/nptl/Makefile: ... here.
	* nptl/sysdeps/sparc/tcb-offsets.sym: Moved ...
	* sysdeps/sparc/nptl/tcb-offsets.sym: ... here.
	* nptl/sysdeps/sparc/tls.h: Moved ...
	* sysdeps/sparc/nptl/tls.h: ... here.
	* nptl/sysdeps/sparc/sparc32/pthread_spin_lock.S: Moved ...
	* sysdeps/sparc/sparc32/nptl/pthread_spin_lock.S: ... here.
	* nptl/sysdeps/sparc/sparc32/pthread_spin_trylock.S: Moved ...
	* sysdeps/sparc/sparc32/nptl/pthread_spin_trylock.S: ... here.
	* nptl/sysdeps/sparc/sparc32/pthreaddef.h: Moved ...
	* sysdeps/sparc/sparc32/nptl/pthreaddef.h: ... here.
	* nptl/sysdeps/sparc/sparc64/pthread_spin_init.c: Moved ...
	* sysdeps/sparc/sparc64/nptl/pthread_spin_init.c: ... here.
	* nptl/sysdeps/sparc/sparc64/pthread_spin_lock.S: Moved ...
	* sysdeps/sparc/sparc64/nptl/pthread_spin_lock.S: ... here.
	* nptl/sysdeps/sparc/sparc64/pthread_spin_trylock.S: Moved ...
	* sysdeps/sparc/sparc64/nptl/pthread_spin_trylock.S: ... here.
	* nptl/sysdeps/sparc/sparc64/pthread_spin_unlock.S: Moved ...
	* sysdeps/sparc/sparc64/nptl/pthread_spin_unlock.S: ... here.
	* nptl/sysdeps/sparc/sparc64/pthreaddef.h: Moved ...
	* sysdeps/sparc/sparc64/nptl/pthreaddef.h: ... here.
	* nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_init.c: Moved ...
	* sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_init.c: ... here.
	Update #include.
	* nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_lock.S: Moved ...
	* sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_lock.S: ... here.
	Update #include.
	* nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_trylock.S: Moved ...
	* sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_trylock.S: ... here.
	Update #include.
	* nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_unlock.S: Moved ...
	* sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_unlock.S: ... here.
	Update #include.

Comments

David Miller June 12, 2014, 10:24 p.m. UTC | #1
From: Roland McGrath <roland@hack.frob.com>
Date: Thu, 12 Jun 2014 14:36:33 -0700 (PDT)

> Not tested at all, but this should not change anything.
> Note it presumes the previously-posted patches on the same branch.

Sorry for not reviewing this earlier, I planned to do it earlier today.

It looks fine to me, and if there should be some fallout I'll fix it up
for you.
Roland McGrath June 12, 2014, 10:32 p.m. UTC | #2
> Sorry for not reviewing this earlier, I planned to do it earlier today.
> 
> It looks fine to me, and if there should be some fallout I'll fix it up
> for you.

Thanks.  I've merged the branch, getting these four commits:

	febf6cc Move SPARC public headers out of nptl/
	e268224 Move SPARC code out of nptl/sysdeps/sparc/.
	b0725f4 SPARC: Define TLS_DEFINE_INIT_TP
	b14368b SPARC: Convert fork.c to arch-fork.h

What would be really great is if you could RSN do the clone and vfork
consolidation as most other machines have done now.  That should leave only
the simple renames left to entirely get rid of
nptl/sysdeps/unix/sysv/linux/sparc/.


Thanks,
Roland
David Miller June 12, 2014, 10:58 p.m. UTC | #3
From: Roland McGrath <roland@hack.frob.com>
Date: Thu, 12 Jun 2014 15:32:22 -0700 (PDT)

>> Sorry for not reviewing this earlier, I planned to do it earlier today.
>> 
>> It looks fine to me, and if there should be some fallout I'll fix it up
>> for you.
> 
> Thanks.  I've merged the branch, getting these four commits:
> 
> 	febf6cc Move SPARC public headers out of nptl/
> 	e268224 Move SPARC code out of nptl/sysdeps/sparc/.
> 	b0725f4 SPARC: Define TLS_DEFINE_INIT_TP
> 	b14368b SPARC: Convert fork.c to arch-fork.h
> 
> What would be really great is if you could RSN do the clone and vfork
> consolidation as most other machines have done now.  That should leave only
> the simple renames left to entirely get rid of
> nptl/sysdeps/unix/sysv/linux/sparc/.

Ok, I just have to deal with this link failure first:

/home/davem/src/GIT/GLIBC/build-sparcv9/libc_pic.os: In function `fork_handler_alloc':
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:71: undefined reference to `__fork_handlers'
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:71: undefined reference to `__fork_handlers'
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:71: undefined reference to `__fork_handlers'
/home/davem/src/GIT/GLIBC/build-sparcv9/libc_pic.os: In function `__linkin_atfork':
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:114: undefined reference to `__fork_handlers'
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:114: undefined reference to `__fork_handlers'
/home/davem/src/GIT/GLIBC/build-sparcv9/libc_pic.os:/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:114: more un\
defined references to `__fork_handlers' follow
/home/davem/src/GIT/GLIBC/build-sparcv9/libc_pic.os: In function `__libc_pthread_init':
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/libc_pthread_init.c:44: undefined reference to `__fork_generation_pointer'
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/libc_pthread_init.c:44: undefined reference to `__fork_generation_pointer'
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/libc_pthread_init.c:44: undefined reference to `__fork_generation_pointer'
/home/davem/src/GIT/GLIBC/build-sparcv9/libc_pic.os: In function `free_mem':
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:128: undefined reference to `__fork_handlers'
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:128: undefined reference to `__fork_handlers'
/home/davem/src/GIT/GLIBC/glibc/nptl/../nptl/sysdeps/unix/sysv/linux/register-atfork.c:128: undefined reference to `__fork_handlers'
/usr/bin/ld: /home/davem/src/GIT/GLIBC/build-sparcv9/libc.so: hidden symbol `__fork_generation_pointer' isn't defined
/usr/bin/ld: final link failed: Bad value
collect2: ld returned 1 exit status
David Miller June 12, 2014, 11:18 p.m. UTC | #4
From: David Miller <davem@davemloft.net>
Date: Thu, 12 Jun 2014 15:58:20 -0700 (PDT)

> From: Roland McGrath <roland@hack.frob.com>
> Date: Thu, 12 Jun 2014 15:32:22 -0700 (PDT)
> 
>>> Sorry for not reviewing this earlier, I planned to do it earlier today.
>>> 
>>> It looks fine to me, and if there should be some fallout I'll fix it up
>>> for you.
>> 
>> Thanks.  I've merged the branch, getting these four commits:
>> 
>> 	febf6cc Move SPARC public headers out of nptl/
>> 	e268224 Move SPARC code out of nptl/sysdeps/sparc/.
>> 	b0725f4 SPARC: Define TLS_DEFINE_INIT_TP
>> 	b14368b SPARC: Convert fork.c to arch-fork.h
>> 
>> What would be really great is if you could RSN do the clone and vfork
>> consolidation as most other machines have done now.  That should leave only
>> the simple renames left to entirely get rid of
>> nptl/sysdeps/unix/sysv/linux/sparc/.
> 
> Ok, I just have to deal with this link failure first:

It seems that removing sysdeps/unix/sysv/linux/sparc/fork.S fixes
the problem, doing full 32-bit and 64-bit test builds now.
Joseph Myers June 12, 2014, 11:49 p.m. UTC | #5
On Thu, 12 Jun 2014, Roland McGrath wrote:

> What would be really great is if you could RSN do the clone and vfork
> consolidation as most other machines have done now.  That should leave only

If there are particular changes in this series you'd like machine 
maintainers to do rather than planning to do yourself, please put a 
description of them (with links to any relevant further details) on the 
PortStatus wiki page, listing the machines for which it is applicable.  
My presumption is that if you haven't put something there but have done 
the change for some machines, you intend to do it for other machines as 
well yourself (and that this applies generally when anyone is making such 
cross-machine changes for which it's predictable what machines need 
updates - the person making the changes will either do them for all 
machines, or add details to PortStatus).
Roland McGrath June 12, 2014, 11:52 p.m. UTC | #6
> It seems that removing sysdeps/unix/sysv/linux/sparc/fork.S fixes
> the problem, doing full 32-bit and 64-bit test builds now.

Oh, yeah.  That is something that would have been handled by doing the fork
consolidation first.
Roland McGrath June 12, 2014, 11:53 p.m. UTC | #7
I had planned to at least make the attempt to do all those changes myself
just so as not to make too much work for people and then harangue them
about doing it for me.  Some machine maintainers have beat me to the punch,
which is great.  Dave appeared somewhat eager to help, so I thought he
might take it on.
David Miller June 13, 2014, 1:31 a.m. UTC | #8
From: Roland McGrath <roland@hack.frob.com>
Date: Thu, 12 Jun 2014 16:52:06 -0700 (PDT)

>> It seems that removing sysdeps/unix/sysv/linux/sparc/fork.S fixes
>> the problem, doing full 32-bit and 64-bit test builds now.
> 
> Oh, yeah.  That is something that would have been handled by doing the fork
> consolidation first.

I think I have the consolidation done properly now too, will commit after
build/check runs complete.
Roland McGrath June 13, 2014, 11:46 p.m. UTC | #9
Looks great!

Thanks,
Roland
diff mbox

Patch

diff --git a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_init.c b/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_init.c
deleted file mode 100644
index d27e550..0000000
--- a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_init.c
+++ /dev/null
@@ -1 +0,0 @@ 
-#include <sparc64/pthread_spin_init.c>
diff --git a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_lock.S b/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_lock.S
deleted file mode 100644
index d33b375..0000000
--- a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_lock.S
+++ /dev/null
@@ -1 +0,0 @@ 
-#include <sparc64/pthread_spin_lock.S>
diff --git a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_trylock.S b/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_trylock.S
deleted file mode 100644
index af5b151..0000000
--- a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_trylock.S
+++ /dev/null
@@ -1 +0,0 @@ 
-#include <sparc64/pthread_spin_trylock.S>
diff --git a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_unlock.S b/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_unlock.S
deleted file mode 100644
index c40d80f..0000000
--- a/nptl/sysdeps/sparc/sparc32/sparcv9/pthread_spin_unlock.S
+++ /dev/null
@@ -1 +0,0 @@ 
-#include <sparc64/pthread_spin_unlock.S>
diff --git a/nptl/sysdeps/sparc/Makefile b/sysdeps/sparc/nptl/Makefile
similarity index 100%
rename from nptl/sysdeps/sparc/Makefile
rename to sysdeps/sparc/nptl/Makefile
diff --git a/nptl/sysdeps/sparc/tcb-offsets.sym b/sysdeps/sparc/nptl/tcb-offsets.sym
similarity index 100%
rename from nptl/sysdeps/sparc/tcb-offsets.sym
rename to sysdeps/sparc/nptl/tcb-offsets.sym
diff --git a/nptl/sysdeps/sparc/tls.h b/sysdeps/sparc/nptl/tls.h
similarity index 100%
rename from nptl/sysdeps/sparc/tls.h
rename to sysdeps/sparc/nptl/tls.h
diff --git a/nptl/sysdeps/sparc/sparc32/pthread_spin_lock.S b/sysdeps/sparc/sparc32/nptl/pthread_spin_lock.S
similarity index 100%
rename from nptl/sysdeps/sparc/sparc32/pthread_spin_lock.S
rename to sysdeps/sparc/sparc32/nptl/pthread_spin_lock.S
diff --git a/nptl/sysdeps/sparc/sparc32/pthread_spin_trylock.S b/sysdeps/sparc/sparc32/nptl/pthread_spin_trylock.S
similarity index 100%
rename from nptl/sysdeps/sparc/sparc32/pthread_spin_trylock.S
rename to sysdeps/sparc/sparc32/nptl/pthread_spin_trylock.S
diff --git a/nptl/sysdeps/sparc/sparc32/pthreaddef.h b/sysdeps/sparc/sparc32/nptl/pthreaddef.h
similarity index 100%
rename from nptl/sysdeps/sparc/sparc32/pthreaddef.h
rename to sysdeps/sparc/sparc32/nptl/pthreaddef.h
diff --git a/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_init.c b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_init.c
new file mode 100644
index 0000000..c8edcb0
--- /dev/null
+++ b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_init.c
@@ -0,0 +1 @@ 
+#include <sysdeps/sparc/sparc64/nptl/pthread_spin_init.c>
diff --git a/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_lock.S b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_lock.S
new file mode 100644
index 0000000..f2b898d
--- /dev/null
+++ b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_lock.S
@@ -0,0 +1 @@ 
+#include <sysdeps/sparc/sparc64/nptl/pthread_spin_lock.S>
diff --git a/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_trylock.S b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_trylock.S
new file mode 100644
index 0000000..b324906
--- /dev/null
+++ b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_trylock.S
@@ -0,0 +1 @@ 
+#include <sysdeps/sparc/sparc64/nptl/pthread_spin_trylock.S>
diff --git a/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_unlock.S b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_unlock.S
new file mode 100644
index 0000000..8c56ebe
--- /dev/null
+++ b/sysdeps/sparc/sparc32/sparcv9/nptl/pthread_spin_unlock.S
@@ -0,0 +1 @@ 
+#include <sysdeps/sparc/sparc64/nptl/pthread_spin_unlock.S>
diff --git a/nptl/sysdeps/sparc/sparc64/pthread_spin_init.c b/sysdeps/sparc/sparc64/nptl/pthread_spin_init.c
similarity index 100%
rename from nptl/sysdeps/sparc/sparc64/pthread_spin_init.c
rename to sysdeps/sparc/sparc64/nptl/pthread_spin_init.c
diff --git a/nptl/sysdeps/sparc/sparc64/pthread_spin_lock.S b/sysdeps/sparc/sparc64/nptl/pthread_spin_lock.S
similarity index 100%
rename from nptl/sysdeps/sparc/sparc64/pthread_spin_lock.S
rename to sysdeps/sparc/sparc64/nptl/pthread_spin_lock.S
diff --git a/nptl/sysdeps/sparc/sparc64/pthread_spin_trylock.S b/sysdeps/sparc/sparc64/nptl/pthread_spin_trylock.S
similarity index 100%
rename from nptl/sysdeps/sparc/sparc64/pthread_spin_trylock.S
rename to sysdeps/sparc/sparc64/nptl/pthread_spin_trylock.S
diff --git a/nptl/sysdeps/sparc/sparc64/pthread_spin_unlock.S b/sysdeps/sparc/sparc64/nptl/pthread_spin_unlock.S
similarity index 100%
rename from nptl/sysdeps/sparc/sparc64/pthread_spin_unlock.S
rename to sysdeps/sparc/sparc64/nptl/pthread_spin_unlock.S
diff --git a/nptl/sysdeps/sparc/sparc64/pthreaddef.h b/sysdeps/sparc/sparc64/nptl/pthreaddef.h
similarity index 100%
rename from nptl/sysdeps/sparc/sparc64/pthreaddef.h
rename to sysdeps/sparc/sparc64/nptl/pthreaddef.h