Re: [PATCH] ARM: KVM: Don't handle PSCI calls via SMC

2013-05-02 Thread Dave Martin
On Wed, May 01, 2013 at 10:59:13AM -0700, Christoffer Dall wrote: > On Wed, May 1, 2013 at 9:49 AM, Dave P Martin wrote: > > Currently, kvmtool unconditionally declares that HVC should be used > > to call PSCI, so the function numbers in the DT tell the guest > > nothing about the function ID name

Re: [PATCH v4 14/14] KVM: ARM: Handle I/O aborts

2012-11-30 Thread Dave Martin
On Mon, Nov 19, 2012 at 03:09:24PM +, Will Deacon wrote: > On Sat, Nov 10, 2012 at 03:43:49PM +, Christoffer Dall wrote: > > When the guest accesses I/O memory this will create data abort > > exceptions and they are handled by decoding the HSR information > > (physical address, read/write,

Re: RFC: Move kvm's instruction decoding into generic code.

2012-10-24 Thread Dave Martin
On Wed, Oct 24, 2012 at 09:55:13PM +1030, Rusty Russell wrote: > Compile-tested only, but shows what it looks like (ugly). > > I just used the existing code and didn't crack open my ARM books to > determine if there's a better way. The struct names are pretty vague, > too, as a result. > Thanks

Re: [PATCH 14/15] KVM: ARM: Handle I/O aborts

2012-10-08 Thread Dave Martin
On Fri, Oct 05, 2012 at 10:00:25AM +0100, Russell King - ARM Linux wrote: > On Mon, Oct 01, 2012 at 01:53:26PM +0100, Dave Martin wrote: > > A good starting point would be load/store emulation as this seems to be a > > common theme, and we would need a credible deployment for any ne

Re: [PATCH 14/15] KVM: ARM: Handle I/O aborts

2012-10-01 Thread Dave Martin
On Mon, Oct 01, 2012 at 04:12:09PM +0100, Jon Medhurst (Tixy) wrote: > On Mon, 2012-10-01 at 13:53 +0100, Dave Martin wrote: > > On Sun, Sep 30, 2012 at 05:49:21PM -0400, Christoffer Dall wrote: > > > On Thu, Sep 27, 2012 at 11:11 AM, Will Deacon wrote: > > > > On S

Re: [PATCH 14/15] KVM: ARM: Handle I/O aborts

2012-10-01 Thread Dave Martin
On Sun, Sep 30, 2012 at 05:49:21PM -0400, Christoffer Dall wrote: > On Thu, Sep 27, 2012 at 11:11 AM, Will Deacon wrote: > > On Sat, Sep 15, 2012 at 04:35:59PM +0100, Christoffer Dall wrote: > >> When the guest accesses I/O memory this will create data abort > >> exceptions and they are handled by

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-03-09 Thread Dave Martin
On Thu, Mar 8, 2012 at 6:47 PM, Richard Earnshaw wrote: > On 08/03/12 17:21, Nicolas Pitre wrote: >> On Thu, 8 Mar 2012, Richard Earnshaw wrote: >> >>> On 02/03/12 21:15, Nicolas Pitre wrote: So, to me, the gcc documentation is perfectly clear on this topic. there really _is_ a guarantee

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-03-08 Thread Dave Martin
On Thu, Mar 08, 2012 at 09:58:23AM +, Richard Earnshaw wrote: > On 02/03/12 21:15, Nicolas Pitre wrote: > > [ coming back from vacation and trying to catch up ] > > > > On Wed, 29 Feb 2012, Dave Martin wrote: > > > >> Just had a chat with some tools guys

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-03-01 Thread Dave Martin
On Thu, Mar 01, 2012 at 10:10:29AM +, Russell King - ARM Linux wrote: > On Wed, Feb 29, 2012 at 12:58:26PM +0000, Dave Martin wrote: > > On Wed, Feb 29, 2012 at 09:56:02AM +, Ian Campbell wrote: > > > On Wed, 2012-02-29 at 09:34 +, Dave Martin wrote: > > > &

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-03-01 Thread Dave Martin
On Wed, Feb 29, 2012 at 02:52:38PM +, Stefano Stabellini wrote: > On Wed, 29 Feb 2012, Dave Martin wrote: > > On Wed, Feb 29, 2012 at 09:56:02AM +, Ian Campbell wrote: > > > On Wed, 2012-02-29 at 09:34 +, Dave Martin wrote: > > > > On Tue, Feb 28, 201

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-03-01 Thread Dave Martin
On Wed, Feb 29, 2012 at 02:44:24PM +, Ian Campbell wrote: > On Wed, 2012-02-29 at 12:58 +0000, Dave Martin wrote: > > On Wed, Feb 29, 2012 at 09:56:02AM +, Ian Campbell wrote: > > > On Wed, 2012-02-29 at 09:34 +, Dave Martin wrote: > > > > On Tue, Fe

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-02-29 Thread Dave Martin
On Wed, Feb 29, 2012 at 09:56:02AM +, Ian Campbell wrote: > On Wed, 2012-02-29 at 09:34 +0000, Dave Martin wrote: > > On Tue, Feb 28, 2012 at 12:28:29PM +, Stefano Stabellini wrote: > > > > I don't have a very strong opinion on which register we should use, but &

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-02-29 Thread Dave Martin
On Wed, Feb 29, 2012 at 09:56:02AM +, Ian Campbell wrote: > On Wed, 2012-02-29 at 09:34 +0000, Dave Martin wrote: > > On Tue, Feb 28, 2012 at 12:28:29PM +, Stefano Stabellini wrote: > > > > I don't have a very strong opinion on which register we should use, but &

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-02-29 Thread Dave Martin
On Tue, Feb 28, 2012 at 12:28:29PM +, Stefano Stabellini wrote: > On Tue, 28 Feb 2012, Ian Campbell wrote: > > On Tue, 2012-02-28 at 10:20 +0000, Dave Martin wrote: > > > On Mon, Feb 27, 2012 at 07:33:39PM +, Ian Campbell wrote: > > > > On Mon, 2012-02-27 at

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-02-28 Thread Dave Martin
On Mon, Feb 27, 2012 at 07:33:39PM +, Ian Campbell wrote: > On Mon, 2012-02-27 at 18:03 +0000, Dave Martin wrote: > > On Mon, Feb 27, 2012 at 04:27:23PM +, Ian Campbell wrote: > > > On Thu, 2012-02-23 at 17:48 +, Stefano Stabellini wrote: > > > >

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-02-28 Thread Dave Martin
On Mon, Feb 27, 2012 at 07:48:45PM +, Ian Campbell wrote: > On Mon, 2012-02-27 at 17:53 +0000, Dave Martin wrote: > > On Thu, Feb 23, 2012 at 05:48:22PM +, Stefano Stabellini wrote: > > > We need a register to pass the hypercall number because we might not > > >

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-02-27 Thread Dave Martin
On Mon, Feb 27, 2012 at 04:27:23PM +, Ian Campbell wrote: > On Thu, 2012-02-23 at 17:48 +, Stefano Stabellini wrote: > > We need a register to pass the hypercall number because we might not > > know it at compile time and HVC only takes an immediate argument. > > > > Among the available re

Re: [PATCH-WIP 01/13] xen/arm: use r12 to pass the hypercall number to the hypervisor

2012-02-27 Thread Dave Martin
On Thu, Feb 23, 2012 at 05:48:22PM +, Stefano Stabellini wrote: > We need a register to pass the hypercall number because we might not > know it at compile time and HVC only takes an immediate argument. > > Among the available registers r12 seems to be the best choice because it > is defined a