[SRU] flash-kernel to support xilinx kria platforms with noble kernel

Bug #2069802 reported by Talha Can Havadar
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
flash-kernel (Ubuntu)
In Progress
Undecided
Talha Can Havadar

Bug Description

[ Impact ]

 * With the current version of flash-kernel, users wont be able to boot into xilinx devices using optimised noble kernel because of the renaming of device-trees in kernel and cma configuration changes.

 * We need to fix this issue to keep support for xilinx devices in noble as well. This must not be back-ported into jammy since the kernel in jammy keeps the old names for device-trees and cma configuration in there didn't change

 * Patch will fix the issue by updating bootscript and `its` file to point correct device-tree files.

[ Test Plan ]

 * Flash the new image generated with new flash-kernel that has this fix
 * Try booting with new image
 * You should be able to reach the login prompt and be able to login using default username and password
 * This must work for all Kria devices listed below:
  - KV260
  - KR260
  - KD240

[ Where problems could occur ]
 * This change could only impact Xilinx Kria devices since it will only touch the files used by Xilinx Kria platforms including the ones listed above in test plan
 * Since this is an enablement patch for Ubuntu Noble image on Xilinx Kria platforms, there shouldnt be an impact to devices in the field.
 * If the patch is broken then the devices listed above will still continue to not work.

[ Other Info ]

 * This is based on changes in new optimised kernel: https://code.launchpad.net/~canonical-kernel/ubuntu/+source/linux-xilinx/+git/noble

Tags: patch
Changed in flash-kernel (Ubuntu):
assignee: nobody → Talha Can Havadar (tchavadar)
status: New → In Progress
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
summary: - [SRU] flash-kernel to support xilinx platforms with noble kernel
+ [SRU] flash-kernel to support xilinx kria platforms with noble kernel
Revision history for this message
Talha Can Havadar (tchavadar) wrote :
tags: added: patch
Revision history for this message
Loïc Minier (lool) wrote :

Hi Talha,

This is subjective, but I would suggest not dropping the xilinx-zynqmp flavor from the list immediately for two reasons:
- might be useful across upgrades (e.g. I upgrade f-k but not the kernel)
- in the case of a backport of f-k

The f-k kernel flavors are an useful protection in the face of trying to run the raspi kernel on a xilinx platform, but I don't know if we'd gain much by enforcing the removal of the old kernel flavor name. We could drop it after the next LTS though.

Revision history for this message
Loïc Minier (lool) wrote :

Actually, sorry, I see that your other change on device tree names might require the new kernel flavor; are the new and old device tree names available in both old and new kernel? How do we ensure people don't upgrade to a broken combination?

Revision history for this message
Loïc Minier (lool) wrote :

For the CMA change, I guess it's easy enough to release note this in oracular, but I'm less clear on noble: are we going to reroll existing images with the change? How will we communicate on the differences between old and new images with larger and smaller CMA?

Revision history for this message
Talha Can Havadar (tchavadar) wrote :

Hi Loic,

new kernel doesnt have old device-trees at all. (this raises questions for the one upgrading from jammy, I feel like we need to create a transition package for linux-xilinx-zynqmp in noble so when people upgrade it it will install linux-xilinx)

But indeed for noble we should keep old kernel flavor as well. (so it will be like `Kernel-Flavors: xilinx xilinx-zynqmp`)

About the cma change, cma change will be documented in Xilinx Wiki.

Revision history for this message
Loïc Minier (lool) wrote :

I'm not sure if f-k supports having multiple device tree options; you probably also can't have multiple entries for the same machine but different kernels/device trees.

My recommendation would be to add both transitional packages for upgrades, and provide the old device tree name in the new kernel.

Revision history for this message
Talha Can Havadar (tchavadar) wrote :
Revision history for this message
Talha Can Havadar (tchavadar) wrote :

kept the old kernel flavor for noble to make sure it is backwards compatible and attached new debdiff noble-2.debdiff

I will talk with kernel team about keeping the old device trees still in new kernel

Revision history for this message
Loïc Minier (lool) wrote :

Alternatively, you can add a versioned conflict on f-k in the kernel package (Breaks << versions of f-k prior to the change)

Revision history for this message
Talha Can Havadar (tchavadar) wrote :

I think breaks is better in this case makes kernel clean

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.