HTTP cache v2: optimize CacheIOThread::Target()

RESOLVED FIXED in mozilla30

Status

()

defect
--
major
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: mayhemer, Assigned: mayhemer)

Tracking

Trunk
mozilla30
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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
Posted 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: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.