Thread (127 messages) 127 messages, 14 authors, 2022-12-01

Re: [PATCH 02/44] KVM: Initialize IRQ FD after arch hardware setup

From: Sean Christopherson <seanjc@google.com>
Date: 2022-11-04 20:15:45
Also in: kvm, kvm-riscv, kvmarm, linux-mips, linux-riscv, linux-s390, linuxppc-dev, lkml

On Fri, Nov 04, 2022, Chao Gao wrote:
On Wed, Nov 02, 2022 at 11:18:29PM +0000, Sean Christopherson wrote:
quoted
+	r = kvm_irqfd_init();
+	if (r)
+		goto err_irqfd;
+
	r = kvm_async_pf_init();
	if (r)
-		goto out_free_4;
+		goto err_async_pf;

	kvm_chardev_ops.owner = module;
@@ -5927,6 +5926,9 @@ int kvm_init(void *opaque, unsigned vcpu_size, unsigned vcpu_align,
	kvm_vfio_ops_exit();
err_vfio:
	kvm_async_pf_deinit();
+err_async_pf:
+	kvm_irqfd_exit();
quoted
+err_irqfd:
out_free_4:
Do you mind removing one of the two labels?
Ah, I meant to tack on a patch at the very end to clean up these labels once the
dust had settled, e.g. to also resolve the "err" vs. "out" mess I created (on
purpose, because trying to describe the "out" path was frustrating and generated
too much churn).

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help