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

Reply via email to