RE: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-07 Thread Bhushan Bharat-R65777
t; >> Sent: Monday, October 07, 2013 9:16 PM > >> To: Bhushan Bharat-R65777 > >> Cc: Wood Scott-B07421; kvm-ppc@vger.kernel.org; k...@vger.kernel.org > >> Subject: Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to > >> epapr_hypercall() > >> &g

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-07 Thread Alexander Graf
arat-R65777 >> Cc: Wood Scott-B07421; kvm-ppc@vger.kernel.org; k...@vger.kernel.org >> Subject: Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to >> epapr_hypercall() >> >> >> On 07.10.2013, at 17:43, Bhushan Bharat-R65777 wrote: >> >>>>

RE: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-07 Thread Bhushan Bharat-R65777
.org > Subject: Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to > epapr_hypercall() > > > On 07.10.2013, at 17:43, Bhushan Bharat-R65777 wrote: > > >>>>>>>>> at least when I can avoid it. With the current code the > >>>>>&

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-07 Thread Alexander Graf
On 07.10.2013, at 17:43, Bhushan Bharat-R65777 wrote: > at least when I can avoid it. With the current code the compiler > would be >> smart enough to just optimize out the complete branch. Sure. My point is, where would you be calling that where the >

RE: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-07 Thread Bhushan Bharat-R65777
> >>> at least when I can avoid it. With the current code the compiler > >>> would be > smart enough to just optimize out the complete branch. > >> > >> Sure. My point is, where would you be calling that where the > >> entire file isn't predicated on (or selecting) CONFIG_

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-07 Thread Alexander Graf
t; From: Wood Scott-B07421 >>>> Sent: Thursday, October 03, 2013 12:04 AM >>>> To: Alexander Graf >>>> Cc: Bhushan Bharat-R65777; kvm-ppc@vger.kernel.org; >>>> k...@vger.kernel.org; Bhushan >>>> Bharat-R65777 >>>> Subject: R

RE: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-07 Thread Bhushan Bharat-R65777
gt; >> Cc: Bhushan Bharat-R65777; kvm-ppc@vger.kernel.org; > >> k...@vger.kernel.org; Bhushan > >> Bharat-R65777 > >> Subject: Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to > >> epapr_hypercall() > >> > >> On Wed, 2013-10-02 at 19:54 +0

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-04 Thread Alexander Graf
el.org; >> Bhushan >> Bharat-R65777 >> Subject: Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to >> epapr_hypercall() >> >> On Wed, 2013-10-02 at 19:54 +0200, Alexander Graf wrote: >>> On 02.10.2013, at 19:49, Scott Wood wrote: >>> >>

RE: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-03 Thread Bhushan Bharat-R65777
> -Original Message- > From: Wood Scott-B07421 > Sent: Thursday, October 03, 2013 12:04 AM > To: Alexander Graf > Cc: Bhushan Bharat-R65777; kvm-ppc@vger.kernel.org; k...@vger.kernel.org; > Bhushan > Bharat-R65777 > Subject: Re: [PATCH 1/2] kvm/powerpc: r

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Scott Wood
On Wed, 2013-10-02 at 19:54 +0200, Alexander Graf wrote: > On 02.10.2013, at 19:49, Scott Wood wrote: > > > On Wed, 2013-10-02 at 19:46 +0200, Alexander Graf wrote: > >> On 02.10.2013, at 19:42, Scott Wood wrote: > >> > >>> On Wed, 2013-10-02 at 19:17 +0200, Alexander Graf wrote: > On 02.10.

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Alexander Graf
On 02.10.2013, at 19:49, Scott Wood wrote: > On Wed, 2013-10-02 at 19:46 +0200, Alexander Graf wrote: >> On 02.10.2013, at 19:42, Scott Wood wrote: >> >>> On Wed, 2013-10-02 at 19:17 +0200, Alexander Graf wrote: On 02.10.2013, at 19:04, Scott Wood wrote: > On Wed, 2013-10-02 at 18

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Scott Wood
On Wed, 2013-10-02 at 19:46 +0200, Alexander Graf wrote: > On 02.10.2013, at 19:42, Scott Wood wrote: > > > On Wed, 2013-10-02 at 19:17 +0200, Alexander Graf wrote: > >> On 02.10.2013, at 19:04, Scott Wood wrote: > >> > >>> On Wed, 2013-10-02 at 18:53 +0200, Alexander Graf wrote: > On 02.10.

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Alexander Graf
On 02.10.2013, at 19:42, Scott Wood wrote: > On Wed, 2013-10-02 at 19:17 +0200, Alexander Graf wrote: >> On 02.10.2013, at 19:04, Scott Wood wrote: >> >>> On Wed, 2013-10-02 at 18:53 +0200, Alexander Graf wrote: On 02.10.2013, at 18:40, Scott Wood wrote: > On Wed, 2013-10-02 at 16

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Scott Wood
On Wed, 2013-10-02 at 19:17 +0200, Alexander Graf wrote: > On 02.10.2013, at 19:04, Scott Wood wrote: > > > On Wed, 2013-10-02 at 18:53 +0200, Alexander Graf wrote: > >> On 02.10.2013, at 18:40, Scott Wood wrote: > >> > >>> On Wed, 2013-10-02 at 16:19 +0200, Alexander Graf wrote: > Won't thi

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Alexander Graf
On 02.10.2013, at 19:04, Scott Wood wrote: > On Wed, 2013-10-02 at 18:53 +0200, Alexander Graf wrote: >> On 02.10.2013, at 18:40, Scott Wood wrote: >> >>> On Wed, 2013-10-02 at 16:19 +0200, Alexander Graf wrote: On 23.09.2013, at 07:23, Bharat Bhushan wrote: > static inline long kvm_hyp

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Scott Wood
On Wed, 2013-10-02 at 18:53 +0200, Alexander Graf wrote: > On 02.10.2013, at 18:40, Scott Wood wrote: > > > On Wed, 2013-10-02 at 16:19 +0200, Alexander Graf wrote: > >> On 23.09.2013, at 07:23, Bharat Bhushan wrote: > >>> static inline long kvm_hypercall0_1(unsigned int nr, unsigned long *r2) > >

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Alexander Graf
On 02.10.2013, at 18:40, Scott Wood wrote: > On Wed, 2013-10-02 at 16:19 +0200, Alexander Graf wrote: >> On 23.09.2013, at 07:23, Bharat Bhushan wrote: >>> static inline long kvm_hypercall0_1(unsigned int nr, unsigned long *r2) >>> @@ -65,7 +54,7 @@ static inline long kvm_hypercall0_1(unsigned in

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Scott Wood
On Wed, 2013-10-02 at 16:19 +0200, Alexander Graf wrote: > On 23.09.2013, at 07:23, Bharat Bhushan wrote: > > static inline long kvm_hypercall0_1(unsigned int nr, unsigned long *r2) > > @@ -65,7 +54,7 @@ static inline long kvm_hypercall0_1(unsigned int nr, > > unsigned long *r2) > > unsigned l

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-10-02 Thread Alexander Graf
On 23.09.2013, at 07:23, Bharat Bhushan wrote: > kvm_hypercall() have nothing KVM specific, so renamed to epapr_hypercall(). > Also this in moved to arch/powerpc/include/asm/epapr_hcalls.h > > Signed-off-by: Bharat Bhushan > --- > arch/powerpc/include/asm/epapr_hcalls.h | 36 +

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-09-23 Thread Scott Wood
On Mon, 2013-09-23 at 17:45 -0500, Scott Wood wrote: > On Mon, 2013-09-23 at 10:53 +0530, Bharat Bhushan wrote: > > kvm_hypercall() have nothing KVM specific, so renamed to epapr_hypercall(). > > Also this in moved to arch/powerpc/include/asm/epapr_hcalls.h > [snip] > > + out[0] = r4; > > + out

Re: [PATCH 1/2] kvm/powerpc: rename kvm_hypercall() to epapr_hypercall()

2013-09-23 Thread Scott Wood
On Mon, 2013-09-23 at 10:53 +0530, Bharat Bhushan wrote: > kvm_hypercall() have nothing KVM specific, so renamed to epapr_hypercall(). > Also this in moved to arch/powerpc/include/asm/epapr_hcalls.h [snip] > + out[0] = r4; > + out[1] = r5; > + out[2] = r6; > + out[3] = r7; > + o