From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-it0-x232.google.com (mail-it0-x232.google.com [IPv6:2607:f8b0:4001:c0b::232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 625FF21E977FA for ; Wed, 6 Sep 2017 09:45:51 -0700 (PDT) Received: by mail-it0-x232.google.com with SMTP id k186so8166968ith.0 for ; Wed, 06 Sep 2017 09:48:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=cGHhLvXL+Lhh+zGzC/EQKwuI3EMIV5xesy96osM6mWw=; b=jLTwwDJ4X/y5AQMpvR40dwc2hfcME4QsrxYagiWoN8X/KRYd/TWaxoIT61S4m536x5 uBuik7IlCOI6e6SVKjlpMqTEJhKWfniOXs+dFXu+xSrwEAavZ6X5+joTBncqjAiZZ6ly gJL1zqQe8CqLcPkhg2a6GmOHaAe0duPbASq3I= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=cGHhLvXL+Lhh+zGzC/EQKwuI3EMIV5xesy96osM6mWw=; b=tZ/XwsNkVPLa8tSLyJL8k7MFU1CxZPHV5WD66gG/c65CHUNBkxlqfcg9eOxwuxWyFX 8o8597/suTWl413xs7EhQl3xMZRNwXRJyUXfUwy1gS8gMD/88uYm6Ooz3B9bAfHNeiTP 2mzFCnIWrLEzoU/Mi9E5Ix42SMj6P7cRtqws/f8Y8iXhshbMXSEKihY9ZKowIO83M869 qlomM4R8iyXGx5LmTUYcy9ef+28JnmEZd+bkYZhczCg04GpAeTHJTQ6qG924QK1RMsnq KJM20rxli+krmOOFvLch/cGf3Ixu2vI9mZeeQEzWBInNeogsOs52jntRDtGK0VQbTyg5 ndBQ== X-Gm-Message-State: AHPjjUge+meC7WzTQN7pGND1Af7Eg0qddZA8W8qvVa8XHVrtdKjDHmNE WcXsIdmbi23dE9ZIo6+ftC0pmzSxaQFf X-Google-Smtp-Source: ADKCNb6cGB1y4T+hsSjxAPF9bbZvpDFSZJARC886es2U5uYVNArFavIDd/1a82HUkuEgqdu5O22Sjx6nsar7HnkleGg= X-Received: by 10.36.184.4 with SMTP id m4mr419895ite.41.1504716521080; Wed, 06 Sep 2017 09:48:41 -0700 (PDT) MIME-Version: 1.0 Received: by 10.107.162.1 with HTTP; Wed, 6 Sep 2017 09:48:40 -0700 (PDT) In-Reply-To: <20170906153355.foh2ccau2yy3lnsh@bivouac.eciton.net> References: <20170905182249.1230-1-ard.biesheuvel@linaro.org> <20170906153355.foh2ccau2yy3lnsh@bivouac.eciton.net> From: Ard Biesheuvel Date: Wed, 6 Sep 2017 17:48:40 +0100 Message-ID: To: Leif Lindholm Cc: "edk2-devel@lists.01.org" Subject: Re: [PATCH] ArmPkg: add ArmCrashDumpDxe driver X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Sep 2017 16:45:51 -0000 Content-Type: text/plain; charset="UTF-8" On 6 September 2017 at 16:33, Leif Lindholm wrote: > On Tue, Sep 05, 2017 at 07:22:49PM +0100, Ard Biesheuvel wrote: >> Even though RELEASE builds produce some diagnostics when a crash >> occurs, they can be rather unhelpful: >> >> Synchronous Exception at 0x0000000000000000 >> >> and sometimes, it would be useful to get a full register dump from >> a production machine without having to modify the firmware. >> >> This can be achieved very easily by incorporating a DEBUG build of >> ARM's DefaultExceptionHandlerLib into a DXE driver, and registering >> its DefaultExceptionHandler entry point as the synchronous exception >> handler, overriding the default one. If we then build this driver >> using the UefiDebugLibConOut DebugLib implementation, we end up >> with a module than can simply be loaded via the Shell on any system. >> >> Shell> load fs0:ArmCrashDumpDxe.efi >> >> As a bonus, the crash dump will also appear on the graphical display, >> not only on the serial port. >> >> Contributed-under: TianoCore Contribution Agreement 1.1 >> Signed-off-by: Ard Biesheuvel > > Really useful thing. > Reviewed-by: Leif Lindholm > Thanks. Pushed as a94081fb266b