[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2] xen/arm: Correctly support WARN_ON

Hi Ian,

On 09/09/14 02:17, Ian Campbell wrote:
On Mon, 2014-09-08 at 12:19 -0700, Julien Grall wrote:
Hi Ian,

On 08/09/14 06:01, Ian Campbell wrote:
+    /* PC is always 4-byte align, as Xen is using ARM instruction set */



Is it worth a check here? I presume the nested fault if PC were
misaligned would be pretty exciting, print+goto die would seem

I don't think so. The "undefined instruction" is only used when the
processor is unable to decode the instruction. If it fails to load the
instruction (because a wrong memory address, pc misaligned), Xen would
have received a "prefetch abort".

I was thinking of something either wrongly or rightly transitioning to
thumb mode. It'd have to be something horrible like a broken firmware
returning from a PSCI call in the wrong mode or something like that

The thumb mode is stored in SPSR which is a banked registers. Your use case can only happen if the secure mode decides to modify the registers of the hypervisor. It would be very stupid from the firmware and other bad things could happen.

   }> +#ifdef CONFIG_ARM_64
+static void do_trap_brk(struct cpu_user_regs *regs, union hsr hsr)
+    /* HCR_EL2.TGE and MDCR_EL2.TDE are not set so we never receive
+     * software breakpoint exception for EL1 and EL0 here
+     */


Sounds a good plan.

Just be careful that it won't recurse ;-)

Panic should never go in exception mode, so I will use it instead.


Julien Grall

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.