Re: [PATCH 1/2] KVM: PPC: Book3S HV: Correct tlbie usage

2013-07-17 Thread Aneesh Kumar K.V
Paul Mackerras pau...@samba.org writes: This corrects the usage of the tlbie (TLB invalidate entry) instruction in HV KVM. The tlbie instruction changed between PPC970 and POWER7. On the PPC970, the bit to select large vs. small page is in the instruction, not in the RB register value. This

Re: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Gleb Natapov
On Tue, Jul 16, 2013 at 06:04:34PM -0500, Scott Wood wrote: On 07/16/2013 01:35:55 AM, Gleb Natapov wrote: On Mon, Jul 15, 2013 at 01:17:33PM -0500, Scott Wood wrote: On 07/15/2013 06:30:20 AM, Gleb Natapov wrote: There is no much sense to share hypercalls between architectures. There is

Re: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Alexander Graf
On 17.07.2013, at 13:00, Gleb Natapov wrote: On Tue, Jul 16, 2013 at 06:04:34PM -0500, Scott Wood wrote: On 07/16/2013 01:35:55 AM, Gleb Natapov wrote: On Mon, Jul 15, 2013 at 01:17:33PM -0500, Scott Wood wrote: On 07/15/2013 06:30:20 AM, Gleb Natapov wrote: There is no much sense to share

[PATCH] KVM: PPC: Book3S PR: return appropriate error when allocation fails

2013-07-17 Thread Thadeu Lima de Souza Cascardo
err was overwritten by a previous function call, and checked to be 0. If the following page allocation fails, 0 is going to be returned instead of -ENOMEM. Signed-off-by: Thadeu Lima de Souza Cascardo casca...@linux.vnet.ibm.com --- arch/powerpc/kvm/book3s_pr.c |5 +++-- 1 files changed, 3

Re: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Alexander Graf
On 17.07.2013, at 17:19, Yoder Stuart-B08248 wrote: -Original Message- From: Alexander Graf [mailto:ag...@suse.de] Sent: Wednesday, July 17, 2013 7:19 AM To: Gleb Natapov Cc: Wood Scott-B07421; Bhushan Bharat-R65777; k...@vger.kernel.org; kvm-ppc@vger.kernel.org; Yoder

RE: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Yoder Stuart-B08248
-Original Message- From: Alexander Graf [mailto:ag...@suse.de] Sent: Wednesday, July 17, 2013 7:19 AM To: Gleb Natapov Cc: Wood Scott-B07421; Bhushan Bharat-R65777; k...@vger.kernel.org; kvm-ppc@vger.kernel.org; Yoder Stuart-B08248; Bhushan Bharat-R65777 Subject: Re: [PATCH 3/5]

RE: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Yoder Stuart-B08248
-Original Message- From: Alexander Graf [mailto:ag...@suse.de] Sent: Wednesday, July 17, 2013 10:21 AM To: Yoder Stuart-B08248 Cc: Wood Scott-B07421; Bhushan Bharat-R65777; k...@vger.kernel.org; kvm-ppc@vger.kernel.org; Gleb Natapov Subject: Re: [PATCH 3/5] booke: define reset

RE: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Bhushan Bharat-R65777
On 17.07.2013, at 13:00, Gleb Natapov wrote: On Tue, Jul 16, 2013 at 06:04:34PM -0500, Scott Wood wrote: On 07/16/2013 01:35:55 AM, Gleb Natapov wrote: On Mon, Jul 15, 2013 at 01:17:33PM -0500, Scott Wood wrote: On 07/15/2013 06:30:20 AM, Gleb Natapov wrote: There is no much sense to

Re: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Alexander Graf
On 17.07.2013, at 18:21, Bhushan Bharat-R65777 wrote: On 17.07.2013, at 13:00, Gleb Natapov wrote: On Tue, Jul 16, 2013 at 06:04:34PM -0500, Scott Wood wrote: On 07/16/2013 01:35:55 AM, Gleb Natapov wrote: On Mon, Jul 15, 2013 at 01:17:33PM -0500, Scott Wood wrote: On 07/15/2013 06:30:20

Re: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Scott Wood
On 07/17/2013 11:04:08 AM, Alexander Graf wrote: On 17.07.2013, at 17:59, Bhushan Bharat-R65777 wrote: Ah, so we can choose any address range in ccsr space of a PV machine (-M ppce500 / e500plat). No, we don't put it in CCSR space. It'd just be orthogonal to CCSR. I'd rather we put it

Re: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Alexander Graf
On 17.07.2013, at 18:59, Scott Wood wrote: On 07/17/2013 11:04:08 AM, Alexander Graf wrote: On 17.07.2013, at 17:59, Bhushan Bharat-R65777 wrote: Ah, so we can choose any address range in ccsr space of a PV machine (-M ppce500 / e500plat). No, we don't put it in CCSR space. It'd just be

Re: [PATCH 3/5] booke: define reset and shutdown hcalls

2013-07-17 Thread Scott Wood
On 07/17/2013 12:05:41 PM, Alexander Graf wrote: On 17.07.2013, at 18:59, Scott Wood wrote: On 07/17/2013 11:04:08 AM, Alexander Graf wrote: On 17.07.2013, at 17:59, Bhushan Bharat-R65777 wrote: Ah, so we can choose any address range in ccsr space of a PV machine (-M ppce500 / e500plat).