Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Gleb Natapov
On Mon, Jan 14, 2013 at 10:36:38PM +, Will Deacon wrote: > On Mon, Jan 14, 2013 at 07:12:49PM +, Christoffer Dall wrote: > > On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon wrote: > > > On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: > > >> On 01/14/2013 07:50 PM, Will Deacon

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 5:36 PM, Will Deacon wrote: > On Mon, Jan 14, 2013 at 07:12:49PM +, Christoffer Dall wrote: >> On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon wrote: >> > On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: >> >> On 01/14/2013 07:50 PM, Will Deacon wrote: >> >

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 07:12:49PM +, Christoffer Dall wrote: > On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon wrote: > > On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: > >> On 01/14/2013 07:50 PM, Will Deacon wrote: > >> > FWIW, KVM only needs this code for handling complex MMI

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 2:00 PM, Will Deacon wrote: > On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: >> On 01/14/2013 07:50 PM, Will Deacon wrote: >> > FWIW, KVM only needs this code for handling complex MMIO instructions, >> > which >> > aren't even generated by recent guest ker

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Will Deacon
On Mon, Jan 14, 2013 at 06:53:14PM +, Alexander Graf wrote: > On 01/14/2013 07:50 PM, Will Deacon wrote: > > FWIW, KVM only needs this code for handling complex MMIO instructions, which > > aren't even generated by recent guest kernels. I'm inclined to suggest > > removing > > this emulation c

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Christoffer Dall
On Mon, Jan 14, 2013 at 1:53 PM, Alexander Graf wrote: > On 01/14/2013 07:50 PM, Will Deacon wrote: >> >> On Mon, Jan 14, 2013 at 06:43:19PM +, Russell King - ARM Linux wrote: >>> >>> On Mon, Jan 14, 2013 at 01:25:39PM -0500, Christoffer Dall wrote: However, unifying all instruction

Re: [kvmarm] [PATCH v5 13/14] KVM: ARM: Handle I/O aborts

2013-01-14 Thread Alexander Graf
On 01/14/2013 07:50 PM, Will Deacon wrote: On Mon, Jan 14, 2013 at 06:43:19PM +, Russell King - ARM Linux wrote: On Mon, Jan 14, 2013 at 01:25:39PM -0500, Christoffer Dall wrote: However, unifying all instruction decoding within arch/arm is quite the heavy task, and requires agreeing on som