Skip to content

Commit 488927a

Browse files
russell-islamrbradford
authored andcommitted
performance-metrics: Remove duplicate direct_kernel_boot_path()
Remove the local direct_kernel_boot_path() function and unused PathBuf import from performance_tests.rs. The identical public function from test_infra is already available via wildcard import. Signed-off-by: Muminul Islam <muislam@microsoft.com>
1 parent 5ae3293 commit 488927a

1 file changed

Lines changed: 0 additions & 17 deletions

File tree

performance-metrics/src/performance_tests.rs

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
// Performance tests
77

8-
use std::path::PathBuf;
98
use std::time::Duration;
109
use std::{fs, thread};
1110

@@ -130,22 +129,6 @@ fn performance_test_new_guest(disk_config: Box<dyn DiskConfig>) -> Guest {
130129
const DIRECT_KERNEL_BOOT_CMDLINE: &str =
131130
"root=/dev/vda1 console=hvc0 rw systemd.journald.forward_to_console=1";
132131

133-
// Creates the path for direct kernel boot and return the path.
134-
// For x86_64, this function returns the vmlinux kernel path.
135-
// For AArch64, this function returns the PE kernel path.
136-
fn direct_kernel_boot_path() -> PathBuf {
137-
let mut workload_path = dirs::home_dir().unwrap();
138-
workload_path.push("workloads");
139-
140-
let mut kernel_path = workload_path;
141-
#[cfg(target_arch = "x86_64")]
142-
kernel_path.push("vmlinux-x86_64");
143-
#[cfg(target_arch = "aarch64")]
144-
kernel_path.push("Image-arm64");
145-
146-
kernel_path
147-
}
148-
149132
fn remote_command(api_socket: &str, command: &str, arg: Option<&str>) -> bool {
150133
let mut cmd = std::process::Command::new(clh_command("ch-remote"));
151134
cmd.args([&format!("--api-socket={api_socket}"), command]);

0 commit comments

Comments
 (0)