Skip to content

Commit

Permalink
selftests/net: remove ARRAY_SIZE define from individual tests
Browse files Browse the repository at this point in the history
ARRAY_SIZE is defined in several selftests. Remove definitions from
individual test files and include header file for the define instead.
ARRAY_SIZE define is added in a separate patch to prepare for this
change.

Remove ARRAY_SIZE from net tests and pickup the one defined in
kselftest.h.

Signed-off-by: Shuah Khan <[email protected]>
  • Loading branch information
shuahkh committed Dec 11, 2021
1 parent 5a69d33 commit 1329e40
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 11 deletions.
3 changes: 2 additions & 1 deletion tools/testing/selftests/net/gro.c
Original file line number Diff line number Diff line change
Expand Up @@ -57,10 +57,11 @@
#include <string.h>
#include <unistd.h>

#include "../kselftest.h"

#define DPORT 8000
#define SPORT 1500
#define PAYLOAD_LEN 100
#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
#define NUM_PACKETS 4
#define START_SEQ 100
#define START_ACK 100
Expand Down
1 change: 0 additions & 1 deletion tools/testing/selftests/net/ipsec.c
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@

#define pr_err(fmt, ...) printk(fmt ": %m", ##__VA_ARGS__)

#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
#define BUILD_BUG_ON(condition) ((void)sizeof(char[1 - 2*!!(condition)]))

#define IPV4_STR_SZ 16 /* xxx.xxx.xxx.xxx is longest + \0 */
Expand Down
4 changes: 1 addition & 3 deletions tools/testing/selftests/net/reuseport_bpf.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,7 @@
#include <sys/resource.h>
#include <unistd.h>

#ifndef ARRAY_SIZE
#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
#endif
#include "../kselftest.h"

struct test_params {
int recv_family;
Expand Down
2 changes: 1 addition & 1 deletion tools/testing/selftests/net/rxtimestamp.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
#include <linux/net_tstamp.h>
#include <linux/errqueue.h>

#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
#include "../kselftest.h"

struct options {
int so_timestamp;
Expand Down
3 changes: 2 additions & 1 deletion tools/testing/selftests/net/socket.c
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@
#include <sys/socket.h>
#include <netinet/in.h>

#include "../kselftest.h"

struct socket_testcase {
int domain;
int type;
Expand All @@ -31,7 +33,6 @@ static struct socket_testcase tests[] = {
{ AF_INET, SOCK_STREAM, IPPROTO_UDP, -EPROTONOSUPPORT, 1 },
};

#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
#define ERR_STRING_SZ 64

static int run_tests(void)
Expand Down
6 changes: 2 additions & 4 deletions tools/testing/selftests/net/tcp_fastopen_backup_key.c
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@
#include <fcntl.h>
#include <time.h>

#include "../kselftest.h"

#ifndef TCP_FASTOPEN_KEY
#define TCP_FASTOPEN_KEY 33
#endif
Expand All @@ -34,10 +36,6 @@
#define PROC_FASTOPEN_KEY "/proc/sys/net/ipv4/tcp_fastopen_key"
#define KEY_LENGTH 16

#ifndef ARRAY_SIZE
#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
#endif

static bool do_ipv6;
static bool do_sockopt;
static bool do_rotate;
Expand Down

0 comments on commit 1329e40

Please sign in to comment.