| Message ID | 20200202110200.31024-1-mrostecki@opensuse.org |
|---|---|
| State | Accepted |
| Delegated to: | BPF Maintainers |
| Headers | show |
| Series | [bpf] bpftool: Remove redundant "HAVE" prefix from the large INSN limit check | expand |
On 2/2/20 3:02 AM, Michal Rostecki wrote: > "HAVE" prefix is already applied by default to feature macros and before > this change, the large INSN limit macro had the incorrect name with > double "HAVE". > > Fixes: 2faef64aa6b3 ("bpftool: Add misc section and probe for large INSN limit") > Signed-off-by: Michal Rostecki <mrostecki@opensuse.org> Acked-by: Yonghong Song <yhs@fb.com>
On 2/2/20 12:02 PM, Michal Rostecki wrote: > "HAVE" prefix is already applied by default to feature macros and before > this change, the large INSN limit macro had the incorrect name with > double "HAVE". > > Fixes: 2faef64aa6b3 ("bpftool: Add misc section and probe for large INSN limit") > Signed-off-by: Michal Rostecki <mrostecki@opensuse.org> Applied, thanks!
diff --git a/tools/bpf/bpftool/feature.c b/tools/bpf/bpftool/feature.c index 446ba891f1e2..941873d778d8 100644 --- a/tools/bpf/bpftool/feature.c +++ b/tools/bpf/bpftool/feature.c @@ -580,7 +580,7 @@ probe_large_insn_limit(const char *define_prefix, __u32 ifindex) res = bpf_probe_large_insn_limit(ifindex); print_bool_feature("have_large_insn_limit", "Large program size limit", - "HAVE_LARGE_INSN_LIMIT", + "LARGE_INSN_LIMIT", res, define_prefix); }
"HAVE" prefix is already applied by default to feature macros and before this change, the large INSN limit macro had the incorrect name with double "HAVE". Fixes: 2faef64aa6b3 ("bpftool: Add misc section and probe for large INSN limit") Signed-off-by: Michal Rostecki <mrostecki@opensuse.org> --- tools/bpf/bpftool/feature.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)