Automate testing of CLs with disk_cache_perftest as much as possible. |
|
Issue descriptionRight now the process is pretty work intensive, and fiddly. Besides getting all the CLs for tests in, a bit of scripting may help --- it also sets enough path knobs that the Android tests doesn't look for the JVM. Attached is a start --- runs the bench for a given platform using the current source tree. (Might be an appropriate place for it in git, but I am not sure of where, and this is public and more durable than my hard drive.)
,
May 16 2017
Made it summarize stuff into a single .csv, rather than 5.
,
May 17 2017
xunjieli@ pointed me at: https://codereview.chromium.org/2748073003/ which sets up automated running of net_perftests (on Linux)
,
May 18 2017
And fixing some quoting that was breaking RUNS= on Android..
,
May 19 2017
Made column names shorter and another helper to interleave columns of two runs --- makes plugging them into a spreadsheet and doing graphs really easy.
,
Jun 1 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1e701e32a7785efb6b77d42e6d2b838b4d4590c3 commit 1e701e32a7785efb6b77d42e6d2b838b4d4590c3 Author: morlovich <morlovich@chromium.org> Date: Thu Jun 01 14:40:08 2017 Provide base::EvictFileFromSystemCache on Android on ARM32 This is needed to be able to run disk_cache_perftest on the platform. This just uses the Linux implementation, but provides a syscall wrapper since NDK won't provide one at the API level we build for. BUG=722885 Review-Url: https://codereview.chromium.org/2885423002 Cr-Commit-Position: refs/heads/master@{#476281} [modify] https://crrev.com/1e701e32a7785efb6b77d42e6d2b838b4d4590c3/base/test/BUILD.gn [modify] https://crrev.com/1e701e32a7785efb6b77d42e6d2b838b4d4590c3/base/test/test_file_util_linux.cc [modify] https://crrev.com/1e701e32a7785efb6b77d42e6d2b838b4d4590c3/base/test/test_file_util_posix.cc
,
Jun 1 2017
Version that hand-applies a .diff, and alternates patched and unpatched versions. (That can help cancel some trends, but the test is probably too slow for it to help much).
,
Jun 1 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d29ed392cc031d617f582ff019e555e50c311e37 commit d29ed392cc031d617f582ff019e555e50c311e37 Author: morlovich <morlovich@chromium.org> Date: Thu Jun 01 15:54:37 2017 diskcache_perftest: fully use base::EvictFileFromSystemCache on android, now it's available there. BUG=722885 Review-Url: https://codereview.chromium.org/2914203003 Cr-Commit-Position: refs/heads/master@{#476310} [modify] https://crrev.com/d29ed392cc031d617f582ff019e555e50c311e37/net/disk_cache/disk_cache_perftest.cc
,
Jun 1 2017
And fixes Windows support. Need to use gn.bat there, rather than have cygwin trying to Unix-up the usual gn wrapper.
,
Dec 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f3860655759a60ce589514e8c22e04abb79f171b commit f3860655759a60ce589514e8c22e04abb79f171b Author: Maks Orlovich <morlovich@chromium.org> Date: Wed Dec 13 18:03:16 2017 disk_cache_perf_test: Actually be parallel This is 99.9% work by Gavin Peters, at https://codereview.chromium.org/2501353002, with me just tweaking some parameters and applying the tiny things I suggested on that review. Bug: 611464, 722885 Change-Id: I5bdc7d60ec4bc54e1099f408ea09e0573c176fca Reviewed-on: https://chromium-review.googlesource.com/800972 Commit-Queue: Maks Orlovich <morlovich@chromium.org> Reviewed-by: Julia Tuttle <juliatuttle@chromium.org> Cr-Commit-Position: refs/heads/master@{#523814} [modify] https://crrev.com/f3860655759a60ce589514e8c22e04abb79f171b/net/disk_cache/backend_unittest.cc [modify] https://crrev.com/f3860655759a60ce589514e8c22e04abb79f171b/net/disk_cache/disk_cache_perftest.cc [modify] https://crrev.com/f3860655759a60ce589514e8c22e04abb79f171b/net/disk_cache/disk_cache_test_base.cc [modify] https://crrev.com/f3860655759a60ce589514e8c22e04abb79f171b/net/disk_cache/disk_cache_test_base.h [modify] https://crrev.com/f3860655759a60ce589514e8c22e04abb79f171b/net/disk_cache/disk_cache_test_util.cc [modify] https://crrev.com/f3860655759a60ce589514e8c22e04abb79f171b/net/disk_cache/disk_cache_test_util.h |
|
►
Sign in to add a comment |
|
Comment 1 by morlovich@chromium.org
, May 16 20172.2 KB
2.2 KB View Download