Closed Bug 1350955 Opened 5 years ago Closed 5 years ago
Enable adaptive mutex on Free
59 bytes, text/x-review-board-request
PTHREAD_MUTEX_ADAPTIVE_NP is supported since FreeBSD 8.0. Getting patches accepted in NSPR is too hard: very few reviewers + a lot of cruft. I'll try but let's not block on that. https://github.com/freebsd/freebsd/commit/bbfd76f8722c569a2be8de7fd01318fa54fa750c
Comment on attachment 8851631 [details] Bug 1350955 - Enable adaptive mutex usage on FreeBSD. https://reviewboard.mozilla.org/r/123902/#review126404 Cool, I didn't know FreeBSD supported these. ::: mozglue/misc/Mutex_posix.cpp:30 (Diff revision 1) > // glibc's adaptive mutexes spin for a short number of tries before sleeping. > // NSPR's locks did this, too, and it seems like a reasonable thing to do. Please modify this comment to read: "Linux with glibc and FreedBSD support adaptive mutexes that spin for a short number of tries before sleeping. NSPR's locks did this, too, and it seems like a reasonable thing to do."
Attachment #8851631 - Flags: review?(nfroyd) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/f6fbee70b32c Enable adaptive mutex usage on FreeBSD. r=froydnj
You need to log in before you can comment on or make changes to this bug.