Hi,

On 15/07/2021 16:18, Jan Beulich wrote:
On 14.07.2021 22:37, Bobby Eshleman wrote:
--- a/xen/arch/arm/traps.c
+++ b/xen/arch/arm/traps.c
@@ -16,6 +16,7 @@
   * GNU General Public License for more details.
   */
+#include <xen/debugger.h>

I don't think this is needed here; instead I think ...

@@ -41,7 +42,6 @@
  #include <asm/acpi.h>
  #include <asm/cpuerrata.h>
  #include <asm/cpufeature.h>
-#include <asm/debugger.h>

... this wants to be done in patch 1 already.

+1. I was actually going to ask in patch 1 to drop <asm/debugger.h> as there should be no more callers of the debugger helpers.

Cheers,

--
Julien Grall

Reply via email to