[PATCH xserver v2] inputthread: On Linux leave the main thread's name as-is
Peter Hutterer
peter.hutterer at who-t.net
Wed Oct 26 23:53:44 UTC 2016
On Wed, Oct 26, 2016 at 12:21:16PM +0200, Hans de Goede wrote:
> From: Peter Hutterer <peter.hutterer at who-t.net>
>
> On Linux, setting the main thread's name changes the program name
> (/proc/self/comm). Setting it to MainThread breaks scripts that rely on
> the command name, e.g. ps -C Xorg.
>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
pushed, thanks.
007f8ee..5cb3283 master -> master
Cheers,
Peter
> ---
> Changes in v2 (hdegoede):
> -Only leave the main thread as-is in Linux, naming it is not an issue on
> other platforms
> ---
> os/inputthread.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/os/inputthread.c b/os/inputthread.c
> index ddafa7f..8e7f2ed 100644
> --- a/os/inputthread.c
> +++ b/os/inputthread.c
> @@ -431,11 +431,13 @@ InputThreadPreInit(void)
> }
> hotplugPipeWrite = hotplugPipe[1];
>
> +#ifndef __linux__ /* Linux does not deal well with renaming the main thread */
> #if defined(HAVE_PTHREAD_SETNAME_NP_WITH_TID)
> pthread_setname_np (pthread_self(), "MainThread");
> #elif defined(HAVE_PTHREAD_SETNAME_NP_WITHOUT_TID)
> pthread_setname_np ("MainThread");
> #endif
> +#endif
>
> }
>
> --
> 2.9.3
>
More information about the xorg-devel
mailing list