]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
hv_netvsc: use kmap_local_page in netvsc_copy_to_send_buf
authorAnton Leontev <leontyevantony@gmail.com>
Thu, 4 Jun 2026 16:59:38 +0000 (19:59 +0300)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 9 Jun 2026 11:16:30 +0000 (13:16 +0200)
netvsc_copy_to_send_buf() copies page buffer entries into the VMBus
send buffer using phys_to_virt() on the entry PFN. Entries for the
RNDIS header and the skb linear data come from kmalloc'd memory and
are always in the kernel direct map, but entries for skb fragments
reference page cache or user pages, which on 32-bit x86 with
CONFIG_HIGHMEM=y can live above the LOWMEM boundary. For such a page
phys_to_virt() returns an address outside the direct map and the
subsequent memcpy() faults on the transmit softirq path, which is
fatal.

Map the pages with kmap_local_page() instead, handling two properties
of the page buffer entries:

 - pb[i].pfn is a Hyper-V PFN at HV_HYP_PAGE_SIZE (4K) granularity,
   not a native PFN. Reconstruct the physical address first and derive
   the native page from it, so the mapping stays correct where
   PAGE_SIZE > HV_HYP_PAGE_SIZE (e.g. arm64 with 64K pages).

 - Since commit 41a6328b2c55 ("hv_netvsc: Preserve contiguous PFN
   grouping in the page buffer array"), an entry describes a full
   physically contiguous fragment and pb[i].len can exceed PAGE_SIZE,
   while kmap_local_page() maps a single page. Copy page by page,
   splitting at native page boundaries.

The copy path only handles packets smaller than the send section size
(6144 bytes by default); larger packets take the cp_partial path where
only the RNDIS header is copied. So entries here are bounded by the
section size and a copy is split at most once on 4K-page systems. On
!CONFIG_HIGHMEM configs kmap_local_page() folds to page_address() and
no mapping work is added.

Fixes: c25aaf814a63 ("hyperv: Enable sendbuf mechanism on the send path")
Cc: stable@vger.kernel.org
Signed-off-by: Anton Leontev <leontyevantony@gmail.com>
Link: https://patch.msgid.link/20260604165938.32033-1-leontyevantony@gmail.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/hyperv/netvsc.c

index 59e95341f9b1e56d66052814c04f2d76ae5fe8e1..4d319c50955ed0510df45d8fc4e2e4871661d154 100644 (file)
@@ -12,6 +12,7 @@
 #include <linux/sched.h>
 #include <linux/wait.h>
 #include <linux/mm.h>
+#include <linux/highmem.h>
 #include <linux/delay.h>
 #include <linux/io.h>
 #include <linux/slab.h>
@@ -965,12 +966,22 @@ static void netvsc_copy_to_send_buf(struct netvsc_device *net_device,
        }
 
        for (i = 0; i < page_count; i++) {
-               char *src = phys_to_virt(pb[i].pfn << HV_HYP_PAGE_SHIFT);
-               u32 offset = pb[i].offset;
+               phys_addr_t paddr = (pb[i].pfn << HV_HYP_PAGE_SHIFT) +
+                                   pb[i].offset;
                u32 len = pb[i].len;
 
-               memcpy(dest, (src + offset), len);
-               dest += len;
+               while (len) {
+                       struct page *page = phys_to_page(paddr);
+                       u32 off = offset_in_page(paddr);
+                       u32 chunk = min_t(u32, len, PAGE_SIZE - off);
+                       char *src = kmap_local_page(page);
+
+                       memcpy(dest, src + off, chunk);
+                       kunmap_local(src);
+                       dest += chunk;
+                       paddr += chunk;
+                       len -= chunk;
+               }
        }
 
        if (padding)