xdp-forward: Don't use errno as program exit code

Converting errno returns to program exit status means we may end up
returning SKIPPED_TEST, which messes with the selftests. Change this to
always use EXIT_SUCCESS/EXIT_FAILURE instead.

Signed-off-by: Toke Høiland-Jørgensen <[email protected]>

Author: tohojo

xdp-forward/xdp-forward.c

@@ -202,9 +202,7 @@ static int do_load(const void *cfg, __unused const char *pin_root_path)
 	opts.obj = obj;
 	xdp_prog = xdp_program__create(&opts);
 	if (!xdp_prog) {
-		ret = -errno;
-		pr_warn("Couldn't open XDP program: %s\n",
-			strerror(-ret));
+		pr_warn("Couldn't open XDP program: %s\n", strerror(errno));
 		goto end_destroy;
 	}
 
@@ -245,6 +243,8 @@ static int do_load(const void *cfg, __unused const char *pin_root_path)
 		pr_info("Loaded on interface %s\n", iface->ifname);
 	}
 
+	ret = EXIT_SUCCESS;
+
 end_destroy:
 	if (opt->fwd_mode == FWD_FLOWTABLE)
 		xdp_flowtable__destroy(skel);
@@ -254,6 +254,7 @@ static int do_load(const void *cfg, __unused const char *pin_root_path)
 	return ret;
 
 end_detach:
+	ret = EXIT_FAILURE;
 	for (iface = opt->ifaces; iface; iface = iface->next)
 		xdp_program__detach(xdp_prog, iface->ifindex, opt->xdp_mode, 0);
 	goto end_destroy;