qemu.git uses an array of 100 entries for the msr list, which is arguably large
enough (tm).
I propose we follow the same path, for two reasons:
1) ease future merge.
2) avoid stack overflow problems that had already began to appear
Signed-off-by: Glauber Costa <glommer@xxxxxxxxxx>
---
qemu-kvm-x86.c | 10 ++--------
1 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/qemu-kvm-x86.c b/qemu-kvm-x86.c
index c8e37ed..350e5fd 100644
--- a/qemu-kvm-x86.c
+++ b/qemu-kvm-x86.c
@@ -808,12 +808,6 @@ static int get_msr_entry(struct kvm_msr_entry *entry, CPUState *env)
return 0;
}
-#ifdef TARGET_X86_64
-#define MSR_COUNT 9
-#else
-#define MSR_COUNT 5
-#endif
-
static void set_v8086_seg(struct kvm_segment *lhs, const SegmentCache *rhs)
{
lhs->selector = rhs->selector;
@@ -868,7 +862,7 @@ void kvm_arch_load_regs(CPUState *env)
struct kvm_regs regs;
struct kvm_fpu fpu;
struct kvm_sregs sregs;
- struct kvm_msr_entry msrs[MSR_COUNT];
+ struct kvm_msr_entry msrs[100];
int rc, n, i;
regs.rax = env->regs[R_EAX];
@@ -1021,7 +1015,7 @@ void kvm_arch_save_regs(CPUState *env)
struct kvm_regs regs;
struct kvm_fpu fpu;
struct kvm_sregs sregs;
- struct kvm_msr_entry msrs[MSR_COUNT];
+ struct kvm_msr_entry msrs[100];
uint32_t hflags;
uint32_t i, n, rc;
--
1.6.2.5
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
