Commit 279bdf6b authored by Magnus Karlsson's avatar Magnus Karlsson Committed by Alexei Starovoitov
Browse files

selftests: xsk: Make enums lower case 



Make enums lower case as that is the standard. Also drop the
unnecessary TEST_MODE_UNCONFIGURED mode.

Signed-off-by: default avatarMagnus Karlsson <magnus.karlsson@intel.com>
Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20210825093722.10219-16-magnus.karlsson@gmail.com
parent 29f128b3

tools/testing/selftests/bpf/xdpxceiver.c

+3 −8
Original line number Diff line number Diff line
@@ -105,15 +105,10 @@ static const u16 UDP_PORT2 = 2121; 


static void __exit_with_error(int error, const char *file, const char *func, int line)

{

	if (configured_mode == TEST_MODE_UNCONFIGURED) {

		ksft_exit_fail_msg

		("[%s:%s:%i]: ERROR: %d/\"%s\"\n", file, func, line, error, strerror(error));

	} else {

		ksft_test_result_fail

		("[%s:%s:%i]: ERROR: %d/\"%s\"\n", file, func, line, error, strerror(error));

	ksft_test_result_fail("[%s:%s:%i]: ERROR: %d/\"%s\"\n", file, func, line, error,

			      strerror(error));

	ksft_exit_xfail();

}

}



#define exit_with_error(error) __exit_with_error(error, __FILE__, __func__, __LINE__)

tools/testing/selftests/bpf/xdpxceiver.h

+4 −5
Original line number Diff line number Diff line
@@ -44,14 +44,13 @@ 


#define print_verbose(x...) do { if (opt_verbose) ksft_print_msg(x); } while (0)



enum TEST_MODES {

	TEST_MODE_UNCONFIGURED = -1,

enum test_mode {

	TEST_MODE_SKB,

	TEST_MODE_DRV,

	TEST_MODE_MAX

};



enum TEST_TYPES {

enum test_type {

	TEST_TYPE_NOPOLL,

	TEST_TYPE_POLL,

	TEST_TYPE_TEARDOWN,
@@ -61,7 +60,7 @@ enum TEST_TYPES { 
	TEST_TYPE_MAX

};



enum STAT_TEST_TYPES {

enum stat_test_type {

	STAT_TEST_RX_DROPPED,

	STAT_TEST_TX_INVALID,

	STAT_TEST_RX_FULL,
@@ -69,7 +68,7 @@ enum STAT_TEST_TYPES { 
	STAT_TEST_TYPE_MAX

};



static int configured_mode = TEST_MODE_UNCONFIGURED;

static int configured_mode;

static u8 debug_pkt_dump;

static u32 num_frames = DEFAULT_PKT_CNT / 4;

static bool second_step;