456603 - get rid of Windows stat cache

Status: REOPENED

(Core :: XPCOM, defect)

Description

[Josh Aas]

This is the Windows part of bug 307815, we don't have to remove stat caches for all platforms at the same time. Our current stat caches are fundamentally flawed, we should remove them.

Comment 1

[Josh Aas]

Attachment: fix v1.0

Basic try server run shows no significant perf impact, we'll see better numbers when this lands on trunk.

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 340064 [details] [diff] [review]
fix v1.0

drop the braces for one liners.

     if (aFileSize != -1 && SetEndOfFile(hFile))
     {
-        MakeDirty();
         rv = NS_OK;
     }




assuming that this does not cause any performance regression as mentioned in bug 307815, this is fine.

Comment 3

[Josh Aas]

landed on trunk

Comment 4

[Josh Aas]

backed out because of tbox issues, not necessarily anything wrong with the patch

Comment 5

[Dave Townsend [:mossop]]

This might have caused a performance regression on Tp on Vista (some 30% increase). It is confusing though since the first talos runs also supposedly included the backout, but Tp consistently went up on all 4 Vista talos machines then back down again one cycle later. This was the only patch liable to have impacted, but not sure what is going on. Might be good to land in a quiet tree next time.

Comment 6

[Josh Aas]

Attachment: windows errors

this is a text file containing the unit test errors that came up