Received: by 2002:ac0:bc90:0:0:0:0:0 with SMTP id a16csp300991img; Wed, 20 Mar 2019 00:36:57 -0700 (PDT) X-Google-Smtp-Source: APXvYqwaVSqNtw5LEoJECVEANFs2JIXQ5OMoalsdFm2wwhJg5UjBoy5qV3mipIqPWR1EmQJkOXuM X-Received: by 2002:aa7:87c5:: with SMTP id i5mr6209562pfo.20.1553067417754; Wed, 20 Mar 2019 00:36:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1553067417; cv=none; d=google.com; s=arc-20160816; b=bU1TJSStm/y+RIVZmuPepL6LsGwZg4N3dmPoft7xLZPx3o4nkMqwYRwU9Rl2mVW2Mn SyaAF3oTTNgBj1MG8A6rRrLy613M+RAuMAK7M1Dnok3fF+nYPd4M1Af6xyC587r6sx4C Jhx2c0DFJqRIpj4XunC55zJvgNyvo297tP029/JbTVaV6OwfPDHuzQ/EUQM92hVpT4mr hNybXFAYIfnW5y1BQUYOL4liskv3grFTA2JRuuanzXVXpKCaDt5EK4mm4FwVEHu8Hbzv TAAIz+dN1WsFeXkCOS2wO1k+8lGJW3u0EoVNMHPUn+yDWuUtLlxV6dRrvidfiVgMyj4P EW1A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from; bh=w9cFJm+HEeCHcX9JqiAPkwlfm2xd5GqjMiIlH+9+Qg0=; b=uext5bYMooCkpVttpCB74GazJQOEXXnroGGHVxZTxXrPdmshfgM04tlvgEXAynEmmc BsK4DHNi4E12/U6opjrEVjwCjhZTR9D+g2z68sXcBevxy12c0D8nJRVQckPmpwrejZ0i oFKcQCN8Se1/LbwIeqCC1MZ0LPPVAW0tCTorSZCt9TVzg9FgPBjriyAEA5SaBmGcekrF ZgtfDJg74n1IEtdZwJhInWRMXhitFouBj/K/yWD3R0b8YXM8dBbfwi211FY1GMAJ3OZ1 c/hjy7TU4+IQNwSBlz2Om9TW7cHN7Xzt1eJJsfg2GphN229Qupu29zSpUmDdKpaK7JGn XHGQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id a1si1069390pfn.245.2019.03.20.00.36.42; Wed, 20 Mar 2019 00:36:57 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727544AbfCTHf6 (ORCPT + 99 others); Wed, 20 Mar 2019 03:35:58 -0400 Received: from mx1.redhat.com ([209.132.183.28]:55384 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726303AbfCTHf5 (ORCPT ); Wed, 20 Mar 2019 03:35:57 -0400 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 9578D31688EB; Wed, 20 Mar 2019 07:35:57 +0000 (UTC) Received: from MiWiFi-R3L-srv.redhat.com (ovpn-12-38.pek2.redhat.com [10.72.12.38]) by smtp.corp.redhat.com (Postfix) with ESMTP id C8C4F60CD3; Wed, 20 Mar 2019 07:35:54 +0000 (UTC) From: Baoquan He To: linux-kernel@vger.kernel.org Cc: akpm@linux-foundation.org, pasha.tatashin@oracle.com, mhocko@suse.com, rppt@linux.vnet.ibm.com, richard.weiyang@gmail.com, linux-mm@kvack.org, Baoquan He Subject: [PATCH 2/3] mm/sparse: Optimize sparse_add_one_section() Date: Wed, 20 Mar 2019 15:35:39 +0800 Message-Id: <20190320073540.12866-2-bhe@redhat.com> In-Reply-To: <20190320073540.12866-1-bhe@redhat.com> References: <20190320073540.12866-1-bhe@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.41]); Wed, 20 Mar 2019 07:35:57 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Reorder the allocation of usemap and memmap since usemap allocation is much smaller and simpler. Otherwise hard work is done to make memmap ready, then have to rollback just because of usemap allocation failure. Signed-off-by: Baoquan He --- mm/sparse.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/mm/sparse.c b/mm/sparse.c index 0a0f82c5d969..054b99f74181 100644 --- a/mm/sparse.c +++ b/mm/sparse.c @@ -697,16 +697,17 @@ int __meminit sparse_add_one_section(int nid, unsigned long start_pfn, ret = sparse_index_init(section_nr, nid); if (ret < 0 && ret != -EEXIST) return ret; - ret = 0; - memmap = kmalloc_section_memmap(section_nr, nid, altmap); - if (!memmap) - return -ENOMEM; + usemap = __kmalloc_section_usemap(); - if (!usemap) { - __kfree_section_memmap(memmap, altmap); + if (!usemap) + return -ENOMEM; + memmap = kmalloc_section_memmap(section_nr, nid, altmap); + if (!memmap) { + kfree(usemap); return -ENOMEM; } + ret = 0; ms = __pfn_to_section(start_pfn); if (ms->section_mem_map & SECTION_MARKED_PRESENT) { ret = -EEXIST; -- 2.17.2