From: "Steven Rostedt (Google)" <rostedt@xxxxxxxxxxx>
kvm_scaling_check_vm_cpu() uses read_ll_from_file() to determine if the
kvm scaling and fraction files exist. The read_ll_from_file() does not
return if the contents of the files are legit or not. No need to read
them in this file. Just use stat().
Signed-off-by: Steven Rostedt (Google) <rostedt@xxxxxxxxxxx>
---
lib/trace-cmd/trace-timesync-kvm.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/trace-cmd/trace-timesync-kvm.c b/lib/trace-cmd/trace-timesync-kvm.c
index 12a22d4c4d6a..a645fa2cc70b 100644
--- a/lib/trace-cmd/trace-timesync-kvm.c
+++ b/lib/trace-cmd/trace-timesync-kvm.c
@@ -75,22 +75,22 @@ static int read_ll_from_file(char *file, long long *res)
static bool kvm_scaling_check_vm_cpu(char *vname, char *cpu)
{
- long long scaling, frac;
bool has_scaling = false;
bool has_frac = false;
+ struct stat st;
char *path;
int ret;
if (asprintf(&path, "%s/%s/%s", vname, cpu, KVM_DEBUG_SCALING_FILE) < 0)
return false;
- ret = read_ll_from_file(path, &scaling);
+ ret = stat(path, &st);
free(path);
if (!ret)
has_scaling = true;
if (asprintf(&path, "%s/%s/%s", vname, cpu, KVM_DEBUG_FRACTION_FILE) < 0)
return false;
- ret = read_ll_from_file(path, &frac);
+ ret = stat(path, &st);
free(path);
if (!ret)
has_frac = true;
--
2.35.1
|