KVM: fix AUDIT code

PATCH In review Ingo Molnar 2 participants 3 messages Mar 11, 2007 2 messages not in the archive KVM

lore

Share & tools

lore

Ingo Molnar

author

2007-03-11 10:37 UTC #0 lore raw

PATCH 1/1 KVM: fix AUDIT code

From: Ingo Molnar <[email protected]>
Subject: [patch] KVM: fix AUDIT code

fix build bug with AUDIT defined. (it's off by default)

Signed-off-by Ingo Molnar <[email protected]>

drivers/kvm/mmu.c +1 -1

Index: linux/drivers/kvm/mmu.c
===================================================================
@@ -1410,7 +1410,7 @@ static void audit_mappings_page(struct k
 
 static void audit_mappings(struct kvm_vcpu *vcpu)
 {
-	unsigned i;
+	unsigned i, j;
 
 	if (vcpu->mmu.root_level == 4) {
 		audit_mappings_page(vcpu, vcpu->mmu.root_hpa, 0, 4);

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

A message that is not in the archive (sent off-list or not yet received): [email protected]

Avi Kivity

2007-03-11 10:41 UTC #2 lore raw

Quoted 13 lines · Ingo Molnar

From: Ingo Molnar <[email protected]>
Subject: [patch] KVM: fix AUDIT code
fix build bug with AUDIT defined. (it's off by default)
Signed-off-by Ingo Molnar <[email protected]>
Index: linux/drivers/kvm/mmu.c
===================================================================

drivers/kvm/mmu.c static void audit_mappings_page(struct k · line 1416

Quoted 8 lines

drivers/kvm/mmu.c +1 -1

@@ -1410,7 +1410,7 @@ static void audit_mappings_page(struct k
 
 static void audit_mappings(struct kvm_vcpu *vcpu)
 {
-	unsigned i;
+	unsigned i, j;
 
 	if (vcpu->mmu.root_level == 4) {
 		audit_mappings_page(vcpu, vcpu->mmu.root_hpa, 0, 4);

My audit_mappings() doesn't have a j. Is this a cr3 cache tree?

Signature · 8 lines

error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

A message that is not in the archive (sent off-list or not yet received): 45F3DCBE.8010203-atKUWr5tajBWk0Htik3J/[email protected]

Ingo Molnar

author

2007-03-11 10:42 UTC #4 lore raw

Quoted 10 lines · Avi Kivity

Quoted 8 lines
static void audit_mappings(struct kvm_vcpu *vcpu)
{
drivers/kvm/mmu.c +1 -1
-	unsigned i;
+	unsigned i, j;
	if (vcpu->mmu.root_level == 4) {
		audit_mappings_page(vcpu, vcpu->mmu.root_hpa, 0, 4);
My audit_mappings() doesn't have a j. Is this a cr3 cache tree?

ah, indeed. Sorry about the noise :)

Ingo