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

public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed

* RE: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description
@ 2013-04-25  9:09 Caraman Mihai Claudiu-B02008
  2013-04-25  9:16 ` tiejun.chen
  0 siblings, 1 reply; 7+ messages in thread
From: Caraman Mihai Claudiu-B02008 @ 2013-04-25  9:09 UTC (permalink / raw)
  To: tiejun.chen@windriver.com; +Cc: kvm-ppc@vger.kernel.org, kvm@vger.kernel.org

> -----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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description
  2013-04-25  9:09 [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description Caraman Mihai Claudiu-B02008
@ 2013-04-25  9:16 ` tiejun.chen
       [not found]   ` <300B73AA675FCE4A93EB4FC1D42459FF3D967A@039-SN2MPN1-013.039d.mgd.msft.net>
  0 siblings, 1 reply; 7+ messages in thread
From: tiejun.chen @ 2013-04-25  9:16 UTC (permalink / raw)
  To: Caraman Mihai Claudiu-B02008; +Cc: kvm-ppc@vger.kernel.org, kvm@vger.kernel.org

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

[parent not found: <300B73AA675FCE4A93EB4FC1D42459FF3D967A@039-SN2MPN1-013.039d.mgd.msft.net>]

* Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description
       [not found]   ` <300B73AA675FCE4A93EB4FC1D42459FF3D967A@039-SN2MPN1-013.039d.mgd.msft.net>
@ 2013-04-25  9:49     ` tiejun.chen
       [not found]       ` <300B73AA675FCE4A93EB4FC1D42459FF3D997F@039-SN2MPN1-013.039d.mgd.msft.net>
  0 siblings, 1 reply; 7+ messages in thread
From: tiejun.chen @ 2013-04-25  9:49 UTC (permalink / raw)
  To: Caraman Mihai Claudiu-B02008; +Cc: kvm-ppc@vger.kernel.org, kvm@vger.kernel.org

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

[parent not found: <300B73AA675FCE4A93EB4FC1D42459FF3D997F@039-SN2MPN1-013.039d.mgd.msft.net>]

[parent not found: <51791195.7090909@windriver.com>]

[parent not found: <300B73AA675FCE4A93EB4FC1D42459FF3D9A8A@039-SN2MPN1-013.039d.mgd.msft.net>]

* Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description
       [not found]           ` <300B73AA675FCE4A93EB4FC1D42459FF3D9A8A@039-SN2MPN1-013.039d.mgd.msft.net>
@ 2013-04-26  3:11             ` tiejun.chen
  2013-04-26  9:38               ` tiejun.chen
  0 siblings, 1 reply; 7+ messages in thread
From: tiejun.chen @ 2013-04-26  3:11 UTC (permalink / raw)
  To: Caraman Mihai Claudiu-B02008; +Cc: kvm-ppc@vger.kernel.org, kvm@vger.kernel.org

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description
  2013-04-26  3:11             ` tiejun.chen
@ 2013-04-26  9:38               ` tiejun.chen
  0 siblings, 0 replies; 7+ messages in thread
From: tiejun.chen @ 2013-04-26  9:38 UTC (permalink / raw)
  To: Caraman Mihai Claudiu-B02008; +Cc: kvm-ppc@vger.kernel.org, kvm@vger.kernel.org

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 0/8 v3] KVM: PPC: e500: Enable FSL e6500 core
@ 2013-04-11 10:03 Mihai Caraman
  2013-04-11 10:03 ` [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description Mihai Caraman
  0 siblings, 1 reply; 7+ messages in thread
From: Mihai Caraman @ 2013-04-11 10:03 UTC (permalink / raw)
  To: kvm-ppc; +Cc: kvm, linuxppc-dev, Mihai Caraman

Enable basic support for Freescale e6500 core, adding MAV 2.0 support.
Validated on T4240QDS platfrom. Altivec, Multithreading and HW Tablewalk
are not addressed by this patchset.

Mihai Caraman (8):
  KVM: PPC: Book3E: Refactor ONE_REG ioctl implementation
  KVM: PPC: e500: Expose MMU registers via ONE_REG
  KVM: PPC: e500: Move vcpu's MMU configuration to dedicated functions
  KVM: PPC: e500: Add support for TLBnPS registers
  KVM: PPC: e500: Add support for EPTCFG register
  KVM: PPC: e500: Remove E.PT and E.HV.LRAT categories from VCPUs
  KVM: PPC: e500mc: Enable e6500 cores
  KVM: PPC: e500: Add e6500 core to Kconfig description

 Documentation/virtual/kvm/api.txt   |   16 +++
 arch/powerpc/include/asm/kvm_host.h |    2 +
 arch/powerpc/include/uapi/asm/kvm.h |   22 ++++
 arch/powerpc/kvm/44x.c              |   12 ++
 arch/powerpc/kvm/Kconfig            |    6 +-
 arch/powerpc/kvm/booke.c            |  102 ++++++++++---------
 arch/powerpc/kvm/e500.c             |   14 +++
 arch/powerpc/kvm/e500.h             |   22 ++++
 arch/powerpc/kvm/e500_emulate.c     |   19 ++++
 arch/powerpc/kvm/e500_mmu.c         |  192 +++++++++++++++++++++++++++++++----
 arch/powerpc/kvm/e500mc.c           |   16 +++
 11 files changed, 351 insertions(+), 72 deletions(-)

-- 
1.7.4.1



^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description
  2013-04-11 10:03 [PATCH 0/8 v3] KVM: PPC: e500: Enable FSL e6500 core Mihai Caraman
@ 2013-04-11 10:03 ` Mihai Caraman
  2013-04-19 10:03   ` tiejun.chen
  0 siblings, 1 reply; 7+ messages in thread
From: Mihai Caraman @ 2013-04-11 10:03 UTC (permalink / raw)
  To: kvm-ppc; +Cc: kvm, linuxppc-dev, 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



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description
  2013-04-11 10:03 ` [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description Mihai Caraman
@ 2013-04-19 10:03   ` tiejun.chen
  0 siblings, 0 replies; 7+ messages in thread
From: tiejun.chen @ 2013-04-19 10:03 UTC (permalink / raw)
  To: Mihai Caraman; +Cc: kvm-ppc, kvm

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.
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2013-04-26  9:38 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-25  9:09 [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description Caraman Mihai Claudiu-B02008
2013-04-25  9:16 ` tiejun.chen
     [not found]   ` <300B73AA675FCE4A93EB4FC1D42459FF3D967A@039-SN2MPN1-013.039d.mgd.msft.net>
2013-04-25  9:49     ` tiejun.chen
     [not found]       ` <300B73AA675FCE4A93EB4FC1D42459FF3D997F@039-SN2MPN1-013.039d.mgd.msft.net>
     [not found]         ` <51791195.7090909@windriver.com>
     [not found]           ` <300B73AA675FCE4A93EB4FC1D42459FF3D9A8A@039-SN2MPN1-013.039d.mgd.msft.net>
2013-04-26  3:11             ` tiejun.chen
2013-04-26  9:38               ` tiejun.chen
  -- strict thread matches above, loose matches on Subject: below --
2013-04-11 10:03 [PATCH 0/8 v3] KVM: PPC: e500: Enable FSL e6500 core Mihai Caraman
2013-04-11 10:03 ` [PATCH 8/8 v3] KVM: PPC: e500: Add e6500 core to Kconfig description Mihai Caraman
2013-04-19 10:03   ` tiejun.chen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox