<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<div dir="ltr">
<div dir="ltr" style="font-family: Aptos, Arial, Helvetica, sans-serif; font-size: 12pt;">
I’m fine with these patches.<span></span></div>
<div dir="ltr"><br>
</div>
<div id="mail-editor-reference-message-container">
<div class="ms-outlook-mobile-reference-message skipProofing"><span style="mso-bookmark:_MailOriginalBody">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
<div style="font-family: Aptos; font-size: 12pt; text-align: left; border-width: 1pt medium medium; border-style: solid none none; border-color: rgb(181, 196, 223) currentcolor currentcolor; padding: 3pt 0in 0in; color: black;">
<span style="font-weight:bold">From: </span>kernel-team <kernel-team-bounces@lists.ubuntu.com> on behalf of John Cabaj <john.cabaj@canonical.com><br>
<span style="font-weight:bold">Date: </span>Wednesday, October 16, 2024 at 8:49 AM<br>
<span style="font-weight:bold">To: </span>kernel-team@lists.ubuntu.com <kernel-team@lists.ubuntu.com><br>
<span style="font-weight:bold">Subject: </span>ACK: [SRU][N:nvidia][PATCH v2 0/8] MANA: include driver fixes and enable module on ARM64<br>
<br>
</div>
<font size="2"><span style="font-size:11pt;">
<div class="PlainText">External email: Use caution opening links or attachments<br>
<br>
<br>
On 10/16/24 9:43 AM, Jacob Martin wrote:<br>
> BugLink: <a href="https://bugs.launchpad.net/bugs/2084598">https://bugs.launchpad.net/bugs/2084598</a><br>
><br>
> SRU Justification<br>
><br>
> [Impact]<br>
><br>
> NVIDIA kernels are used on systems that require support for the Microsoft Azure<br>
> Network Adapter, including on ARM64 systems with support for 64K and 4K page<br>
> sizes. Build the MANA driver on both arches and include the listed fixes for<br>
> the MANA driver to support both 64K and 4K page sizes and address a race<br>
> condition.<br>
><br>
> [Fix]<br>
><br>
> UBUNTU: [Packaging] nvidia: Include mana.ko in linux-modules-ABIVER package<br>
> UBUNTU: [Config] nvidia: Enable MANA configs on x86 and arm64<br>
> RDMA/mana_ib: use the correct page size for mapping user-mode doorbell page<br>
> RDMA/mana_ib: use the correct page table index based on hardware page size<br>
> net: mana: Fix RX buf alloc_size alignment and atomic op panic<br>
> net: mana: Fix race of mana_hwc_post_rx_wqe and new hwc response<br>
> net: mana: Add support for page sizes other than 4KB on ARM64<br>
> net: mana: Enable MANA driver on ARM64 with 4K page size<br>
><br>
> [Test Plan]<br>
><br>
> Compile tested on x86 and arm64. Kernel regression testing will be run to<br>
> verify there are no regressions from these patches on NVIDIA hardware.<br>
><br>
> [Where problems could occur]<br>
><br>
> These changes are specific to the MANA networking driver. Issues with this<br>
> patch would manifest as misbehavior of that driver, likely on ARM64 platforms.<br>
><br>
> v2:<br>
>   - Remove netdev and linux-next from the cherry pick notes of "net: mana: Fix race<br>
>     of mana_hwc_post_rx_wqe and new hwc response" and "net: mana: Add support for<br>
>     page sizes other than 4KB on ARM64", these commits exist in mainline with the<br>
>     same hashes.<br>
><br>
> Haiyang Zhang (4):<br>
>    net: mana: Enable MANA driver on ARM64 with 4K page size<br>
>    net: mana: Add support for page sizes other than 4KB on ARM64<br>
>    net: mana: Fix race of mana_hwc_post_rx_wqe and new hwc response<br>
>    net: mana: Fix RX buf alloc_size alignment and atomic op panic<br>
><br>
> Jacob Martin (2):<br>
>    UBUNTU: [Config] nvidia: Enable MANA configs on x86 and arm64<br>
>    UBUNTU: [Packaging] nvidia: Include mana.ko in linux-modules-ABIVER<br>
>      package<br>
><br>
> Long Li (2):<br>
>    RDMA/mana_ib: use the correct page table index based on hardware page<br>
>      size<br>
>    RDMA/mana_ib: use the correct page size for mapping user-mode doorbell<br>
>      page<br>
><br>
>   debian.nvidia/config/annotations              |  6 ++<br>
>   debian.nvidia/control.d/nvidia.inclusion-list |  1 +<br>
>   drivers/infiniband/hw/mana/main.c             |  8 +-<br>
>   drivers/net/ethernet/microsoft/Kconfig        |  3 +-<br>
>   .../net/ethernet/microsoft/mana/gdma_main.c   | 10 +--<br>
>   .../net/ethernet/microsoft/mana/hw_channel.c  | 76 ++++++++++---------<br>
>   drivers/net/ethernet/microsoft/mana/mana_en.c | 14 ++--<br>
>   .../net/ethernet/microsoft/mana/shm_channel.c | 13 ++--<br>
>   include/net/mana/gdma.h                       | 10 ++-<br>
>   include/net/mana/mana.h                       |  3 +-<br>
>   10 files changed, 86 insertions(+), 58 deletions(-)<br>
><br>
<br>
Acked-by: John Cabaj <john.cabaj@canonical.com><br>
<br>
<br>
--<br>
kernel-team mailing list<br>
kernel-team@lists.ubuntu.com<br>
<a href="https://lists.ubuntu.com/mailman/listinfo/kernel-team">https://lists.ubuntu.com/mailman/listinfo/kernel-team</a><br>
</div>
</span></font></span></div>
</div>
</div>
</body>
</html>