Move the hint about CGROUP_BPF enabled to prog_load failure since it fails before the attach. Update the existing error message to print to stderr.
Signed-off-by: David Ahern <d...@cumulusnetworks.com> --- ip/ipvrf.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ip/ipvrf.c b/ip/ipvrf.c index 4d59845416cd..44ad7e07024a 100644 --- a/ip/ipvrf.c +++ b/ip/ipvrf.c @@ -170,14 +170,15 @@ static int vrf_configure_cgroup(const char *path, int ifindex) */ prog_fd = prog_load(ifindex); if (prog_fd < 0) { - printf("Failed to load BPF prog: '%s'\n", strerror(errno)); + fprintf(stderr, "Failed to load BPF prog: '%s'\n", + strerror(errno)); + fprintf(stderr, "Kernel compiled with CGROUP_BPF enabled?\n"); goto out; } if (bpf_prog_attach_fd(prog_fd, cg_fd, BPF_CGROUP_INET_SOCK_CREATE)) { fprintf(stderr, "Failed to attach prog to cgroup: '%s'\n", strerror(errno)); - fprintf(stderr, "Kernel compiled with CGROUP_BPF enabled?\n"); goto out; } -- 2.1.4