Closed
Bug 26511
Opened 25 years ago
Closed 21 years ago
eliminate nsPipe2 locks
Categories
(Core :: XPCOM, defect, P3)
Tracking
()
RESOLVED
WONTFIX
Future
People
(Reporter: warrensomebody, Assigned: darin.moz)
Details
(Keywords: perf)
I've noticed during profiling that a huge amount of time is spent in the locks
in xpcom/io/nsPipe2.cpp. In theory, I think we could eliminate entering and
exiting locks when we can notice that the read cursor is in a different segment
from the write cursor. Although tricky, this could give us a huge win.
Since I wrote it, I'll look into it.
Reporter | ||
Updated•25 years ago
|
Reporter | ||
Comment 1•25 years ago
|
||
Warren: how much benefit are we going to get from this? Also -- what's the
likeliness that you'll get to it before beta?
Reporter | ||
Comment 4•25 years ago
|
||
Not feasible for beta1 -- what was I thinking?
Reporter | ||
Comment 5•25 years ago
|
||
Moving non-essential, non-beta2 and performance-related bugs to M17.
Target Milestone: M15 → M17
Comment 7•24 years ago
|
||
Warren, I am reassigning some bugs which you own over to me (dougt). If you
think that you will have time to address them please reassign them back to
yourself. This reassignment was suggested by choffman.
Assignee: warsome → dougt
Updated•24 years ago
|
QA Contact: leger → kandrot
Assignee | ||
Comment 9•24 years ago
|
||
Pushing milestone back to 0.9.1
Target Milestone: M18 → mozilla0.9.1
Assignee | ||
Comment 10•24 years ago
|
||
with the somewhat recent change waterson made to stop holding the pipe lock
while calling out to the ns{Write,Read}SegmentFun in {Read,Write}Segments, i
think we solved 80% of this bug. so, i'm going to move this out to Future
as we have much bigger fish to fry.
Target Milestone: mozilla0.9.1 → Future
Assignee | ||
Comment 11•21 years ago
|
||
WONTFIX .. i'm not convinced the gain would be worth the pain at this point.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•