diff options
author | Sean Christopherson <sean.j.christopherson@intel.com> | 2019-04-16 22:32:47 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2019-04-30 21:32:17 +0200 |
commit | 4ca88b3f86cd03deecd48ca9880a7c2e7c6fb788 (patch) | |
tree | 8f34de9438c5c9c4fa50df89a8c1496d014499bb /fs/configfs | |
parent | KVM: lapic: Refactor ->set_hv_timer to use an explicit expired param (diff) | |
download | linux-4ca88b3f86cd03deecd48ca9880a7c2e7c6fb788.tar.xz linux-4ca88b3f86cd03deecd48ca9880a7c2e7c6fb788.zip |
KVM: lapic: Check for a pending timer intr prior to start_hv_timer()
Checking for a pending non-periodic interrupt in start_hv_timer() leads
to restart_apic_timer() making an unnecessary call to start_sw_timer()
due to start_hv_timer() returning false.
Alternatively, start_hv_timer() could return %true when there is a
pending non-periodic interrupt, but that approach is less intuitive,
i.e. would require a beefy comment to explain an otherwise simple check.
Cc: Liran Alon <liran.alon@oracle.com>
Cc: Wanpeng Li <wanpengli@tencent.com>
Suggested-by: Liran Alon <liran.alon@oracle.com>
Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'fs/configfs')
0 files changed, 0 insertions, 0 deletions