From: "Zoltan Boszormenyi" <zboszor@pr.hu>
To: Jon Mason <jdmason@kudzu.us>, Andrey Zhizhikin <andrey.z@gmail.com>
Cc: "Böszörményi Zoltán" <zboszor@gmail.com>,
"OE Core mailing list" <openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH v4 3/4] kernel.bbclass: Use full versions for inter-package dependencies
Date: Tue, 31 Aug 2021 06:28:47 +0200 [thread overview]
Message-ID: <f52a6dae-426c-34f3-448d-e4ba50b23438@pr.hu> (raw)
In-Reply-To: <CAPoiz9xDN1Mx6B_TWFn0FYArM1+Y1PoXmY5TBqMBo4CHk3KfiA@mail.gmail.com>
2021. 08. 30. 21:51 keltezéssel, Jon Mason írta:
> On Mon, Aug 30, 2021 at 6:26 AM Andrey Zhizhikin <andrey.z@gmail.com> wrote:
>>
>> On Mon, Aug 30, 2021 at 12:06 PM Böszörményi Zoltán <zboszor@gmail.com> wrote:
>>>
>>> 2021. 08. 30. 11:30 keltezéssel, Andrey Zhizhikin írta:
>>>> Hello Zoltan,
>>>>
>>>> On Fri, Aug 27, 2021 at 9:37 AM Zoltan Boszormenyi via
>>>> lists.openembedded.org <zboszor=pr.hu@lists.openembedded.org> wrote:
>>>>> From: Zoltán Böszörményi <zboszor@gmail.com>
>>>>>
>>>>> If the kernel configuration enables module signing but no key
>>>>> is provided, then the kernel generates one during the kernel build.
>>>>>
>>>>> The current runtime-dependency references (with only package names
>>>>> without full versions) allow mixed package installations from different
>>>>> rebuilds of the same kernel version.
>>>>>
>>>>> This creates an issue because then the modules either don't work
>>>>> or taint the kernel.
>>>>>
>>>>> Tighten RDEPENDS with the full package version, i.e. use (= ${EXTENDPKGV})
>>>>> markers for inter-package dependencies.
>>>>>
>>>>> The kernel will pull in the kernel-modules subpackage of the same
>>>>> exact version automatically if KERNEL_SPLIT_MODULES="0" is set.
>>>>> Otherwise the situation is the same as with the old default with
>>>>> one subpackage per kernel module where they have to be upgraded
>>>>> manually.
>>>>>
>>>>> Signed-off-by: Zoltán Böszörményi <zboszor@gmail.com>
>>>>> ---
>>>>> meta/classes/kernel.bbclass | 13 +++++++------
>>>>> 1 file changed, 7 insertions(+), 6 deletions(-)
>>>>>
>>>> I'm seeing errors during the do_rootfs() with this patch applied,
>>>> there are few messages like this:
>>>>
>>>> * Solver encountered 1 problem(s):
>>>> * Problem 1/1:
>>>> * - package kernel-module-libchacha-5.13.13+g91381833a4e2-5.13.13+git0+91381833a4-r0.imx8mp_lpddr4_evk
>>>> requires kernel-5.13.13+g91381833a4e2, but none of the providers can
>>>> be installed
>>>> * - package kernel-modules-5.13.13+git0+91381833a4-r0.imx8mp_lpddr4_evk
>>>> requires kernel-module-libchacha-5.13.13+g91381833a4e2, but none of
>>>> the providers can be installed
>>>> * - package kernel-5.13.13+g91381833a4e2-5.13.13+git0+91381833a4-r0.imx8mp_lpddr4_evk
>>>> requires kernel-image-5.13.13+g91381833a4e2 =
>>>> 5.13.13+git0+91381833a4-r0, but none of the providers can be installed
>>>> * - conflicting requests
>>>> * - nothing provides kernel-image-image-5.13.13+g91381833a4e2 =
>>>> 5.13.13+gitAUTOINC+91381833a4-r0 needed by
>>>
>>> This seems to be the problem.
>>> Is there a "kernel-image-image-5.13.13" built from your kernel recipe?
>>
>> Yes, it is produced. But for some reasons opkg cannot resolve it
>> during the do_rootfs(), which is quite odd.
>
> I'm seeing the same issue. All of the BSPs that I set the kernel to
> not be 5.13 (i.e., 5.10, 5.4, etc) fail. For example,
> https://gitlab.com/jonmason00/meta-arm/-/jobs/1544819828
> If I set the PACKAGE_CLASS to be rpm instead of ipk, everything works
> as expected. So, there must be some difference in the dep calculation
> in ipk.
Interesting.
Can you both please try setting KERNEL_SPLIT_MODULES="0" in your kernel recipe?
In my testing, it works for both ipk and rpm properly.
I am thinking that probably the full version dependency should
only be used in the KERNEL_SPLIT_MODULES="0" case.
This can be easily tested, unlike the packaging method.
>
>
> Thanks,
> Jon
>
>>> For me, a kernel-image-bzimage-x.y.z is built.
>>>
>>>> kernel-image-5.13.13+g91381833a4e2-5.13.13+git0+91381833a4-r0.imx8mp_lpddr4_evk
>>>> *
>>>> * Solution 1:
>>>> * - do not ask to install a package providing kernel-modules
>>>>
>>>> Reverting this commit does solve it, but I would like to understand
>>>> more on what should be done in order to use this new mechanism you've
>>>> introduced. Do you have any suggestion on what should be adapted in
>>>> the BSP layer to have those RDEPENDS properly resolved?
>>>>
>>>> Package manager is set for me to IPK.
>>>>
>>>> Setting KERNEL_SPLIT_MODULES="1" does not solve the issue.
>>>
>>> This is the default.
>>> But setting it to "0" won't change whether the
>>> kernel-image-image-x.y.z subpackage gets built or not.
>>>
>>
>>
>> --
>> Regards,
>> Andrey.
>>
>>
>>
>>
>>
>>
>>
next prev parent reply other threads:[~2021-08-31 4:29 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-23 12:23 Kernel and RPM related bbclass changes Zoltan Boszormenyi
2021-08-23 12:23 ` [PATCH 1/6] package_rpm.bbclass: Handle posttrans scriptlets Zoltan Boszormenyi
2021-08-23 12:34 ` [OE-core] " Alexander Kanavin
2021-08-23 13:14 ` Zoltan Boszormenyi
2021-08-23 13:23 ` Richard Purdie
2021-08-23 13:42 ` Zoltan Boszormenyi
2021-08-23 12:23 ` [PATCH 2/6] kernel-module-split.bbclass: Support zstd-compressed modules Zoltan Boszormenyi
2021-08-23 12:23 ` [PATCH 3/6] kernel-module-split.bbclass: Allow opt-out of split kernel modules Zoltan Boszormenyi
2021-08-23 12:55 ` [OE-core] " Bruce Ashfield
2021-08-23 13:18 ` Zoltan Boszormenyi
2021-08-23 13:27 ` Bruce Ashfield
2021-08-23 13:31 ` Zoltan Boszormenyi
2021-08-23 13:35 ` Bruce Ashfield
[not found] ` <169DF0BA24F74A88.27647@lists.openembedded.org>
2021-08-23 12:56 ` Bruce Ashfield
2021-08-23 12:23 ` [PATCH 4/6] kernel.bbclass: Adapt to KERNEL_SPLIT_MODULES != "1" case Zoltan Boszormenyi
2021-08-23 12:23 ` [PATCH 5/6] kernel.bbclass: Use full versions for inter-package dependencies Zoltan Boszormenyi
2021-08-23 13:03 ` [OE-core] " Bruce Ashfield
2021-08-23 13:29 ` Zoltan Boszormenyi
2021-08-23 13:38 ` Bruce Ashfield
2021-08-23 13:48 ` Zoltan Boszormenyi
2021-08-23 12:23 ` [PATCH 6/6] Support zstd-compressed squashfs and cpio initramfs Zoltan Boszormenyi
2021-08-23 14:47 ` Kernel related bbclass changes Zoltan Boszormenyi
2021-08-23 14:47 ` [PATCH v2 1/4] kernel-module-split.bbclass: Support zstd-compressed modules Zoltan Boszormenyi
2021-08-23 14:47 ` [PATCH v2 2/4] Allow opt-out of split kernel modules Zoltan Boszormenyi
2021-08-23 14:47 ` [PATCH v2 3/4] kernel.bbclass: Use full versions for inter-package dependencies Zoltan Boszormenyi
2021-08-23 14:47 ` [PATCH v2 4/4] Support zstd-compressed squashfs and cpio initramfs Zoltan Boszormenyi
2021-08-23 14:54 ` Kernel related bbclass changes Zoltan Boszormenyi
2021-08-27 7:37 ` Kernel and image " Zoltan Boszormenyi
2021-08-27 7:37 ` [PATCH v4 1/4] kernel-module-split.bbclass: Support zstd-compressed modules Zoltan Boszormenyi
2021-08-27 7:37 ` [PATCH v4 2/4] Allow opt-out of split kernel modules Zoltan Boszormenyi
2021-08-27 7:37 ` [PATCH v4 3/4] kernel.bbclass: Use full versions for inter-package dependencies Zoltan Boszormenyi
2021-08-30 9:30 ` [OE-core] " Andrey Zhizhikin
2021-08-30 10:06 ` Zoltan Boszormenyi
[not found] ` <b75df7be-03d8-e454-60b8-1651fc935ca5@gmail.com>
2021-08-30 10:25 ` Andrey Zhizhikin
2021-08-30 19:51 ` Jon Mason
2021-08-31 4:28 ` Zoltan Boszormenyi [this message]
2021-08-31 13:32 ` Richard Purdie
2021-08-31 13:54 ` Andrey Zhizhikin
2021-08-27 7:37 ` [PATCH v4 4/4] Support zstd-compressed squashfs and cpio initramfs Zoltan Boszormenyi
2021-08-23 14:54 ` [PATCH v3 1/4] kernel-module-split.bbclass: Support zstd-compressed modules Zoltan Boszormenyi
2021-08-23 14:54 ` [PATCH v3 2/4] Allow opt-out of split kernel modules Zoltan Boszormenyi
2021-08-23 14:54 ` [PATCH v3 3/4] kernel.bbclass: Use full versions for inter-package dependencies Zoltan Boszormenyi
2021-08-23 14:54 ` [PATCH v3 4/4] Support zstd-compressed squashfs and cpio initramfs Zoltan Boszormenyi
2021-08-26 12:04 ` [OE-core] " Richard Purdie
2021-08-27 7:24 ` Zoltan Boszormenyi
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=f52a6dae-426c-34f3-448d-e4ba50b23438@pr.hu \
--to=zboszor@pr.hu \
--cc=andrey.z@gmail.com \
--cc=jdmason@kudzu.us \
--cc=openembedded-core@lists.openembedded.org \
--cc=zboszor@gmail.com \
/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