From 9ba96eda70ecf6f86f74580b23fc387d8fd1cd77 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Wed, 17 Sep 2003 19:15:35 +0000 Subject: Update. * elf/Makefile (distribute): Add tst-alignmod.c. (tests): Add tst-align. (modules-names): Add tst-alignmod. (CFLAGS-tst-align.c, CFLAGS-tst-alignmod.c): Add $(stack-align-test-flags). ($(objpfx)tst-align): Depend on libdl. ($(objpfx)tst-align.out): Depend on tst-alignmod.so. * elf/tst-align.c: New file. * elf/tst-alignmod.c: New file. * misc/Makefile (CFLAGS-tst-tsearch.c): Add $(stack-align-test-flags). * misc/tst-tsearch.c: Include tst-stack-align.h. (stack_align_check): New array. (cmp_fn, walk_action): Use TEST_STACK_ALIGN (). (main): Report error if stack was not enough aligned in cmp_fn or walk_action. * stdlib/Makefile (CFLAGS-tst-qsort.c): Add $(stack-align-test-flags). * stdlib/tst-qsort.c: Include tst-stack-align.h. (align_check): New var. (compare): Use TEST_STACK_ALIGN () macro. (main): Report error if stack was not enough aligned in compare. * sysdeps/i386/i686/Makefile (stack-align-test-flags): Remove unneeded -mpreferred-stack-boundary=4. * sysdeps/x86_64/tst-stack-align.h: New file. * libio/memstream.c (_IO_wmem_jumps): Remove unused structure. (_IO_wmem_sync, _IO_wmem_finish): Remove unused functions. 2003-09-17 Philip Blundell * sysdeps/arm/dl-machine.h (CALL_ROUTINE): Deleted. (BX): Define. (ELF_MACHINE_RUNTIME_TRAMPOLINE): Optimise a little. * sysdeps/arm/sysdep.h (DO_RET): New. * sysdeps/unix/sysv/linux/arm/vfork.S: Add #error if __NR_vfork required but not defined. Improve test of error code. 2003-09-17 Jakub Jelinek --- ChangeLog | 40 ++++++++++++++++++++++ nptl/ChangeLog | 4 +++ nptl/sysdeps/pthread/pthread.h | 4 ++- .../unix/sysv/linux/pthread_attr_setaffinity.c | 18 +++++++--- 4 files changed, 60 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 4ba2bcb282..7de86d1a22 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,43 @@ +2003-09-17 Jakub Jelinek + + * elf/Makefile (distribute): Add tst-alignmod.c. + (tests): Add tst-align. + (modules-names): Add tst-alignmod. + (CFLAGS-tst-align.c, CFLAGS-tst-alignmod.c): Add + $(stack-align-test-flags). + ($(objpfx)tst-align): Depend on libdl. + ($(objpfx)tst-align.out): Depend on tst-alignmod.so. + * elf/tst-align.c: New file. + * elf/tst-alignmod.c: New file. + * misc/Makefile (CFLAGS-tst-tsearch.c): Add $(stack-align-test-flags). + * misc/tst-tsearch.c: Include tst-stack-align.h. + (stack_align_check): New array. + (cmp_fn, walk_action): Use TEST_STACK_ALIGN (). + (main): Report error if stack was not enough aligned + in cmp_fn or walk_action. + * stdlib/Makefile (CFLAGS-tst-qsort.c): Add $(stack-align-test-flags). + * stdlib/tst-qsort.c: Include tst-stack-align.h. + (align_check): New var. + (compare): Use TEST_STACK_ALIGN () macro. + (main): Report error if stack was not enough aligned in compare. + * sysdeps/i386/i686/Makefile (stack-align-test-flags): Remove + unneeded -mpreferred-stack-boundary=4. + * sysdeps/x86_64/tst-stack-align.h: New file. + + * libio/memstream.c (_IO_wmem_jumps): Remove unused structure. + (_IO_wmem_sync, _IO_wmem_finish): Remove unused functions. + +2003-09-17 Philip Blundell + + * sysdeps/arm/dl-machine.h (CALL_ROUTINE): Deleted. + (BX): Define. + (ELF_MACHINE_RUNTIME_TRAMPOLINE): Optimise a little. + + * sysdeps/arm/sysdep.h (DO_RET): New. + + * sysdeps/unix/sysv/linux/arm/vfork.S: Add #error if __NR_vfork + required but not defined. Improve test of error code. + 2003-09-17 Jakub Jelinek * sysdeps/i386/Makefile (stack-align-test-flags): Add diff --git a/nptl/ChangeLog b/nptl/ChangeLog index e7341f1394..bb73d0f20c 100644 --- a/nptl/ChangeLog +++ b/nptl/ChangeLog @@ -1,5 +1,9 @@ 2003-09-17 Jakub Jelinek + * sysdeps/pthread/pthread.h (pthread_getattr_np): Clarify usage. + * sysdeps/unix/sysv/linux/pthread_attr_setaffinity.c + (pthread_attr_setaffinity_np): Handle cpuset == NULL. + * sysdeps/unix/sysv/linux/pthread_attr_getaffinity.c (pthread_attr_getaffinity_np): Don't segfault if iattr->cpuset is NULL. diff --git a/nptl/sysdeps/pthread/pthread.h b/nptl/sysdeps/pthread/pthread.h index af0a22a111..6e8da67753 100644 --- a/nptl/sysdeps/pthread/pthread.h +++ b/nptl/sysdeps/pthread/pthread.h @@ -334,7 +334,9 @@ extern int pthread_attr_getaffinity_np (__const pthread_attr_t *__attr, cpu_set_t *__cpuset) __THROW; -/* Get thread attributes corresponding to the already running thread TH. */ +/* Initialize thread attribute *ATTR with attributes corresponding to the + already running thread TH. It shall be called on unitialized ATTR + and destroyed with pthread_attr_destroy when no longer needed. */ extern int pthread_getattr_np (pthread_t __th, pthread_attr_t *__attr) __THROW; #endif diff --git a/nptl/sysdeps/unix/sysv/linux/pthread_attr_setaffinity.c b/nptl/sysdeps/unix/sysv/linux/pthread_attr_setaffinity.c index f25ccb213e..de8f54fb97 100644 --- a/nptl/sysdeps/unix/sysv/linux/pthread_attr_setaffinity.c +++ b/nptl/sysdeps/unix/sysv/linux/pthread_attr_setaffinity.c @@ -34,14 +34,22 @@ pthread_attr_setaffinity_np (attr, cpuset) assert (sizeof (*attr) >= sizeof (struct pthread_attr)); iattr = (struct pthread_attr *) attr; - if (iattr->cpuset == NULL) + if (cpuset == NULL) { - iattr->cpuset = (cpu_set_t *) malloc (sizeof (cpu_set_t)); - if (iattr->cpuset == NULL) - return ENOMEM; + free (iattr->cpuset); + iattr->cpuset = NULL; } + else + { + if (iattr->cpuset == NULL) + { + iattr->cpuset = (cpu_set_t *) malloc (sizeof (cpu_set_t)); + if (iattr->cpuset == NULL) + return ENOMEM; + } - memcpy (iattr->cpuset, cpuset, sizeof (cpu_set_t)); + memcpy (iattr->cpuset, cpuset, sizeof (cpu_set_t)); + } return 0; } -- cgit 1.4.1