Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp793459pxb; Tue, 3 Nov 2020 12:41:18 -0800 (PST) X-Google-Smtp-Source: ABdhPJy1lhIsL9SfH0Q50D+AqRLBiZEfMOpSGezaL3+iVaag0QrWcGw+FOoWJsGczwpcUf3j7v57 X-Received: by 2002:aa7:d1c6:: with SMTP id g6mr5900600edp.130.1604436078731; Tue, 03 Nov 2020 12:41:18 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1604436078; cv=none; d=google.com; s=arc-20160816; b=KeKS7gx3y02tlp+6Ka9gK4F7lcdQVzZ8SnkhZxAa7QL1KeXJzSV20jVry/1AqENpeu 9hMjqe/T5y6AXc2WVf0NXEdYbmP30f/RiSE4aI6ab7xEsnsE35bx/NA6rTuine6Xp0ka NLpTZLRelkyRsuGbXq3VtHAadCpY9aNjkIcCBvkJ2u9BQ79r4K4hxL3hQMnXkDdBaVup wvA/s8KSW4IIRhWb2GNhL+AztE5uPAqOAbQfInoC6APTDLAkhSXpGevyYrYhnTE4RUvR oi82aB9fRePlmOePwbRfBRFlHyESS1srySR9lRJvLZvRyJkeYzzeIl1OMNr1dPtJtJY+ Acug== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=v+rL5mzGv9eyMkC4+5wlBSrKrxje1Bc4lB8ixgnWAxQ=; b=c8Ne2UqYufN0fpz5UMauOXDydDuzZAHdg/wIBeLuZ84bs//5Mvd6ZxoYlUaD/EoOfU +NngZ7G52OuDGjfOyqEPe5Kqh2AgsPvUxX1yeVriAMT9z5RhmoO3Eihpfc466jCKQnTS XnJLmb9XIVLVODIX6stAZNH+lQOEvlvoPWC7vB58SWF7LobDpXGOKgxr9kaui/YN9OsJ oW0MTeq6Dj3CqrHt1qpqNQzxoZD2l0K85j+hU65TZ67q1aKOoIpSjJUJnRqn8iwB1EUC +hDr5RNPdNMgkMofx1y0hnkd3Mnnj+NHoeyAEcHjFeK9z/LISuo9oygl0Psj83YFKDdZ jZwg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=V8RkMYhc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id r25si4137037edy.166.2020.11.03.12.40.55; Tue, 03 Nov 2020 12:41:18 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=V8RkMYhc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730014AbgKCUiu (ORCPT + 99 others); Tue, 3 Nov 2020 15:38:50 -0500 Received: from mail.kernel.org ([198.145.29.99]:49262 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729403AbgKCUis (ORCPT ); Tue, 3 Nov 2020 15:38:48 -0500 Received: from localhost (83-86-74-64.cable.dynamic.v4.ziggo.nl [83.86.74.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5AB0122277; Tue, 3 Nov 2020 20:38:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1604435927; bh=kgeIlsTT05XlXvQJ8pWD+5Y2tPCb2uZZkCHsNJwG7fI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=V8RkMYhcfosBxc+D47hV9DMLfF9K7EUUYJknDZQnD9sO0LNtgNxMSgr2FxZa9tDxH 1pxxvU2ZkyGE1QNngQCDiR5DnOoqSO5igkZWvu7xn47FAJNGTQCC7vQPcbt8z0BiLs lq4uDv3m8tAOvmMgVPLVXx+WEpvqclmAktyIcH54= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, "Aneesh Kumar K.V" , Michael Ellerman , Sasha Levin Subject: [PATCH 5.9 043/391] powerpc/vmemmap: Fix memory leak with vmemmap list allocation failures. Date: Tue, 3 Nov 2020 21:31:34 +0100 Message-Id: <20201103203350.517420128@linuxfoundation.org> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20201103203348.153465465@linuxfoundation.org> References: <20201103203348.153465465@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Aneesh Kumar K.V [ Upstream commit ccaea15296f9773abd43aaa17ee4b88848e4a505 ] If we fail to allocate vmemmap list, we don't keep track of allocated vmemmap block buf. Hence on section deactivate we skip vmemmap block buf free. This results in memory leak. Signed-off-by: Aneesh Kumar K.V Signed-off-by: Michael Ellerman Link: https://lore.kernel.org/r/20200731113500.248306-1-aneesh.kumar@linux.ibm.com Signed-off-by: Sasha Levin --- arch/powerpc/mm/init_64.c | 35 ++++++++++++++++++++++++++++------- 1 file changed, 28 insertions(+), 7 deletions(-) diff --git a/arch/powerpc/mm/init_64.c b/arch/powerpc/mm/init_64.c index 8459056cce671..2ae42c2a5cf04 100644 --- a/arch/powerpc/mm/init_64.c +++ b/arch/powerpc/mm/init_64.c @@ -162,16 +162,16 @@ static __meminit struct vmemmap_backing * vmemmap_list_alloc(int node) return next++; } -static __meminit void vmemmap_list_populate(unsigned long phys, - unsigned long start, - int node) +static __meminit int vmemmap_list_populate(unsigned long phys, + unsigned long start, + int node) { struct vmemmap_backing *vmem_back; vmem_back = vmemmap_list_alloc(node); if (unlikely(!vmem_back)) { - WARN_ON(1); - return; + pr_debug("vmemap list allocation failed\n"); + return -ENOMEM; } vmem_back->phys = phys; @@ -179,6 +179,7 @@ static __meminit void vmemmap_list_populate(unsigned long phys, vmem_back->list = vmemmap_list; vmemmap_list = vmem_back; + return 0; } static bool altmap_cross_boundary(struct vmem_altmap *altmap, unsigned long start, @@ -199,6 +200,7 @@ static bool altmap_cross_boundary(struct vmem_altmap *altmap, unsigned long star int __meminit vmemmap_populate(unsigned long start, unsigned long end, int node, struct vmem_altmap *altmap) { + bool altmap_alloc; unsigned long page_size = 1 << mmu_psize_defs[mmu_vmemmap_psize].shift; /* Align to the page size of the linear mapping. */ @@ -228,13 +230,32 @@ int __meminit vmemmap_populate(unsigned long start, unsigned long end, int node, p = vmemmap_alloc_block_buf(page_size, node, altmap); if (!p) pr_debug("altmap block allocation failed, falling back to system memory"); + else + altmap_alloc = true; } - if (!p) + if (!p) { p = vmemmap_alloc_block_buf(page_size, node, NULL); + altmap_alloc = false; + } if (!p) return -ENOMEM; - vmemmap_list_populate(__pa(p), start, node); + if (vmemmap_list_populate(__pa(p), start, node)) { + /* + * If we don't populate vmemap list, we don't have + * the ability to free the allocated vmemmap + * pages in section_deactivate. Hence free them + * here. + */ + int nr_pfns = page_size >> PAGE_SHIFT; + unsigned long page_order = get_order(page_size); + + if (altmap_alloc) + vmem_altmap_free(altmap, nr_pfns); + else + free_pages((unsigned long)p, page_order); + return -ENOMEM; + } pr_debug(" * %016lx..%016lx allocated at %p\n", start, start + page_size, p); -- 2.27.0