1252246 - Enable the CrossProcessMutex code on more Unices

Status: RESOLVED FIXED

(Core :: IPC, defect)

Description

[Jan Beich]

+++ This bug was initially created as a clone of Bug #1072093 +++

The CrossProcessMutex is used in the APZ/tiling code for implementing progressive painting. On Desktop platforms layers.async-pan-zoom.enabled;true since bug 1178298 while layers.enable-tiles;true is limited to OS X. Let's prepare for layers.progressive-paint;true in advance by always building _posix.cpp where pthread_mutexattr_setpshared() is supported. If PTHREAD_PROCESS_SHARED doesn't work yet it'd only trade NS_RUNTIMEABORT() with MOZ_CRASH().

https://dxr.mozilla.org/mozilla-central/search?q=CrossProcessMutex
https://svnweb.freebsd.org/changeset/base/296162 DragonFly may want to adopt
bug 1072093 comment 16 (simple test program)

Comment 1

[Jan Beich]

Attachment: v1: Solaris, GNU/kFreeBSD, FreeBSD, DragonFly, etc

Partially backs out bug 1098607. According to ipc/chromium/chromium-config.mozbuild anything non-Windows is OS_POSIX, so it can be omitted thus avoiding ugliness in moz.build.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=0715f6d11c4e

Comment 2

[Jan Beich]

Attachment: v1: Solaris, GNU/kFreeBSD, FreeBSD, DragonFly, etc

Oops, forgot to sort OS_ARCH values in moz.build.

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 8724976 [details] [diff] [review]
v1: Solaris, GNU/kFreeBSD, FreeBSD, DragonFly, etc

Review of attachment 8724976 [details] [diff] [review]:
-----------------------------------------------------------------

I have no objections to this, but I'll defer to glandium.

::: ipc/glue/CrossProcessMutex.h
@@ +8,5 @@
>  
>  #include "base/process.h"
>  #include "mozilla/Mutex.h"
>  
> +#if (!defined(OS_NETBSD) && !defined(OS_OPENBSD))

This one needs a !defined(OS_WIN) as well since there's no #if branch for windows like in the other sites.

Comment 4

[Jan Beich]

Attachment: v1.1: Solaris, GNU/kFreeBSD, FreeBSD, DragonFly, etc

Carrying over f=kats after adding missing !defined(OS_WIN) per comment 3. Also, dropped unnecessary parentheses when stacking negation:

  !defined(OS_WIN) && !defined(OS_NETBSD) && !defined(OS_OPENBSD) 

vs.

  defined(OS_POSIX) && (!defined(OS_NETBSD) && !defined(OS_OPENBSD))

https://treeherder.mozilla.org/#/jobs?repo=try&revision=de279a712c9d

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ee9285c872a5

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ee9285c872a5