From: Phil Sutter <phil@nwl.cc>
To: Pablo Neira Ayuso <pablo@netfilter.org>
Cc: netfilter-devel@vger.kernel.org
Subject: [iptables PATCH v4 06/12] nft: Eliminate pointless calls to nft_family_ops_lookup()
Date: Thu, 21 Nov 2019 18:36:41 +0100 [thread overview]
Message-ID: <20191121173647.31488-7-phil@nwl.cc> (raw)
In-Reply-To: <20191121173647.31488-1-phil@nwl.cc>
If nft_handle is available, use its 'ops' field instead of performing a
new lookup. For the same reason, there is no need to pass ops pointer to
__nft_print_header().
Signed-off-by: Phil Sutter <phil@nwl.cc>
---
iptables/nft.c | 15 +++++----------
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/iptables/nft.c b/iptables/nft.c
index 36c3bc2d41764..22a4dc8904126 100644
--- a/iptables/nft.c
+++ b/iptables/nft.c
@@ -1336,12 +1336,10 @@ static const char *policy_name[NF_ACCEPT+1] = {
int nft_chain_save(struct nft_handle *h, struct nftnl_chain_list *list)
{
+ struct nft_family_ops *ops = h->ops;
struct nftnl_chain_list_iter *iter;
- struct nft_family_ops *ops;
struct nftnl_chain *c;
- ops = nft_family_ops_lookup(h->family);
-
iter = nftnl_chain_list_iter_create(list);
if (iter == NULL)
return 0;
@@ -2165,7 +2163,6 @@ static int nft_rule_count(struct nft_handle *h, struct nftnl_chain *c)
}
static void __nft_print_header(struct nft_handle *h,
- const struct nft_family_ops *ops,
struct nftnl_chain *c, unsigned int format)
{
const char *chain_name = nftnl_chain_get_str(c, NFTNL_CHAIN_NAME);
@@ -2181,14 +2178,14 @@ static void __nft_print_header(struct nft_handle *h,
if (nftnl_chain_is_set(c, NFTNL_CHAIN_POLICY))
pname = policy_name[nftnl_chain_get_u32(c, NFTNL_CHAIN_POLICY)];
- ops->print_header(format, chain_name, pname,
+ h->ops->print_header(format, chain_name, pname,
&ctrs, basechain, refs - entries, entries);
}
int nft_rule_list(struct nft_handle *h, const char *chain, const char *table,
int rulenum, unsigned int format)
{
- const struct nft_family_ops *ops;
+ const struct nft_family_ops *ops = h->ops;
struct nftnl_chain_list *list;
struct nftnl_chain_list_iter *iter;
struct nftnl_chain *c;
@@ -2197,8 +2194,6 @@ int nft_rule_list(struct nft_handle *h, const char *chain, const char *table,
nft_xt_builtin_init(h, table);
nft_assert_table_compatible(h, table, chain);
- ops = nft_family_ops_lookup(h->family);
-
list = nft_chain_list_get(h, table, chain);
if (!list)
return 0;
@@ -2211,7 +2206,7 @@ int nft_rule_list(struct nft_handle *h, const char *chain, const char *table,
if (!rulenum) {
if (ops->print_table_header)
ops->print_table_header(table);
- __nft_print_header(h, ops, c, format);
+ __nft_print_header(h, c, format);
}
__nft_rule_list(h, c, rulenum, format, ops->print_rule);
return 1;
@@ -2229,7 +2224,7 @@ int nft_rule_list(struct nft_handle *h, const char *chain, const char *table,
if (found)
printf("\n");
- __nft_print_header(h, ops, c, format);
+ __nft_print_header(h, c, format);
__nft_rule_list(h, c, rulenum, format, ops->print_rule);
found = true;
--
2.24.0
next prev parent reply other threads:[~2019-11-21 17:37 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-21 17:36 [iptables PATCH v4 00/12] Implement among match support Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 01/12] nft: family_ops: Pass nft_handle to 'add' callback Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 02/12] nft: family_ops: Pass nft_handle to 'rule_find' callback Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 03/12] nft: family_ops: Pass nft_handle to 'print_rule' callback Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 04/12] nft: family_ops: Pass nft_handle to 'rule_to_cs' callback Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 05/12] nft: Keep nft_handle pointer in nft_xt_ctx Phil Sutter
2019-11-21 17:36 ` Phil Sutter [this message]
2019-11-21 17:36 ` [iptables PATCH v4 07/12] nft: Introduce NFT_CL_SETS cache level Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 08/12] nft: Support NFT_COMPAT_SET_ADD Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 09/12] nft: Bore up nft_parse_payload() Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 10/12] nft: Embed rule's table name in nft_xt_ctx Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 11/12] nft: Support parsing lookup expression Phil Sutter
2019-11-21 17:36 ` [iptables PATCH v4 12/12] nft: bridge: Rudimental among extension support Phil Sutter
2019-11-25 20:41 ` [iptables PATCH v4 00/12] Implement among match support Pablo Neira Ayuso
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191121173647.31488-7-phil@nwl.cc \
--to=phil@nwl.cc \
--cc=netfilter-devel@vger.kernel.org \
--cc=pablo@netfilter.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).