RE: [PATCH 2/4] [v2] kvm: qemu: Fix leak of ioperm data

2008-12-10 Thread Han, Weidong
Mark McLoughlin wrote: > On Wed, 2008-12-10 at 21:22 +0800, Han, Weidong wrote: >> >> +void kvm_remove_ioperm_data(unsigned long start_port, unsigned long >> num) +{ +struct ioperm_data *data; >> + >> +data = LIST_FIRST(&ioperm_head); >> +while (data) { >> +if (data->start_port

RE: [PATCH 2/4] [v2] kvm: qemu: Fix leak of ioperm data

2008-12-10 Thread Han, Weidong
Mark McLoughlin wrote: > On Wed, 2008-12-10 at 21:22 +0800, Han, Weidong wrote: >> >> +void kvm_remove_ioperm_data(unsigned long start_port, unsigned long >> num) +{ +struct ioperm_data *data; >> + >> +data = LIST_FIRST(&ioperm_head); >> +while (data) { >> +if (data->start_port

Re: [PATCH 2/4] [v2] kvm: qemu: Fix leak of ioperm data

2008-12-10 Thread Mark McLoughlin
On Wed, 2008-12-10 at 21:22 +0800, Han, Weidong wrote: > > +void kvm_remove_ioperm_data(unsigned long start_port, unsigned long num) > +{ > +struct ioperm_data *data; > + > +data = LIST_FIRST(&ioperm_head); > +while (data) { > +if (data->start_port == start_port && data->num ==

[PATCH 2/4] [v2] kvm: qemu: Fix leak of ioperm data

2008-12-10 Thread Han, Weidong
Free ioperm data in free_assigned_device. And also, define ioperm_data structure and declare related functions when USE_KVM_DEVICE_ASSIGNMENT is defined, because ioperm data is only used by device assignment. Signed-off-by: Weidong Han <[EMAIL PROTECTED]> --- qemu/hw/device-assignment.c |2