bpf_progs: use unsigned instead of int
This gets rid of a bunch of warnings (enabled by -Wextra)
about signed vs unsigned comparisons.
For the for loops, the compiler really should be smarter...
the so-called 'unsigned' values are *small* compile time
constants like 20 / 2 = 10 or 40 / 2 = 20.
For the 'int len' argument, it's always going to be a positive
value anyway (negative values or zero are meaningless)
Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I57fae9add1d2aea7f47ed6a897f1b0f3fb874113
diff --git a/bpf_progs/offload.c b/bpf_progs/offload.c
index 4f152bf..3a713a0 100644
--- a/bpf_progs/offload.c
+++ b/bpf_progs/offload.c
@@ -593,7 +593,7 @@
// Calculate the IPv4 one's complement checksum of the IPv4 header.
__wsum sum4 = 0;
- for (int i = 0; i < sizeof(*ip) / sizeof(__u16); ++i) {
+ for (unsigned i = 0; i < sizeof(*ip) / sizeof(__u16); ++i) {
sum4 += ((__u16*)ip)[i];
}
// Note that sum4 is guaranteed to be non-zero by virtue of ip4->version == 4