From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1c2Irx-000574-MR for mharc-grub-devel@gnu.org; Thu, 03 Nov 2016 10:19:53 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35295) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c2Irs-00055v-M8 for grub-devel@gnu.org; Thu, 03 Nov 2016 10:19:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c2Iro-00077z-Nb for grub-devel@gnu.org; Thu, 03 Nov 2016 10:19:48 -0400 Received: from boksu.net-space.pl ([185.15.1.105]:60976) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_3DES_EDE_CBC_SHA1:24) (Exim 4.71) (envelope-from ) id 1c2Iro-00073D-Bg for grub-devel@gnu.org; Thu, 03 Nov 2016 10:19:44 -0400 Received: (from localhost user: 'dkiper' uid#4000 fake: STDIN (dkiper@boksu.net-space.pl)) by router-fw-old.local.net-space.pl id S1657002AbcKCOTf (ORCPT ); Thu, 3 Nov 2016 15:19:35 +0100 Date: Thu, 3 Nov 2016 15:19:35 +0100 From: Daniel Kiper To: grub-devel@gnu.org, julien.grall@linaro.org Cc: steve.capper@arm.com, phcoder@gmail.com, xen-devel@lists.xen.org, sstabellini@kernel.org, fu.wei@linaro.org Subject: Re: [PATCH] arm64: xen_boot: Fix xen boot using Grub on AARCH64 Message-ID: <20161103141935.GA8022@router-fw-old.local.net-space.pl> References: <1455899332-9054-1-git-send-email-julien.grall@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1455899332-9054-1-git-send-email-julien.grall@linaro.org> User-Agent: Mutt/1.3.28i X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 185.15.1.105 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Nov 2016 14:19:52 -0000 On Fri, Feb 19, 2016 at 04:28:52PM +0000, Julien Grall wrote: > Xen is currently crashing because of malformed compatible property for > the boot module. This is because the property string is not > null-terminated as requested by the ePAR spec. Just two nitpicks. Could you change subject to "arm64/xen_boot: Fix xen boot using Grub on AARCH64" and add your SOB? You can also repost this patch with: Reviewed-by: Daniel Kiper Daniel