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 X-Spam-Level: X-Spam-Status: No, score=-14.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 387A8C433F5 for ; Thu, 9 Sep 2021 10:39:25 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 67297604E9 for ; Thu, 9 Sep 2021 10:39:24 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 67297604E9 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmx.de Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 14842832CC; Thu, 9 Sep 2021 12:39:22 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=fail (p=none dis=none) header.from=gmx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; secure) header.d=gmx.net header.i=@gmx.net header.b="F+WCLlBO"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 28D028048A; Thu, 9 Sep 2021 12:39:20 +0200 (CEST) Received: from mout.gmx.net (mout.gmx.net [212.227.17.20]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 08235832CC for ; Thu, 9 Sep 2021 12:39:17 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=gmx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=xypron.glpk@gmx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1631183954; bh=qRt6w7U/kfNEDrquxOkuYHxSb9Vb3BncZtx6RMktubM=; h=X-UI-Sender-Class:Subject:To:Cc:References:From:Date:In-Reply-To; b=F+WCLlBONsdE4DRSo3ogz8Pvv7+qZQiQQHK794WhZ3QTvyVngABptsrz/CZ+5nrVw Z6iOiubvC3clTPPljV878SQWoAnDbfzL+/dP7CQ3qUk7cNN2jeuw9E3EEcIJG4yjHZ +18rLZ2/R11v191mv7jzp/DvjTEtohdLppOkh9fk= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from [192.168.123.55] ([88.152.144.157]) by mail.gmx.net (mrgmx104 [212.227.17.168]) with ESMTPSA (Nemesis) id 1Ma20q-1mTgph3PPQ-00Vx6z; Thu, 09 Sep 2021 12:39:13 +0200 Subject: Re: [PATCH 34/35] efi: Show when allocated pages are used To: Simon Glass Cc: Ilias Apalodimas , Bin Meng , Tom Rini , Christian Melki , Alexander Graf , U-Boot Mailing List References: <20210908133405.696481-1-sjg@chromium.org> <20210908073355.34.I2eff1cd4f12753d0d86e9751fa350458ef5789e0@changeid> <6cf95832-b162-bdfd-b8d2-901ba7e63b74@gmx.de> From: Heinrich Schuchardt Message-ID: <0dd5a1b3-b5ae-6f8e-d1db-5f5cdf60dd51@gmx.de> Date: Thu, 9 Sep 2021 12:39:13 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:3U0wutq4mNfJkN+mR6w3u2Q2vLEe7Q6l5FaHWJAQj3haYn0H/3P f5DZze0vjtz9VoEmvycQqoxbuV9aGQ2cxp7LUTODg+YXCI6Dw3bcBmcPFEREAcftE1YXIEi dJ8rNiPN2BwFSgmReFeBOcMsamFZXdRlV0w6eV3K1gyURhrY9OTkjYhfAI4urJybFCh3Crn dRAYWLIfBqnQQ5lmMnTbg== X-UI-Out-Filterresults: notjunk:1;V03:K0:8bN2uGfpzmQ=:Fme0Pvj2HloTiL9ErXxCDi JO3pIKasX35ggUeh4ceTcS5IaGegJXwYjDcPAez3Gf1vwpWa3lVNAXq1zFZjw9fVK3eLXh2Zc GDptlzZZW/KCiYSfc0qSNvv+f6pmYgkgY1jtWfUeiKaj3u7nZBs9p+Evwyc+/1+Z5gMNOzTtC 8uFyyB++xU/YTKBuAjf7EnZ0JbXPAH3fQENX1kjNDBPnKLadB0y+b2t+UDN29F+ZiWNqaHn+/ xap3SbfqoHDIsDORuNBX6ESdcRyP/IHPNx6uSzvrG1FswhMzn8k0ZFqr2y8pvoF/yUytjMRw3 lC5VBiN4MhNTsOx9NHd2114dy0IVuOXBEHIvGOjkK1pkcko7bowrEndsvkEGbvQN2FKfjEVt3 F8dZ5SbjOY9NmhInTEVO4dumlGum3A/BSfsNa25/o4Y0q+fYkj7RezM/o1kVBjnZEFRS7FyN0 a1+lUsL0SsMWQ8dV/9h0OUAuLJ6kPKvCwtjNCY2J4oCrMBCVd2ZCko028r2G3UAyeiVFDte1R okaiAmuG8aSk9w+FEF01ZjqO9n9chAS2ywLlAHwI6joiORPnxU5ypcV+hpeXzroLTy8B70tGx FNWIPe90LgqlNeIeCibOoy9TksHgNeAI0Vx8E08AxtQCtVs4ar60E7JI0kxUaBbCvN7KTHjVT RtTUPT7zn0lA4jcb61bFgc+Wq7+WvTwmKPePEM24+KpPtODbF9oeOS3KO3WNCjiXUz4dkThbC ty7+dC1TU0kmv1IVbipHG4iLH3CSdCBf0gZHcHzsCql593f2pbboscRru1ZXUp5mEznJBzrir 8NzMG9Shvihov7VandUmjq3c3piA7azLOPDq45s7D/BG4YQI272zm/6wAZZ1sftGqYxHrC0l0 N3kQ+WZywaKH99r6VuCEJ14fbkFX2lC5t1xo22/gxyR6hiLUJyu4+kHCleTF0xx5NF6QbJ6OE ZWEK23qJud/bn/01MkoA0lTBk7FL8rySs8AT1kC5xJBNVVF8NM97SBoC3/ne1QNnkwosqrxs9 Wu40qsj07sPvYvZYIb8Aq3a9o1b+LKrQ5OeZWmVrLqYlIVCYUxa2TgV7X7rhSBj6VjZ7k69Nq DLAX2DuCDInbs4= X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean On 9/9/21 10:57 AM, Simon Glass wrote: > Hi Heinrich, > > On Wed, 8 Sept 2021 at 12:25, Heinrich Schuchardt w= rote: >> >> >> >> On 9/8/21 3:34 PM, Simon Glass wrote: >>> Add a message here so that both paths of memory allocation are reporte= d. >>> >>> Signed-off-by: Simon Glass >>> --- >>> >>> lib/efi/efi_app.c | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/lib/efi/efi_app.c b/lib/efi/efi_app.c >>> index e0666d1fae2..c8f4784eb7d 100644 >>> --- a/lib/efi/efi_app.c >>> +++ b/lib/efi/efi_app.c >>> @@ -97,6 +97,7 @@ static efi_status_t setup_memory(struct efi_priv *pr= iv) >>> return ret; >>> priv->use_pool_for_malloc =3D true; >>> } else { >>> + printf("(using allocated RAM address %lx) ", (ulong)addr= ); >> >> Shouldn't this be log_debug()? > > It matches the one a few lines higher. I suppose it could be a debug > statement, but allocation is an area that seems to different on > different UEFIs, so I have it more prominent for the moment, until we > figure it out. If you want it more prominent, isn't it log_info() that you were promoting= ? Best regards Heinrich