From patchwork Fri Jan 20 18:20:50 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: SF Markus Elfring X-Patchwork-Id: 717866 X-Patchwork-Delegate: paulus@samba.org Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3v4pwq3rBHz9sCX for ; Sat, 21 Jan 2017 05:23:07 +1100 (AEDT) Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 3v4pwq31jmzDqlh for ; Sat, 21 Jan 2017 05:23:07 +1100 (AEDT) X-Original-To: linuxppc-dev@lists.ozlabs.org Delivered-To: linuxppc-dev@lists.ozlabs.org Received: from mout.web.de (mout.web.de [212.227.15.3]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3v4pvP0qH3zDqcl for ; Sat, 21 Jan 2017 05:21:52 +1100 (AEDT) Received: from [192.168.1.2] ([77.182.231.71]) by smtp.web.de (mrweb004 [213.165.67.108]) with ESMTPSA (Nemesis) id 0MPaxV-1cZIVb21o1-004hXE; Fri, 20 Jan 2017 19:20:54 +0100 Subject: [PATCH 02/11] KVM: PPC: Book3S HV: Improve a size determination in kvmppc_alloc_hpt() To: kvm@vger.kernel.org, kvm-ppc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Alexander Graf , Benjamin Herrenschmidt , Michael Ellerman , Paolo Bonzini , =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= References: From: SF Markus Elfring Message-ID: <755544bd-b29c-f240-d526-8a8aec6d620a@users.sourceforge.net> Date: Fri, 20 Jan 2017 19:20:50 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: X-Provags-ID: V03:K0:VQ3fnq2X2AA1shqEiCeq7WlMRjy6EppxtEgbrOcQt5k3VgWrRKT K9iQWAzC6iSnqOAGEaI9UJTrBf7+7zY9UvfTqcL856W/uDxPEvqDJle9dh2DgoaSZ1rnxPj epUz1AqpDJJ15FUT1xsXX2H78tvB5ct7k1r55baQCRlV6oNAd8/e2T6DgbvtByRPSLq5w4+ zjXIXa2DH4U8Hs+NmeyvQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:oWC4zE/lwGI=:BmFw+FqGHnxjiI63qsO5ST 5WDMlON0jDPM5WAB/NatqamE+KpquCekux8yoleSVZPeVWCuXnhAuhGAaf7yDa3+uMOFID3qo LsmX3nSegIaGmHYaKwx0sYiWsvdafZF0tGHIefEcSCNT6xSCjawCJ2zfd9G+s2xOj8iTNwKH3 Nwh706lr3q88hQErHCWkA2AKDEFvw80m4Y+0CBfh9mT+Y0DA+74PheE985ShxhMCqScoYa/CD ZLJ6X2XgEccGApuy6OzthuKfahnA244SW/glI1+S6YD0QBxWPE5P0Bisq4xbadcCvo9p633V8 ij3MADPfJivzrebfe/QWZPXEGHiz8dM3MfzFBOdnTOUu3c6AZEWHR77h53JS1r63B2ZRUPuq+ kU6uOLO1APGNQ1TBf+gF+k/Cl2XtU6NGcJGZL3mIAP1deuPiPy1k9vZjr7rOJgj38lfwmd8V0 OJN/S7TjexNNAPk/aSF2cGEmpaGs+CIY+LUTH3Lv6dJXXMre8sply973QURL9L6L2NtR5TPbs JEbaogbNv+Umkh7XtuMXwH7agPSImFVB+H+3N4OJPv7o9cnQDJ4qac+NzCulA6oYV3AFkh6t2 Tx1R6kNw49xWx4dJQgNr1odKRTHOx2UWMV5k+f3PcL12CdHf35QSdApKOy1KV2B6mT8LU97gH AeZxlrhgxOlpzCUhbCg4NK7nWFVdi6ZHoQoLO3udkoMbGmSbynriAGBneOUYgdy0N56xx0LZi NuhM6qPvWzi/N5pBzz3yaqfTeEtXSwdeMgUcWDbaWYo+jfQNJ4hLSwAZRESZMrmwkg1AGPWxT Kd8lU3c X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kernel-janitors@vger.kernel.org, LKML Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: "Linuxppc-dev" From: Markus Elfring Date: Fri, 20 Jan 2017 10:00:13 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus Elfring --- arch/powerpc/kvm/book3s_64_mmu_hv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kvm/book3s_64_mmu_hv.c b/arch/powerpc/kvm/book3s_64_mmu_hv.c index dc34729e4ad0..4655f27e0518 100644 --- a/arch/powerpc/kvm/book3s_64_mmu_hv.c +++ b/arch/powerpc/kvm/book3s_64_mmu_hv.c @@ -91,7 +91,7 @@ long kvmppc_alloc_hpt(struct kvm *kvm, u32 *htab_orderp) atomic64_set(&kvm->arch.mmio_update, 0); /* Allocate reverse map array */ - rev = vmalloc(sizeof(struct revmap_entry) * kvm->arch.hpt_npte); + rev = vmalloc(sizeof(*rev) * kvm->arch.hpt_npte); if (!rev) { pr_err("kvmppc_alloc_hpt: Couldn't alloc reverse map array\n"); goto out_freehpt;