Bug 88534

Summary:	include/c11/threads_posix.h PTHREAD_MUTEX_RECURSIVE_NP not defined
Product:	Mesa	Reporter:	felix.janda
Component:	Mesa core	Assignee:	mesa-dev
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium
Version:	git
Hardware:	Other
OS:	Linux (All)
Whiteboard:
i915 platform:		i915 features:
Attachments:	Proposed patch

Description felix.janda 2015-01-17 14:24:48 UTC

Created attachment 112394 [details] [review]
Proposed patch

The non-portable version of PTHREAD_MUTEX_RECURSIVE is used since older glibc didn't have the POSIX version. The attached patch makes the code only fall back to PTHREAD_MUTEX_RECURSIVE_NP if PTHREAD_MUTEX_RECURSIVE is not defined. This fixes compilation with other libcs such as musl, which don't have the nonstandard version.

Comment 1 felix.janda 2015-01-31 18:27:38 UTC

Patch now sent instead to mesa-dev.

Comment 2 Emil Velikov 2015-03-16 21:01:29 UTC

Thanks for the patch. Pushed to master

commit aead7fe2e2b6c89258f80a25299f4ec0fece2d95
Author: Felix Janda <felix.janda@posteo.de>
Date:   Mon Feb 2 20:04:16 2015 +0100

    c11/threads: Use PTHREAD_MUTEX_RECURSIVE by default

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.