Closed Bug 975255 Opened 6 years ago Closed 6 years ago

HTTP cache v2: optimize CacheIOThread::Target()

Categories

(Core :: Networking: Cache, defect)

defect
Not set
major

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

Attachments

(1 file)

I realized we need to enter the monitor only to wait for the target to be published to the member (so very rarely soon after the startup).  Otherwise, it's safe to just take it.
OS: Windows 7 → All
Hardware: x86_64 → All
This is called too often.  Should be fixed for first version (patch trivial).
Blocks: cache2enable
No longer blocks: cache2afterenable
Severity: normal → major
Attached patch v1Splinter Review
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Attachment #8379684 - Flags: review?(michal.novotny)
Attachment #8379684 - Flags: review?(michal.novotny) → review+
There is no direct test, but this is a generally used code tested a lot during any test run.
Flags: in-testsuite+
https://hg.mozilla.org/mozilla-central/rev/75c53d96a9c9
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.