selftests: xsk: Rename worker_* functions that are not thread entry points
authorMagnus Karlsson <magnus.karlsson@intel.com>
Wed, 25 Aug 2021 09:37:14 +0000 (11:37 +0200)
committerAlexei Starovoitov <ast@kernel.org>
Wed, 25 Aug 2021 19:22:00 +0000 (12:22 -0700)
Rename worker_* functions that are not thread entry points to
something else. This was confusing. Now only thread entry points are
worker_something.

Signed-off-by: Magnus Karlsson <magnus.karlsson@intel.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20210825093722.10219-9-magnus.karlsson@gmail.com
tools/testing/selftests/bpf/xdpxceiver.c

index ebed88c..17956fd 100644 (file)
@@ -589,7 +589,7 @@ static void tx_only_all(struct ifobject *ifobject)
        complete_tx_only_all(ifobject);
 }
 
-static void worker_pkt_dump(void)
+static void pkt_dump(void)
 {
        struct ethhdr *ethhdr;
        struct iphdr *iphdr;
@@ -631,7 +631,7 @@ static void worker_pkt_dump(void)
        }
 }
 
-static void worker_stats_validate(struct ifobject *ifobject)
+static void stats_validate(struct ifobject *ifobject)
 {
        struct xdp_statistics stats;
        socklen_t optlen;
@@ -673,7 +673,7 @@ static void worker_stats_validate(struct ifobject *ifobject)
        }
 }
 
-static void worker_pkt_validate(void)
+static void pkt_validate(void)
 {
        u32 payloadseqnum = -2;
        struct iphdr *iphdr;
@@ -833,9 +833,9 @@ static void *worker_testapp_validate_rx(void *arg)
        while (1) {
                if (test_type != TEST_TYPE_STATS) {
                        rx_pkt(ifobject->xsk, fds);
-                       worker_pkt_validate();
+                       pkt_validate();
                } else {
-                       worker_stats_validate(ifobject);
+                       stats_validate(ifobject);
                }
                if (sigvar)
                        break;
@@ -873,7 +873,7 @@ static void testapp_validate(void)
        pthread_join(t0, NULL);
 
        if (debug_pkt_dump && test_type != TEST_TYPE_STATS) {
-               worker_pkt_dump();
+               pkt_dump();
                for (int iter = 0; iter < num_frames; iter++) {
                        free(pkt_buf[iter]->payload);
                        free(pkt_buf[iter]);