This is the mail archive of the glibc-bugs@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug nptl/21672] sys-libs/glibc on ia64 crashes on thread exit: signal SIGSEGV, Segmentation fault: pthread_create.c:432: __madvise (pd->stackblock, freesize - PTHREAD_STACK_MIN, MADV_DONTNEED);


https://sourceware.org/bugzilla/show_bug.cgi?id=21672

--- Comment #4 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  01b87c656f670863ce437421b8e9278200965d38 (commit)
      from  16f138a49ad1815e113d2b5b7a87f26999ade811 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=01b87c656f670863ce437421b8e9278200965d38

commit 01b87c656f670863ce437421b8e9278200965d38
Author: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Date:   Mon Aug 28 11:24:35 2017 -0300

    ia64: Fix thread stack allocation permission set (BZ #21672)

    This patch fixes ia64 failures on thread exit by madvise the required
    area taking in consideration its disjoing stacks
    (NEED_SEPARATE_REGISTER_STACK).  Also the snippet that setup the
    madvise call to advertise kernel the area won't be used anymore in
    near future is reallocated in allocatestack.c (for consistency to
    put all stack management function in one place).

    Checked on x86_64-linux-gnu and i686-linux-gnu for sanity (since
    it is not expected code changes for architecture that do not
    define NEED_SEPARATE_REGISTER_STACK) and also got a report that
    it fixes ia64-linux-gnu failures from Sergei Trofimovich
    <slyfox@gentoo.org>.

        [BZ #21672]
        * nptl/allocatestack.c [_STACK_GROWS_DOWN] (setup_stack_prot):
        Set to use !NEED_SEPARATE_REGISTER_STACK as well.
        (advise_stack_range): New function.
        * nptl/pthread_create.c (START_THREAD_DEFN): Move logic to mark
        stack non required to advise_stack_range at allocatestack.c

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog             |    9 +++++++++
 nptl/allocatestack.c  |   29 ++++++++++++++++++++++++++++-
 nptl/pthread_create.c |   27 ++-------------------------
 3 files changed, 39 insertions(+), 26 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]