lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAM0EoMkAns=QXc-Lh3gkRmJ4SrGbbKpSbaSy5Btv7uw0NTUYUA@mail.gmail.com>
Date: Tue, 15 Oct 2024 18:16:28 -0400
From: Jamal Hadi Salim <jhs@...atatu.com>
To: Kuniyuki Iwashima <kuniyu@...zon.com>
Cc: "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, 
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, 
	Kuniyuki Iwashima <kuni1840@...il.com>, netdev@...r.kernel.org, 
	Cong Wang <xiyou.wangcong@...il.com>, Jiri Pirko <jiri@...nulli.us>
Subject: Re: [PATCH v2 net-next 04/11] net: sched: Use rtnl_register_many().

On Mon, Oct 14, 2024 at 4:20 PM Kuniyuki Iwashima <kuniyu@...zon.com> wrote:
>
> We will remove rtnl_register() in favour of rtnl_register_many().
>
> When it succeeds, rtnl_register_many() guarantees all rtnetlink types
> in the passed array are supported, and there is no chance that a part
> of message types is not supported.
>
> Let's use rtnl_register_many() instead.
>
> Signed-off-by: Kuniyuki Iwashima <kuniyu@...zon.com>

LGTM.
Acked-by: Jamal Hadi Salim <jhs@...atatu.com>

cheers,
jamal
> ---
> Cc: Jamal Hadi Salim <jhs@...atatu.com>
> Cc: Cong Wang <xiyou.wangcong@...il.com>
> Cc: Jiri Pirko <jiri@...nulli.us>
>
> v2:
>   * Add __initconst
>   * Use C99 initialisation
> ---
>  net/sched/act_api.c | 13 ++++++++-----
>  net/sched/cls_api.c | 25 ++++++++++++++-----------
>  net/sched/sch_api.c | 20 ++++++++++++--------
>  3 files changed, 34 insertions(+), 24 deletions(-)
>
> diff --git a/net/sched/act_api.c b/net/sched/act_api.c
> index 2714c4ed928e..5bbfb83ed600 100644
> --- a/net/sched/act_api.c
> +++ b/net/sched/act_api.c
> @@ -2243,13 +2243,16 @@ static int tc_dump_action(struct sk_buff *skb, struct netlink_callback *cb)
>         return skb->len;
>  }
>
> +static const struct rtnl_msg_handler tc_action_rtnl_msg_handlers[] __initconst = {
> +       {.msgtype = RTM_NEWACTION, .doit = tc_ctl_action},
> +       {.msgtype = RTM_DELACTION, .doit = tc_ctl_action},
> +       {.msgtype = RTM_GETACTION, .doit = tc_ctl_action,
> +        .dumpit = tc_dump_action},
> +};
> +
>  static int __init tc_action_init(void)
>  {
> -       rtnl_register(PF_UNSPEC, RTM_NEWACTION, tc_ctl_action, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_DELACTION, tc_ctl_action, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_GETACTION, tc_ctl_action, tc_dump_action,
> -                     0);
> -
> +       rtnl_register_many(tc_action_rtnl_msg_handlers);
>         return 0;
>  }
>
> diff --git a/net/sched/cls_api.c b/net/sched/cls_api.c
> index 17d97bbe890f..7637f979d689 100644
> --- a/net/sched/cls_api.c
> +++ b/net/sched/cls_api.c
> @@ -4055,6 +4055,19 @@ static struct pernet_operations tcf_net_ops = {
>         .size = sizeof(struct tcf_net),
>  };
>
> +static const struct rtnl_msg_handler tc_filter_rtnl_msg_handlers[] __initconst = {
> +       {.msgtype = RTM_NEWTFILTER, .doit = tc_new_tfilter,
> +        .flags = RTNL_FLAG_DOIT_UNLOCKED},
> +       {.msgtype = RTM_DELTFILTER, .doit = tc_del_tfilter,
> +        .flags = RTNL_FLAG_DOIT_UNLOCKED},
> +       {.msgtype = RTM_GETTFILTER, .doit = tc_get_tfilter,
> +        .dumpit = tc_dump_tfilter, .flags = RTNL_FLAG_DOIT_UNLOCKED},
> +       {.msgtype = RTM_NEWCHAIN, .doit = tc_ctl_chain},
> +       {.msgtype = RTM_DELCHAIN, .doit = tc_ctl_chain},
> +       {.msgtype = RTM_GETCHAIN, .doit = tc_ctl_chain,
> +        .dumpit = tc_dump_chain},
> +};
> +
>  static int __init tc_filter_init(void)
>  {
>         int err;
> @@ -4068,17 +4081,7 @@ static int __init tc_filter_init(void)
>                 goto err_register_pernet_subsys;
>
>         xa_init_flags(&tcf_exts_miss_cookies_xa, XA_FLAGS_ALLOC1);
> -
> -       rtnl_register(PF_UNSPEC, RTM_NEWTFILTER, tc_new_tfilter, NULL,
> -                     RTNL_FLAG_DOIT_UNLOCKED);
> -       rtnl_register(PF_UNSPEC, RTM_DELTFILTER, tc_del_tfilter, NULL,
> -                     RTNL_FLAG_DOIT_UNLOCKED);
> -       rtnl_register(PF_UNSPEC, RTM_GETTFILTER, tc_get_tfilter,
> -                     tc_dump_tfilter, RTNL_FLAG_DOIT_UNLOCKED);
> -       rtnl_register(PF_UNSPEC, RTM_NEWCHAIN, tc_ctl_chain, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_DELCHAIN, tc_ctl_chain, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_GETCHAIN, tc_ctl_chain,
> -                     tc_dump_chain, 0);
> +       rtnl_register_many(tc_filter_rtnl_msg_handlers);
>
>         return 0;
>
> diff --git a/net/sched/sch_api.c b/net/sched/sch_api.c
> index 2eefa4783879..da2da2ab858b 100644
> --- a/net/sched/sch_api.c
> +++ b/net/sched/sch_api.c
> @@ -2420,6 +2420,17 @@ static struct pernet_operations psched_net_ops = {
>  DEFINE_STATIC_KEY_FALSE(tc_skip_wrapper);
>  #endif
>
> +static const struct rtnl_msg_handler psched_rtnl_msg_handlers[] __initconst = {
> +       {.msgtype = RTM_NEWQDISC, .doit = tc_modify_qdisc},
> +       {.msgtype = RTM_DELQDISC, .doit = tc_get_qdisc},
> +       {.msgtype = RTM_GETQDISC, .doit = tc_get_qdisc,
> +        .dumpit = tc_dump_qdisc},
> +       {.msgtype = RTM_NEWTCLASS, .doit = tc_ctl_tclass},
> +       {.msgtype = RTM_DELTCLASS, .doit = tc_ctl_tclass},
> +       {.msgtype = RTM_GETTCLASS, .doit = tc_ctl_tclass,
> +        .dumpit = tc_dump_tclass},
> +};
> +
>  static int __init pktsched_init(void)
>  {
>         int err;
> @@ -2438,14 +2449,7 @@ static int __init pktsched_init(void)
>         register_qdisc(&mq_qdisc_ops);
>         register_qdisc(&noqueue_qdisc_ops);
>
> -       rtnl_register(PF_UNSPEC, RTM_NEWQDISC, tc_modify_qdisc, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_DELQDISC, tc_get_qdisc, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_GETQDISC, tc_get_qdisc, tc_dump_qdisc,
> -                     0);
> -       rtnl_register(PF_UNSPEC, RTM_NEWTCLASS, tc_ctl_tclass, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_DELTCLASS, tc_ctl_tclass, NULL, 0);
> -       rtnl_register(PF_UNSPEC, RTM_GETTCLASS, tc_ctl_tclass, tc_dump_tclass,
> -                     0);
> +       rtnl_register_many(psched_rtnl_msg_handlers);
>
>         tc_wrapper_init();
>
> --
> 2.39.5 (Apple Git-154)
>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ