diff mbox

silence kernel messages that can be triggered by a guest

Message ID 20090708082654.GK28046@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Gleb Natapov July 8, 2009, 8:26 a.m. UTC 
Some Linux versions (f8) try to read EOI register that is write only.

Signed-off-by: Gleb Natapov <gleb@redhat.com>
--
			Gleb.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Marcelo Tosatti July 14, 2009, 11:11 p.m. UTC | #1 
On Wed, Jul 08, 2009 at 11:26:54AM +0300, Gleb Natapov wrote:
> Some Linux versions (f8) try to read EOI register that is write only.
> 
> Signed-off-by: Gleb Natapov <gleb@redhat.com>
> diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c
> index ccafe0d..6b5dad4 100644
> --- a/arch/x86/kvm/lapic.c
> +++ b/arch/x86/kvm/lapic.c
> @@ -590,14 +590,14 @@ static int apic_reg_read(struct kvm_lapic *apic, u32 offset, int len,
>  	static const u64 rmask = 0x43ff01ffffffe70cULL;
>  
>  	if ((alignment + len) > 4) {
> -		printk(KERN_ERR "KVM_APIC_READ: alignment error %x %d\n",
> -				offset, len);
> +		apic_debug("KVM_APIC_READ: alignment error %x %d\n",
> +			   offset, len);
>  		return 1;
>  	}
>  
>  	if (offset > 0x3f0 || !(rmask & (1ULL << (offset >> 4)))) {
> -		printk(KERN_ERR "KVM_APIC_READ: read reserved register %x\n",
> -				offset);
> +		apic_debug("KVM_APIC_READ: read reserved register %x\n",
> +			   offset);
>  		return 1;
>  	}
>  
> --
> 			Gleb.

Applied, thanks.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c
index ccafe0d..6b5dad4 100644
--- a/arch/x86/kvm/lapic.c
+++ b/arch/x86/kvm/lapic.c
@@ -590,14 +590,14 @@  static int apic_reg_read(struct kvm_lapic *apic, u32 offset, int len,
 	static const u64 rmask = 0x43ff01ffffffe70cULL;
 
 	if ((alignment + len) > 4) {
-		printk(KERN_ERR "KVM_APIC_READ: alignment error %x %d\n",
-				offset, len);
+		apic_debug("KVM_APIC_READ: alignment error %x %d\n",
+			   offset, len);
 		return 1;
 	}
 
 	if (offset > 0x3f0 || !(rmask & (1ULL << (offset >> 4)))) {
-		printk(KERN_ERR "KVM_APIC_READ: read reserved register %x\n",
-				offset);
+		apic_debug("KVM_APIC_READ: read reserved register %x\n",
+			   offset);
 		return 1;
 	}