Revert "xen/arm64: io: Handle the abort due to access to stage1 translation table"
authorJulien Grall <jgrall@amazon.com>
Mon, 14 Mar 2022 22:57:54 +0000 (22:57 +0000)
committerJulien Grall <jgrall@amazon.com>
Mon, 14 Mar 2022 23:00:05 +0000 (23:00 +0000)
This reverts commit fef5531fd8f311b347970568d77e7bf4feffd7fb.

This depends on patch 9e5a68a6652cc54ce3cb3b0ce208eeed79d5aeb5 which
will need to be reverted as it break boot on arm32.

https://lore.kernel.org/xen-devel/osstest-168589-mainreport@xen.org/T/#u

Signed-off-by: Julien Grall <jgrall@amazon.com>
xen/arch/arm/io.c

index 53729a342bdabbf32cf735b1ad335cb4f263aca7..fd903b7b035a0565a74f61e81367c55219259e80 100644 (file)
@@ -128,17 +128,6 @@ void try_decode_instruction(const struct cpu_user_regs *regs,
         return;
     }
 
-    /*
-     * At this point, we know that the stage1 translation table is either in
-     * an emulated MMIO region or its address is invalid. This is not
-     * expected by Xen and thus it forwards the abort to the guest.
-     */
-    if ( info->dabt.s1ptw )
-    {
-        info->dabt_instr.state = INSTR_ERROR;
-        return;
-    }
-
     /*
      * Armv8 processor does not provide a valid syndrome for decoding some
      * instructions. So in order to process these instructions, Xen must