ACK/Cmnt: NAK: [jammy xilinx-zynqmp 0/2] Turn off non-zynqmp arch support

[Tim Gardner]

On 1/3/24 5:14 PM, Portia Stephens wrote:
> My fault, I should have marked the bug as public. It should be accessible now.
> 
> On Thu, Jan 4, 2024 at 3:23 AM Tim Gardner <tim.gardner at canonical.com> wrote:
>>
>> Regardless, it still ain't working for me.
>>
>> On 1/3/24 9:06 AM, Dimitri John Ledkov wrote:
>>> i think we had two buglinks, and the other one should have been
>>> working or marked public now that this is being submitted.
>>>
>>> On Wed, 3 Jan 2024 at 15:56, Tim Gardner <tim.gardner at canonical.com> wrote:
>>>>
>>>> On 12/12/23 10:24 PM, Portia Stephens wrote:
>>>>> [ Impact ]
>>>>>
>>>>> * This is an optimized kernel for zynqmp range of devices.
>>>>> * Currently about 14% of kernel build is targeting non-zynqmp devices, which
>>>>> is dead code unused in production
>>>>> * This reduces kernel build time, kernel image & modules size, amount of
>>>>> required firmware, and makes initrd smaller.
>>>>> * This reduces peak memory usage on boot as well
>>>>> * This will loose ability to execute this exact same kernel build on non-zynqmp
>>>>> device, for benchmarking purposes generic kernel should be used instead.
>>>>>
>>>>> [ Test Plan ]
>>>>>
>>>>> * Ensure newly produced kernel installs and boots on Ubuntu Server and Ubuntu
>>>>> Core target devices without any loss of peripherals and stock devices.
>>>>>
>>>>> [ Where problems could occur ]
>>>>>
>>>>> * This changes may limit ability to smoke-test this kernel on non-zynqmp
>>>>> devices, if needed support for individual CPUs can be turned back on. Or
>>>>> ideally any such testing should move to zynqmp devices.
>>>>> (examples of potentially interesting things to turn back on upon request
>>>>> are QCOM APPLE suppport for running this kernel on Arm Laptops for developer
>>>>> convenience)
>>>>> * It is expected for KVM and QEMU to continue to work as both host and guest
>>>>>
>>>>> Dimitri John Ledkov (2):
>>>>>      UBUNTU: [Config] Turn off non-zynqmp arch support
>>>>>      UBUNTU: [Packaging] Update new baseline abi checks
>>>>>
>>>>>     debian.zynqmp/abi/arm64/xilinx-zynqmp         | 39937 ++++++++--------
>>>>>     debian.zynqmp/abi/arm64/xilinx-zynqmp.modules |   921 +-
>>>>>     .../abi/arm64/xilinx-zynqmp.modules.builtin   |   152 -
>>>>>     debian.zynqmp/abi/fwinfo                      |    68 +-
>>>>>     debian.zynqmp/abi/fwinfo.builtin              |     3 -
>>>>>     debian.zynqmp/config/annotations              |  1623 +-
>>>>>     6 files changed, 20917 insertions(+), 21787 deletions(-)
>>>>>
>>>>
>>>> That BugLink isn't working for me.
>>>> --
>>>> -----------
>>>> Tim Gardner
>>>> Canonical, Inc
>>>>
>>>>
>>>> --
>>>> kernel-team mailing list
>>>> kernel-team at lists.ubuntu.com
>>>> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>>>
>>>
>>>
>>
>> --
>> -----------
>> Tim Gardner
>> Canonical, Inc
>>
Acked-by: Tim Gardner <tim.gardner at canonical.com>

BugLink works much better.
-- 
-----------
Tim Gardner
Canonical, Inc