Closed Bug 975255 Opened 11 years ago Closed 11 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 v1 (deleted) — Splinter 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+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: