On 9/24/20 4:25 AM, bimmy.puj...@intel.com wrote:
From: Bimmy Pujari <bimmy.puj...@intel.com>
Test xdping measures RTT from xdp using monotonic time helper.
Extending xdping test to use real time helper function in order
to verify this helper function.
Signed-off-by: Bimmy Pujari <bimmy.puj...@intel.com>
[...]
diff --git a/tools/testing/selftests/bpf/progs/xdping_realtime_kern.c
b/tools/testing/selftests/bpf/progs/xdping_realtime_kern.c
new file mode 100644
index 000000000000..85f9d9bfc5b7
--- /dev/null
+++ b/tools/testing/selftests/bpf/progs/xdping_realtime_kern.c
@@ -0,0 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0
+#define REALTIME
+
Instead of moving everything into xdping_kern.h and then adding a REALTIME
define just
for doing ...
+#ifdef REALTIME
+ recvtime = bpf_ktime_get_real_ns();
+#else
+ recvtime = bpf_ktime_get_ns();
+#endif
... why not simply just provide a small inline helper with the bpf_ktime_*()
implementation
instead so we can avoid the ugly ifdef altogether..
+#include "xdping_kern.h"
diff --git a/tools/testing/selftests/bpf/test_xdping.sh
b/tools/testing/selftests/bpf/test_xdping.sh
index c2f0ddb45531..3e357755b279 100755
--- a/tools/testing/selftests/bpf/test_xdping.sh
[...]