Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 13 Aug 2004 22:06:42 GMT
From:      David Xu <davidxu@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 59594 for review
Message-ID:  <200408132206.i7DM6gpM006439@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=59594

Change 59594 by davidxu@davidxu_alona on 2004/08/13 22:06:25

	Call _set_curthread in _thread_start to set arch dependent environment for
	current thread, this is needed on AMD64.

Affected files ...

.. //depot/projects/davidxu_ksedbg/src/lib/libthr/thread/thr_create.c#5 edit
.. //depot/projects/davidxu_ksedbg/src/lib/libthr/thread/thr_private.h#5 edit

Differences ...

==== //depot/projects/davidxu_ksedbg/src/lib/libthr/thread/thr_create.c#5 (text+ko) ====

@@ -99,9 +99,10 @@
 
 	/* Initialise the machine context: */
 	getcontext(&new_thread->ctx);
+	new_thread->savedsig = new_thread->ctx.uc_sigmask;
 	new_thread->ctx.uc_stack.ss_sp = new_thread->stack;
 	new_thread->ctx.uc_stack.ss_size = pattr->stacksize_attr;
-	makecontext(&new_thread->ctx, _thread_start, 0);
+	makecontext(&new_thread->ctx, (void (*)(void))_thread_start, 1, new_thread);
 	new_thread->arch_id = _set_curthread(&new_thread->ctx, new_thread, &ret);
 	if (ret != 0) {
 		if (pattr->stackaddr_attr == NULL) {
@@ -145,7 +146,11 @@
 	 */
 	if (pattr->suspend == PTHREAD_CREATE_SUSPENDED)
 		new_thread->flags |= PTHREAD_FLAGS_SUSPENDED;
+	/* new thread inherits signal mask in kernel */
+	_thread_sigblock();
 	ret = thr_create(&new_thread->ctx, &new_thread->thr_id, flags);
+	/* restore my signal mask */
+	_thread_sigunblock();
 	if (ret != 0) {
 		_thread_printf(STDERR_FILENO, "thr_create() == %d\n", ret);
 		PANIC("thr_create");
@@ -160,10 +165,15 @@
 }
 
 void
-_thread_start(void)
+_thread_start(pthread_t td)
 {
+	int ret;
+
+	_set_curthread(NULL, td, &ret);
+	__sys_sigprocmask(SIG_SETMASK, &td->savedsig, NULL);
 	if ((curthread->flags & PTHREAD_FLAGS_SUSPENDED) != 0)
 		_thread_suspend(curthread, NULL);
+
 	pthread_exit(curthread->start_routine(curthread->arg));
 
 	/* This point should never be reached. */

==== //depot/projects/davidxu_ksedbg/src/lib/libthr/thread/thr_private.h#5 (text+ko) ====

@@ -797,7 +797,7 @@
 void    _thread_dump_info(void);
 void    _thread_init(void);
 void	_thread_printf(int fd, const char *, ...);
-void    _thread_start(void);
+void    _thread_start(pthread_t);
 void	_thread_seterrno(pthread_t, int);
 void	_thread_enter_cancellation_point(void);
 void	_thread_leave_cancellation_point(void);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200408132206.i7DM6gpM006439>