On 3/27/2020 4:53 PM, Stefano Garzarella wrote:
> On Fri, Mar 27, 2020 at 11:56:50AM +0800, Pan Nengyuan wrote:
>> req_vq/event_vq forgot to free in unrealize(). Fix that.
>>
>> Signed-off-by: Pan Nengyuan <pannengy...@huawei.com>
>> ---
>> Cc: Eric Auger <eric.au...@redhat.com>
>> ---
>> hw/virtio/virtio-iommu.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c
>> index 4cee8083bc..9d2ff0693c 100644
>> --- a/hw/virtio/virtio-iommu.c
>> +++ b/hw/virtio/virtio-iommu.c
>> @@ -696,6 +696,8 @@ static void virtio_iommu_device_unrealize(DeviceState
>> *dev, Error **errp)
>> g_tree_destroy(s->domains);
>> g_tree_destroy(s->endpoints);
>>
>> + virtio_delete_queue(s->req_vq);
>> + virtio_delete_queue(s->event_vq);
>> virtio_cleanup(vdev);
>> }
>
> Hi Pan,
> thanks for this fix.
>
> Since we are here, should we also clean 's->as_by_busptr' hash table?
>
> Maybe adding this in the unrealize:
>
> g_hash_table_destroy(s->as_by_busptr);
Yes, you are right. I will add it.
Thanks.
>
> Thanks,
> Stefano
>