From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [RFC 6/6] xen/arm: Replace early_printk call to printk call Date: Thu, 20 Feb 2014 11:14:16 +0000 Message-ID: <5305E388.2030609@linaro.org> References: <1388957191-10337-1-git-send-email-julien.grall@linaro.org> <1388957191-10337-7-git-send-email-julien.grall@linaro.org> <1392808847.23084.138.camel@kazak.uk.xensource.com> <5304F058.6090503@linaro.org> <1392887074.22494.7.camel@kazak.uk.xensource.com> <5305E090.4030602@linaro.org> <1392894351.23342.22.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1WGRa9-0000yB-4T for xen-devel@lists.xenproject.org; Thu, 20 Feb 2014 11:14:21 +0000 Received: by mail-we0-f174.google.com with SMTP id w61so1328106wes.19 for ; Thu, 20 Feb 2014 03:14:19 -0800 (PST) In-Reply-To: <1392894351.23342.22.camel@kazak.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: xen-devel@lists.xenproject.org, stefano.stabellini@eu.citrix.com, tim@xen.org, patches@linaro.org List-Id: xen-devel@lists.xenproject.org On 20/02/14 11:05, Ian Campbell wrote: > On Thu, 2014-02-20 at 11:01 +0000, Julien Grall wrote: >> >> On 20/02/14 09:04, Ian Campbell wrote: >>> I was actually thinking more along the lines of a .word at a defined >>> offset which you could hex edit to a specific value to activate a >>> particular flavour of early printk handling. That would be sufficient >>> e.g. for osstest to activate the appropriate stuff for the specific >>> platform. >> >> I don't see useful use case to have a such early printk implementation >> in Xen. When the board is fully supported, failed at early stage (e.g >> before console is initialized) is very unlikely. At least if you don't >> play with memory. > > a) there are boards which aren't fully supported, getting some debug out > of a distro package might be useful Few months ago we have decided to allow early printk only when Xen is compiled with debug enabled. It seems a big mistake to ship distro with debug enabled :). > b) even for boards which are fully supported there may still be bugs > which only appear under particular circumstances. I understand this use case. If I understand your previous mail the solution would me "Hex editing manually the Xen binary to set the early printk", right? If so, you are assuming that the distro (or anything else) is proving the zImage. Otherwise the developper has to: - unpack from the uImage - editing the zImage - recreate the uImage That seems very complicated compare to recompiling Xen... -- Julien Grall