diff options
author | Gregory Haskins <ghaskins@novell.com> | 2009-06-01 12:54:45 -0400 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2009-09-10 01:32:45 -0400 |
commit | 787a660a4f03325a0e00493ac39017e53fd345fa (patch) | |
tree | 1732fc0b8687f2ca9ab25066c364301803898b0c | |
parent | 2986b8c72c272ea58edd37903b042c6da985627d (diff) |
KVM: Clean up coalesced_mmio destruction
We invoke kfree() on a data member instead of the structure. This works today
because the kvm_io_device is the first element of the private structure, but
this could change in the future, so lets clean this up.
Signed-off-by: Gregory Haskins <ghaskins@novell.com>
Acked-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: Avi Kivity <avi@redhat.com>
-rw-r--r-- | virt/kvm/coalesced_mmio.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/virt/kvm/coalesced_mmio.c b/virt/kvm/coalesced_mmio.c index 5ae620d32fac..03ea2807b1f2 100644 --- a/virt/kvm/coalesced_mmio.c +++ b/virt/kvm/coalesced_mmio.c | |||
@@ -80,7 +80,10 @@ static void coalesced_mmio_write(struct kvm_io_device *this, | |||
80 | 80 | ||
81 | static void coalesced_mmio_destructor(struct kvm_io_device *this) | 81 | static void coalesced_mmio_destructor(struct kvm_io_device *this) |
82 | { | 82 | { |
83 | kfree(this); | 83 | struct kvm_coalesced_mmio_dev *dev = |
84 | (struct kvm_coalesced_mmio_dev *)this->private; | ||
85 | |||
86 | kfree(dev); | ||
84 | } | 87 | } |
85 | 88 | ||
86 | int kvm_coalesced_mmio_init(struct kvm *kvm) | 89 | int kvm_coalesced_mmio_init(struct kvm *kvm) |