Re: [OE-core] [PATCH 1/5] qemuarm: Swap for an arm7ve (A15) configuration
On Thu, 2019-03-21 at 16:35 -0400, Denys Dmytriyenko wrote: > On Wed, Mar 20, 2019 at 12:25:02AM +, > richard.pur...@linuxfoundation.org wrote: > > On Tue, 2019-03-19 at 20:22 -0400, Denys Dmytriyenko wrote: > > > On Mon, Mar 11, 2019 at 03:12:26AM +, Richard Purdie wrote: > > > > From: Jon Mason > > > > > > > > Add new QEMU BSP for a Arm Cortex-A15 system and use this as > > > > qemuarm, > > > > moving the old armv5te Versatile PB based machine to qemuarmv5. > > > > > > > > The new machine uses the QEMU virt machine type, which should > > > > be > > > > faster to emulate and updates the qemuarm support to a modern > > > > architecture. > > > > > > > > Signed-off-by: Jon Mason > > > > Signed-off-by: Richard Purdie < > > > > richard.pur...@linuxfoundation.org> > > > > --- > > > > meta/conf/machine/qemuarm.conf| 34 > > > > + > > > > -- > > > > meta/conf/machine/qemuarmv5.conf | 23 > > > > + > > > > .../linux/linux-yocto-rt_4.19.bb | 2 +- > > > > .../linux/linux-yocto-rt_5.0.bb | 2 +- > > > > .../linux/linux-yocto-tiny_4.19.bb| 2 +- > > > > .../linux/linux-yocto-tiny_5.0.bb | 2 +- > > > > meta/recipes-kernel/linux/linux-yocto_4.19.bb | 2 +- > > > > meta/recipes-kernel/linux/linux-yocto_5.0.bb | 2 +- > > > > 8 files changed, 52 insertions(+), 17 deletions(-) > > > > create mode 100644 meta/conf/machine/qemuarmv5.conf > > > > > > > > diff --git a/meta/conf/machine/qemuarm.conf > > > > b/meta/conf/machine/qemuarm.conf > > > > index a5443126273..95550461899 100644 > > > > --- a/meta/conf/machine/qemuarm.conf > > > > +++ b/meta/conf/machine/qemuarm.conf > > > ... > > > > +KMACHINE_qemuarmv5 = "qemuarm" > > > > \ No newline at end of file > > > > diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > > b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > > index 07dc5b77a14..36761b99437 100644 > > > > --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > > +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > > @@ -31,7 +31,7 @@ KCONF_BSP_AUDIT_LEVEL = "2" > > > > > > > > LINUX_KERNEL_TYPE = "preempt-rt" > > > > > > > > -COMPATIBLE_MACHINE = "(qemux86|qemux86- > > > > 64|qemuarm|qemuarm64|qemuppc|qemumips)" > > > > +COMPATIBLE_MACHINE = "(qemux86|qemux86- > > > > 64|qemuarm|qemuarma15|qemuarm64|qemuppc|qemumips)" > > > > > > Did you mean to list here "qemuarmv5" instead of "qemuarma15"? > > > > Yes! > > > > > > @@ -40,7 +40,7 @@ KCONF_BSP_AUDIT_LEVEL = "2" > > > > > > > > KERNEL_DEVICETREE_qemuarm = "versatile-pb.dtb" > > > > > > > > -COMPATIBLE_MACHINE = > > > > "qemuarm|qemuarm64|qemux86|qemuppc|qemumips|qemumips64|qemux86- > > > > 64" > > > > +COMPATIBLE_MACHINE = > > > > "qemuarm|qemuarmv5|qemuarm64|qemux86|qemuppc|qemumips|qemumips6 > > > > 4|qe > > > > mux86-64" > > > > > > Ok, "qemuarmv5" - I see a pattern here! Was it a challenge? :) > > > > It definitely was a challenge, perhaps not the way you mean it > > though! > > :) > > > > Patches to fix up the mess I made of this gratefully received, or > > I'll > > sort it out tomorrow, thanks for highlighting it. > > Submitted a patch last night: > http://lists.openembedded.org/pipermail/openembedded-core/2019-March/280345.html > > Not sure if you got to it sooner and fixed it yourself (don't see it > in git yet). Either way is fine. Let me know if you need anything > else. Thanks. Looks good, thanks. I'm struggling a bit after some travel unfortunately but doing my best to keep the patches moving. Its testing in -next now. Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/5] qemuarm: Swap for an arm7ve (A15) configuration
On Wed, Mar 20, 2019 at 12:25:02AM +, richard.pur...@linuxfoundation.org wrote: > On Tue, 2019-03-19 at 20:22 -0400, Denys Dmytriyenko wrote: > > On Mon, Mar 11, 2019 at 03:12:26AM +, Richard Purdie wrote: > > > From: Jon Mason > > > > > > Add new QEMU BSP for a Arm Cortex-A15 system and use this as > > > qemuarm, > > > moving the old armv5te Versatile PB based machine to qemuarmv5. > > > > > > The new machine uses the QEMU virt machine type, which should be > > > faster to emulate and updates the qemuarm support to a modern > > > architecture. > > > > > > Signed-off-by: Jon Mason > > > Signed-off-by: Richard Purdie > > > --- > > > meta/conf/machine/qemuarm.conf| 34 + > > > -- > > > meta/conf/machine/qemuarmv5.conf | 23 + > > > .../linux/linux-yocto-rt_4.19.bb | 2 +- > > > .../linux/linux-yocto-rt_5.0.bb | 2 +- > > > .../linux/linux-yocto-tiny_4.19.bb| 2 +- > > > .../linux/linux-yocto-tiny_5.0.bb | 2 +- > > > meta/recipes-kernel/linux/linux-yocto_4.19.bb | 2 +- > > > meta/recipes-kernel/linux/linux-yocto_5.0.bb | 2 +- > > > 8 files changed, 52 insertions(+), 17 deletions(-) > > > create mode 100644 meta/conf/machine/qemuarmv5.conf > > > > > > diff --git a/meta/conf/machine/qemuarm.conf > > > b/meta/conf/machine/qemuarm.conf > > > index a5443126273..95550461899 100644 > > > --- a/meta/conf/machine/qemuarm.conf > > > +++ b/meta/conf/machine/qemuarm.conf > > ... > > > +KMACHINE_qemuarmv5 = "qemuarm" > > > \ No newline at end of file > > > diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > index 07dc5b77a14..36761b99437 100644 > > > --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > @@ -31,7 +31,7 @@ KCONF_BSP_AUDIT_LEVEL = "2" > > > > > > LINUX_KERNEL_TYPE = "preempt-rt" > > > > > > -COMPATIBLE_MACHINE = "(qemux86|qemux86- > > > 64|qemuarm|qemuarm64|qemuppc|qemumips)" > > > +COMPATIBLE_MACHINE = "(qemux86|qemux86- > > > 64|qemuarm|qemuarma15|qemuarm64|qemuppc|qemumips)" > > > > Did you mean to list here "qemuarmv5" instead of "qemuarma15"? > > Yes! > > > > @@ -40,7 +40,7 @@ KCONF_BSP_AUDIT_LEVEL = "2" > > > > > > KERNEL_DEVICETREE_qemuarm = "versatile-pb.dtb" > > > > > > -COMPATIBLE_MACHINE = > > > "qemuarm|qemuarm64|qemux86|qemuppc|qemumips|qemumips64|qemux86-64" > > > +COMPATIBLE_MACHINE = > > > "qemuarm|qemuarmv5|qemuarm64|qemux86|qemuppc|qemumips|qemumips64|qe > > > mux86-64" > > > > Ok, "qemuarmv5" - I see a pattern here! Was it a challenge? :) > > It definitely was a challenge, perhaps not the way you mean it though! > :) > > Patches to fix up the mess I made of this gratefully received, or I'll > sort it out tomorrow, thanks for highlighting it. Submitted a patch last night: http://lists.openembedded.org/pipermail/openembedded-core/2019-March/280345.html Not sure if you got to it sooner and fixed it yourself (don't see it in git yet). Either way is fine. Let me know if you need anything else. Thanks. -- Denys -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/5] qemuarm: Swap for an arm7ve (A15) configuration
On Tue, 2019-03-19 at 20:22 -0400, Denys Dmytriyenko wrote: > On Mon, Mar 11, 2019 at 03:12:26AM +, Richard Purdie wrote: > > From: Jon Mason > > > > Add new QEMU BSP for a Arm Cortex-A15 system and use this as > > qemuarm, > > moving the old armv5te Versatile PB based machine to qemuarmv5. > > > > The new machine uses the QEMU virt machine type, which should be > > faster to emulate and updates the qemuarm support to a modern > > architecture. > > > > Signed-off-by: Jon Mason > > Signed-off-by: Richard Purdie > > --- > > meta/conf/machine/qemuarm.conf| 34 + > > -- > > meta/conf/machine/qemuarmv5.conf | 23 + > > .../linux/linux-yocto-rt_4.19.bb | 2 +- > > .../linux/linux-yocto-rt_5.0.bb | 2 +- > > .../linux/linux-yocto-tiny_4.19.bb| 2 +- > > .../linux/linux-yocto-tiny_5.0.bb | 2 +- > > meta/recipes-kernel/linux/linux-yocto_4.19.bb | 2 +- > > meta/recipes-kernel/linux/linux-yocto_5.0.bb | 2 +- > > 8 files changed, 52 insertions(+), 17 deletions(-) > > create mode 100644 meta/conf/machine/qemuarmv5.conf > > > > diff --git a/meta/conf/machine/qemuarm.conf > > b/meta/conf/machine/qemuarm.conf > > index a5443126273..95550461899 100644 > > --- a/meta/conf/machine/qemuarm.conf > > +++ b/meta/conf/machine/qemuarm.conf > ... > > +KMACHINE_qemuarmv5 = "qemuarm" > > \ No newline at end of file > > diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > index 07dc5b77a14..36761b99437 100644 > > --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > @@ -31,7 +31,7 @@ KCONF_BSP_AUDIT_LEVEL = "2" > > > > LINUX_KERNEL_TYPE = "preempt-rt" > > > > -COMPATIBLE_MACHINE = "(qemux86|qemux86- > > 64|qemuarm|qemuarm64|qemuppc|qemumips)" > > +COMPATIBLE_MACHINE = "(qemux86|qemux86- > > 64|qemuarm|qemuarma15|qemuarm64|qemuppc|qemumips)" > > Did you mean to list here "qemuarmv5" instead of "qemuarma15"? Yes! > > @@ -40,7 +40,7 @@ KCONF_BSP_AUDIT_LEVEL = "2" > > > > KERNEL_DEVICETREE_qemuarm = "versatile-pb.dtb" > > > > -COMPATIBLE_MACHINE = > > "qemuarm|qemuarm64|qemux86|qemuppc|qemumips|qemumips64|qemux86-64" > > +COMPATIBLE_MACHINE = > > "qemuarm|qemuarmv5|qemuarm64|qemux86|qemuppc|qemumips|qemumips64|qe > > mux86-64" > > Ok, "qemuarmv5" - I see a pattern here! Was it a challenge? :) It definitely was a challenge, perhaps not the way you mean it though! :) Patches to fix up the mess I made of this gratefully received, or I'll sort it out tomorrow, thanks for highlighting it. Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/5] qemuarm: Swap for an arm7ve (A15) configuration
On Mon, Mar 11, 2019 at 03:12:26AM +, Richard Purdie wrote:
> From: Jon Mason
>
> Add new QEMU BSP for a Arm Cortex-A15 system and use this as qemuarm,
> moving the old armv5te Versatile PB based machine to qemuarmv5.
>
> The new machine uses the QEMU virt machine type, which should be
> faster to emulate and updates the qemuarm support to a modern
> architecture.
>
> Signed-off-by: Jon Mason
> Signed-off-by: Richard Purdie
> ---
> meta/conf/machine/qemuarm.conf| 34 +--
> meta/conf/machine/qemuarmv5.conf | 23 +
> .../linux/linux-yocto-rt_4.19.bb | 2 +-
> .../linux/linux-yocto-rt_5.0.bb | 2 +-
> .../linux/linux-yocto-tiny_4.19.bb| 2 +-
> .../linux/linux-yocto-tiny_5.0.bb | 2 +-
> meta/recipes-kernel/linux/linux-yocto_4.19.bb | 2 +-
> meta/recipes-kernel/linux/linux-yocto_5.0.bb | 2 +-
> 8 files changed, 52 insertions(+), 17 deletions(-)
> create mode 100644 meta/conf/machine/qemuarmv5.conf
>
> diff --git a/meta/conf/machine/qemuarm.conf b/meta/conf/machine/qemuarm.conf
> index a5443126273..95550461899 100644
> --- a/meta/conf/machine/qemuarm.conf
> +++ b/meta/conf/machine/qemuarm.conf
...
> +KMACHINE_qemuarmv5 = "qemuarm"
> \ No newline at end of file
> diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> index 07dc5b77a14..36761b99437 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> @@ -31,7 +31,7 @@ KCONF_BSP_AUDIT_LEVEL = "2"
>
> LINUX_KERNEL_TYPE = "preempt-rt"
>
> -COMPATIBLE_MACHINE =
> "(qemux86|qemux86-64|qemuarm|qemuarm64|qemuppc|qemumips)"
> +COMPATIBLE_MACHINE =
> "(qemux86|qemux86-64|qemuarm|qemuarma15|qemuarm64|qemuppc|qemumips)"
Did you mean to list here "qemuarmv5" instead of "qemuarma15"?
>
> KERNEL_DEVICETREE_qemuarm = "versatile-pb.dtb"
>
> diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> b/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> index 2350973fef2..52c289903be 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> @@ -31,7 +31,7 @@ KCONF_BSP_AUDIT_LEVEL = "2"
>
> LINUX_KERNEL_TYPE = "preempt-rt"
>
> -COMPATIBLE_MACHINE =
> "(qemux86|qemux86-64|qemuarm|qemuarm64|qemuppc|qemumips)"
> +COMPATIBLE_MACHINE =
> "(qemux86|qemux86-64|qemuarm|qemuarmv5|qemuarm64|qemuppc|qemumips)"
This one uses the correct "qemuarmv5", though.
> KERNEL_DEVICETREE_qemuarm = "versatile-pb.dtb"
>
> diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_4.19.bb
> b/meta/recipes-kernel/linux/linux-yocto-tiny_4.19.bb
> index c6d1594a828..52dbabd7a47 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-tiny_4.19.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-tiny_4.19.bb
> @@ -24,7 +24,7 @@ PV = "${LINUX_VERSION}+git${SRCPV}"
> SRC_URI =
> "git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \
>
> git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-4.19;destsuffix=${KMETA}"
>
> -COMPATIBLE_MACHINE = "qemux86|qemux86-64|qemuarm"
> +COMPATIBLE_MACHINE = "qemux86|qemux86-64|qemuarm|qemuarma15"
Here again looks like a typo - "qemuarma15"
> # Functionality flags
> KERNEL_FEATURES = ""
> diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_5.0.bb
> b/meta/recipes-kernel/linux/linux-yocto-tiny_5.0.bb
> index e0a68014461..b6401199a41 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-tiny_5.0.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-tiny_5.0.bb
> @@ -24,7 +24,7 @@ PV = "${LINUX_VERSION}+git${SRCPV}"
> SRC_URI =
> "git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine \
>
> git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-5.0;destsuffix=${KMETA}"
>
> -COMPATIBLE_MACHINE = "qemux86|qemux86-64|qemuarm"
> +COMPATIBLE_MACHINE = "qemux86|qemux86-64|qemuarm|qemuarmv5"
Correct "qemuarmv5" here.
> # Functionality flags
> KERNEL_FEATURES = ""
> diff --git a/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> index 8a4355ee833..3ef26a12baa 100644
> --- a/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> @@ -37,7 +37,7 @@ KCONF_BSP_AUDIT_LEVEL = "2"
>
> KERNEL_DEVICETREE_qemuarm = "versatile-pb.dtb"
>
> -COMPATIBLE_MACHINE =
> "qemuarm|qemuarm64|qemux86|qemuppc|qemumips|qemumips64|qemux86-64"
> +COMPATIBLE_MACHINE =
> "qemuarm|qemuarma15|qemuarm64|qemux86|qemuppc|qemumips|qemumips64|qemux86-64"
"qemuarma15"?
> # Functionality flags
> KERNEL_EXTRA_FEATURES ?= "features/netfilter/netfilter.scc"
> diff --git a/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> b/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> index f2c066a1503..ae6e80c0108 100644
> ---
Re: [OE-core] [PATCH 1/5] qemuarm: Swap for an arm7ve (A15) configuration
On Mon, Mar 11, 2019 at 08:13:21AM -0700, Richard Purdie wrote: > On Mon, 2019-03-11 at 11:09 -0400, Tom Rini wrote: > > On Mon, Mar 11, 2019 at 03:12:26AM +, Richard Purdie wrote: > > > > > From: Jon Mason > > > > > > Add new QEMU BSP for a Arm Cortex-A15 system and use this as > > > qemuarm, > > > moving the old armv5te Versatile PB based machine to qemuarmv5. > > > > > > The new machine uses the QEMU virt machine type, which should be > > > faster to emulate and updates the qemuarm support to a modern > > > architecture. > > > > > > Signed-off-by: Jon Mason > > > Signed-off-by: Richard Purdie > > > > Should we also not set UBOOT_MACHINE here to qemu_arm_defconfig for > > the new machine? We dropped versatilepb support back in 2015 > > however. > > Thanks! > > For which machine, the new qemuarm or the qemuarmv5? I'm guessing you > mean the latter? For the new v7 machine that uses the virt machine upstream. > We perhaps could/should but it isn't set now so I'd suggest it be done > in a separate patch. I see we aren't doing it for qemuarm64 either which is also supported in U-Boot under qemu_arm64_defconfig, so yes, I'll do a follow-up to enable in both of these. And to qemumips* too. > I would like to see u-boot being used in more builds but if we don't > actually test/use the binary, its not so useful and that would mean > further changes? So, humm. As a serious question, how do we test wic images / ovmf today? On the U-Boot side of things, we test all of our qemu images frequently with our functional tests. What we don't have atm is "boot linux to login prompt" or similar. -- Tom signature.asc Description: PGP signature -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/5] qemuarm: Swap for an arm7ve (A15) configuration
On Mon, 2019-03-11 at 11:09 -0400, Tom Rini wrote: > On Mon, Mar 11, 2019 at 03:12:26AM +, Richard Purdie wrote: > > > From: Jon Mason > > > > Add new QEMU BSP for a Arm Cortex-A15 system and use this as > > qemuarm, > > moving the old armv5te Versatile PB based machine to qemuarmv5. > > > > The new machine uses the QEMU virt machine type, which should be > > faster to emulate and updates the qemuarm support to a modern > > architecture. > > > > Signed-off-by: Jon Mason > > Signed-off-by: Richard Purdie > > Should we also not set UBOOT_MACHINE here to qemu_arm_defconfig for > the new machine? We dropped versatilepb support back in 2015 > however. > Thanks! For which machine, the new qemuarm or the qemuarmv5? I'm guessing you mean the latter? We perhaps could/should but it isn't set now so I'd suggest it be done in a separate patch. I would like to see u-boot being used in more builds but if we don't actually test/use the binary, its not so useful and that would mean further changes? Cheers, Richard -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH 1/5] qemuarm: Swap for an arm7ve (A15) configuration
On Mon, Mar 11, 2019 at 03:12:26AM +, Richard Purdie wrote: > From: Jon Mason > > Add new QEMU BSP for a Arm Cortex-A15 system and use this as qemuarm, > moving the old armv5te Versatile PB based machine to qemuarmv5. > > The new machine uses the QEMU virt machine type, which should be > faster to emulate and updates the qemuarm support to a modern > architecture. > > Signed-off-by: Jon Mason > Signed-off-by: Richard Purdie Should we also not set UBOOT_MACHINE here to qemu_arm_defconfig for the new machine? We dropped versatilepb support back in 2015 however. Thanks! -- Tom signature.asc Description: PGP signature -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
