On 2015-08-07 09:59 AM, Paolo Bonzini wrote:
>
>
> On 07/08/2015 15:47, Nicholas Krause wrote:
>> - kvm_set_irq_routing(opp->kvm, routing, 0, 0);
>> + ret = kvm_set_irq_routing(opp->kvm, routing, 0, 0);
>> + if (ret) {
>> + kfree(routing);
>> + return ret;
>> + }
>>
>> kfree(routing);
>> return 0;
>
> You could just return ret here. The new "if" is not necessary.
>
> Paolo
>
Ok sure that seems fine to be and much simpler then this.
Sorry about the wasting your time with this patch.
Nick
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
