On Wed, 2020-06-24 at 23:57 +0800, Xu, Yanfei wrote:
> 
> On 6/24/20 9:08 PM, Richard Purdie wrote:
> > On Wed, 2020-06-24 at 09:47 +0800, Xu, Yanfei wrote:
> > > On 6/24/20 5:51 AM, Richard Purdie wrote:
> > > > On Tue, 2020-06-23 at 10:31 -0400, Bruce Ashfield wrote:
> > > > > On Fri, Jun 19, 2020 at 7:47 AM <[email protected]>
> > > > > wrote:
> > > > > > From: Yanfei Xu <[email protected]>
> > > > > > 
> > > > > > Some filesystems don't support symlink, then you will get
> > > > > > failure
> > > > > > when
> > > > > > you install or update the kernel rpm package. Now we use a
> > > > > > copy
> > > > > > of
> > > > > > image for these filesystems instead of symlink.
> > > > > > 
> > > > > > Suggested-by: Bruce Ashfield <[email protected]>
> > > > > > Suggested-by: Richard Purdie <
> > > > > > [email protected]>
> > > > > > Signed-off-by: Yanfei Xu <[email protected]>
> > > > > > ---
> > > > > >    meta/classes/kernel.bbclass | 19 ++++++++++++++++---
> > > > > >    1 file changed, 16 insertions(+), 3 deletions(-)
> > > > > > 
> > > > > > diff --git a/meta/classes/kernel.bbclass
> > > > > > b/meta/classes/kernel.bbclass
> > > > > > index 20a0135fc9..1269b54343 100644
> > > > > > --- a/meta/classes/kernel.bbclass
> > > > > > +++ b/meta/classes/kernel.bbclass
> > > > > > @@ -94,6 +94,22 @@ python __anonymous () {
> > > > > >            d.appendVar('RDEPENDS_%s-image' % kname, ' %s-
> > > > > > image-
> > > > > > %s' %
> > > > > > (kname, typelower))
> > > > > >            d.setVar('PKG_%s-image-%s' % (kname,typelower),
> > > > > > '%s-
> > > > > > image-
> > > > > > %s-${KERNEL_VERSION_PKG_NAME}' % (kname, typelower))
> > > > > >            d.setVar('ALLOW_EMPTY_%s-image-%s' % (kname,
> > > > > > typelower),
> > > > > > '1')
> > > > > > +        d.setVar('pkg_postinst_ontarget_%s-image-%s' %
> > > > > > (kname,typelower), """
> > > > > 
> > > > > You were previously concerned about boot issues if the
> > > > > unversioned
> > > > > image link/copy was deferred until boot. I don't see a
> > > > > summary of
> > > > > why
> > > > > that's no longer a concern in the v3 of the patch. Can you
> > > > > confirm
> > > > > that it isn't an issue ? Is it simply because the booted
> > > > > image
> > > > > isn't
> > > > > installed via this package, and hence we are fine ?
> > > > 
> > > > I'm suspecting this patch is why we're seeing selftest
> > > > failures:
> > > > 
> > > > https://autobuilder.yoctoproject.org/typhoon/#/builders/86/builds/1055
> > > > 
> > > Do you mean this patch has been applied? and introduce new
> > > issues?
> > 
> > Yes, I reverted it and retested and confirmed this patch causes the
> > selftest failure in the link to the build above.
> > 
> OK, I am trying to analyse the log. But I haven't figured out what
> cause
> it fialed, and how to reproduce it on my mechine. I'd apprecate any
> information.

The command to reproduce should be:

oe-selftest -r efibootpartition.GenericEFITest.test_boot_efi

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#139893): 
https://lists.openembedded.org/g/openembedded-core/message/139893
Mute This Topic: https://lists.openembedded.org/mt/74977721/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to