From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:4864:20::342; helo=mail-wm1-x342.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wm1-x342.google.com (mail-wm1-x342.google.com [IPv6:2a00:1450:4864:20::342]) (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 E3049211B63F0 for ; Tue, 15 Jan 2019 00:23:50 -0800 (PST) Received: by mail-wm1-x342.google.com with SMTP id p6so2228776wmc.1 for ; Tue, 15 Jan 2019 00:23:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=UF01awAUsXiX87BpjrSz/17jHtv10NQtbAvmzI1O/U0=; b=Zxan1+nQiRXUkFboNnpMZwkESmVgjoj01OzdQfwvdvDZAIM5dq7OLoBCbhb8kiWTra yR/C5l47CnQ/xjCI72LTm8wblusd+IoZTBSa+6CWnVCN0DXE+ZH8zora9Tlje6qYwMCl GKo+bfom4f5RouXvOMalkivptVwQaudOIDKuQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=UF01awAUsXiX87BpjrSz/17jHtv10NQtbAvmzI1O/U0=; b=n3oF/j9EM/zUSoLHyXqYr87DUVf3oLwgyXFVQlJUv+lS+XzqvLHcXlxpdcVKxZuDAa NO7O1tN+ndCY+2Ckn7tYsdTtQ5x9vZrOFhTCE7uOHC0SDZjJWmdD6J8wo93grlaTfyvd kVqOK8AWOauPh2qjWujc7Yot/LWhgxDCVN8WD8X8gUJWOPNXnO0mRcpTG5MX5lzsqeVC O8bUoU09vBKQlPYLgDeusu661DnEOOjtz8Q3O6W7C6br8d76J6fPzKrtgejyBYp2i7LA IKVaHdeQ36n5LnLWMzrEubw+6UXD5b7DLCbpdiZCl3YMoZ7yGUWdOoewFp/Q2yK2OIy/ R8dA== X-Gm-Message-State: AJcUukdfwA68r+Qx5u/FMBsa2yueqzVyIzb2ZbTBHtrWhrbZgPdIx7hA pgUmaxmkdZOAwGF90PvH0cUYocmC+U9N0Q== X-Google-Smtp-Source: ALg8bN6ubW16mC5T0Csgfv1KjGBSZn2FMq+4p27ayvyMhr4WSl8ocdO9zuTViQBfwqQeotMj60Jqlg== X-Received: by 2002:a1c:b14:: with SMTP id 20mr2356899wml.103.1547540628942; Tue, 15 Jan 2019 00:23:48 -0800 (PST) Received: from dogfood.home ([2a01:cb1d:112:6f00:c8d1:b905:1bdd:adb6]) by smtp.gmail.com with ESMTPSA id 67sm120697983wra.37.2019.01.15.00.23.47 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 15 Jan 2019 00:23:48 -0800 (PST) From: Ard Biesheuvel To: edk2-devel@lists.01.org Cc: leif.lindholm@linaro.org, lersek@redhat.com, Ard Biesheuvel Date: Tue, 15 Jan 2019 09:23:40 +0100 Message-Id: <20190115082345.3711-1-ard.biesheuvel@linaro.org> X-Mailer: git-send-email 2.17.1 Subject: [PATCH v2 0/5] ArmPkg: use console for minimal 'exception occurred' message X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 15 Jan 2019 08:23:51 -0000 When running with a graphical console, no message whatsoever is printed when the systems hits an unexpected exception and hangs, because even the minimal 'exception occurred' message is only sent to the serial port. So let's fix that, by updating DefaultExceptionHandlerLib to take the availability of a console into account. (#5) This requires some preparatory decruftication so that we can safely refer to the system table and console (#1 .. #4). Changes since v1: - split off ArmVirtPkg patch (#3) - always send minimal error message to the serial port before attempting to send it to console->stdout as well (which is more likely to fail) (#5) - add Leif's R-b to the series Ard Biesheuvel (5): ArmPkg/DebugAgentSymbolsBaseLib: remove exception handling ArmPkg/DefaultExceptionHandlerLib: declare the permitted usage context ArmVirtPkg: drop reference to ArmPkg/DefaultExceptionHandlerLibBase ArmPkg/DefaultExceptionHandlerLib: drop BASE variant ArmPkg/DefaultExceptionHandlerLib: use console if available ArmPkg/ArmPkg.dsc | 1 - .../AArch64/DebugAgentException.S | 96 ------ .../Arm/DebugAgentException.S | 277 ------------------ .../Arm/DebugAgentException.asm | 273 ----------------- .../DebugAgentSymbolsBaseLib.c | 7 - .../DebugAgentSymbolsBaseLib.inf | 9 - .../AArch64/DefaultExceptionHandler.c | 12 +- .../Arm/DefaultExceptionHandler.c | 7 +- .../DefaultExceptionHandlerBase.c | 35 --- .../DefaultExceptionHandlerLib.inf | 5 +- .../DefaultExceptionHandlerLibBase.inf | 45 --- ArmVirtPkg/ArmVirt.dsc.inc | 1 - 12 files changed, 19 insertions(+), 749 deletions(-) delete mode 100644 ArmPkg/Library/DebugAgentSymbolsBaseLib/AArch64/DebugAgentException.S delete mode 100644 ArmPkg/Library/DebugAgentSymbolsBaseLib/Arm/DebugAgentException.S delete mode 100644 ArmPkg/Library/DebugAgentSymbolsBaseLib/Arm/DebugAgentException.asm delete mode 100644 ArmPkg/Library/DefaultExceptionHandlerLib/DefaultExceptionHandlerBase.c delete mode 100644 ArmPkg/Library/DefaultExceptionHandlerLib/DefaultExceptionHandlerLibBase.inf -- 2.17.1