Enable adaptive mutex on FreeBSD

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jbeich, Unassigned)

Tracking

Trunk
mozilla55
Unspecified
FreeBSD
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
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 hidden (mozreview-request)
(Reporter)

Updated

2 years ago
Depends on: 1350966
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+
Comment hidden (mozreview-request)
(Reporter)

Updated

2 years ago
Keywords: checkin-needed

Comment 4

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/f6fbee70b32c
Enable adaptive mutex usage on FreeBSD. r=froydnj
Keywords: checkin-needed

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f6fbee70b32c
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox55: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.