Closed Bug 975255 Opened 10 years ago Closed 10 years ago

HTTP cache v2: optimize CacheIOThread::Target()

Categories

(Core :: Networking: Cache, defect)

Product:
Core
Component:
Networking: Cache
Type:
defect
Priority:
Not set
Severity:
major

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla30

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

Attachments

(1 file)

v1
677 bytes, patch
michal
: review+
Details | Diff | Splinter Review 
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 — — 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+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/75c53d96a9c9
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30

          You need to log in
          before you can comment on or make changes to this bug.