Commit a0517a0f authored by Stanislav Fomichev's avatar Stanislav Fomichev Committed by Alexei Starovoitov

selftests/bpf: use __bpf_constant_htons in test_prog.c

For some reason, my older GCC (< 4.8) isn't smart enough to optimize the
!__builtin_constant_p() branch in bpf_htons, I see:
  error: implicit declaration of function '__builtin_bswap16'

Let's use __bpf_constant_htons as suggested by Daniel Borkmann.

I tried to use simple htons, but it produces the following:
  test_progs.c:54:17: error: braced-group within expression allowed only
  inside a function
    .eth.h_proto = htons(ETH_P_IP),
Signed-off-by: default avatarStanislav Fomichev <sdf@google.com>
Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
parent 37ab566c
...@@ -51,10 +51,10 @@ static struct { ...@@ -51,10 +51,10 @@ static struct {
struct iphdr iph; struct iphdr iph;
struct tcphdr tcp; struct tcphdr tcp;
} __packed pkt_v4 = { } __packed pkt_v4 = {
.eth.h_proto = bpf_htons(ETH_P_IP), .eth.h_proto = __bpf_constant_htons(ETH_P_IP),
.iph.ihl = 5, .iph.ihl = 5,
.iph.protocol = 6, .iph.protocol = 6,
.iph.tot_len = bpf_htons(MAGIC_BYTES), .iph.tot_len = __bpf_constant_htons(MAGIC_BYTES),
.tcp.urg_ptr = 123, .tcp.urg_ptr = 123,
}; };
...@@ -64,9 +64,9 @@ static struct { ...@@ -64,9 +64,9 @@ static struct {
struct ipv6hdr iph; struct ipv6hdr iph;
struct tcphdr tcp; struct tcphdr tcp;
} __packed pkt_v6 = { } __packed pkt_v6 = {
.eth.h_proto = bpf_htons(ETH_P_IPV6), .eth.h_proto = __bpf_constant_htons(ETH_P_IPV6),
.iph.nexthdr = 6, .iph.nexthdr = 6,
.iph.payload_len = bpf_htons(MAGIC_BYTES), .iph.payload_len = __bpf_constant_htons(MAGIC_BYTES),
.tcp.urg_ptr = 123, .tcp.urg_ptr = 123,
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment