On Mon, 19 Nov 2018 18:25:36 +0100
Michael Mueller <mimu@xxxxxxxxxxxxx> wrote:
> By initializing the GIB, it will we used by the kvm host.
>
> Signed-off-by: Michael Mueller <mimu@xxxxxxxxxxxxx>
> ---
> arch/s390/kvm/kvm-s390.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
> index 386f98029a3f..08341be4d0aa 100644
> --- a/arch/s390/kvm/kvm-s390.c
> +++ b/arch/s390/kvm/kvm-s390.c
> @@ -417,6 +417,8 @@ static void kvm_s390_cpu_feat_init(void)
>
> int kvm_arch_init(void *opaque)
> {
> + int rc;
> +
> kvm_s390_dbf = debug_register("kvm-trace", 32, 1, 7 * sizeof(long));
> if (!kvm_s390_dbf)
> return -ENOMEM;
> @@ -426,6 +428,10 @@ int kvm_arch_init(void *opaque)
> return -ENOMEM;
> }
>
> + rc = kvm_s390_gib_init(GAL_ISC);
> + if (rc)
> + return rc;
Is a quick exit the right thing here? Doesn't that leave a memory leak
because you did not unregister the dbf again?
> +
> kvm_s390_cpu_feat_init();
>
> /* Register floating interrupt controller interface. */
Further below, we'll have the same problem (dbf leak) if registering
the flic should fail. This is extremely unlikely... but don't we also
need to clean up the gib again in that case? For all later failure
cases in kvm init, the exit function is called, but that one may be a
problem.
