From patchwork Wed Nov 29 16:38:26 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Serhii Popovych X-Patchwork-Id: 842646 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=kvm-ppc-owner@vger.kernel.org; receiver=) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3yn5pZ0Qf6z9t3m for ; Thu, 30 Nov 2017 03:39:18 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933685AbdK2Qis (ORCPT ); Wed, 29 Nov 2017 11:38:48 -0500 Received: from mx1.redhat.com ([209.132.183.28]:58270 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933664AbdK2Qia (ORCPT ); Wed, 29 Nov 2017 11:38:30 -0500 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id CC061C02738B; Wed, 29 Nov 2017 16:38:30 +0000 (UTC) Received: from tuxracer.localdomain (ibm-p8-virt-03.lab.eng.rdu.redhat.com [10.12.2.14]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 480955D964; Wed, 29 Nov 2017 16:38:30 +0000 (UTC) From: Serhii Popovych To: linux-kernel@vger.kernel.org Cc: michael@ellerman.id.au, paulus@samba.org, linuxppc-dev@lists.ozlabs.org, kvm-ppc@vger.kernel.org, david@gibson.dropbear.id.au Subject: [PATCH 4/4] KVM: PPC: Book3S HV: Remove redundant parameter from resize_hpt_release() Date: Wed, 29 Nov 2017 11:38:26 -0500 Message-Id: <1511973506-65683-5-git-send-email-spopovyc@redhat.com> In-Reply-To: <1511973506-65683-1-git-send-email-spopovyc@redhat.com> References: <1511973506-65683-1-git-send-email-spopovyc@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.31]); Wed, 29 Nov 2017 16:38:30 +0000 (UTC) Sender: kvm-ppc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm-ppc@vger.kernel.org There is no need to pass it explicitly from the caller: struct kvm_resize_hpt already contains it. Additional benefit from this change is that BUG_ON() assertion now checks that mutex is held on kvm instance associated with resize structure we going to release. Also kill check for resize being NULL to make code simpler and we called with resize != NULL in all places except kvm_vm_ioctl_resize_hpt_commit(). Signed-off-by: Serhii Popovych Reviewed-by: David Gibson --- arch/powerpc/kvm/book3s_64_mmu_hv.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/arch/powerpc/kvm/book3s_64_mmu_hv.c b/arch/powerpc/kvm/book3s_64_mmu_hv.c index 690f061..a74a0ad 100644 --- a/arch/powerpc/kvm/book3s_64_mmu_hv.c +++ b/arch/powerpc/kvm/book3s_64_mmu_hv.c @@ -1415,12 +1415,11 @@ static void resize_hpt_pivot(struct kvm_resize_hpt *resize) resize_hpt_debug(resize, "resize_hpt_pivot() done\n"); } -static void resize_hpt_release(struct kvm *kvm, struct kvm_resize_hpt *resize) +static void resize_hpt_release(struct kvm_resize_hpt *resize) { - BUG_ON(!mutex_is_locked(&kvm->lock)); + struct kvm *kvm = resize->kvm; - if (!resize) - return; + BUG_ON(!mutex_is_locked(&kvm->lock)); if (resize->error != -EBUSY) { kvmppc_free_hpt(&resize->hpt); @@ -1469,7 +1468,7 @@ static void resize_hpt_prepare_work(struct work_struct *work) resize->error = err; if (kvm->arch.resize_hpt != resize) - resize_hpt_release(kvm, resize); + resize_hpt_release(resize); mutex_unlock(&kvm->lock); } @@ -1499,13 +1498,13 @@ long kvm_vm_ioctl_resize_hpt_prepare(struct kvm *kvm, if (ret == -EBUSY) ret = 100; /* estimated time in ms */ else if (ret) - resize_hpt_release(kvm, resize); + resize_hpt_release(resize); goto out; } /* not suitable, cancel it */ - resize_hpt_release(kvm, resize); + resize_hpt_release(resize); } ret = 0; @@ -1590,7 +1589,8 @@ long kvm_vm_ioctl_resize_hpt_commit(struct kvm *kvm, kvm->arch.mmu_ready = 1; smp_mb(); out_no_hpt: - resize_hpt_release(kvm, resize); + if (resize) + resize_hpt_release(resize); mutex_unlock(&kvm->lock); return ret; }