492117 - pthread_condattr_init on darwin

Status: RESOLVED FIXED

(NSPR :: NSPR, defect)

Description

[Jeremy Huddleston]

pthread_condattr_init is available on 10.5 and later

Comment 1

[glen beasley]

jeremy on a 10.5.6 Mac OS X I can "man pthread_condattr_init" and I see 
that it exists. 

I also read:
http://developer.apple.com/MacOsX/multithreadedprogramming.html

Where did you find that pthread_condattr_init is available on 10.5 and later
meaning not on 10.4 or 10.3?

Comment 2

[Jeremy Huddleston]

I didn't say it wasn't available earlier.  In fact, I didn't check.  Right now, the source is never using it at all on darwin.  This patch changes that.

Comment 3

[Jeremy Huddleston]

Attachment: patch-pthread.diff

Comment 4

[Jeremy Huddleston]

Original reports are quite dated.  It looks like it was never reported upstream, so I'm just passing it along:

http://trac.macports.org/attachment/ticket/13150
http://trac.macports.org/attachment/ticket/13150/firefox-leopard.patch

Comment 5

[glen beasley]

Comment on attachment 376499 [details] [diff] [review]
patch-pthread.diff

bug 153525 stated pthread_condattr_init and pthread_kill were not on Darwin in
patch https://bugzilla.mozilla.org/attachment.cgi?id=92470&action=diff

because Mac OS X 10.2 appeared to not have them. 
although according to 
http://developer.apple.com/technotes/tn2002/tn2071.html
pthread_kill was introduced in 10.2 man pthread_kill shows PTHREAD_KILL(2). 

bug 363092 allowed pthread_kill on Darwin.

I think we can assume  pthread_condattr_init(3) was introduced for MAC OS X 10.3?

Comment 6

[Jeremy Huddleston]

That sounds plausible.  It's probably safe just to remove this check entirely and just use pthread_condattr_init unconditionally since 10.2 isn't really supported by anyone at this point...

Comment 7

[Wan-Teh Chang]

Comment on attachment 376499 [details] [diff] [review]
patch-pthread.diff

If all the versions of Mac OS X that Firefox 3.0
supports have pthread_condattr_init, we don't need
any ifdef at all.

The "Firefox 3 System Requirements" page at
http://www.mozilla.com/en-US/firefox/system-requirements.html
says:

  Mac
  ---------------------------------

  Operating Systems
  * Mac OS X 10.4 and later
  Minimum Hardware
  * Macintosh computer with an Intel x86 or PowerPC
    G3, G4, or G5 processor
  * 128 MB RAM (Recommended: 256 MB RAM or greater)
  * 200 MB hard drive space

So I believe we can just delete #if defined(DARWIN).

Comment 8

[glen beasley]

Attachment: use pthread_condattr_init unconditionally

Comment 9

[glen beasley]

Attachment: use pthread_condattr_init unconditionally2

sorry forgot -dup

Comment 10

[Wan-Teh Chang]

Comment on attachment 376515 [details] [diff] [review]
 use pthread_condattr_init unconditionally2

r=wtc.

Comment 11

[glen beasley]

cvs commit -m "492117 allow pthread_condattr_init on Mac OS X patch from jHuddleston r=wan-teh" _pth.h 
Checking in _pth.h;
/cvsroot/mozilla/nsprpub/pr/include/md/_pth.h,v  <--  _pth.h
new revision: 3.36; previous revision: 3.35
done

Comment 12

[glen beasley]

nspr version is 4.8 beta 1 changing target milestone to 4.8

Comment 13

[Wan-Teh Chang]

Thanks, Jeremy and Glen.

It's actually NSPR 4.8 Beta 2 now.  I just tagged NSPR 4.8 Beta 1
this morning.