[PATCH] lib/result.rb: Restore PATH_SCHEME of LOCAL_RUN in ResultPath

Commit cd1d0c6cff23 ("run-local: simplify result path") changes LOCAL_RUN path scheme to "%w[path_params run]" like below: /root/.cache/lkp/result/unixbench/1-100-syscall/0 /root/.cache/lkp/result/unixbench/1-100-syscall/1 /root/.cache/lkp/result/unixbench/96-100-syscall/0 According to the lkp compare usage, user can use the -d option with "commit, kconfig, fs, etc" dimensions to compare results generated by different kernel versions with different commits, kconfigs, or rootfs, it's useful for evaluating the impact of different dimensions(like commits) on performance by running benchmarks on local server. " [root@localhost ~]# lkp compare --help Usage: compare [options] RESULT_ROOT... options: -d, --dimension DIMENSION DIMENSION to compare: commit, kconfig, fs, etc. " But with the simplified result path scheme for local run, this usage is not supported, as the remove of commit, kconfig and rootfs elements, the result_root path will not contain these sub-directories and when setup_result_root_hash greps "dim value" in the "~/.cache/lkp/paths", nothing will be matched, then $_result_root_tuple is empty and no stats can be compared and shown. This patch changes the PATH_SCHEME of LOCAL_RUN to support this usage. Signed-off-by: Zheng Zengkai <zhengzengkai@huawei.com> --- lib/result.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/result.rb b/lib/result.rb index 492e43b82..8260c8671 100755 --- a/lib/result.rb +++ b/lib/result.rb @@ -28,7 +28,7 @@ class ResultPath < Hash AXIS_KEYS = (MAXIS_KEYS + ['run']).freeze PATH_SCHEME = { - 'LOCAL_RUN' => %w[path_params run], + 'LOCAL_RUN' => %w[path_params tbox_group rootfs kconfig compiler commit run], 'default' => %w[path_params tbox_group rootfs kconfig compiler commit run], 'kvm:default' => %w[path_params tbox_group rootfs kconfig compiler commit run], 'health-stats' => %w[path_params run], -- 2.43.0
participants (1)
-
Zheng Zengkai