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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id E99C3C3DA7D for ; Thu, 5 Jan 2023 09:12:35 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 821B1855A0; Thu, 5 Jan 2023 10:12:33 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="aZNotSFe"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id E495585589; Thu, 5 Jan 2023 10:12:31 +0100 (CET) Received: from mail-ej1-x636.google.com (mail-ej1-x636.google.com [IPv6:2a00:1450:4864:20::636]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 6183B855B0 for ; Thu, 5 Jan 2023 10:12:28 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=ilias.apalodimas@linaro.org Received: by mail-ej1-x636.google.com with SMTP id fy8so24644292ejc.13 for ; Thu, 05 Jan 2023 01:12:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=pJBXG5AxoZ/71GfdCbKaE0Cua4hG7u48GYCaL7JunVg=; b=aZNotSFeN1Z35VrTjGSOKVzelk9nKBSQs8UJVVu34SafcbAaFSnbfeGlmlqmJ1CF/K BWFNnYNieAAMNmU4CSpJ6yyUvwhF+LvlDaCJ8CUPxg6K3QePD1ppf2uNr8MElP35Dpht qFOixk2QtGdd6nju8zeVog/p0wUZJiQPXWMV9NN6rNzZwlxQC5w4SQ9MMdiMKAEaAHe/ rqI5+cxEZ8czCE5jwsAbyCqfzTDkUxqjM3cABpAdDB8Nv1lIrx+TYPOLYhKl2mTE9nbJ 5CW4UICIXklbYnXq2UI1morjLJQpMg1EdS1HNCOpMGa5bwGGsJXHgysb0xA7QekeMCwJ Wr1Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=pJBXG5AxoZ/71GfdCbKaE0Cua4hG7u48GYCaL7JunVg=; b=T65q5kyBcZqvgykHIL/N/aoZufsuNdrvTrFOCrRcLHwXwU1lEPirb16WZqk7UWH4n1 C90Iknohe6bC75Cx0GEskZHCxINGx6VkO1CvaTxIr+9yKvko4c+i8eiF52oxFJXCn2nL pkT6xou1swj2NUH0TbgI5qID14yv8xeJEQWfiqsIYdLPfEoQF36zVNZGStAWLrfVJEHU L4Jc2B8vieb9Q2D9lsyN/2lyk0qZUhbel1an4SR9eKoY2K90Nd44Q/fVwZt5U4F1hKRt 2j2vJRvGNRJuSgctpvs2TmLR1OllCfugeTAOUNk5y/OkUsVT6Gwh744dWzCI4GTwJrNk EFww== X-Gm-Message-State: AFqh2ko/CM/LVkj3EpIYGEX4G27u6CosmtgE1ZMJxFPjjTEtSbP2/aIV GDZmTCS8NQcFEFcNoPSpD1aOOzdlQvfTS8HG X-Google-Smtp-Source: AMrXdXv8dst6zuL3PQ6xLnuMKrH5gxW+fsBjbnUXfFA6ws315MkJYR8JEfNck820ZpMJlCkgroaxMQ== X-Received: by 2002:a17:907:a705:b0:7bd:ece7:ae66 with SMTP id vw5-20020a170907a70500b007bdece7ae66mr47868721ejc.34.1672909947994; Thu, 05 Jan 2023 01:12:27 -0800 (PST) Received: from hades (ppp079167090036.access.hol.gr. [79.167.90.36]) by smtp.gmail.com with ESMTPSA id 23-20020a170906309700b00809e33ba33dsm16539001ejv.19.2023.01.05.01.12.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 05 Jan 2023 01:12:27 -0800 (PST) Date: Thu, 5 Jan 2023 11:12:25 +0200 From: Ilias Apalodimas To: Heinrich Schuchardt Cc: u-boot@lists.denx.de Subject: Re: [PATCH 1/1] efi_loader: make .data section of *_efi.so files RW Message-ID: References: <20230104224308.37244-1-heinrich.schuchardt@canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230104224308.37244-1-heinrich.schuchardt@canonical.com> X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 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.6 at phobos.denx.de X-Virus-Status: Clean Hi Heinrich On Wed, Jan 04, 2023 at 11:43:08PM +0100, Heinrich Schuchardt wrote: > When building with binutils 2.39 warnings > > *_efi.so has a LOAD segment with RWX permissions > > occur. > > Use SHF_WRITE | SHF_ALLOC as section flags for the .data section. > > Signed-off-by: Heinrich Schuchardt > --- > arch/arm/lib/elf_aarch64_efi.lds | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/lib/elf_aarch64_efi.lds b/arch/arm/lib/elf_aarch64_efi.lds > index ffc6f6e604..3e3da47d6a 100644 > --- a/arch/arm/lib/elf_aarch64_efi.lds > +++ b/arch/arm/lib/elf_aarch64_efi.lds > @@ -7,6 +7,12 @@ > > OUTPUT_FORMAT("elf64-littleaarch64", "elf64-littleaarch64", "elf64-littleaarch64") > OUTPUT_ARCH(aarch64) > + > +PHDRS > +{ > + data PT_LOAD FLAGS(3); /* SHF_WRITE | SHF_ALLOC */ > +} > + > ENTRY(_start) > SECTIONS > { > @@ -49,7 +55,7 @@ SECTIONS > . = ALIGN(512); > _bss_end = .; > _edata = .; > - } > + } :data Is this only to signal the end of the .data section? If so let's get rid of it since no other section has that > _data_size = _edata - _data; > .rela.dyn : { *(.rela.dyn) } > .rela.plt : { *(.rela.plt) } > -- > 2.37.2 > other than that Reviewed-by: Ilias Apalodimas