|
|
Description[SyncFS] CountDirtyTracke() does not run a full scan.
It reduces disk I/O cost on counting dirty trackers.
BUG=347425
TEST=./unit_tests --gtest_filter="MetadataDatabase*"
Committed: https://crrev.com/a38fa22d030b7688544a5f41f4fc9a5b5dfe22c2
Cr-Commit-Position: refs/heads/master@{#293447}
Patch Set 1 : #Patch Set 2 : Move CountDirtyTrackerInternal in .cc #Patch Set 3 : Do not store on disk #Patch Set 4 : Rebase #
Messages
Total messages: 31 (13 generated)
Patchset #1 (id:1) has been deleted
Patchset #1 (id:20001) has been deleted
peria@chromium.org changed reviewers: + nhiroki@chromium.org, tzik@chromium.org
PTL
Can we just hold it on memory..?
On 2014/09/04 06:48:44, tzik wrote: > Can we just hold it on memory..? Yes, we can. Though, it needs to sweep all dirty tracker entries on construction. I'll update this CL.
On 2014/09/04 06:50:41, peria wrote: > On 2014/09/04 06:48:44, tzik wrote: > > Can we just hold it on memory..? > > Yes, we can. > Though, it needs to sweep all dirty tracker entries on construction. > > I'll update this CL. Done.
lgtm
lgtm
The CQ bit was checked by peria@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/peria@chromium.org/536343002/80001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: android_clang_dbg_recipe on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
The CQ bit was checked by peria@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/peria@chromium.org/536343002/80001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: android_clang_dbg_recipe on tryserver.chromium.linux (http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
The CQ bit was checked by peria@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/peria@chromium.org/536343002/80001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: win_gpu_triggered_tests on tryserver.chromium.gpu (http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)
The CQ bit was checked by peria@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/peria@chromium.org/536343002/80001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: win_gpu_triggered_tests on tryserver.chromium.gpu (http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)
The CQ bit was checked by peria@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/peria@chromium.org/536343002/80001
The CQ bit was checked by peria@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/peria@chromium.org/536343002/100001
Message was sent while issue was closed.
Committed patchset #4 (id:100001) as b23dbc0aea7ed99b06a5cdddb3349c258b295347
Message was sent while issue was closed.
Patchset 4 (id:??) landed as https://crrev.com/a38fa22d030b7688544a5f41f4fc9a5b5dfe22c2 Cr-Commit-Position: refs/heads/master@{#293447} |