cls_bpf: propagate extack to offload delete callback
authorJakub Kicinski <jakub.kicinski@netronome.com>
Wed, 24 Jan 2018 20:54:17 +0000 (12:54 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 24 Jan 2018 21:01:10 +0000 (16:01 -0500)
Propagate extack on removal of offloaded filter.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Simon Horman <simon.horman@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/cls_bpf.c

index 323b01f76a4cdbeda743a7c7965c5265eac47f63..8e5326bc64402fcdfeef403dc16b85791ec689dd 100644 (file)
@@ -212,11 +212,12 @@ static int cls_bpf_offload(struct tcf_proto *tp, struct cls_bpf_prog *prog,
 }
 
 static void cls_bpf_stop_offload(struct tcf_proto *tp,
-                                struct cls_bpf_prog *prog)
+                                struct cls_bpf_prog *prog,
+                                struct netlink_ext_ack *extack)
 {
        int err;
 
-       err = cls_bpf_offload_cmd(tp, NULL, prog, NULL);
+       err = cls_bpf_offload_cmd(tp, NULL, prog, extack);
        if (err)
                pr_err("Stopping hardware offload failed: %d\n", err);
 }
@@ -289,12 +290,13 @@ static void cls_bpf_delete_prog_rcu(struct rcu_head *rcu)
        tcf_queue_work(&prog->work);
 }
 
-static void __cls_bpf_delete(struct tcf_proto *tp, struct cls_bpf_prog *prog)
+static void __cls_bpf_delete(struct tcf_proto *tp, struct cls_bpf_prog *prog,
+                            struct netlink_ext_ack *extack)
 {
        struct cls_bpf_head *head = rtnl_dereference(tp->root);
 
        idr_remove_ext(&head->handle_idr, prog->handle);
-       cls_bpf_stop_offload(tp, prog);
+       cls_bpf_stop_offload(tp, prog, extack);
        list_del_rcu(&prog->link);
        tcf_unbind_filter(tp, &prog->res);
        if (tcf_exts_get_net(&prog->exts))
@@ -308,7 +310,7 @@ static int cls_bpf_delete(struct tcf_proto *tp, void *arg, bool *last,
 {
        struct cls_bpf_head *head = rtnl_dereference(tp->root);
 
-       __cls_bpf_delete(tp, arg);
+       __cls_bpf_delete(tp, arg, extack);
        *last = list_empty(&head->plist);
        return 0;
 }
@@ -320,7 +322,7 @@ static void cls_bpf_destroy(struct tcf_proto *tp,
        struct cls_bpf_prog *prog, *tmp;
 
        list_for_each_entry_safe(prog, tmp, &head->plist, link)
-               __cls_bpf_delete(tp, prog);
+               __cls_bpf_delete(tp, prog, extack);
 
        idr_destroy(&head->handle_idr);
        kfree_rcu(head, rcu);