Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp1271629pxa; Thu, 20 Aug 2020 07:13:38 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw0p9zSoLvb1ahnidZvTvD0eKO79e/RpVxFAvbIJRrjAsgVaxu8asn65sdA18dYjiQacwW7 X-Received: by 2002:a17:906:2cc2:: with SMTP id r2mr3384532ejr.482.1597932818492; Thu, 20 Aug 2020 07:13:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1597932818; cv=none; d=google.com; s=arc-20160816; b=m5cpy6sh0aX7uBvjj7mXia743zPEn6nmMZJrLM8OjQZDyuCh9ubF70oQKfpKF2bLfG QOhg6RrBTh709g9lFrNzZDj6snXFuvFnrfeXFZ6D0JnutjIFSMhGJErfqe/ADKlktVVc j8QGYG6nUDAhjJBLfEfbc1zJd/f/qUQyHSTFMXgkMXYoK0NKA0aD1stAjohze1cgQGoC s6lMCUzp9kYdyFJYAOUWQcH+lqhoEc1cd6fQADJJziuspt1JFKhGjrPV+Y2wBm7LbN1e QB+liLuEoQT0GOY3gvSt+iMkJMhkfiKQPZEesm9xoCrFNkKLmq/MWxhCG2qdu7UNtm9g 5UZg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=Mjkc+uOFGofk3IvrwN78/tWylXT0mT1Xub0o04o3B+A=; b=cXBpIk5gTPQVagH3HKEfxoI1B9UGg3GDihQIPasrf5NZkvoHbaTQvbOQ77UynDLQV+ HGKFYK7gHpWFyxrAT4F6Lu7z9ZOuVj6PyuA6mB6WhVwuSux2iEj3C+ARx3UwXanSWy1g 8qLgEr4maDQqn2v4qbSTpSsgW1woASJkr/MGV2MbClA5UfNfZEwe4lTRXTbXzf/AKSdR wyoramfgtt/suYqTvlca8t8s7AoktTne6Vt4MQUJtmf/VXNI+dQC5jFaod5OlsdopVV2 0rv7WW4SwC8ciA8Qbqsrg4kK9Mx5iw1Kte+kI6bDC6c3LDkEFRi4SIMfui4d7NsdZpum RnYg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@cloudflare.com header.s=google header.b=RMZGaVEW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=cloudflare.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id oz27si1281866ejb.27.2020.08.20.07.13.14; Thu, 20 Aug 2020 07:13:38 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@cloudflare.com header.s=google header.b=RMZGaVEW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=cloudflare.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729107AbgHTOJc (ORCPT + 99 others); Thu, 20 Aug 2020 10:09:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42154 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731200AbgHTN6f (ORCPT ); Thu, 20 Aug 2020 09:58:35 -0400 Received: from mail-wr1-x442.google.com (mail-wr1-x442.google.com [IPv6:2a00:1450:4864:20::442]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1F8ABC061357 for ; Thu, 20 Aug 2020 06:58:15 -0700 (PDT) Received: by mail-wr1-x442.google.com with SMTP id r2so2111762wrs.8 for ; Thu, 20 Aug 2020 06:58:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cloudflare.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=Mjkc+uOFGofk3IvrwN78/tWylXT0mT1Xub0o04o3B+A=; b=RMZGaVEW5IVGuTA6PbP9MNBnRo5TEV7o9K9/YR0QKsq+59hK1vhRjJIMb0omsq4lUk vIryb+QTQUt6ZxOjsFBRj9QteTdzHtsCJOEV8spsa6Bd4G9psvbwGgeCFtv1Eg0dMerz cQNzWwlaS+eWMFvlZn0EfDOaeen8+lWXnEIPU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=Mjkc+uOFGofk3IvrwN78/tWylXT0mT1Xub0o04o3B+A=; b=FZBFKbXgxf8USQBHuPsnx3xjd+TZ0Q1CRBTPWEvoXufBNWk+4LJkE/JSOoFHnd+QJx 8PtR7XrqQb7ka8BM21KH+xeVgPetVzh/Y/OHatYSBaozSSY8T9wMIGwtpRXKLjn9Yxrz rBhEpreQj2lIrWaHspoPrfzONZda1rbeCxwwCBzBhQfQQPjNpjYWGLHL3usOTTD2JFZm a4AWQe25xKC9cSvd/ckuTzmueIzCeQcUOW2k3f/tpSoVwUjWok1m+hmaMrUc+ycCyaBE ryjV7tGQ9Z2Hjkj+YTO+F1R9PcsMPiqFb1DqOc92I7fkOq0WTUTrxdv+NQyqNu3jv6oX P2Wg== X-Gm-Message-State: AOAM531KpxtwtIpUjV+GCyatPZ+LwfXhPaH3n7UvH/cGW0Ym3qp5Qe2a k+vJHWaLr+ryHUkGAbgaXHShGQ== X-Received: by 2002:a5d:4c83:: with SMTP id z3mr3297909wrs.359.1597931893722; Thu, 20 Aug 2020 06:58:13 -0700 (PDT) Received: from antares.lan (d.0.f.e.b.c.7.2.d.c.3.8.4.8.d.9.f.f.6.2.a.5.a.7.0.b.8.0.1.0.0.2.ip6.arpa. [2001:8b0:7a5a:26ff:9d84:83cd:27cb:ef0d]) by smtp.gmail.com with ESMTPSA id l81sm4494215wmf.4.2020.08.20.06.58.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 20 Aug 2020 06:58:12 -0700 (PDT) From: Lorenz Bauer To: jakub@cloudflare.com, john.fastabend@gmail.com, Shuah Khan , Alexei Starovoitov , Daniel Borkmann Cc: kernel-team@cloudflare.com, Lorenz Bauer , linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org Subject: [PATCH bpf-next v2 6/6] selftests: bpf: test sockmap update from BPF Date: Thu, 20 Aug 2020 14:57:29 +0100 Message-Id: <20200820135729.135783-7-lmb@cloudflare.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20200820135729.135783-1-lmb@cloudflare.com> References: <20200820135729.135783-1-lmb@cloudflare.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Add a test which copies a socket from a sockmap into another sockmap or sockhash. This excercises bpf_map_update_elem support from BPF context. Compare the socket cookies from source and destination to ensure that the copy succeeded. Signed-off-by: Lorenz Bauer --- .../selftests/bpf/prog_tests/sockmap_basic.c | 71 +++++++++++++++++++ .../selftests/bpf/progs/test_sockmap_copy.c | 48 +++++++++++++ 2 files changed, 119 insertions(+) create mode 100644 tools/testing/selftests/bpf/progs/test_sockmap_copy.c diff --git a/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c b/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c index 96e7b7f84c65..cd05ff5e88e1 100644 --- a/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c +++ b/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c @@ -4,6 +4,7 @@ #include "test_progs.h" #include "test_skmsg_load_helpers.skel.h" +#include "test_sockmap_copy.skel.h" #define TCP_REPAIR 19 /* TCP sock is under repair right now */ @@ -101,6 +102,72 @@ static void test_skmsg_helpers(enum bpf_map_type map_type) test_skmsg_load_helpers__destroy(skel); } +static void test_sockmap_copy(enum bpf_map_type map_type) +{ + struct bpf_prog_test_run_attr tattr; + struct test_sockmap_copy *skel; + __u64 src_cookie, dst_cookie; + int err, prog, src, dst; + const __u32 zero = 0; + char dummy[14] = {0}; + __s64 sk; + + sk = connected_socket_v4(); + if (CHECK_FAIL(sk == -1)) + return; + + skel = test_sockmap_copy__open_and_load(); + if (CHECK_FAIL(!skel)) { + close(sk); + perror("test_sockmap_copy__open_and_load"); + return; + } + + prog = bpf_program__fd(skel->progs.copy_sock_map); + src = bpf_map__fd(skel->maps.src); + if (map_type == BPF_MAP_TYPE_SOCKMAP) + dst = bpf_map__fd(skel->maps.dst_sock_map); + else + dst = bpf_map__fd(skel->maps.dst_sock_hash); + + err = bpf_map_update_elem(src, &zero, &sk, BPF_NOEXIST); + if (CHECK_FAIL(err)) { + perror("bpf_map_update"); + goto out; + } + + err = bpf_map_lookup_elem(src, &zero, &src_cookie); + if (CHECK_FAIL(err)) { + perror("bpf_map_lookup_elem(src)"); + goto out; + } + + tattr = (struct bpf_prog_test_run_attr){ + .prog_fd = prog, + .repeat = 1, + .data_in = dummy, + .data_size_in = sizeof(dummy), + }; + + err = bpf_prog_test_run_xattr(&tattr); + if (CHECK_ATTR(err || !tattr.retval, "bpf_prog_test_run", + "errno=%u retval=%u\n", errno, tattr.retval)) + goto out; + + err = bpf_map_lookup_elem(dst, &zero, &dst_cookie); + if (CHECK_FAIL(err)) { + perror("bpf_map_lookup_elem(dst)"); + goto out; + } + + if (dst_cookie != src_cookie) + PRINT_FAIL("cookie %llu != %llu\n", dst_cookie, src_cookie); + +out: + close(sk); + test_sockmap_copy__destroy(skel); +} + void test_sockmap_basic(void) { if (test__start_subtest("sockmap create_update_free")) @@ -111,4 +178,8 @@ void test_sockmap_basic(void) test_skmsg_helpers(BPF_MAP_TYPE_SOCKMAP); if (test__start_subtest("sockhash sk_msg load helpers")) test_skmsg_helpers(BPF_MAP_TYPE_SOCKHASH); + if (test__start_subtest("sockmap copy")) + test_sockmap_copy(BPF_MAP_TYPE_SOCKMAP); + if (test__start_subtest("sockhash copy")) + test_sockmap_copy(BPF_MAP_TYPE_SOCKHASH); } diff --git a/tools/testing/selftests/bpf/progs/test_sockmap_copy.c b/tools/testing/selftests/bpf/progs/test_sockmap_copy.c new file mode 100644 index 000000000000..9d0c9f28cab2 --- /dev/null +++ b/tools/testing/selftests/bpf/progs/test_sockmap_copy.c @@ -0,0 +1,48 @@ +// SPDX-License-Identifier: GPL-2.0 +// Copyright (c) 2020 Cloudflare +#include "vmlinux.h" +#include + +struct { + __uint(type, BPF_MAP_TYPE_SOCKMAP); + __uint(max_entries, 1); + __type(key, __u32); + __type(value, __u64); +} src SEC(".maps"); + +struct { + __uint(type, BPF_MAP_TYPE_SOCKMAP); + __uint(max_entries, 1); + __type(key, __u32); + __type(value, __u64); +} dst_sock_map SEC(".maps"); + +struct { + __uint(type, BPF_MAP_TYPE_SOCKHASH); + __uint(max_entries, 1); + __type(key, __u32); + __type(value, __u64); +} dst_sock_hash SEC(".maps"); + +SEC("classifier/copy_sock_map") +int copy_sock_map(void *ctx) +{ + struct bpf_sock *sk; + bool failed = false; + __u32 key = 0; + + sk = bpf_map_lookup_elem(&src, &key); + if (!sk) + return SK_DROP; + + if (bpf_map_update_elem(&dst_sock_map, &key, sk, 0)) + failed = true; + + if (bpf_map_update_elem(&dst_sock_hash, &key, sk, 0)) + failed = true; + + bpf_sk_release(sk); + return failed ? SK_DROP : SK_PASS; +} + +char _license[] SEC("license") = "GPL"; -- 2.25.1