From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cmL2q-000805-UD for barebox@lists.infradead.org; Fri, 10 Mar 2017 13:57:28 +0000 Message-ID: <1489154222.24921.24.camel@pengutronix.de> From: Lucas Stach Date: Fri, 10 Mar 2017 14:57:02 +0100 In-Reply-To: <20170310135403.GA19458@mail.ovh.net> References: <20170309143117.GI4120@mail.ovh.net> <1489070050-16024-1-git-send-email-plagnioj@jcrosoft.com> <1489070050-16024-2-git-send-email-plagnioj@jcrosoft.com> <1489080293.24921.0.camel@pengutronix.de> <1489143938.24921.17.camel@pengutronix.de> <20170310135403.GA19458@mail.ovh.net> Mime-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 2/5] efi: fix lds for secure boot support To: Jean-Christophe PLAGNIOL-VILLARD Cc: barebox@lists.infradead.org Am Freitag, den 10.03.2017, 14:54 +0100 schrieb Jean-Christophe PLAGNIOL-VILLARD: > On 12:05 Fri 10 Mar , Lucas Stach wrote: > > Am Freitag, den 10.03.2017, 18:17 +0800 schrieb Jean-Christophe > > PLAGNIOL-VILLARD: > > > > On Mar 10, 2017, at 1:24 AM, Lucas Stach wrote: > > > > > > > > Am Donnerstag, den 09.03.2017, 15:34 +0100 schrieb Jean-Christophe > > > > PLAGNIOL-VILLARD: > > > >> everythink need to be aligned to 4096 > > > > > > > > Why? The commit message isn't really telling anything. > > > This is a requierment by EFI > > > > This is in no way an EFI requirement. > > > > Googling tells me that the signing procedure for EFI secure boot is > > built around a PE binary. PE in turn is based on the COFF binary format > > which, unlike ELF, has no section descriptions in the header and > > therefore requires the sections to be placed page aligned (4K on x86, 4K > > or 64K on ARM64). > No COFF does not require to have section "page aligned" which is wrong here > The is a Requirement by EFI from secure boot > > Otherwise EFI will not work today > > > > > All of the above is what should have been included in the commit > > message, to allow other people to understand the commit and not require > > them to google their way to the justification of this commit. > And read the Spec is required to undrestant EFI no? > Or I need to put the spec in the commit too? > At least a pointer to the relevant part of the spec would be appreciated. That we are even having this conversation shows that your commit messages are a bit too terse. Being a bit more verbose would be really helpful for other folks trying to understand your changes. Thanks, Lucas _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox