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

Re: [Xen-devel] [PATCH 6 of 8] Xen: remove run_in_exception_handler() and recode its only caller


  • To: Tim Deegan <Tim.Deegan@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir.xen@xxxxxxxxx>
  • Date: Mon, 07 Mar 2011 15:05:28 +0000
  • Cc:
  • Delivery-date: Mon, 07 Mar 2011 07:06:09 -0800
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=user-agent:date:subject:from:to:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=pulDLShugRY7hgjwjrqeS1eRDwKMfHlDsQYZ2oAvT2yepTzdiZ+U/HF9OKAYOqYFmJ c/8G30Wrx07SFthjTm9n1QkttgBOE9v1sLymytJ8Xxs2fWmnjcm7QtHUTf1WcQEbwe3x 5crm4w9MccAVtFkc0ypHdsGIMgsatRCQg91T0=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Acvc2RgRC3Rwv6POPUWUo2Mg1G952Q==
  • Thread-topic: [Xen-devel] [PATCH 6 of 8] Xen: remove run_in_exception_handler() and recode its only caller

On 07/03/2011 11:26, "Tim Deegan" <Tim.Deegan@xxxxxxxxxx> wrote:

> (dump_execution_state()) as its own bug-trap.
> 
> This is needed to compile xen with clang, which can't handle using a
> function name in an asm immediate.

Actually run_in_exception_handler() does have another user, in ns16550.c.
Although non-essential, it makes the 'd' debug key much more useful when
running the UART in polled mode.

So I suggest we keep run_in_exception_handler but modify it to pass the
function pointer in (say) rAX. I think we won't easily be able to use
BUG_STR() logic but r_i_e_h is only used (directly or indirectly) in a few
places so the BUG_STR optimisation is unimportant. The only other
disadvantage is that rAX is less interesting in the state dump, but any
value the function pointer displaces can still be found in the stack dump,
albeit with likely a little extra effort.

Sound good?

 -- Keir 

> Signed-off-by: Tim Deegan <Tim.Deegan@xxxxxxxxxx>
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

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