Re: [Xen-devel] [PATCH] xen/arm: Init traps very early

[Naresh Bhat <naresh.bhat@xxxxxxxxxx>]

Tested/Acked by: Naresh Bhat <naresh.bhat@xxxxxxxxxx>

On 28 May 2014 19:03, Julien Grall <julien.grall@xxxxxxxxxx> wrote:

The function init_traps setups the handler taken when Xen hits a BUG_ON/ASSERT.

If an error happen before init_traps is called, we loose the backtrace.

As the function doesn't require any specific setup, we can call it just
after Xen has jumped in C code.

Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>
---
Âxen/arch/arm/setup.c | Â Â3 +--
Â1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c
index b9ce7a9..5bf8686 100644
--- a/xen/arch/arm/setup.c
+++ b/xen/arch/arm/setup.c
@@ -666,6 +666,7 @@ void __init start_xen(unsigned long boot_phys_offset,
  Âstruct domain *dom0;

  Âsetup_cache();
+ Â Âinit_traps();

  Âpercpu_init_areas();
  Âset_processor_id(0); /* needed early, for smp_processor_id() */
@@ -709,8 +710,6 @@ void __init start_xen(unsigned long boot_phys_offset,
  Âset_current((struct vcpu *)0xfffff000); /* debug sanity */
  Âidle_vcpu[0] = current;

- Â Âinit_traps();
-
  Âsetup_virt_paging();

  Âp2m_vmid_allocator_init();
--
1.7.10.4

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel