diff options
author | Binbin Wu <binbin.wu@linux.intel.com> | 2023-03-22 07:58:22 +0300 |
---|---|---|
committer | Sean Christopherson <seanjc@google.com> | 2023-03-22 20:18:11 +0300 |
commit | bede6eb4db19424477c36dace426ae12255f4a0d (patch) | |
tree | ac2e27614a571387bcbfb7a68e0bd636119fd7d3 /tools/perf/scripts/python/exported-sql-viewer.py | |
parent | 607475cfa0f753862c0030de836875fe4ac5cf8a (diff) | |
download | linux-bede6eb4db19424477c36dace426ae12255f4a0d.tar.xz |
KVM: x86: Use boolean return value for is_{pae,pse,paging}()
Convert is_{pae,pse,paging}() to use kvm_is_cr{0,4}_bit_set() and return
bools. Returning an "int" requires not one, but two implicit casts, first
from "unsigned long" to "int", and then again to a "bool". Both casts are
more than a bit dangerous; the ulong=>int casts would drop a bit on 64-bit
kernels _if_ the bits in question weren't in the lower 32 bits, and the
int=>bool cast can result in false negatives/positives, e.g. see commit
0c928ff26bd6 ("KVM: SVM: Fix benign "bool vs. int" comparison in
svm_set_cr0()").
Suggested-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Binbin Wu <binbin.wu@linux.intel.com>
Link: https://lore.kernel.org/r/20230322045824.22970-3-binbin.wu@linux.intel.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
Diffstat (limited to 'tools/perf/scripts/python/exported-sql-viewer.py')
0 files changed, 0 insertions, 0 deletions