Re: [OE-core] [PATCH 2/2] linux-yocto: Add scsi_debug module when ptest is in DISTRO_FEATURES

2019-06-15 Thread Mariano Lopez


On 6/15/19 10:15 AM, Richard Purdie wrote:

On Thu, 2019-06-13 at 22:32 -0500, Mariano López wrote:

util-linux ptest requires the scsi_debug module to perform eject/mount
tests. This will conditionally add scsi_debug module when ptest is in
DISTRO_FEATURES.

This doesn't include linux-yocto-tiny because the resulting image will
be too big and do_image would complain about this.

[YOCTO #13301]

Signed-off-by: Mariano López 
---
  meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb | 1 +
  meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb  | 1 +
  meta/recipes-kernel/linux/linux-yocto_4.19.bb| 1 +
  meta/recipes-kernel/linux/linux-yocto_5.0.bb | 1 +
  4 files changed, 4 insertions(+)

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 6604bdf90c..0836dc7ea0 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
@@ -41,3 +41,4 @@ KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
  KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
+KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
features/scsi/scsi-debug.scc", "" ,d)}"
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 1fe28b1bf0..b5e415f935 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
@@ -41,3 +41,4 @@ KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
  KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
+KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
features/scsi/scsi-debug.scc", "" ,d)}"
diff --git a/meta/recipes-kernel/linux/linux-yocto_4.19.bb 
b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
index a5fdafe6b2..cda4ecf89e 100644
--- a/meta/recipes-kernel/linux/linux-yocto_4.19.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
@@ -47,3 +47,4 @@ KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", 
"" ,d)}"
+KERNEL_FEATURES_append = " ${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
features/scsi/scsi-debug.scc", "" ,d)}"
diff --git a/meta/recipes-kernel/linux/linux-yocto_5.0.bb 
b/meta/recipes-kernel/linux/linux-yocto_5.0.bb
index da795d97aa..8aec315084 100644
--- a/meta/recipes-kernel/linux/linux-yocto_5.0.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_5.0.bb
@@ -50,3 +50,4 @@ KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", 
"" ,d)}"
+KERNEL_FEATURES_append = " ${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
features/scsi/scsi-debug.scc", "" ,d)}"

I merged this but have just realised its causing warnings:

https://autobuilder.yoctoproject.org/typhoon/#/builders/93/builds/31

which suggest its not quite doing what we hope/need? :/

Cheers,

Richard


Indeed, it seems the configuration is not applied for the qemuarm64 
MACHINE:(


Let me look into this and send another patch

Thanks,
Mairano

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 2/2] linux-yocto: Add scsi_debug module when ptest is in DISTRO_FEATURES

2019-06-15 Thread Richard Purdie
On Thu, 2019-06-13 at 22:32 -0500, Mariano López wrote:
> util-linux ptest requires the scsi_debug module to perform eject/mount
> tests. This will conditionally add scsi_debug module when ptest is in
> DISTRO_FEATURES.
> 
> This doesn't include linux-yocto-tiny because the resulting image will
> be too big and do_image would complain about this.
> 
> [YOCTO #13301]
> 
> Signed-off-by: Mariano López 
> ---
>  meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb | 1 +
>  meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb  | 1 +
>  meta/recipes-kernel/linux/linux-yocto_4.19.bb| 1 +
>  meta/recipes-kernel/linux/linux-yocto_5.0.bb | 1 +
>  4 files changed, 4 insertions(+)
> 
> 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 6604bdf90c..0836dc7ea0 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> @@ -41,3 +41,4 @@ KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
>  KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
> +KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
> features/scsi/scsi-debug.scc", "" ,d)}"
> 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 1fe28b1bf0..b5e415f935 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> @@ -41,3 +41,4 @@ KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
>  KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
> +KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES", "ptest", " 
> features/scsi/scsi-debug.scc", "" ,d)}"
> diff --git a/meta/recipes-kernel/linux/linux-yocto_4.19.bb 
> b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> index a5fdafe6b2..cda4ecf89e 100644
> --- a/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> @@ -47,3 +47,4 @@ KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " 
> cfg/x32.scc", "" ,d)}"
> +KERNEL_FEATURES_append = " ${@bb.utils.contains("DISTRO_FEATURES", "ptest", 
> " features/scsi/scsi-debug.scc", "" ,d)}"
> diff --git a/meta/recipes-kernel/linux/linux-yocto_5.0.bb 
> b/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> index da795d97aa..8aec315084 100644
> --- a/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> @@ -50,3 +50,4 @@ KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " 
> cfg/x32.scc", "" ,d)}"
> +KERNEL_FEATURES_append = " ${@bb.utils.contains("DISTRO_FEATURES", "ptest", 
> " features/scsi/scsi-debug.scc", "" ,d)}"

I merged this but have just realised its causing warnings:

https://autobuilder.yoctoproject.org/typhoon/#/builders/93/builds/31

which suggest its not quite doing what we hope/need? :/

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/4] util-linux: add setpriv utility

2019-06-15 Thread Richard Purdie
On Fri, 2019-06-14 at 12:45 -0400, Randy MacLeod wrote:
> Enable the setpriv utility for targets only. It will be used in
> the run-ptest script for bash and perhaps other packages where
> the ptest code is expected to run as a user.
> 
> setpriv uses libcap-ng which doesn't build natively so disable
> it for native* builds. Also, busybox has a setpriv implementation
> so ensure that setpriv adheres to the alternatives scheme.
> 
> Signed-off-by: Randy MacLeod 
> ---
>  meta/recipes-core/util-linux/util-linux.inc | 15 +++
>  1 file changed, 11 insertions(+), 4 deletions(-)

No errors but the build is now seeing warnings e.g.:

https://autobuilder.yoctoproject.org/typhoon/#/builders/42/builds/703

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 2/2] linux-yocto: Add scsi_debug module when ptest is in DISTRO_FEATURES

2019-06-15 Thread Bruce Ashfield
On Sat, Jun 15, 2019 at 3:47 PM Mariano Lopez
 wrote:
>
>
> On 6/15/19 10:15 AM, Richard Purdie wrote:
> > On Thu, 2019-06-13 at 22:32 -0500, Mariano López wrote:
> >> util-linux ptest requires the scsi_debug module to perform eject/mount
> >> tests. This will conditionally add scsi_debug module when ptest is in
> >> DISTRO_FEATURES.
> >>
> >> This doesn't include linux-yocto-tiny because the resulting image will
> >> be too big and do_image would complain about this.
> >>
> >> [YOCTO #13301]
> >>
> >> Signed-off-by: Mariano López 
> >> ---
> >>   meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb | 1 +
> >>   meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb  | 1 +
> >>   meta/recipes-kernel/linux/linux-yocto_4.19.bb| 1 +
> >>   meta/recipes-kernel/linux/linux-yocto_5.0.bb | 1 +
> >>   4 files changed, 4 insertions(+)
> >>
> >> 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 6604bdf90c..0836dc7ea0 100644
> >> --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> >> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb
> >> @@ -41,3 +41,4 @@ KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
> >>   KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
> >>   KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >>   KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >> +KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES", 
> >> "ptest", " features/scsi/scsi-debug.scc", "" ,d)}"
> >> 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 1fe28b1bf0..b5e415f935 100644
> >> --- a/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> >> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb
> >> @@ -41,3 +41,4 @@ KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
> >>   KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
> >>   KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >>   KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >> +KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES", 
> >> "ptest", " features/scsi/scsi-debug.scc", "" ,d)}"
> >> diff --git a/meta/recipes-kernel/linux/linux-yocto_4.19.bb 
> >> b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> >> index a5fdafe6b2..cda4ecf89e 100644
> >> --- a/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> >> +++ b/meta/recipes-kernel/linux/linux-yocto_4.19.bb
> >> @@ -47,3 +47,4 @@ KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
> >>   KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >>   KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >>   KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", 
> >> " cfg/x32.scc", "" ,d)}"
> >> +KERNEL_FEATURES_append = " ${@bb.utils.contains("DISTRO_FEATURES", 
> >> "ptest", " features/scsi/scsi-debug.scc", "" ,d)}"
> >> diff --git a/meta/recipes-kernel/linux/linux-yocto_5.0.bb 
> >> b/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> >> index da795d97aa..8aec315084 100644
> >> --- a/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> >> +++ b/meta/recipes-kernel/linux/linux-yocto_5.0.bb
> >> @@ -50,3 +50,4 @@ KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc"
> >>   KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >>   KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc cfg/paravirt_kvm.scc"
> >>   KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", 
> >> " cfg/x32.scc", "" ,d)}"
> >> +KERNEL_FEATURES_append = " ${@bb.utils.contains("DISTRO_FEATURES", 
> >> "ptest", " features/scsi/scsi-debug.scc", "" ,d)}"
> > I merged this but have just realised its causing warnings:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/93/builds/31
> >
> > which suggest its not quite doing what we hope/need? :/
> >
> > Cheers,
> >
> > Richard
>
> Indeed, it seems the configuration is not applied for the qemuarm64
> MACHINE:(
>

It would be applied, but it may be missing some dependency on that
machine and is hence not enabled in the final kernel .config.

Bruce

> Let me look into this and send another patch
>
> Thanks,
> Mairano
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core