Re: [Mesa-dev] [PATCH] util: Don't block SIGSYS for new threads

2019-02-25 Thread Marek Olšák 
Reviewed-by: Marek Olšák 

Marek

On Sat, Feb 23, 2019 at 2:05 AM Drew Davenport 
wrote:

> SIGSYS is needed for programs using seccomp for sandboxing.
> ---
>  src/util/u_thread.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/util/u_thread.h b/src/util/u_thread.h
> index 7538d7d634b2..a46c18d3db20 100644
> --- a/src/util/u_thread.h
> +++ b/src/util/u_thread.h
> @@ -44,7 +44,8 @@ static inline thrd_t u_thread_create(int (*routine)(void
> *), void *param)
> int ret;
>
> sigfillset(_set);
> -   pthread_sigmask(SIG_SETMASK, _set, _set);
> +   sigdelset(_set, SIGSYS);
> +   pthread_sigmask(SIG_BLOCK, _set, _set);
> ret = thrd_create( , routine, param );
> pthread_sigmask(SIG_SETMASK, _set, NULL);
>  #else
> --
> 2.20.1
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] util: Don't block SIGSYS for new threads

2019-02-22 Thread Drew Davenport 
SIGSYS is needed for programs using seccomp for sandboxing.
---
 src/util/u_thread.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/util/u_thread.h b/src/util/u_thread.h
index 7538d7d634b2..a46c18d3db20 100644
--- a/src/util/u_thread.h
+++ b/src/util/u_thread.h
@@ -44,7 +44,8 @@ static inline thrd_t u_thread_create(int (*routine)(void *), 
void *param)
int ret;
 
sigfillset(_set);
-   pthread_sigmask(SIG_SETMASK, _set, _set);
+   sigdelset(_set, SIGSYS);
+   pthread_sigmask(SIG_BLOCK, _set, _set);
ret = thrd_create( , routine, param );
pthread_sigmask(SIG_SETMASK, _set, NULL);
 #else
-- 
2.20.1

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev