diff mbox series

[2/2] qemux86/qemuarm: Drop kernel version overrides

Message ID 20231006113641.3248432-2-richard.purdie@linuxfoundation.org
State Accepted, archived
Commit 298fa078fab58b64246376ffd70ad6a0c7589876
Headers show
Series [1/2] cryptodev: Update to latest git for 6.5 kernel fixes | expand

Commit Message

Richard Purdie Oct. 6, 2023, 11:36 a.m. UTC
Drop the version overrides for the kernel for the x86 and arm machines
so we can go back to following the distro versions. The reasons for
these versions is mostly historical at this point as the issues were
resolved.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/conf/machine/include/x86/x86-base.inc | 1 -
 meta/conf/machine/qemuarm.conf             | 4 ----
 meta/conf/machine/qemuarmv5.conf           | 1 -
 3 files changed, 6 deletions(-)

Comments

Alejandro Enedino Hernandez Samaniego Oct. 11, 2023, 1:11 a.m. UTC | #1
On 10/6/23 05:36, Richard Purdie wrote:
> Drop the version overrides for the kernel for the x86 and arm machines
> so we can go back to following the distro versions. The reasons for
> these versions is mostly historical at this point as the issues were
> resolved.
>
> Signed-off-by: Richard Purdie<richard.purdie@linuxfoundation.org>
> ---
>   meta/conf/machine/include/x86/x86-base.inc | 1 -
>   meta/conf/machine/qemuarm.conf             | 4 ----
>   meta/conf/machine/qemuarmv5.conf           | 1 -
>   3 files changed, 6 deletions(-)
>
> diff --git a/meta/conf/machine/include/x86/x86-base.inc b/meta/conf/machine/include/x86/x86-base.inc
> index 050c20af70c..fc6c39148db 100644
> --- a/meta/conf/machine/include/x86/x86-base.inc
> +++ b/meta/conf/machine/include/x86/x86-base.inc
> @@ -18,7 +18,6 @@ SERIAL_CONSOLES ?= "115200;ttyS0"
>   # kernel-related variables
>   #
>   PREFERRED_PROVIDER_virtual/kernel ??= "linux-yocto"
> -PREFERRED_VERSION_linux-yocto ??= "6.4%"
>   
>   #
>   # XSERVER subcomponents, used to build the XSERVER variable
> diff --git a/meta/conf/machine/qemuarm.conf b/meta/conf/machine/qemuarm.conf
> index 04216cdc2e0..881733eb2da 100644
> --- a/meta/conf/machine/qemuarm.conf
> +++ b/meta/conf/machine/qemuarm.conf
> @@ -28,8 +28,4 @@ QB_ROOTFS_OPT = "-drive id=disk0,file=@ROOTFS@,if=none,format=raw -device virtio
>   QB_SERIAL_OPT = "-device virtio-serial-device -chardev null,id=virtcon -device virtconsole,chardev=virtcon"
>   QB_TCPSERIAL_OPT = "-device virtio-serial-device -chardev socket,id=virtcon,port=@PORT@,host=127.0.0.1  -device virtconsole,chardev=virtcon"
>   
> -# 6.4 causes Xorg to fail to read the virtio framebuffer so stick with 6.1 until
> -# that is resolved.
> -PREFERRED_VERSION_linux-yocto ??= "6.1%"
> -
>   KMACHINE:qemuarm = "qemuarma15"
> diff --git a/meta/conf/machine/qemuarmv5.conf b/meta/conf/machine/qemuarmv5.conf
> index deb4869c0aa..f0a86f1e4c7 100644
> --- a/meta/conf/machine/qemuarmv5.conf
> +++ b/meta/conf/machine/qemuarmv5.conf
> @@ -16,5 +16,4 @@ QB_GRAPHICS = "-device virtio-gpu-pci"
>   QB_OPT_APPEND = "-device qemu-xhci -device usb-tablet -device usb-kbd"
>   QB_DTB ="${@oe.utils.version_less_or_equal('PREFERRED_VERSION_linux-yocto', 
> '4.7', '', 'zImage-versatile-pb.dtb', d)}"
>   

Removing this bit results in the above line causing an ExpansionError, since
PREFERRED_VERSION_linux-yocto isnt defined anywhere anymore (if not using poky.conf)

What would be the right approach to solve this issue?, Should QB_DTB be depending on such variable?

Cheers,
Alejandro


> -PREFERRED_VERSION_linux-yocto ??= "6.4%"
>   KMACHINE:qemuarmv5 = "arm-versatile-926ejs"
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#188762):https://lists.openembedded.org/g/openembedded-core/message/188762
> Mute This Topic:https://lists.openembedded.org/mt/101795562/4354175
> Group Owner:openembedded-core+owner@lists.openembedded.org
> Unsubscribe:https://lists.openembedded.org/g/openembedded-core/unsub  [alhe@linux.microsoft.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Richard Purdie Oct. 11, 2023, 9:39 p.m. UTC | #2
On Tue, 2023-10-10 at 19:11 -0600, Alejandro Enedino Hernandez
Samaniego wrote:
>  
> 
>  
>  
> On 10/6/23 05:36, Richard Purdie wrote:
>  
>  
> >  
> > Drop the version overrides for the kernel for the x86 and arm
> > machines
> > so we can go back to following the distro versions. The reasons for
> > these versions is mostly historical at this point as the issues
> > were
> > resolved.
> > 
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > ---
> >  meta/conf/machine/include/x86/x86-base.inc | 1 -
> >  meta/conf/machine/qemuarm.conf             | 4 ----
> >  meta/conf/machine/qemuarmv5.conf           | 1 -
> >  3 files changed, 6 deletions(-)
> > 
> > diff --git a/meta/conf/machine/include/x86/x86-base.inc
> > b/meta/conf/machine/include/x86/x86-base.inc
> > index 050c20af70c..fc6c39148db 100644
> > --- a/meta/conf/machine/include/x86/x86-base.inc
> > +++ b/meta/conf/machine/include/x86/x86-base.inc
> > @@ -18,7 +18,6 @@ SERIAL_CONSOLES ?= "115200;ttyS0"
> >  # kernel-related variables
> >  #
> >  PREFERRED_PROVIDER_virtual/kernel ??= "linux-yocto"
> > -PREFERRED_VERSION_linux-yocto ??= "6.4%"
> >  
> >  #
> >  # XSERVER subcomponents, used to build the XSERVER variable
> > diff --git a/meta/conf/machine/qemuarm.conf
> > b/meta/conf/machine/qemuarm.conf
> > index 04216cdc2e0..881733eb2da 100644
> > --- a/meta/conf/machine/qemuarm.conf
> > +++ b/meta/conf/machine/qemuarm.conf
> > @@ -28,8 +28,4 @@ QB_ROOTFS_OPT = "-drive
> > id=disk0,file=@ROOTFS@,if=none,format=raw -device virtio
> >  QB_SERIAL_OPT = "-device virtio-serial-device -chardev
> > null,id=virtcon -device virtconsole,chardev=virtcon"
> >  QB_TCPSERIAL_OPT = "-device virtio-serial-device -chardev
> > socket,id=virtcon,port=@PORT@,host=127.0.0.1 -device
> > virtconsole,chardev=virtcon"
> >  
> > -# 6.4 causes Xorg to fail to read the virtio framebuffer so stick
> > with 6.1 until
> > -# that is resolved.
> > -PREFERRED_VERSION_linux-yocto ??= "6.1%"
> > -
> >  KMACHINE:qemuarm = "qemuarma15"
> > diff --git a/meta/conf/machine/qemuarmv5.conf
> > b/meta/conf/machine/qemuarmv5.conf
> > index deb4869c0aa..f0a86f1e4c7 100644
> > --- a/meta/conf/machine/qemuarmv5.conf
> > +++ b/meta/conf/machine/qemuarmv5.conf
> > @@ -16,5 +16,4 @@ QB_GRAPHICS = "-device virtio-gpu-pci"
> >  QB_OPT_APPEND = "-device qemu-xhci -device usb-tablet -device usb-
> > kbd"
> >  QB_DTB =
> > "${@oe.utils.version_less_or_equal('PREFERRED_VERSION_linux-yocto',
> > '4.7', '', 'zImage-versatile-pb.dtb', d)}"
> >  
> >  
>  
> Removing this bit results in the above line causing an
> ExpansionError, since
> PREFERRED_VERSION_linux-yocto isnt defined anywhere anymore (if not
> using poky.conf)
> 
> What would be the right approach to solve this issue?, Should QB_DTB
> be depending on such variable?

4.7 is old enough we can probably just drop the conditional now?

Cheers,

Richard
Bruce Ashfield Oct. 11, 2023, 9:54 p.m. UTC | #3
On Wed, Oct 11, 2023 at 5:39 PM Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> On Tue, 2023-10-10 at 19:11 -0600, Alejandro Enedino Hernandez
> Samaniego wrote:
> >
> >
> >
> >
> > On 10/6/23 05:36, Richard Purdie wrote:
> >
> >
> > >
> > > Drop the version overrides for the kernel for the x86 and arm
> > > machines
> > > so we can go back to following the distro versions. The reasons for
> > > these versions is mostly historical at this point as the issues
> > > were
> > > resolved.
> > >
> > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > > ---
> > >  meta/conf/machine/include/x86/x86-base.inc | 1 -
> > >  meta/conf/machine/qemuarm.conf             | 4 ----
> > >  meta/conf/machine/qemuarmv5.conf           | 1 -
> > >  3 files changed, 6 deletions(-)
> > >
> > > diff --git a/meta/conf/machine/include/x86/x86-base.inc
> > > b/meta/conf/machine/include/x86/x86-base.inc
> > > index 050c20af70c..fc6c39148db 100644
> > > --- a/meta/conf/machine/include/x86/x86-base.inc
> > > +++ b/meta/conf/machine/include/x86/x86-base.inc
> > > @@ -18,7 +18,6 @@ SERIAL_CONSOLES ?= "115200;ttyS0"
> > >  # kernel-related variables
> > >  #
> > >  PREFERRED_PROVIDER_virtual/kernel ??= "linux-yocto"
> > > -PREFERRED_VERSION_linux-yocto ??= "6.4%"
> > >
> > >  #
> > >  # XSERVER subcomponents, used to build the XSERVER variable
> > > diff --git a/meta/conf/machine/qemuarm.conf
> > > b/meta/conf/machine/qemuarm.conf
> > > index 04216cdc2e0..881733eb2da 100644
> > > --- a/meta/conf/machine/qemuarm.conf
> > > +++ b/meta/conf/machine/qemuarm.conf
> > > @@ -28,8 +28,4 @@ QB_ROOTFS_OPT = "-drive
> > > id=disk0,file=@ROOTFS@,if=none,format=raw -device virtio
> > >  QB_SERIAL_OPT = "-device virtio-serial-device -chardev
> > > null,id=virtcon -device virtconsole,chardev=virtcon"
> > >  QB_TCPSERIAL_OPT = "-device virtio-serial-device -chardev
> > > socket,id=virtcon,port=@PORT@,host=127.0.0.1 -device
> > > virtconsole,chardev=virtcon"
> > >
> > > -# 6.4 causes Xorg to fail to read the virtio framebuffer so stick
> > > with 6.1 until
> > > -# that is resolved.
> > > -PREFERRED_VERSION_linux-yocto ??= "6.1%"
> > > -
> > >  KMACHINE:qemuarm = "qemuarma15"
> > > diff --git a/meta/conf/machine/qemuarmv5.conf
> > > b/meta/conf/machine/qemuarmv5.conf
> > > index deb4869c0aa..f0a86f1e4c7 100644
> > > --- a/meta/conf/machine/qemuarmv5.conf
> > > +++ b/meta/conf/machine/qemuarmv5.conf
> > > @@ -16,5 +16,4 @@ QB_GRAPHICS = "-device virtio-gpu-pci"
> > >  QB_OPT_APPEND = "-device qemu-xhci -device usb-tablet -device usb-
> > > kbd"
> > >  QB_DTB =
> > > "${@oe.utils.version_less_or_equal('PREFERRED_VERSION_linux-yocto',
> > > '4.7', '', 'zImage-versatile-pb.dtb', d)}"
> > >
> > >
> >
> > Removing this bit results in the above line causing an
> > ExpansionError, since
> > PREFERRED_VERSION_linux-yocto isnt defined anywhere anymore (if not
> > using poky.conf)
> >
> > What would be the right approach to solve this issue?, Should QB_DTB
> > be depending on such variable?
>
> 4.7 is old enough we can probably just drop the conditional now?
>

Yep, that should be safe. It hasn't changed name or format in quite some
time.

(Although there has been some re-organization in 6.5/6.6 around device
trees, but that shouldn't come into play here).

Bruce



>
> Cheers,
>
> Richard
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#188972):
> https://lists.openembedded.org/g/openembedded-core/message/188972
> Mute This Topic: https://lists.openembedded.org/mt/101795562/1050810
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> bruce.ashfield@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Alejandro Enedino Hernandez Samaniego Oct. 13, 2023, 4:21 p.m. UTC | #4
On 10/11/23 15:54, Bruce Ashfield wrote:
>
>
> On Wed, Oct 11, 2023 at 5:39 PM Richard Purdie 
> <richard.purdie@linuxfoundation.org> wrote:
>
>     On Tue, 2023-10-10 at 19:11 -0600, Alejandro Enedino Hernandez
>     Samaniego wrote:
>     >
>     >
>     >
>     >
>     > On 10/6/23 05:36, Richard Purdie wrote:
>     >
>     >
>     > >
>     > > Drop the version overrides for the kernel for the x86 and arm
>     > > machines
>     > > so we can go back to following the distro versions. The
>     reasons for
>     > > these versions is mostly historical at this point as the issues
>     > > were
>     > > resolved.
>     > >
>     > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>     > > ---
>     > >  meta/conf/machine/include/x86/x86-base.inc | 1 -
>     > >  meta/conf/machine/qemuarm.conf             | 4 ----
>     > >  meta/conf/machine/qemuarmv5.conf           | 1 -
>     > >  3 files changed, 6 deletions(-)
>     > >
>     > > diff --git a/meta/conf/machine/include/x86/x86-base.inc
>     > > b/meta/conf/machine/include/x86/x86-base.inc
>     > > index 050c20af70c..fc6c39148db 100644
>     > > --- a/meta/conf/machine/include/x86/x86-base.inc
>     > > +++ b/meta/conf/machine/include/x86/x86-base.inc
>     > > @@ -18,7 +18,6 @@ SERIAL_CONSOLES ?= "115200;ttyS0"
>     > >  # kernel-related variables
>     > >  #
>     > >  PREFERRED_PROVIDER_virtual/kernel ??= "linux-yocto"
>     > > -PREFERRED_VERSION_linux-yocto ??= "6.4%"
>     > >
>     > >  #
>     > >  # XSERVER subcomponents, used to build the XSERVER variable
>     > > diff --git a/meta/conf/machine/qemuarm.conf
>     > > b/meta/conf/machine/qemuarm.conf
>     > > index 04216cdc2e0..881733eb2da 100644
>     > > --- a/meta/conf/machine/qemuarm.conf
>     > > +++ b/meta/conf/machine/qemuarm.conf
>     > > @@ -28,8 +28,4 @@ QB_ROOTFS_OPT = "-drive
>     > > id=disk0,file=@ROOTFS@,if=none,format=raw -device virtio
>     > >  QB_SERIAL_OPT = "-device virtio-serial-device -chardev
>     > > null,id=virtcon -device virtconsole,chardev=virtcon"
>     > >  QB_TCPSERIAL_OPT = "-device virtio-serial-device -chardev
>     > > socket,id=virtcon,port=@PORT@,host=127.0.0.1 -device
>     > > virtconsole,chardev=virtcon"
>     > >
>     > > -# 6.4 causes Xorg to fail to read the virtio framebuffer so stick
>     > > with 6.1 until
>     > > -# that is resolved.
>     > > -PREFERRED_VERSION_linux-yocto ??= "6.1%"
>     > > -
>     > >  KMACHINE:qemuarm = "qemuarma15"
>     > > diff --git a/meta/conf/machine/qemuarmv5.conf
>     > > b/meta/conf/machine/qemuarmv5.conf
>     > > index deb4869c0aa..f0a86f1e4c7 100644
>     > > --- a/meta/conf/machine/qemuarmv5.conf
>     > > +++ b/meta/conf/machine/qemuarmv5.conf
>     > > @@ -16,5 +16,4 @@ QB_GRAPHICS = "-device virtio-gpu-pci"
>     > >  QB_OPT_APPEND = "-device qemu-xhci -device usb-tablet -device
>     usb-
>     > > kbd"
>     > >  QB_DTB =
>     > >
>     "${@oe.utils.version_less_or_equal('PREFERRED_VERSION_linux-yocto',
>     > > '4.7', '', 'zImage-versatile-pb.dtb', d)}"
>     > >
>     > >
>     >
>     > Removing this bit results in the above line causing an
>     > ExpansionError, since
>     > PREFERRED_VERSION_linux-yocto isnt defined anywhere anymore (if not
>     > using poky.conf)
>     >
>     > What would be the right approach to solve this issue?, Should QB_DTB
>     > be depending on such variable?
>
>     4.7 is old enough we can probably just drop the conditional now?
>
>
> Yep, that should be safe. It hasn't changed name or format in quite 
> some time.
>
> (Although there has been some re-organization in 6.5/6.6 around device 
> trees, but that shouldn't come into play here).
>
> Bruce
>
>
>     Cheers,
>
>     Richard
>
Great, thanks for the pointers, sending patch.


Alejandro


>
>
>
>
>
> -- 
> - Thou shalt not follow the NULL pointer, for chaos and madness await 
> thee at its end
> - "Use the force Harry" - Gandalf, Star Trek II
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#188973):https://lists.openembedded.org/g/openembedded-core/message/188973
> Mute This Topic:https://lists.openembedded.org/mt/101795562/4354175
> Group Owner:openembedded-core+owner@lists.openembedded.org
> Unsubscribe:https://lists.openembedded.org/g/openembedded-core/unsub  [alhe@linux.microsoft.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta/conf/machine/include/x86/x86-base.inc b/meta/conf/machine/include/x86/x86-base.inc
index 050c20af70c..fc6c39148db 100644
--- a/meta/conf/machine/include/x86/x86-base.inc
+++ b/meta/conf/machine/include/x86/x86-base.inc
@@ -18,7 +18,6 @@  SERIAL_CONSOLES ?= "115200;ttyS0"
 # kernel-related variables
 #
 PREFERRED_PROVIDER_virtual/kernel ??= "linux-yocto"
-PREFERRED_VERSION_linux-yocto ??= "6.4%"
 
 #
 # XSERVER subcomponents, used to build the XSERVER variable
diff --git a/meta/conf/machine/qemuarm.conf b/meta/conf/machine/qemuarm.conf
index 04216cdc2e0..881733eb2da 100644
--- a/meta/conf/machine/qemuarm.conf
+++ b/meta/conf/machine/qemuarm.conf
@@ -28,8 +28,4 @@  QB_ROOTFS_OPT = "-drive id=disk0,file=@ROOTFS@,if=none,format=raw -device virtio
 QB_SERIAL_OPT = "-device virtio-serial-device -chardev null,id=virtcon -device virtconsole,chardev=virtcon"
 QB_TCPSERIAL_OPT = "-device virtio-serial-device -chardev socket,id=virtcon,port=@PORT@,host=127.0.0.1 -device virtconsole,chardev=virtcon"
 
-# 6.4 causes Xorg to fail to read the virtio framebuffer so stick with 6.1 until
-# that is resolved.
-PREFERRED_VERSION_linux-yocto ??= "6.1%"
-
 KMACHINE:qemuarm = "qemuarma15"
diff --git a/meta/conf/machine/qemuarmv5.conf b/meta/conf/machine/qemuarmv5.conf
index deb4869c0aa..f0a86f1e4c7 100644
--- a/meta/conf/machine/qemuarmv5.conf
+++ b/meta/conf/machine/qemuarmv5.conf
@@ -16,5 +16,4 @@  QB_GRAPHICS = "-device virtio-gpu-pci"
 QB_OPT_APPEND = "-device qemu-xhci -device usb-tablet -device usb-kbd"
 QB_DTB = "${@oe.utils.version_less_or_equal('PREFERRED_VERSION_linux-yocto', '4.7', '', 'zImage-versatile-pb.dtb', d)}"
 
-PREFERRED_VERSION_linux-yocto ??= "6.4%"
 KMACHINE:qemuarmv5 = "arm-versatile-926ejs"