From patchwork Mon Feb 12 20:19:04 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexander Aring X-Patchwork-Id: 872371 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=mojatatu-com.20150623.gappssmtp.com header.i=@mojatatu-com.20150623.gappssmtp.com header.b="1vGssk10"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3zgH8f5Z7Zz9sRm for ; Tue, 13 Feb 2018 07:20:02 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753566AbeBLUTq (ORCPT ); Mon, 12 Feb 2018 15:19:46 -0500 Received: from mail-io0-f195.google.com ([209.85.223.195]:34084 "EHLO mail-io0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752470AbeBLUTk (ORCPT ); Mon, 12 Feb 2018 15:19:40 -0500 Received: by mail-io0-f195.google.com with SMTP id e7so4454676ioj.1 for ; Mon, 12 Feb 2018 12:19:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mojatatu-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=eqBoLnRc5rkV2W+6O3khc0+RpsATU6p+W3b3wFSkRjM=; b=1vGssk10cP2U8DNnS7ygzeBGl82+E5qbbDOjOLjdpnI0wq3Gt+1MEH5c0Uj959QVC+ bWdJ6B4eq8x5KezXsgY+NY7zNOw+pmdU7EZT/xJjm4+UtorG12O0sGx3WZitGDd80CrE Q/ugD1Cios3g9ouZNfZl66DJ1sncknQUGFPCbUVY7xUK1YNkdrUWAx0Q7lqTg9BmYJ1S itFteL3WwkOT04K0kAT8YnWdQYb+U3gDqcM2nNdr6tnWwVRX6L14iguIJL1nw8WjwR3q n9dEEhTl4UgzzgTFc6uO3ktBNhBCdItnxYio652PxdEbkdRRM0L3kQQ5/AGSpAYbUhNa j8Cg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=eqBoLnRc5rkV2W+6O3khc0+RpsATU6p+W3b3wFSkRjM=; b=AIpcDkafoTdv3wnMmYBWoFTsNINJAgzLkcZCQ1UR5n1cdTvZeifQ3Od/6ODjwuvhv6 KtD2uHTGAWPnGfDRMxc8wdY+cZNC76aZmn7n5XfUTA6GqsAEqvjbEW1JBMCKkDVF+97y e9umeagwXk6q87I4wYpORRkJo/l0OBfgLKMOkeZBdYywH3BNuBb1FRekNGxr7GXYEgMH cjM5YPp2F+pPrVZSvBdewjqRYhPTzhngehWigL4hR7TYdwD10DDbHDZbgqqnonta8Ma1 BNMfYEwmpzq3wqnEN5xVgjVL9UNTa9+IwyWibLQ+fKWlp4gUApLNj7fziON+gZ0DXgoK fB7w== X-Gm-Message-State: APf1xPA2R/Jk1Hf1QJdZnsOD6uiBA9UjeVxgyGmVNV0B0Y8MPANeadcZ ZRhPJRxQlT6aV2GQWRzQLFNL+g== X-Google-Smtp-Source: AH8x225Uy7xdtUAZJ8IjZq9r2MSKe01EriinVnWR8VBD1G0uxOnwVCDCuWwCFHe58SShXizzie1CjQ== X-Received: by 10.107.212.24 with SMTP id l24mr15606093iog.40.1518466780186; Mon, 12 Feb 2018 12:19:40 -0800 (PST) Received: from x220t.lan ([64.26.149.125]) by smtp.gmail.com with ESMTPSA id s8sm8213236itb.39.2018.02.12.12.19.39 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 12 Feb 2018 12:19:39 -0800 (PST) From: Alexander Aring To: davem@davemloft.net Cc: jhs@mojatatu.com, xiyou.wangcong@gmail.com, jiri@resnulli.us, netdev@vger.kernel.org, Alexander Aring Subject: [PATCH net-next 1/8] net: sched: act: fix code style Date: Mon, 12 Feb 2018 15:19:04 -0500 Message-Id: <20180212201911.18704-2-aring@mojatatu.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20180212201911.18704-1-aring@mojatatu.com> References: <20180212201911.18704-1-aring@mojatatu.com> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org This patch is used by subsequent patches. It fixes code style issues caught by checkpatch. Signed-off-by: Alexander Aring --- include/net/act_api.h | 5 +++-- net/sched/act_api.c | 12 ++++++------ net/sched/act_mirred.c | 6 +++--- 3 files changed, 12 insertions(+), 11 deletions(-) diff --git a/include/net/act_api.h b/include/net/act_api.h index 6ed9692f20bd..32ef544f4ddc 100644 --- a/include/net/act_api.h +++ b/include/net/act_api.h @@ -87,12 +87,13 @@ struct tc_action_ops { struct tcf_result *); int (*dump)(struct sk_buff *, struct tc_action *, int, int); void (*cleanup)(struct tc_action *); - int (*lookup)(struct net *, struct tc_action **, u32); + int (*lookup)(struct net *net, struct tc_action **a, u32 index); int (*init)(struct net *net, struct nlattr *nla, struct nlattr *est, struct tc_action **act, int ovr, int bind); int (*walk)(struct net *, struct sk_buff *, - struct netlink_callback *, int, const struct tc_action_ops *); + struct netlink_callback *, int, + const struct tc_action_ops *); void (*stats_update)(struct tc_action *, u64, u32, u64); struct net_device *(*get_dev)(const struct tc_action *a); }; diff --git a/net/sched/act_api.c b/net/sched/act_api.c index eba6682727dd..d885b3127e53 100644 --- a/net/sched/act_api.c +++ b/net/sched/act_api.c @@ -621,7 +621,7 @@ struct tc_action *tcf_action_init_1(struct net *net, struct tcf_proto *tp, goto err_out; err = -EINVAL; kind = tb[TCA_ACT_KIND]; - if (kind == NULL) + if (!kind) goto err_out; if (nla_strlcpy(act_name, kind, IFNAMSIZ) >= IFNAMSIZ) goto err_out; @@ -822,7 +822,7 @@ static int tca_get_fill(struct sk_buff *skb, struct list_head *actions, t->tca__pad2 = 0; nest = nla_nest_start(skb, TCA_ACT_TAB); - if (nest == NULL) + if (!nest) goto out_nlmsg_trim; if (tcf_action_dump(skb, actions, bind, ref) < 0) @@ -934,7 +934,7 @@ static int tca_action_flush(struct net *net, struct nlattr *nla, t->tca__pad2 = 0; nest = nla_nest_start(skb, TCA_ACT_TAB); - if (nest == NULL) + if (!nest) goto out_module_put; err = ops->walk(net, skb, &dcb, RTM_DELACTION, ops); @@ -1005,10 +1005,10 @@ tca_action_gd(struct net *net, struct nlattr *nla, struct nlmsghdr *n, return ret; if (event == RTM_DELACTION && n->nlmsg_flags & NLM_F_ROOT) { - if (tb[1] != NULL) + if (tb[1]) return tca_action_flush(net, tb[1], n, portid); - else - return -EINVAL; + + return -EINVAL; } for (i = 1; i <= TCA_ACT_MAX_PRIO && tb[i]; i++) { diff --git a/net/sched/act_mirred.c b/net/sched/act_mirred.c index e6ff88f72900..abcd5f12b913 100644 --- a/net/sched/act_mirred.c +++ b/net/sched/act_mirred.c @@ -80,12 +80,12 @@ static int tcf_mirred_init(struct net *net, struct nlattr *nla, bool exists = false; int ret; - if (nla == NULL) + if (!nla) return -EINVAL; ret = nla_parse_nested(tb, TCA_MIRRED_MAX, nla, mirred_policy, NULL); if (ret < 0) return ret; - if (tb[TCA_MIRRED_PARMS] == NULL) + if (!tb[TCA_MIRRED_PARMS]) return -EINVAL; parm = nla_data(tb[TCA_MIRRED_PARMS]); @@ -117,7 +117,7 @@ static int tcf_mirred_init(struct net *net, struct nlattr *nla, } if (!exists) { - if (dev == NULL) + if (!dev) return -EINVAL; ret = tcf_idr_create(tn, parm->index, est, a, &act_mirred_ops, bind, true);