Hi, all
This is today's KVM test result against kvm.git
90af65e22b7ef28aa0dc6d16ff42dee1b5329a82 and kvm-userspace.git
ad220afdd1473a13dca92d88ea9daae335ba8347.
One issue has been fixed:
1. 64bit host crashes when booting guests
Avi Kivity wrote:
Jan Kiszka wrote:
Hi,
running some oldish 286 protected mode demo in kvm, I came across this
bug of kvm-52:
unhandled vm exit: 0x8021 vcpu_id 0
ds (0010/ p 1 dpl 3 db 0 s 1 type 3 l 0 g 0 avl 0)
ds.base must equal ds.sel4 when entering
In order to make kvm.h arch-independent, some code need to move out from
kvm.h. Now, we move them to x86.h.
This is the preparetion for splitting struct kvm and struct kvm_vcpu.
[1/5] Move some macro definitions out.
[2/5] Move kvm_x86_ops out.
[3/5] Move vcpu regs enumeration defintions out
Xiantao
0002-Move-strcut-kvm_x86_ops-defintion-to-x86.h.patch
Description: 0002-Move-strcut-kvm_x86_ops-defintion-to-x86.h.patch
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio
Xiantao
0003-Move-vcpu-regs-enumeration-defintion-to-x86.h.patch
Description: 0003-Move-vcpu-regs-enumeration-defintion-to-x86.h.patch
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual
Xiantao
0004-Move-some-static-inline-functions-out-from-kvm.h.patch
Description: 0004-Move-some-static-inline-functions-out-from-kvm.h.patch
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R)
Xiantao
0005-Move-some-function-declares-to-x86.h.patch
Description: 0005-Move-some-function-declares-to-x86.h.patch
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
0001-Move-some-macro-definitions-to-x86.h.patch
Description: 0001-Move-some-macro-definitions-to-x86.h.patch
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
Hi Avi,
Based on privious discussion, I made this patch to split struct kvm.
In this patch, strcut kvm only holds common fields, and struct kvm_x86
will keep x86-specific fields. In this way, struct kvm will be a
sub-filed in struct kvm_x86, and we can use to_kvm_x86 to get kvm_x86
from struct
Just Found first KVM-52 regression. SUSE 9.1 ceases to boot.
It turned out this commit, that killed it:
KVM-51-2 - (ab4e018dc95e1b32b7f25746eb9b2c1385aec93e)
Avi: Could you please take a look at it ?
-Alexey
-
This SF.net
Bugs item #1834513, was opened at 2007-11-19 14:40
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=893831aid=1834513group_id=180599
Please note that this message will contain a full copy of
Zhang, Xiantao wrote:
Hi Avi,
Considering your comments,I refined the patch, and use kvm-memslots to
caculate the nr_mmu_pagas every time.
Patch [1/3] Move kvm_vcpu_iotcl_get_dirty_log implementation into arch,
and still keep the interface in common.
Patch [2/3] Move kvm_mmu init and exit
Zhang, Xiantao wrote:
In order to make kvm.h arch-independent, some code need to move out from
kvm.h. Now, we move them to x86.h.
This is the preparetion for splitting struct kvm and struct kvm_vcpu.
[1/5] Move some macro definitions out.
[2/5] Move kvm_x86_ops out.
[3/5] Move vcpu regs
Radisson Hotel claims 30% cut in energy cost due to et GU
EnerBrite Technologies Group, Inc.
ETGU
$0.008
The top five reasons to consider ET gU
1. Energy problems are a Global issue
2. Government sees no real relief in sight. Corporations are finding
their own way out.
3. SensorStat, is already
Jan Kiszka wrote:
Avi Kivity wrote:
Jan Kiszka wrote:
Hi,
running some oldish 286 protected mode demo in kvm, I came across this
bug of kvm-52:
unhandled vm exit: 0x8021 vcpu_id 0
ds (0010/ p 1 dpl 3 db 0 s 1 type 3 l 0 g 0 avl 0)
ds.base must
Zhang, Xiantao wrote:
Based on privious discussion, I made this patch to split struct kvm.
In this patch, strcut kvm only holds common fields, and struct kvm_x86
will keep x86-specific fields. In this way, struct kvm will be a
sub-filed in struct kvm_x86, and we can use to_kvm_x86 to get
Zhang, Xiantao wrote:
Move kvm_vcpu_ioctl_get_dirty_log to arch, and still keep the interface
in common.
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
index 45b18e1..3400265 100644
--- a/drivers/kvm/kvm_main.c
+++ b/drivers/kvm/kvm_main.c
@@ -419,19 +419,14 @@ int
Zhang, Xiantao wrote:
Move kvm_mmu init and exit functionality to arch.
Acked-by: Carsten Otte [EMAIL PROTECTED]
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
Carsten Otte wrote:
Zhang, Xiantao wrote:
Move kvm_vcpu_ioctl_get_dirty_log to arch, and still keep the interface
in common.
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
index 45b18e1..3400265 100644
--- a/drivers/kvm/kvm_main.c
+++ b/drivers/kvm/kvm_main.c
@@ -419,19
Avi Kivity wrote:
Carsten Otte wrote:
Zhang, Xiantao wrote:
Move kvm_vcpu_ioctl_get_dirty_log to arch, and still keep the interface
in common.
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
index 45b18e1..3400265 100644
--- a/drivers/kvm/kvm_main.c
+++
Carsten Otte wrote:
Zhang, Xiantao wrote:
Patch [3/3] Add the hook kvm_arch_set_memory_region. In this patch,
introduce a function kvm_caculate_mmu_pages to compute mmu pages in
total, once memory region changes.diff --git
a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
Carsten Otte wrote:
Yea, I agree that it would make sense in case it works for power too. We
could have an
#ifdef CONFIG_ARCH_S390
return cool_big_iron_get_dirty_log(args);
#endif
at the beginning of that function. It would'nt hurt readability too much.
Stupid suggestion. We could
Zhang, Xiantao wrote:
User-allocation should be what we are heading. But considering
compatibility with old user-space support, I think kernel-allocation
approach should exist for a long time.
That's right. This is why I would prefer to have the corresponding
code out of kvm_main.c: it may
Carsten Otte wrote:
Zhang, Xiantao wrote:
User-allocation should be what we are heading. But considering
compatibility with old user-space support, I think kernel-allocation
approach should exist for a long time.
That's right. This is why I would prefer to have the corresponding
code out
Bugs item #1834585, was opened at 2007-11-19 16:16
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=893831aid=1834585group_id=180599
Please note that this message will contain a full copy of
Zhang, Xiantao wrote:
Carsten Otte wrote:
Zhang, Xiantao wrote:
Based on privious discussion, I made this patch to split struct kvm.
In this patch, strcut kvm only holds common fields, and struct
kvm_x86 will keep x86-specific fields. In this way, struct kvm will
be a sub-filed
Avi Kivity wrote:
Carsten Otte wrote:
Zhang, Xiantao wrote:
User-allocation should be what we are heading. But considering
compatibility with old user-space support, I think kernel-allocation
approach should exist for a long time.
That's right. This is why I would prefer to have the
Avi Kivity wrote:
Zhang, Xiantao wrote:
Carsten Otte wrote:
Zhang, Xiantao wrote:
Based on privious discussion, I made this patch to split struct
kvm. In this patch, strcut kvm only holds common fields, and struct
kvm_x86 will keep x86-specific fields. In this way, struct kvm
will be
The clts code didn't use set_cr0 properly, so our lazy FPU
processing wasn't being done by the clts instruction at all.
This fixes all the FPU leaks, so re-enabling lazy FPU
optimization.
Signed-off-by: Amit Shah [EMAIL PROTECTED]
---
drivers/kvm/svm.c |2 --
drivers/kvm/x86.c |3 +--
2
Bugs item #1834719, was opened at 2007-11-20 02:58
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=893831aid=1834719group_id=180599
Please note that this message will contain a full copy of
this patch discards MSR writes to the Performance Event-Select
Registers, this is the first issue why vista seems to fail although now
vista ends up in an endless loop a bit later.
Qemu currently also discards those writes.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
Signed-off-by: Markus
On Mon, 2007-11-19 at 18:05 +0800, Zhang, Xiantao wrote:
Hi Avi,
Based on privious discussion, I made this patch to split struct kvm.
In this patch, strcut kvm only holds common fields, and struct kvm_x86
will keep x86-specific fields. In this way, struct kvm will be a
sub-filed in struct
Signed-off-by: Hollis Blanchard [EMAIL PROTECTED]
---
1 file changed, 1 deletion(-)
drivers/kvm/kvm.h |1 -
diff --git a/drivers/kvm/kvm.h b/drivers/kvm/kvm.h
--- a/drivers/kvm/kvm.h
+++ b/drivers/kvm/kvm.h
@@ -318,7 +318,6 @@ struct kvm {
unsigned int n_alloc_mmu_pages;
These are a few fixes to minor/cosmetic issues I've stumbled across recently.
3 files changed, 7 insertions(+), 8 deletions(-)
drivers/kvm/kvm.h | 11 +--
drivers/kvm/kvm_main.c |2 +-
drivers/kvm/x86.c |2 +-
Signed-off-by: Hollis Blanchard [EMAIL PROTECTED]
---
1 file changed, 4 insertions(+), 4 deletions(-)
drivers/kvm/kvm.h |8
diff --git a/drivers/kvm/kvm.h b/drivers/kvm/kvm.h
--- a/drivers/kvm/kvm.h
+++ b/drivers/kvm/kvm.h
@@ -135,15 +135,15 @@ struct kvm_mmu {
#define
Signed-off-by: Hollis Blanchard [EMAIL PROTECTED]
---
3 files changed, 3 insertions(+), 3 deletions(-)
drivers/kvm/kvm.h |2 +-
drivers/kvm/kvm_main.c |2 +-
drivers/kvm/x86.c |2 +-
diff --git a/drivers/kvm/kvm.h b/drivers/kvm/kvm.h
--- a/drivers/kvm/kvm.h
+++
On Mon, 19 Nov 2007, Bastian Blank wrote:
On Fri, Nov 09, 2007 at 04:42:48PM -0200, Glauber de Oliveira Costa wrote:
- wrmsrl(MSR_CSTAR, ia32_cstar_target);
+ wrmsrl(MSR_CSTAR, (u64)ia32_cstar_target);
Hmm, why do you add explicit casts? The compiler should convert that
correctly on
This patch moves structure kvm_regs to include/asm-x86/kvm.h.
Each architecture will need to create there own version of this
structure.
Signed-off-by: Jerone Young [EMAIL PROTECTED]
diff --git a/include/asm-x86/kvm.h b/include/asm-x86/kvm.h
index 7a8f4d5..ec99d9a 100644
---
This patch moves structures:
kvm_cpuid_entry
kvm_cpuid
from include/linux/kvm.h to include/asm-x86/kvm.h
Signed-off-by: Jerone Young [EMAIL PROTECTED]
diff --git a/include/asm-x86/kvm.h b/include/asm-x86/kvm.h
index 97edada..5db4d91 100644
--- a/include/asm-x86/kvm.h
+++
Move structures:
kvm_sregs
kvm_msr_entry
kvm_msrs
kvm_msr_list
from include/linux/kvm.h to include/asm-x86/kvm.h
diff --git a/include/asm-x86/kvm.h b/include/asm-x86/kvm.h
index 07e7fee..1970520 100644
--- a/include/asm-x86/kvm.h
+++ b/include/asm-x86/kvm.h
@@
This patch moves sturct kvm_memory_alias from include/linux/kvm.h
to include/asm-x86/kvm.h. Also have include/linux/kvm.h include
include/asm/kvm.h.
Signed-off-by: Jerone Young [EMAIL PROTECTED]
diff --git a/include/asm-x86/kvm.h b/include/asm-x86/kvm.h
new file mode 100644
index
This set of patches move structres definitions that
are x86 specific from include/linux/kvm.h to inclue/asm-x86/kvm.h.
Missing from these patches is exactly how to move the IOCTL definitions
while keeping things in order. I'll disscuss this on the list.
Signed-off-by: Jerone Young [EMAIL
Hollis Blanchard wrote:
On Mon, 2007-11-19 at 18:05 +0800, Zhang, Xiantao wrote:
Hi Avi,
Based on privious discussion, I made this patch to split struct kvm.
In this patch, strcut kvm only holds common fields, and struct
kvm_x86 will keep x86-specific fields. In this way, struct kvm will
From: Zhang xiantao [EMAIL PROTECTED]
Date: Tue, 20 Nov 2007 10:08:19 +0800
Subject: [PATCH] KVM Portability split: Splitting kvm structure.
Use kvm_x86 to hold x86 specific kvm fields, in this way
kvm strcut only contains common fields.
Signed-off-by: Zhang xiantao [EMAIL PROTECTED]
---
From e7a0fcaa7d87c20a4e7155a11ee2afa880bdc471 Mon Sep 17 00:00:00 2001
From: Zhang xiantao [EMAIL PROTECTED]
Date: Tue, 20 Nov 2007 10:44:08 +0800
Subject: [PATCH] KVM Portability split: splitting kvm.h
Moving code related to mmu to x86.h
Signed-off-by: Zhang xiantao [EMAIL PROTECTED]
---
From: Zhang xiantao [EMAIL PROTECTED]
Date: Tue, 20 Nov 2007 10:46:33 +0800
Subject: [PATCH] KVM Portability move: Moving kvm_vcpu_stat to x86.h
Signed-off-by: Zhang xiantao [EMAIL PROTECTED]
---
drivers/kvm/kvm.h | 22 --
drivers/kvm/x86.h | 22 ++
2
This patch is a continuation of the 7 patches sent earlier. This
patch moves all x86 specific macros from include/linux/kvm.h to
include/asm-x86/kvm.h.
Signed-off-by: Jerone Young [EMAIL PROTECTED]
*note just realized none of the other patches except for email 0
had the [PATCH] addendum on the
Bugs item #1834938, was opened at 2007-11-20 13:49
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=893831aid=1834938group_id=180599
Please note that this message will contain a full copy of
* Steven Rostedt [EMAIL PROTECTED] wrote:
On Fri, Nov 09, 2007 at 04:42:48PM -0200, Glauber de Oliveira Costa wrote:
- wrmsrl(MSR_CSTAR, ia32_cstar_target);
+ wrmsrl(MSR_CSTAR, (u64)ia32_cstar_target);
Hmm, why do you add explicit casts? The compiler should convert that
correctly
On Tue, 20 Nov 2007, Ingo Molnar wrote:
i dont think there's ever any true need (and good cause) to force
integer type casts like that at the callee site.
Unless you mean we should do something like this:
static inline void __wrmsrl(unsigned int msr, unsigned long long val);
#define
On Tue, 20 Nov 2007, Ingo Molnar wrote:
* Steven Rostedt [EMAIL PROTECTED] wrote:
With PVOPS on it gives compiler warnings without that explict cast.
Without looking at the code, IIRC with non-PVOPS it is a macro
directly into asm, so it didn't matter what the cast was. But with
PVOPS
On Tuesday 20 November 2007 00:38:05 Markus Rechberger wrote:
this patch discards MSR writes to the Performance Event-Select
Registers, this is the first issue why vista seems to fail although now
vista ends up in an endless loop a bit later.
Qemu currently also discards those writes.
Won't
52 matches
Mail list logo
