From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758233AbYA2VyL (ORCPT ); Tue, 29 Jan 2008 16:54:11 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753752AbYA2Vx4 (ORCPT ); Tue, 29 Jan 2008 16:53:56 -0500 Received: from terminus.zytor.com ([198.137.202.10]:37035 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753636AbYA2Vxz (ORCPT ); Tue, 29 Jan 2008 16:53:55 -0500 Message-ID: <479F9FA4.8020504@zytor.com> Date: Tue, 29 Jan 2008 13:50:28 -0800 From: "H. Peter Anvin" User-Agent: Thunderbird 2.0.0.9 (X11/20071115) MIME-Version: 1.0 To: Ian Campbell CC: Jeremy Fitzhardinge , linux-kernel@vger.kernel.org, "Eric W. Biederman" Subject: Re: PATCH/RFC: bzImage payload as compressed ELF file. References: <1201560133.17053.36.camel@cthulhu.hellion.org.uk> <479E5D22.1070109@goop.org> <1201599251.5301.26.camel@localhost.localdomain> <479F6845.1070902@goop.org> <479F6AD2.8050908@zytor.com> <479F70E9.40905@goop.org> <479F71B6.2010305@zytor.com> <1201639112.9516.23.camel@cthulhu.hellion.org.uk> In-Reply-To: <1201639112.9516.23.camel@cthulhu.hellion.org.uk> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Ian Campbell wrote: > > I finessed the length issue slightly since it happens that the word > before the payload is the payload's length. I'm mulling a variety of > options: > * declare it compression scheme dependant. At least for gzip it > knows when it has reached the end. > * codify the length followed by data thing we happen to have now > * add a length field to the header, using another 4 bytes (is > there any pressure on the size of the header?). > > I'd probably go with the later unless we are pressed for space in the > header. I would agree with that. In fact, while we're adding fields to the header, I would like to add a field that would be the CRC32 (or other simple integrity check) over the whole image. -hpa