From: Trevor Woerner <twoerner@gmail.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: alex.kanavin@gmail.com, paul@pbarker.dev,
openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH v2 1/2] oeqa/sdk: update cryptodev to a revision that builds on kernel 6.18
Date: Sat, 22 Aug 2026 11:00:50 -0400 [thread overview]
Message-ID: <aom5olXslkDa44p8@localhost.localdomain> (raw)
In-Reply-To: <597b689f434fa7eb1b763db08652e393f2f71d18.camel@linuxfoundation.org>
On Mon 2026-08-17 @ 01:48:55 PM, Richard Purdie wrote:
> On Mon, 2026-08-17 at 12:51 +0200, Alexander Kanavin via lists.openembedded.org wrote:
> > On Sun, 16 Aug 2026 at 13:12, Paul Barker via lists.openembedded.org
> > <paul=pbarker.dev@lists.openembedded.org> wrote:
> >
> > > I'm wondering why we haven't spotted this issue previously, we've been
> > > using Linux 6.18 for months now. Are we missing a test case?
> >
> > I suspect it's because the execution of useful cases for sdk and image
> > runtimes tends to regress due to 'soft-skip', e.g. if a needed package
> > isn't present, the test isn't run. I don't think this is quite right,
> > and each particular image and sdk should check that particular tests
> > do run, and if they aren't, that's a fail in itself. I'm not yet sure
> > how to technically implement this.
>
> I've worried about this for a while too.
>
> We really need to mark up the images with the lists of tests we expect
> to pass. It is tricky as that list can vary depending on distro
> features and package classes but should be possible.
Agreed on the goal. Having gone through what the cases actually do, I
want to be careful about how much of it one mechanism can cover, because
the skips are not all the same shape.
Some are a stated requirement. cmake, gtk3, kmod, maturin, meson, perl
and python each name a package and skip when the SDK's manifest does not
list it. Those are already machine-readable, and one narrow check on
them looks worth having by itself: if a case names a requirement, the
manifest satisfies it, and the case skips anyway, that is a bug rather
than a configuration. That is exactly what the twelve above were, and
catching it needs no per-image expected list.
The rest are not declarations at all. autotools, cmake, gcc, makefile
and meson skip on "SDK doesn't contain a supported C library"; gcc, go
and rust skip on not finding a cross-canadian toolchain; go also skips
when the command raises. Those are runtime probes of the SDK that was
installed, and marking up an image cannot say in advance what they will
decide. The go and rust cases are among the ones that skip on a default
SDK, so this is not a rare shape.
So I do not think declarations on their own produce the expected-test
list, and I would rather say that now than discover it half way through.
The configuration-dependence you name looks like the hard part, and I do
not have a design for it.
What I am willing to commit to is the narrow check above, since the
oeqa/sdk change it builds on is already written. If that seems useful I
will send it as its own patch and we can see whether it earns its keep
before anyone builds anything larger on top.
> The other option is to have resulttool have more checks where it
> compares to a list of tests we should see running. The regression
> report is supposed to partly handle that but if you miss a regression,
> the difference will no longer show it...
>
> Cheers,
>
> Richard
next prev parent reply other threads:[~2026-08-22 15:01 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-13 3:05 [PATCH v2 0/2] fix the SDK kernel module test Trevor Woerner
2026-08-13 3:05 ` [PATCH v2 1/2] oeqa/sdk: update cryptodev to a revision that builds on kernel 6.18 Trevor Woerner
2026-08-16 11:12 ` Paul Barker
2026-08-17 10:51 ` [OE-core] " Alexander Kanavin
2026-08-17 12:48 ` Richard Purdie
2026-08-22 15:00 ` Trevor Woerner [this message]
2026-08-22 14:52 ` Trevor Woerner
2026-08-24 7:26 ` Paul Barker
2026-08-13 3:05 ` [PATCH v2 2/2] oeqa/sdk: build the kernel's host tools with a host pkg-config Trevor Woerner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aom5olXslkDa44p8@localhost.localdomain \
--to=twoerner@gmail.com \
--cc=alex.kanavin@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=paul@pbarker.dev \
--cc=richard.purdie@linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox