Add modaliases support

Bug #2021740 reported by You-Sheng Yang
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
ipu6-drivers (Ubuntu)
Status tracked in Mantic
Jammy
In Progress
High
You-Sheng Yang
Kinetic
Won't Fix
Undecided
Unassigned
Lunar
In Progress
High
You-Sheng Yang
Mantic
In Progress
High
You-Sheng Yang
ivsc-driver (Ubuntu)
Status tracked in Mantic
Jammy
In Progress
High
You-Sheng Yang
Kinetic
Won't Fix
Undecided
Unassigned
Lunar
In Progress
High
You-Sheng Yang
Mantic
Fix Released
High
You-Sheng Yang
linux (Ubuntu)
Status tracked in Mantic
Jammy
Won't Fix
Undecided
Unassigned
Kinetic
Won't Fix
Undecided
Unassigned
Lunar
In Progress
High
You-Sheng Yang
Mantic
In Progress
High
You-Sheng Yang
linux-oem-6.1 (Ubuntu)
Status tracked in Mantic
Jammy
In Progress
High
You-Sheng Yang
Kinetic
Invalid
Undecided
Unassigned
Lunar
Invalid
Undecided
Unassigned
Mantic
Invalid
Undecided
Unassigned
linux-oem-6.5 (Ubuntu)
Status tracked in Mantic
Jammy
In Progress
High
You-Sheng Yang
Kinetic
Invalid
Undecided
Unassigned
Lunar
Invalid
Undecided
Unassigned
Mantic
Invalid
Undecided
Unassigned

Bug Description

[SRU Justifications]

========== kernel ==========

[Impact]

Need modaliases info for ubuntu-drivers integration.

[Fix]

This Ubuntu sauced patch copies modaliases info from source dkms deb packages to
standalone `linux-modules-<mod>-<version>-<variant>` ones if available.

[Test Case]

```bash
$ sudo add-apt-repository ppa:canonical-hwe-team/experimental-unstable
$ apt-cache show linux-modules-ivsc-6.6.0-9005-generic | grep Modaliases
Modaliases: mei-vsc(acpi*:INTC1009:*, acpi*:INTC1058:*, acpi*:INTC1094:*, acpi*:INTC10D0:*)
```

[Where problems could occur]

With **XB-Modaliases** field added to deb control, `ubuntu-drivers` will then
give a list of installable `linux-modules-<mod>-<version>-<variant>` candidates
on available platforms. So far based on current `debian/dkms-versions`, the only
one will be enabled will be ivsc-driver, and ipu6-drivers to be SRUed soon.

[Other Info]

To nominate Unstable/Mantic/Lunar/OEM-6.5/OEM-6.1.

Note that this adds a build-dep to dh-modaliases from universe pocket.

========== ipu6-drivers, ivsc-driver ==========

[Impact]

Bug 2012407: kernel build fails without a newly learned directive in /sbin/dkms in bug 2012612 that helps identify whether a dkms modules is supposed to be built at all.

Bug 2021740: To allow dkms driver installation initiated by ubuntu-drivers, packages must carry XB-Modaliases in debian/control.

Bug 2026402: ipu6 dkms driver now fails to build against v6.5-rc1 and on.

[Fix]

* A BUILD_EXCLUSIVE_CONFIG patch for bug 2012407 was added to ivsc-driver/{jammy,lunar} because ivsc-driver/mantic has that already from bug 2012410. Also applied to ipu6-drivers/{mantic,lunar,jammy}.

* Modaliases fixes for bug 2021740 was applied to ivsc-drivers/{mantic,lunar,jammy}, and ipu6-drivers/{mantic,lunar,jammy}

* Additional fixes from upstream ipu6-drivers repo were added to ipu6-drivers/mantic for new hardware platforms.

[Test Case]

Bug 2012407: make sure intel-ipu6-dkms build will be skipped on platforms without CONFIG_VIDEO_V4L2_I2C, and intel-vsc-dkms on platforms without CONFIG_GPIOLIB_IRQCHIP, such as linux-headers-kvm/jammy.

Bug 2021740: check built binary deb has XB-Modaliases tag in `dpkg-deb --info` output.

For rest of the changes, they should be verified on next IPU6 equipped platform.

[Where problems could occur]

For changes for bug 2012407 and bug 2021740, they do no side effect to runtime system but to reinforce the interaction between other components.

For rest of the changes, only in ipu6-drivers/mantic, they're to sync with upstream development for new hardware platforms, and expects bug fixes, updates in the coming development cycle.

[Other Info]

========== original bug report ==========

To be discovered by ubuntu-drivers, dkms packages should have modaliases tag in debian/control as possible.

Tags: patch
Revision history for this message
Anthony Wong (anthonywong) wrote :

modalias should also be added to linux-modules-{ivsc,ipu6)-* packages. Perhaps only to the meta packages but not the ones with built modules?

Revision history for this message
You-Sheng Yang (vicamo) wrote :

debdiff for mantic with following bugs addressed:
* bug 2021740 (add modaliases),
* bug 2012407 (add CONFIG_VIDEO_V4L2_I2C to BUILD_EXCLUSIVE_CONFIG),
* bug 2026402 (fix build against linux v6.5-rc1)

Changed in ivsc-driver (Ubuntu Kinetic):
status: New → Won't Fix
Changed in ipu6-drivers (Ubuntu Kinetic):
status: New → Won't Fix
Changed in ipu6-drivers (Ubuntu Mantic):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "ipu6-drivers_0~git202302081010.7fdfb5eb-0ubuntu0.23.10.2.mantic.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Revision history for this message
You-Sheng Yang (vicamo) wrote :
Revision history for this message
You-Sheng Yang (vicamo) wrote : Re: [Bug 2021740] Re: Add modaliases support

Attach all the debdiff for affected targets. Need sponsoring.

Changed in ipu6-drivers (Ubuntu Lunar):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in ipu6-drivers (Ubuntu Jammy):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in ivsc-driver (Ubuntu Mantic):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in ivsc-driver (Ubuntu Lunar):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in ivsc-driver (Ubuntu Jammy):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

I have sponsored ivsc-driver for mantic. Right now that's as much as I can sponsor however, as the other bits need some questions answered:

 * ipu6-drivers for all series: when looking at the debdiff for mantic, I see that there's quite a lot of changes being added to the package. Besides the aforementioned 3 bug fixes, in the changelog I see entries from Andy Shevchenko, Hans de Goede and liang1.wang which are not clear to how they are related to the upload. Are those required to fix one of the 3 bugs? Can this be made clear in the changelog entry and the bug description? Since without full knowledge it's hard to know what's their purpose.
Also, currently the debdiff is 5k+ lines long. Are all these patches that are added required as-is?

 * ipu6-drivers and ivsc-driver for stable series: for these debdiffs to get sponsored, I would like the bug being prepared from the SRU perspective. Please see the template on https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template and be sure to provide all the necessary information in the bug.

Changed in ivsc-driver (Ubuntu Mantic):
status: In Progress → Fix Committed
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

I'll remove the ubuntu-sponsors subscription for now. Please address my comments as written in comment #7 and re-subscribe in case sponsorship is still required!

Revision history for this message
You-Sheng Yang (vicamo) wrote :

Updated bug descriptions to include details.

As for those huge patches, yes, the upstream project is still in a very early development stage and official releases may not meet the schedule from Ubuntu/OEM's side. Side-loaded patches are often needed to reflect the need from new hardware platforms and bug fixes.

description: updated
Revision history for this message
You-Sheng Yang (vicamo) wrote :

Re-subscribe ubuntu-sponsors after more detailed information along with SRU justifications.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ivsc-driver - 0~git202212210258.94ecb88b-0ubuntu0.23.10.2

---------------
ivsc-driver (0~git202212210258.94ecb88b-0ubuntu0.23.10.2) mantic; urgency=low

  [ You-Sheng Yang ]
  * debian: add modaliases (LP: #2021740)

 -- You-Sheng Yang <email address hidden> Wed, 12 Jul 2023 16:17:37 +0800

Changed in ivsc-driver (Ubuntu Mantic):
status: Fix Committed → Fix Released
Revision history for this message
Robie Basak (racb) wrote :

For ipu6-drivers into Mantic: are these straight cherry-picks from the upstream master branch, or have they been mdified? Please add dep3 headers[1] to the quilt patches that you're adding. This would make it easier to review them. This is our standard in Ubuntu. You might find git-to-dquilt.sh useful[2].

[For other sponsors: my inclination is that if they are direct cherry-picks from upstream then it's probably OK without further review though I'd verify it that far.]

Please adjust your test plan as follows in the SRU documentation.

> Bug 2012407: make sure intel-ipu6-dkms build will be skipped on platforms without CONFIG_VIDEO_V4L2_I2C, and intel-vsc-dkms on platforms without CONFIG_GPIOLIB_IRQCHIP, such as linux-headers-kvm/jammy.

What are the exact steps you will use to verify this please?

> Bug 2021740: check built binary deb has XB-Modaliases tag in `dpkg-deb --info` output.

This test is insufficient. The case you made for SRUing this fix is for support in ubuntu-drivers, so please plan to test that support in ubuntu-drivers is actually working and document how you will do this.

> For rest of the changes, they should be verified on next IPU6 equipped platform.

Please expand on this. What are you intending to test and how exactly?

For bug 2026402, the case for SRUing this fix is that building the DKMS package fails using the upcoming HWE kernel and after this fix that it will work. So please test this end-to-end functionality by checking that with a given kernel the DKMS build succeeds. I would expect that to be tested against a version of the kernel that is in the archive. Is that already the case? If not, then I suggest that we either wait until it is, or split the SRU. If you think that's not practical then we can discuss further.

Since DKMS packages also need to build against the release (non-HWE) kernel version, it's a common area of risk that fixing against the HWE kernel may break the build against the release kernel, so please additionally plan to test that the package still builds against the release kernel.

I'm unsubscribing ~ubuntu-sponsors again; please resubscribe once ready.

[1] https://dep-team.pages.debian.net/deps/dep3/
[2] https://git.launchpad.net/~ubuntu-server/+git/ubuntu-helpers/tree/cpaelzer/git-to-dquilt.sh

Revision history for this message
You-Sheng Yang (vicamo) wrote :
Changed in linux-oem-6.5 (Ubuntu Kinetic):
status: New → Invalid
Changed in linux-oem-6.5 (Ubuntu Lunar):
status: New → Invalid
Changed in linux-oem-6.5 (Ubuntu Mantic):
status: New → Invalid
Revision history for this message
You-Sheng Yang (vicamo) wrote :
Changed in linux-oem-6.5 (Ubuntu Jammy):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in linux-oem-6.1 (Ubuntu Kinetic):
status: New → Invalid
Changed in linux-oem-6.1 (Ubuntu Lunar):
status: New → Invalid
Changed in linux-oem-6.1 (Ubuntu Mantic):
status: New → Invalid
Changed in linux-oem-6.1 (Ubuntu Jammy):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in linux (Ubuntu Jammy):
status: New → Won't Fix
Changed in linux (Ubuntu Kinetic):
status: New → Won't Fix
Changed in linux (Ubuntu Lunar):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in linux (Ubuntu Mantic):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
You-Sheng Yang (vicamo)
description: updated
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.