The following reply was made to PR kern/131597; it has been noted by GNATS.

From: John Baldwin <[email protected]>
To: [email protected],
 [email protected]
Cc: [email protected],
 [email protected]
Subject: Re: kern/131597: [kernel] c++ exceptions very slow on FreeBSD 7.1/amd64
Date: Fri, 28 Jun 2013 08:47:55 -0400

 Looking at this again, the patch committed in 178807 is just wrong and should 
 be reverted.  There is no state in rtld that needs to be protected via a write 
 lock.  GCC is too lazy to use their own locking to protect shared state 
 between threads and wants the runtime linker to enforce this.  Their 
 justification that glibc doesn't allow concurrent execution of this isn't a 
 valid excuse.  For an API like this that just walks a list and invokes a 
 callback, if the callback manipulates shared state owned by the caller, the 
 caller should be responsible for sychronizing access to it, not rtld!
 
 Instead I think we should apply the patch in the original GCC bug to our in-
 tree GCC and to our GCC ports.  This should remove the sigprocmask calls and
 not penalize other users of dl_iterate_phdr() for GCC's poor behavior.
 
 -- 
 John Baldwin
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "[email protected]"

Reply via email to