From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id B9156CFC538 for ; Mon, 14 Oct 2024 10:24:34 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 301AC6B0085; Mon, 14 Oct 2024 06:24:34 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 28AC66B0088; Mon, 14 Oct 2024 06:24:34 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0DBAA6B0089; Mon, 14 Oct 2024 06:24:34 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id DE6346B0085 for ; Mon, 14 Oct 2024 06:24:33 -0400 (EDT) Received: from smtpin20.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id A3A6E120B0B for ; Mon, 14 Oct 2024 10:24:26 +0000 (UTC) X-FDA: 82671823290.20.4E7741B Received: from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by imf28.hostedemail.com (Postfix) with ESMTP id C48CAC0015 for ; Mon, 14 Oct 2024 10:24:24 +0000 (UTC) Authentication-Results: imf28.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b="J1zI4/kG"; dmarc=pass (policy=none) header.from=ibm.com; spf=pass (imf28.hostedemail.com: domain of maddy@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=maddy@linux.ibm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1728901366; a=rsa-sha256; cv=none; b=s0Pg20VcD6DFLlVL9B90jAto43FFncitMyZRtROhTLbWr3Ou4s1qMn37F8eS/W8cSvhrb6 689lBolLxp8c7TxVVBK+OFjI1bLXq23aLs1R0n435RYfT+rQhIo45dj2SUsXTqHs/p7cHJ IytoZh1GF4dH4qR5/8Sog7PgsTw7X88= ARC-Authentication-Results: i=1; imf28.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b="J1zI4/kG"; dmarc=pass (policy=none) header.from=ibm.com; spf=pass (imf28.hostedemail.com: domain of maddy@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=maddy@linux.ibm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1728901366; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=Ocg3L0BVPJRwxCJcj03WGBFydvcJyY9Nl2aza3CZevs=; b=4V1vH2N0fqAkf4sVmwliMxwUMJsZ1+z27h97mSe709DK+ZE+2Bgod/MtYw4bEa27sQv19+ JncAO4aNrFZUajpuKlpzDuRpfiP+VRDdUi8ac1QpuC1MiUL+4dU5U+gASt4Duj91unBIbr yJNTehPUR//HzEPk0ZG7dRfVVRw8ycs= Received: from pps.filterd (m0356516.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 49E7ogVF006067; Mon, 14 Oct 2024 10:24:20 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h= message-id:date:subject:to:cc:references:from:in-reply-to :content-type:content-transfer-encoding:mime-version; s=pp1; bh= Ocg3L0BVPJRwxCJcj03WGBFydvcJyY9Nl2aza3CZevs=; b=J1zI4/kGhDXyQYNQ NA7cBd8+RsnpSRlMo1C0pL6EU/LHnfnDNcuAgEjbLeVHIfyT5ty91FurHJhH6MIC CM7EdPy/l4Us0hNVQ4bdol5g7xx1EDj2WuGRIGtN3SIKx1mE6Jgj9R7WAF0Gk/+E wCAo4EIFYuFSVK2njxE8QuoAKBLWTapagLyvYHhbjibgAUtz+FdxKR9YlzwKb4Zf 8Kjq8OHdwXgkgDLSlfDOuO9eagXIpotBYeMhTtL1dA3qaAHC6PtrsWhbo1+1O9sy QOfYl/9c0O5GjhpnxquVQGZjXe0iT8FOPA8J7nvnBvdCaDIiMOc8ZmxySjnnAzhU aIH8wA== Received: from pps.reinject (localhost [127.0.0.1]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 428y8w0nmf-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 14 Oct 2024 10:24:20 +0000 (GMT) Received: from m0356516.ppops.net (m0356516.ppops.net [127.0.0.1]) by pps.reinject (8.18.0.8/8.18.0.8) with ESMTP id 49EAOJHY029579; Mon, 14 Oct 2024 10:24:19 GMT Received: from ppma22.wdc07v.mail.ibm.com (5c.69.3da9.ip4.static.sl-reverse.com [169.61.105.92]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 428y8w0nmb-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 14 Oct 2024 10:24:19 +0000 (GMT) Received: from pps.filterd (ppma22.wdc07v.mail.ibm.com [127.0.0.1]) by ppma22.wdc07v.mail.ibm.com (8.18.1.2/8.18.1.2) with ESMTP id 49E74jbO027452; Mon, 14 Oct 2024 10:24:19 GMT Received: from smtprelay06.wdc07v.mail.ibm.com ([172.16.1.73]) by ppma22.wdc07v.mail.ibm.com (PPS) with ESMTPS id 4283txe1hf-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 14 Oct 2024 10:24:19 +0000 Received: from smtpav06.wdc07v.mail.ibm.com (smtpav06.wdc07v.mail.ibm.com [10.39.53.233]) by smtprelay06.wdc07v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 49EAOIme27198058 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 14 Oct 2024 10:24:18 GMT Received: from smtpav06.wdc07v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 0551F5804E; Mon, 14 Oct 2024 10:24:18 +0000 (GMT) Received: from smtpav06.wdc07v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id E25D65803F; Mon, 14 Oct 2024 10:24:09 +0000 (GMT) Received: from [9.43.116.47] (unknown [9.43.116.47]) by smtpav06.wdc07v.mail.ibm.com (Postfix) with ESMTP; Mon, 14 Oct 2024 10:24:09 +0000 (GMT) Message-ID: <941875f7-0d7f-4ba3-bc7c-7aedc3b20dae@linux.ibm.com> Date: Mon, 14 Oct 2024 15:54:06 +0530 User-Agent: Mozilla Thunderbird Subject: Re: [RFC v3 1/3] fadump: Refactor and prepare fadump_cma_init for late init To: "Ritesh Harjani (IBM)" , linuxppc-dev@lists.ozlabs.org Cc: linux-mm@kvack.org, Sourabh Jain , Hari Bathini , Zi Yan , David Hildenbrand , "Kirill A . Shutemov" , Mahesh J Salgaonkar , Michael Ellerman , "Aneesh Kumar K . V" , Donet Tom , LKML , Sachin P Bappalige References: <030b6d46fddac126a6cf7e119bea48055338f0ed.1728658614.git.ritesh.list@gmail.com> Content-Language: en-US From: Madhavan Srinivasan In-Reply-To: <030b6d46fddac126a6cf7e119bea48055338f0ed.1728658614.git.ritesh.list@gmail.com> Content-Type: text/plain; charset=UTF-8 X-TM-AS-GCONF: 00 X-Proofpoint-ORIG-GUID: vCjybacVMoVYpyuqbebMZr5tU5ddzP-S X-Proofpoint-GUID: HKCPcImhC0WCwjRQTgK8ZuBP0V1vbSTr Content-Transfer-Encoding: 7bit X-Proofpoint-UnRewURL: 0 URL was un-rewritten MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1051,Hydra:6.0.680,FMLib:17.12.62.30 definitions=2024-10-14_08,2024-10-11_01,2024-09-30_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 bulkscore=0 impostorscore=0 lowpriorityscore=0 phishscore=0 malwarescore=0 adultscore=0 mlxscore=0 priorityscore=1501 clxscore=1011 suspectscore=0 mlxlogscore=999 spamscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2409260000 definitions=main-2410140073 X-Rspamd-Queue-Id: C48CAC0015 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: ps8wnsaqgm1764qh4iyour9im6934j9x X-HE-Tag: 1728901464-725848 X-HE-Meta: U2FsdGVkX18OoJUe2o/LqwiPNgAdGDx7o+MeAxJucBMrqB0uRq1ygrjwlIWqz5tCsrZNP7QWpjvl+3HaiDzmmn8z3K5FQk/JGZrKzFJVkSHo5cf5UZQuLfweg+GX2k8426zVIJWyx3tvU/xx8PhTPt6zQgMdph8KpTBKu5lzbpMUkY+Lgncr1w12aD0zhm1SIhbspQjLJ+wsv1EvBrjlnLRhWHTgowRJ/DuCnT/EQHL1ZI1pFtPosdvZy2rLxlyzQ4uWkB3LF3FOMnvnWQ51S2Sl+auZDmi74tTETMzgMpl+lVXFDToW1tr9VS16UelRvLrpVRaFR86tqAJhy43BRcUK6+dvix91zhlc6AETc4p31uWR/u0/zGRRZl8vFl9y1puPfXp0f4fau3cnAI+7gOS+gLshDF+wOvDJpPvwYV4+7KbEsnSRb/vq3E39QHaZmGTsnX82I7SM/qc+RE9Y3IN22xOhztPb1LyeoEvUm6BlE7vMirK9Dh6mwLmI9Y6XSZog1Qq8zs3C8gUSZWfhqhRVw0IasEzCYgOxHZz+lkgCyJqGY69ziPWDu/UpTR9Ta2BDy5Cb/qGEAlmIkBji83LdxaiISnodJLBgAUmWJX9Yd+IOdrV6lx4yul7IBcTgS3K/2F2KNHKk6u4FU3ez3+S1QPfuzp+qselUnrCKYQcSTCKHixae72zz9zCxHKEZBnmM2wU7wCXYK8RYbkrFP8/5zKI4aXOrp3FT1ZKE6CgFL8bhtOZw7IBy8fpuq3n/ZnnJo5RP2GizMU3f6zdwftWEy3dGwbFmIu4hEDrglxn/E7WQiGDzJOFVAGFvU+2q6Hb6lOqA08IqxbnQ7QDeiqU2kvCRXoVlTsOGl/8sptrzxy3ZpGgvjGPvrwNcB6FQr71M7BsDhQQggfCZFl3U1iMzQTn0cxUpelEJQzatzowwPwtEWBFbppa4EWfamJH5UfpLLi1m2mZ/WLWy6md Q/0hCmvQ 4tnENf2cInS2q4fX7+1jVvyWx+/222yxnQHM0SLWnX8Z8INPpvVvzABRr8BiQKLSniLHhT8NSLK8iOu4d/nAwRv1SVN5FhgYm8m0Vdlt/IE5hEIbiOmvhm8AcNH9KUulWXL0wZ8tpWDgot6ICZBShzj9XYYdo/3OOlQg1/UNp3Vy4itVTkJBUN56rT/z/5uctmINC4Tl3FrjxHYbWizoXsYdDd+gUQM32W0vLAOmCrwPx8Y2b3P0BqYMMqKFaSErvwvgPHefHcYLeNk9bC4nph/yGI1oX79BnJQ+kuHsBIK1714B4izAVAxpuhodZFqADy/xSNuBanbtINB8H9p/1xfRXj80i5yHnlTqILy5KfNAL1eeNJYyvPRtiKoqCnHEI5/pQZuhKI8tU4zcVFXrbkvYmgZxM+btDzWkNkD+nSPtp6Q8= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On 10/11/24 8:30 PM, Ritesh Harjani (IBM) wrote: > We anyway don't use any return values from fadump_cma_init(). Since > fadump_reserve_mem() from where fadump_cma_init() gets called today, > already has the required checks. > This patch makes this function return type as void. Let's also handle > extra cases like return if fadump_supported is false or dump_active, so > that in later patches we can call fadump_cma_init() separately from > setup_arch(). Usually patches to this file are posted with title format of powerpc/fadump:<> > > Acked-by: Hari Bathini > Signed-off-by: Ritesh Harjani (IBM) > --- > v2 -> v3: Separated the series into 2 as discussed in v2. > [v2]: https://lore.kernel.org/linuxppc-dev/cover.1728585512.git.ritesh.list@gmail.com/ > > arch/powerpc/kernel/fadump.c | 23 +++++++++-------------- > 1 file changed, 9 insertions(+), 14 deletions(-) > > diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c > index a612e7513a4f..162327d66982 100644 > --- a/arch/powerpc/kernel/fadump.c > +++ b/arch/powerpc/kernel/fadump.c > @@ -78,27 +78,23 @@ static struct cma *fadump_cma; > * But for some reason even if it fails we still have the memory reservation > * with us and we can still continue doing fadump. > */ > -static int __init fadump_cma_init(void) > +static void __init fadump_cma_init(void) > { > unsigned long long base, size; > int rc; > > - if (!fw_dump.fadump_enabled) > - return 0; > - > + if (!fw_dump.fadump_supported || !fw_dump.fadump_enabled || > + fw_dump.dump_active) > + return; Is these checks even needed here? fadump_reserve_mem() checked for all these already, also dont see any other caller for fadump_cma_init(). > /* > * Do not use CMA if user has provided fadump=nocma kernel parameter. > - * Return 1 to continue with fadump old behaviour. > */ > - if (fw_dump.nocma) > - return 1; > + if (fw_dump.nocma || !fw_dump.boot_memory_size) > + return; > > base = fw_dump.reserve_dump_area_start; > size = fw_dump.boot_memory_size; > > - if (!size) > - return 0; So this is the only place where we return 0, which in turn will make the "ret" in fadump_reserve_mem() as zero forcing to call reserve_crashkernel() in early_init_devtree(). we are removing it, becos we know "size" here will never be zero? > - > rc = cma_init_reserved_mem(base, size, 0, "fadump_cma", &fadump_cma); > if (rc) { > pr_err("Failed to init cma area for firmware-assisted dump,%d\n", rc); > @@ -108,7 +104,7 @@ static int __init fadump_cma_init(void) > * blocked from production system usage. Hence return 1, > * so that we can continue with fadump. > */ > - return 1; > + return; > } > > /* > @@ -125,10 +121,9 @@ static int __init fadump_cma_init(void) > cma_get_size(fadump_cma), > (unsigned long)cma_get_base(fadump_cma) >> 20, > fw_dump.reserve_dump_area_size); > - return 1; > } > #else > -static int __init fadump_cma_init(void) { return 1; } > +static void __init fadump_cma_init(void) { } > #endif /* CONFIG_CMA */ > > /* > @@ -638,7 +633,7 @@ int __init fadump_reserve_mem(void) > pr_info("Reserved %lldMB of memory at %#016llx (System RAM: %lldMB)\n", > (size >> 20), base, (memblock_phys_mem_size() >> 20)); > > - ret = fadump_cma_init(); > + fadump_cma_init(); > } > > return ret; > -- > 2.46.0 >