public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0 of 3] cosmetic fixes
@ 2007-11-19 20:04 Hollis Blanchard
  2007-11-19 20:04 ` [PATCH 1 of 3] Correct consistent typo: "destory" -> "destroy" Hollis Blanchard
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Hollis Blanchard @ 2007-11-19 20:04 UTC (permalink / raw)
  To: Avi Kivity; +Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

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 +-

-------------------------------------------------------------------------
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] 5+ messages in thread

* [PATCH 1 of 3] Correct consistent typo: "destory" -> "destroy"
  2007-11-19 20:04 [PATCH 0 of 3] cosmetic fixes Hollis Blanchard
@ 2007-11-19 20:04 ` Hollis Blanchard
  2007-11-19 20:04 ` [PATCH 2 of 3] Move misplaced comment Hollis Blanchard
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Hollis Blanchard @ 2007-11-19 20:04 UTC (permalink / raw)
  To: Avi Kivity; +Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

Signed-off-by: Hollis Blanchard <hollisb-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>

---
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
+++ b/drivers/kvm/kvm.h
@@ -462,7 +462,7 @@ void kvm_arch_vcpu_load(struct kvm_vcpu 
 void kvm_arch_vcpu_load(struct kvm_vcpu *vcpu, int cpu);
 void kvm_arch_vcpu_put(struct kvm_vcpu *vcpu);
 struct kvm_vcpu *kvm_arch_vcpu_create(struct kvm *kvm, unsigned int id);
-void kvm_arch_vcpu_destory(struct kvm_vcpu *vcpu);
+void kvm_arch_vcpu_destroy(struct kvm_vcpu *vcpu);
 
 int kvm_arch_vcpu_reset(struct kvm_vcpu *vcpu);
 void kvm_arch_hardware_enable(void *garbage);
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
--- a/drivers/kvm/kvm_main.c
+++ b/drivers/kvm/kvm_main.c
@@ -838,7 +838,7 @@ unlink:
 	kvm->vcpus[n] = NULL;
 	mutex_unlock(&kvm->lock);
 vcpu_destroy:
-	kvm_arch_vcpu_destory(vcpu);
+	kvm_arch_vcpu_destroy(vcpu);
 	return r;
 }
 
diff --git a/drivers/kvm/x86.c b/drivers/kvm/x86.c
--- a/drivers/kvm/x86.c
+++ b/drivers/kvm/x86.c
@@ -2513,7 +2513,7 @@ fail:
 	return ERR_PTR(r);
 }
 
-void kvm_arch_vcpu_destory(struct kvm_vcpu *vcpu)
+void kvm_arch_vcpu_destroy(struct kvm_vcpu *vcpu)
 {
 	vcpu_load(vcpu);
 	kvm_mmu_unload(vcpu);

-------------------------------------------------------------------------
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] 5+ messages in thread

* [PATCH 2 of 3] Move misplaced comment
  2007-11-19 20:04 [PATCH 0 of 3] cosmetic fixes Hollis Blanchard
  2007-11-19 20:04 ` [PATCH 1 of 3] Correct consistent typo: "destory" -> "destroy" Hollis Blanchard
@ 2007-11-19 20:04 ` Hollis Blanchard
  2007-11-19 20:04 ` [PATCH 3 of 3] Remove unused "rmap_overflow" variable Hollis Blanchard
  2007-11-20  9:32 ` [PATCH 0 of 3] cosmetic fixes Avi Kivity
  3 siblings, 0 replies; 5+ messages in thread
From: Hollis Blanchard @ 2007-11-19 20:04 UTC (permalink / raw)
  To: Avi Kivity; +Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

Signed-off-by: Hollis Blanchard <hollisb-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>

---
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 KVM_NR_MEM_OBJS 40
 
+/*
+ * We don't want allocation failures within the mmu code, so we preallocate
+ * enough memory for a single page fault in a cache.
+ */
 struct kvm_mmu_memory_cache {
 	int nobjs;
 	void *objects[KVM_NR_MEM_OBJS];
 };
 
-/*
- * We don't want allocation failures within the mmu code, so we preallocate
- * enough memory for a single page fault in a cache.
- */
 struct kvm_guest_debug {
 	int enabled;
 	unsigned long bp[4];

-------------------------------------------------------------------------
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] 5+ messages in thread

* [PATCH 3 of 3] Remove unused "rmap_overflow" variable
  2007-11-19 20:04 [PATCH 0 of 3] cosmetic fixes Hollis Blanchard
  2007-11-19 20:04 ` [PATCH 1 of 3] Correct consistent typo: "destory" -> "destroy" Hollis Blanchard
  2007-11-19 20:04 ` [PATCH 2 of 3] Move misplaced comment Hollis Blanchard
@ 2007-11-19 20:04 ` Hollis Blanchard
  2007-11-20  9:32 ` [PATCH 0 of 3] cosmetic fixes Avi Kivity
  3 siblings, 0 replies; 5+ messages in thread
From: Hollis Blanchard @ 2007-11-19 20:04 UTC (permalink / raw)
  To: Avi Kivity; +Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

Signed-off-by: Hollis Blanchard <hollisb-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>

---
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;
 	struct hlist_head mmu_page_hash[KVM_NUM_MMU_PAGES];
 	struct kvm_vcpu *vcpus[KVM_MAX_VCPUS];
-	unsigned long rmap_overflow;
 	struct list_head vm_list;
 	struct file *filp;
 	struct kvm_io_bus mmio_bus;

-------------------------------------------------------------------------
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] 5+ messages in thread

* Re: [PATCH 0 of 3] cosmetic fixes
  2007-11-19 20:04 [PATCH 0 of 3] cosmetic fixes Hollis Blanchard
                   ` (2 preceding siblings ...)
  2007-11-19 20:04 ` [PATCH 3 of 3] Remove unused "rmap_overflow" variable Hollis Blanchard
@ 2007-11-20  9:32 ` Avi Kivity
  3 siblings, 0 replies; 5+ messages in thread
From: Avi Kivity @ 2007-11-20  9:32 UTC (permalink / raw)
  To: Hollis Blanchard; +Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f

Hollis Blanchard wrote:
> These are a few fixes to minor/cosmetic issues I've stumbled across recently.
>
>   
Thanks, applied all three.


-- 
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] 5+ messages in thread

end of thread, other threads:[~2007-11-20  9:32 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-11-19 20:04 [PATCH 0 of 3] cosmetic fixes Hollis Blanchard
2007-11-19 20:04 ` [PATCH 1 of 3] Correct consistent typo: "destory" -> "destroy" Hollis Blanchard
2007-11-19 20:04 ` [PATCH 2 of 3] Move misplaced comment Hollis Blanchard
2007-11-19 20:04 ` [PATCH 3 of 3] Remove unused "rmap_overflow" variable Hollis Blanchard
2007-11-20  9:32 ` [PATCH 0 of 3] cosmetic fixes Avi Kivity

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