* [PATCH] KVM: MTRR: Use default type for non-MTRR-covered gfn before WARN_ON
@ 2015-08-04 16:58 Alex Williamson
2015-08-05 2:22 ` Xiao Guangrong
2015-08-05 9:58 ` Paolo Bonzini
0 siblings, 2 replies; 3+ messages in thread
From: Alex Williamson @ 2015-08-04 16:58 UTC (permalink / raw)
To: pbonzini, kvm; +Cc: linux-kernel, guangrong.xiao
The patch was munged on commit to re-order these tests resulting in
excessive warnings when trying to do device assignment. Return to
original ordering: https://lkml.org/lkml/2015/7/15/769
Fixes: 3e5d2fdceda1 ("KVM: MTRR: simplify kvm_mtrr_get_guest_memory_type")
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
---
For v4.2
arch/x86/kvm/mtrr.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/x86/kvm/mtrr.c b/arch/x86/kvm/mtrr.c
index dc0a84a..9e8bf13 100644
--- a/arch/x86/kvm/mtrr.c
+++ b/arch/x86/kvm/mtrr.c
@@ -672,16 +672,16 @@ u8 kvm_mtrr_get_guest_memory_type(struct kvm_vcpu *vcpu, gfn_t gfn)
if (iter.mtrr_disabled)
return mtrr_disabled_type();
+ /* not contained in any MTRRs. */
+ if (type == -1)
+ return mtrr_default_type(mtrr_state);
+
/*
* We just check one page, partially covered by MTRRs is
* impossible.
*/
WARN_ON(iter.partial_map);
- /* not contained in any MTRRs. */
- if (type == -1)
- return mtrr_default_type(mtrr_state);
-
return type;
}
EXPORT_SYMBOL_GPL(kvm_mtrr_get_guest_memory_type);
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] KVM: MTRR: Use default type for non-MTRR-covered gfn before WARN_ON
2015-08-04 16:58 [PATCH] KVM: MTRR: Use default type for non-MTRR-covered gfn before WARN_ON Alex Williamson
@ 2015-08-05 2:22 ` Xiao Guangrong
2015-08-05 9:58 ` Paolo Bonzini
1 sibling, 0 replies; 3+ messages in thread
From: Xiao Guangrong @ 2015-08-05 2:22 UTC (permalink / raw)
To: Alex Williamson, pbonzini, kvm; +Cc: linux-kernel, guangrong.xiao
On 08/05/2015 12:58 AM, Alex Williamson wrote:
> The patch was munged on commit to re-order these tests resulting in
> excessive warnings when trying to do device assignment. Return to
> original ordering: https://lkml.org/lkml/2015/7/15/769
>
Reviewed-by: Xiao Guangrong <guangrong.xiao@linux.intel.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] KVM: MTRR: Use default type for non-MTRR-covered gfn before WARN_ON
2015-08-04 16:58 [PATCH] KVM: MTRR: Use default type for non-MTRR-covered gfn before WARN_ON Alex Williamson
2015-08-05 2:22 ` Xiao Guangrong
@ 2015-08-05 9:58 ` Paolo Bonzini
1 sibling, 0 replies; 3+ messages in thread
From: Paolo Bonzini @ 2015-08-05 9:58 UTC (permalink / raw)
To: Alex Williamson, kvm; +Cc: linux-kernel, guangrong.xiao
On 04/08/2015 18:58, Alex Williamson wrote:
> The patch was munged on commit to re-order these tests resulting in
> excessive warnings when trying to do device assignment. Return to
> original ordering: https://lkml.org/lkml/2015/7/15/769
>
> Fixes: 3e5d2fdceda1 ("KVM: MTRR: simplify kvm_mtrr_get_guest_memory_type")
> Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Applied, thanks. Sorry for the delay.
Paolo
> ---
>
> For v4.2
>
> arch/x86/kvm/mtrr.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/x86/kvm/mtrr.c b/arch/x86/kvm/mtrr.c
> index dc0a84a..9e8bf13 100644
> --- a/arch/x86/kvm/mtrr.c
> +++ b/arch/x86/kvm/mtrr.c
> @@ -672,16 +672,16 @@ u8 kvm_mtrr_get_guest_memory_type(struct kvm_vcpu *vcpu, gfn_t gfn)
> if (iter.mtrr_disabled)
> return mtrr_disabled_type();
>
> + /* not contained in any MTRRs. */
> + if (type == -1)
> + return mtrr_default_type(mtrr_state);
> +
> /*
> * We just check one page, partially covered by MTRRs is
> * impossible.
> */
> WARN_ON(iter.partial_map);
>
> - /* not contained in any MTRRs. */
> - if (type == -1)
> - return mtrr_default_type(mtrr_state);
> -
> return type;
> }
> EXPORT_SYMBOL_GPL(kvm_mtrr_get_guest_memory_type);
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-08-05 9:58 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-04 16:58 [PATCH] KVM: MTRR: Use default type for non-MTRR-covered gfn before WARN_ON Alex Williamson
2015-08-05 2:22 ` Xiao Guangrong
2015-08-05 9:58 ` Paolo Bonzini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).