Re: [RFC v2 01/12] KVM: arm/arm64: Set dist->spis to NULL after kfree

2018-03-28 Thread Auger Eric
Hi Marc, On 27/03/18 19:10, Marc Zyngier wrote: > On 27/03/18 15:04, Eric Auger wrote: >> in case kvm_vgic_map_resources() fails, typically if the vgic >> distributor is not defined, __kvm_vgic_destroy will be called >> several times. Indeed kvm_vgic_map_resources() is called on >> first vcpu run.

Re: [RFC v2 01/12] KVM: arm/arm64: Set dist->spis to NULL after kfree

2018-03-28 Thread Auger Eric
Hi Marc, On 27/03/18 19:10, Marc Zyngier wrote: > On 27/03/18 15:04, Eric Auger wrote: >> in case kvm_vgic_map_resources() fails, typically if the vgic >> distributor is not defined, __kvm_vgic_destroy will be called >> several times. Indeed kvm_vgic_map_resources() is called on >> first vcpu run.

Re: [RFC v2 01/12] KVM: arm/arm64: Set dist->spis to NULL after kfree

2018-03-27 Thread Marc Zyngier
On 27/03/18 15:04, Eric Auger wrote: > in case kvm_vgic_map_resources() fails, typically if the vgic > distributor is not defined, __kvm_vgic_destroy will be called > several times. Indeed kvm_vgic_map_resources() is called on > first vcpu run. As a result dist->spis is freeed more than once > and

Re: [RFC v2 01/12] KVM: arm/arm64: Set dist->spis to NULL after kfree

2018-03-27 Thread Marc Zyngier
On 27/03/18 15:04, Eric Auger wrote: > in case kvm_vgic_map_resources() fails, typically if the vgic > distributor is not defined, __kvm_vgic_destroy will be called > several times. Indeed kvm_vgic_map_resources() is called on > first vcpu run. As a result dist->spis is freeed more than once > and

[RFC v2 01/12] KVM: arm/arm64: Set dist->spis to NULL after kfree

2018-03-27 Thread Eric Auger
in case kvm_vgic_map_resources() fails, typically if the vgic distributor is not defined, __kvm_vgic_destroy will be called several times. Indeed kvm_vgic_map_resources() is called on first vcpu run. As a result dist->spis is freeed more than once and on the second time it causes a "kernel BUG at

[RFC v2 01/12] KVM: arm/arm64: Set dist->spis to NULL after kfree

2018-03-27 Thread Eric Auger
in case kvm_vgic_map_resources() fails, typically if the vgic distributor is not defined, __kvm_vgic_destroy will be called several times. Indeed kvm_vgic_map_resources() is called on first vcpu run. As a result dist->spis is freeed more than once and on the second time it causes a "kernel BUG at