[PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[Mihai Caraman]

Add e6500 core to Kconfig description.

Signed-off-by: Mihai Caraman <mihai.caraman@freescale.com>
---
v3:
 - No change

 arch/powerpc/kvm/Kconfig |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/powerpc/kvm/Kconfig b/arch/powerpc/kvm/Kconfig
index 63c67ec..4489520 100644
--- a/arch/powerpc/kvm/Kconfig
+++ b/arch/powerpc/kvm/Kconfig
@@ -136,15 +136,15 @@ config KVM_E500V2
 	  If unsure, say N.
 
 config KVM_E500MC
-	bool "KVM support for PowerPC E500MC/E5500 processors"
+	bool "KVM support for PowerPC E500MC/E5500/E6500 processors"
 	depends on PPC_E500MC
 	select KVM
 	select KVM_MMIO
 	select KVM_BOOKE_HV
 	select MMU_NOTIFIER
 	---help---
-	  Support running unmodified E500MC/E5500 (32-bit) guest kernels in
-	  virtual machines on E500MC/E5500 host processors.
+	  Support running unmodified E500MC/E5500/E6500 guest kernels in
+	  virtual machines on E500MC/E5500/E6500 host processors.
 
 	  This module provides access to the hardware capabilities through
 	  a character device node named /dev/kvm.
-- 
1.7.4.1

Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[tiejun.chen]

On 04/11/2013 06:03 PM, Mihai Caraman wrote:
> Add e6500 core to Kconfig description.
>
> Signed-off-by: Mihai Caraman <mihai.caraman@freescale.com>
> ---
> v3:
>   - No change
>
>   arch/powerpc/kvm/Kconfig |    6 +++---
>   1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/powerpc/kvm/Kconfig b/arch/powerpc/kvm/Kconfig
> index 63c67ec..4489520 100644
> --- a/arch/powerpc/kvm/Kconfig
> +++ b/arch/powerpc/kvm/Kconfig
> @@ -136,15 +136,15 @@ config KVM_E500V2
>   	  If unsure, say N.
>
>   config KVM_E500MC
> -	bool "KVM support for PowerPC E500MC/E5500 processors"
> +	bool "KVM support for PowerPC E500MC/E5500/E6500 processors"
>   	depends on PPC_E500MC
>   	select KVM
>   	select KVM_MMIO
>   	select KVM_BOOKE_HV
>   	select MMU_NOTIFIER
>   	---help---
> -	  Support running unmodified E500MC/E5500 (32-bit) guest kernels in

I ever tried p5040ds but failed with 64-bit, but looks are you saying this patch 
set can make e5500/e6500 work well with 64-bit? If so, will we need to upgrade 
qemu or something else like dtb?

Tiejun

> -	  virtual machines on E500MC/E5500 host processors.
> +	  Support running unmodified E500MC/E5500/E6500 guest kernels in
> +	  virtual machines on E500MC/E5500/E6500 host processors.
>
>   	  This module provides access to the hardware capabilities through
>   	  a character device node named /dev/kvm.
>

RE: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[Caraman Mihai Claudiu-B02008]

> -----Original Message-----
> From: tiejun.chen [mailto:tiejun.chen@windriver.com]
> Sent: Friday, April 19, 2013 1:03 PM
> To: Caraman Mihai Claudiu-B02008
> Cc: kvm-ppc@vger.kernel.org; kvm@vger.kernel.org
> Subject: Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig
> description
> 
> On 04/11/2013 06:03 PM, Mihai Caraman wrote:
> > Add e6500 core to Kconfig description.
> >
> > Signed-off-by: Mihai Caraman <mihai.caraman@freescale.com>
> > ---
> > v3:
> >   - No change
> >
> >   arch/powerpc/kvm/Kconfig |    6 +++---
> >   1 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/powerpc/kvm/Kconfig b/arch/powerpc/kvm/Kconfig
> > index 63c67ec..4489520 100644
> > --- a/arch/powerpc/kvm/Kconfig
> > +++ b/arch/powerpc/kvm/Kconfig
> > @@ -136,15 +136,15 @@ config KVM_E500V2
> >   	  If unsure, say N.
> >
> >   config KVM_E500MC
> > -	bool "KVM support for PowerPC E500MC/E5500 processors"
> > +	bool "KVM support for PowerPC E500MC/E5500/E6500 processors"
> >   	depends on PPC_E500MC
> >   	select KVM
> >   	select KVM_MMIO
> >   	select KVM_BOOKE_HV
> >   	select MMU_NOTIFIER
> >   	---help---
> > -	  Support running unmodified E500MC/E5500 (32-bit) guest kernels in
> 
> I ever tried p5040ds but failed with 64-bit, but looks are you saying
> this patch
> set can make e5500/e6500 work well with 64-bit? If so, will we need to
> upgrade
> qemu or something else like dtb?

KVM should work on p5040ds with and without this patchset. The latest 
qemu requires this patch: "powerpc: Add paravirt idle loop for 64-bit Book-E",
you will not pass guest udev without it.
Please details what fails on p5040ds.

-Mike

Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[tiejun.chen]

On 04/25/2013 05:09 PM, Caraman Mihai Claudiu-B02008 wrote:
>> -----Original Message-----
>> From: tiejun.chen [mailto:tiejun.chen@windriver.com]
>> Sent: Friday, April 19, 2013 1:03 PM
>> To: Caraman Mihai Claudiu-B02008
>> Cc: kvm-ppc@vger.kernel.org; kvm@vger.kernel.org
>> Subject: Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig
>> description
>>
>> On 04/11/2013 06:03 PM, Mihai Caraman wrote:
>>> Add e6500 core to Kconfig description.
>>>
>>> Signed-off-by: Mihai Caraman <mihai.caraman@freescale.com>
>>> ---
>>> v3:
>>>    - No change
>>>
>>>    arch/powerpc/kvm/Kconfig |    6 +++---
>>>    1 files changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/arch/powerpc/kvm/Kconfig b/arch/powerpc/kvm/Kconfig
>>> index 63c67ec..4489520 100644
>>> --- a/arch/powerpc/kvm/Kconfig
>>> +++ b/arch/powerpc/kvm/Kconfig
>>> @@ -136,15 +136,15 @@ config KVM_E500V2
>>>    	  If unsure, say N.
>>>
>>>    config KVM_E500MC
>>> -	bool "KVM support for PowerPC E500MC/E5500 processors"
>>> +	bool "KVM support for PowerPC E500MC/E5500/E6500 processors"
>>>    	depends on PPC_E500MC
>>>    	select KVM
>>>    	select KVM_MMIO
>>>    	select KVM_BOOKE_HV
>>>    	select MMU_NOTIFIER
>>>    	---help---
>>> -	  Support running unmodified E500MC/E5500 (32-bit) guest kernels in
>>
>> I ever tried p5040ds but failed with 64-bit, but looks are you saying
>> this patch
>> set can make e5500/e6500 work well with 64-bit? If so, will we need to
>> upgrade
>> qemu or something else like dtb?
>
> KVM should work on p5040ds with and without this patchset. The latest
> qemu requires this patch: "powerpc: Add paravirt idle loop for 64-bit Book-E",
> you will not pass guest udev without it.

Which should qemu tree be used here?

My tree is cloned from:

git://repo.or.cz/qemu/agraf.git ppc-next

But I can't find this commit.

Tiejun

Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[tiejun.chen]

On 04/25/2013 05:32 PM, Caraman Mihai Claudiu-B02008 wrote:
>> -----Original Message-----
>> From: tiejun.chen [mailto:tiejun.chen@windriver.com]
>> Sent: Thursday, April 25, 2013 12:17 PM
>> To: Caraman Mihai Claudiu-B02008
>> Cc: kvm-ppc@vger.kernel.org; kvm@vger.kernel.org
>> Subject: Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig
>> description
>>
>> On 04/25/2013 05:09 PM, Caraman Mihai Claudiu-B02008 wrote:
>>>> -----Original Message-----
>>>> From: tiejun.chen [mailto:tiejun.chen@windriver.com]
>>>> Sent: Friday, April 19, 2013 1:03 PM
>>>> To: Caraman Mihai Claudiu-B02008
>>>> Cc: kvm-ppc@vger.kernel.org; kvm@vger.kernel.org
>>>> Subject: Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig
>>>> description
>>>>
>>>> On 04/11/2013 06:03 PM, Mihai Caraman wrote:
>>>>> Add e6500 core to Kconfig description.
>>>>>
>>>>> Signed-off-by: Mihai Caraman <mihai.caraman@freescale.com>
>>>>> ---
>>>>> v3:
>>>>>     - No change
>>>>>
>>>>>     arch/powerpc/kvm/Kconfig |    6 +++---
>>>>>     1 files changed, 3 insertions(+), 3 deletions(-)
>>>>>
>>>>> diff --git a/arch/powerpc/kvm/Kconfig b/arch/powerpc/kvm/Kconfig
>>>>> index 63c67ec..4489520 100644
>>>>> --- a/arch/powerpc/kvm/Kconfig
>>>>> +++ b/arch/powerpc/kvm/Kconfig
>>>>> @@ -136,15 +136,15 @@ config KVM_E500V2
>>>>>     	  If unsure, say N.
>>>>>
>>>>>     config KVM_E500MC
>>>>> -	bool "KVM support for PowerPC E500MC/E5500 processors"
>>>>> +	bool "KVM support for PowerPC E500MC/E5500/E6500 processors"
>>>>>     	depends on PPC_E500MC
>>>>>     	select KVM
>>>>>     	select KVM_MMIO
>>>>>     	select KVM_BOOKE_HV
>>>>>     	select MMU_NOTIFIER
>>>>>     	---help---
>>>>> -	  Support running unmodified E500MC/E5500 (32-bit) guest kernels in
>>>>
>>>> I ever tried p5040ds but failed with 64-bit, but looks are you saying
>>>> this patch
>>>> set can make e5500/e6500 work well with 64-bit? If so, will we need to
>>>> upgrade
>>>> qemu or something else like dtb?
>>>
>>> KVM should work on p5040ds with and without this patchset. The latest
>>> qemu requires this patch: "powerpc: Add paravirt idle loop for 64-bit
>> Book-E",
>>> you will not pass guest udev without it.
>
> This is a kernel patch required by latest qemu.

Looks this commit is applied only into galak/powerpc.git, next, but still not 
merged into agraf/linux-2.6.git, so I'm confused which tree can support 64bit 
Book3E KVM as you point.

Tiejun

Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[tiejun.chen]

On 04/25/2013 07:32 PM, Caraman Mihai Claudiu-B02008 wrote:
>> Is the flowing is fine with that generic machine, ppce500, to boot
>> P5040DS with
>> 64bit,
>>
>> ./qemu-system-ppc64 -enable-kvm -m 1048 -nographic -M ppce500 -kernel
>> uImage
>> -initrd ramdisk.gz  -L . -append "root=/dev/ram rw console=ttyS0,115200"
>> -cpu
>> e5500 -dtb p5040ds.dtb
>>
>> Thanks,
>>
>> Tiejun
>
> There is no need for -dtb.

With your comment, I use kvm-ppc-queue which top commit is be28a27c, "kvm/ppc: 
don't call complete_mmio_load when it's a store", in plus that patch you 
pointed	to build one uImage based on corenet64_smp_defconfig, but we need to 
enable CONFIG_PPC_QEMU_E500 manually, and select CONFIG_TICK_CPU_ACCOUNTING 
since the default CONFIG_VIRT_CPU_ACCOUNTING_NATIVE would introduce some trace 
when boot VM.

And perform as follows:

./qemu-system-ppc64 -enable-kvm -m 1048 -nographic -M ppce500 -kernel uImage 
-initrd ramdisk.gz  -L . -append "root=/dev/ram rw console=ttyS0,115200" -cpu e5500

But I can't see anything in the serial port.

Tiejun

Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description

[tiejun.chen]

On 04/26/2013 11:11 AM, tiejun.chen wrote:
> On 04/25/2013 07:32 PM, Caraman Mihai Claudiu-B02008 wrote:
>>> Is the flowing is fine with that generic machine, ppce500, to boot
>>> P5040DS with
>>> 64bit,
>>>
>>> ./qemu-system-ppc64 -enable-kvm -m 1048 -nographic -M ppce500 -kernel
>>> uImage
>>> -initrd ramdisk.gz  -L . -append "root=/dev/ram rw console=ttyS0,115200"
>>> -cpu
>>> e5500 -dtb p5040ds.dtb
>>>
>>> Thanks,
>>>
>>> Tiejun
>>
>> There is no need for -dtb.
>
> With your comment, I use kvm-ppc-queue which top commit is be28a27c, "kvm/ppc:
> don't call complete_mmio_load when it's a store", in plus that patch you
> pointed    to build one uImage based on corenet64_smp_defconfig, but we need to
> enable CONFIG_PPC_QEMU_E500 manually, and select CONFIG_TICK_CPU_ACCOUNTING
> since the default CONFIG_VIRT_CPU_ACCOUNTING_NATIVE would introduce some trace
> when boot VM.
>
> And perform as follows:
>
> ./qemu-system-ppc64 -enable-kvm -m 1048 -nographic -M ppce500 -kernel uImage
> -initrd ramdisk.gz  -L . -append "root=/dev/ram rw console=ttyS0,115200" -cpu e5500
>
> But I can't see anything in the serial port.

Please ignore this since e5500 is okay now.

Thanks for your reply.

Tiejun