bpf/helpers: Introduce bpf_dynptr_copy kfunc
authorMykyta Yatsenko <yatsenko@meta.com>
Wed, 26 Feb 2025 18:32:00 +0000 (18:32 +0000)
committerAlexei Starovoitov <ast@kernel.org>
Sat, 15 Mar 2025 18:48:16 +0000 (11:48 -0700)
Introducing bpf_dynptr_copy kfunc allowing copying data from one dynptr to
another. This functionality is useful in scenarios such as capturing XDP
data to a ring buffer.
The implementation consists of 4 branches:
  * A fast branch for contiguous buffer capacity in both source and
destination dynptrs
  * 3 branches utilizing __bpf_dynptr_read and __bpf_dynptr_write to copy
data to/from non-contiguous buffer

Signed-off-by: Mykyta Yatsenko <yatsenko@meta.com>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/20250226183201.332713-3-mykyta.yatsenko5@gmail.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
kernel/bpf/helpers.c

index 6600aa4492ec4a6e03d1a5dd4528c023aeed1fbc..5449756ba102e32228a34cf054e085b4fefee24c 100644 (file)
@@ -2770,6 +2770,61 @@ __bpf_kfunc int bpf_dynptr_clone(const struct bpf_dynptr *p,
        return 0;
 }
 
+/**
+ * bpf_dynptr_copy() - Copy data from one dynptr to another.
+ * @dst_ptr: Destination dynptr - where data should be copied to
+ * @dst_off: Offset into the destination dynptr
+ * @src_ptr: Source dynptr - where data should be copied from
+ * @src_off: Offset into the source dynptr
+ * @size: Length of the data to copy from source to destination
+ *
+ * Copies data from source dynptr to destination dynptr.
+ * Returns 0 on success; negative error, otherwise.
+ */
+__bpf_kfunc int bpf_dynptr_copy(struct bpf_dynptr *dst_ptr, u32 dst_off,
+                               struct bpf_dynptr *src_ptr, u32 src_off, u32 size)
+{
+       struct bpf_dynptr_kern *dst = (struct bpf_dynptr_kern *)dst_ptr;
+       struct bpf_dynptr_kern *src = (struct bpf_dynptr_kern *)src_ptr;
+       void *src_slice, *dst_slice;
+       char buf[256];
+       u32 off;
+
+       src_slice = bpf_dynptr_slice(src_ptr, src_off, NULL, size);
+       dst_slice = bpf_dynptr_slice_rdwr(dst_ptr, dst_off, NULL, size);
+
+       if (src_slice && dst_slice) {
+               memmove(dst_slice, src_slice, size);
+               return 0;
+       }
+
+       if (src_slice)
+               return __bpf_dynptr_write(dst, dst_off, src_slice, size, 0);
+
+       if (dst_slice)
+               return __bpf_dynptr_read(dst_slice, size, src, src_off, 0);
+
+       if (bpf_dynptr_check_off_len(dst, dst_off, size) ||
+           bpf_dynptr_check_off_len(src, src_off, size))
+               return -E2BIG;
+
+       off = 0;
+       while (off < size) {
+               u32 chunk_sz = min_t(u32, sizeof(buf), size - off);
+               int err;
+
+               err = __bpf_dynptr_read(buf, chunk_sz, src, src_off + off, 0);
+               if (err)
+                       return err;
+               err = __bpf_dynptr_write(dst, dst_off + off, buf, chunk_sz, 0);
+               if (err)
+                       return err;
+
+               off += chunk_sz;
+       }
+       return 0;
+}
+
 __bpf_kfunc void *bpf_cast_to_kern_ctx(void *obj)
 {
        return obj;
@@ -3218,6 +3273,7 @@ BTF_ID_FLAGS(func, bpf_dynptr_is_null)
 BTF_ID_FLAGS(func, bpf_dynptr_is_rdonly)
 BTF_ID_FLAGS(func, bpf_dynptr_size)
 BTF_ID_FLAGS(func, bpf_dynptr_clone)
+BTF_ID_FLAGS(func, bpf_dynptr_copy)
 #ifdef CONFIG_NET
 BTF_ID_FLAGS(func, bpf_modify_return_test_tp)
 #endif