public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] KVM: x86_emulator: no writeback for bt
@ 2007-09-24 10:07 He, Qing
       [not found] ` <37E52D09333DE2469A03574C88DBF40FA9C24C-wq7ZOvIWXbM/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: He, Qing @ 2007-09-24 10:07 UTC (permalink / raw)
  To: kvm-devel

[-- Attachment #1: Type: text/plain, Size: 694 bytes --]

(p.s., this patch is before Laurent's patch of no_wb removal, the change
is minor to adapt to OP_NONE)


Signed-off-by: Qing He <qing.he-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

---
 drivers/kvm/x86_emulate.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/kvm/x86_emulate.c b/drivers/kvm/x86_emulate.c
index f294a49..6ab67fe 100644
--- a/drivers/kvm/x86_emulate.c
+++ b/drivers/kvm/x86_emulate.c
@@ -1549,6 +1549,7 @@ twobyte_insn:
 		break;
 	case 0xa3:
 	      bt:		/* bt */
+		no_wb = 1;
 		/* only subword offset */
 		c->src.val &= (c->dst.bytes << 3) - 1;
 		emulate_2op_SrcV_nobyte("bt", c->src, c->dst, _eflags);
-- 
1.5.1.2


[-- Attachment #2: 0002-KVM-x86_emulator-no-writeback-for-bt.patch --]
[-- Type: application/octet-stream, Size: 734 bytes --]

From 4d78d9bf330c478496765184196aeaa085b968a8 Mon Sep 17 00:00:00 2001
From: Qing He <qing.he@intel.com>
Date: Mon, 24 Sep 2007 17:22:13 +0800
Subject: [PATCH] KVM: x86_emulator: no writeback for bt

Signed-off-by: Qing He <qing.he@intel.com>

---
 drivers/kvm/x86_emulate.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/kvm/x86_emulate.c b/drivers/kvm/x86_emulate.c
index f294a49..6ab67fe 100644
--- a/drivers/kvm/x86_emulate.c
+++ b/drivers/kvm/x86_emulate.c
@@ -1549,6 +1549,7 @@ twobyte_insn:
 		break;
 	case 0xa3:
 	      bt:		/* bt */
+		no_wb = 1;
 		/* only subword offset */
 		c->src.val &= (c->dst.bytes << 3) - 1;
 		emulate_2op_SrcV_nobyte("bt", c->src, c->dst, _eflags);
-- 
1.5.1.2


[-- Attachment #3: Type: text/plain, Size: 228 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

[-- Attachment #4: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] KVM: x86_emulator: no writeback for bt
       [not found] ` <37E52D09333DE2469A03574C88DBF40FA9C24C-wq7ZOvIWXbM/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
@ 2007-09-24 13:24   ` Avi Kivity
       [not found]     ` <46F7BA79.8010305-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Avi Kivity @ 2007-09-24 13:24 UTC (permalink / raw)
  To: He, Qing; +Cc: kvm-devel

He, Qing wrote:
> (p.s., this patch is before Laurent's patch of no_wb removal, the change
> is minor to adapt to OP_NONE)
>
>   

Applied, thanks.  Does it solve some specific issue with a guest?  I'll 
have to send it for 2.6.23 as well if it does.

-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] KVM: x86_emulator: no writeback for bt
       [not found]     ` <46F7BA79.8010305-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
@ 2007-09-26  1:53       ` He, Qing
  0 siblings, 0 replies; 3+ messages in thread
From: He, Qing @ 2007-09-26  1:53 UTC (permalink / raw)
  To: Avi Kivity; +Cc: kvm-devel

[-- Attachment #1: Type: text/plain, Size: 759 bytes --]



>-----Original Message-----
>From: Avi Kivity [mailto:avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org]
>Sent: 2007年9月24日 21:24
>To: He, Qing
>Cc: kvm-devel
>Subject: Re: [kvm-devel] [PATCH] KVM: x86_emulator: no writeback for bt
>
>He, Qing wrote:
>> (p.s., this patch is before Laurent's patch of no_wb removal, the change
>> is minor to adapt to OP_NONE)
>>
>>
>
>Applied, thanks.  Does it solve some specific issue with a guest?  I'll
>have to send it for 2.6.23 as well if it does.

No, it doesn't solve any major issue. We just found that in some occasions, apic irr is written to when this instruction is emulated. It reduces MMIO emulation overhead as well.

>
>--
>error compiling committee.c: too many arguments to function


[-- Attachment #2: Type: text/plain, Size: 228 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

[-- Attachment #3: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-09-26  1:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-09-24 10:07 [PATCH] KVM: x86_emulator: no writeback for bt He, Qing
     [not found] ` <37E52D09333DE2469A03574C88DBF40FA9C24C-wq7ZOvIWXbM/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2007-09-24 13:24   ` Avi Kivity
     [not found]     ` <46F7BA79.8010305-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-09-26  1:53       ` He, Qing

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox