From: Pablo Neira Ayuso <pablo@netfilter•org>
To: netdev@vger•kernel.org
Cc: davem@davemloft•net, jiri@resnulli•us, horms@verge•net.au,
john.fastabend@gmail•com
Subject: [PATCH RFC 1/3] net: ixgbe: add struct igxbe_filter
Date: Thu, 25 Feb 2016 18:37:22 +0100 [thread overview]
Message-ID: <1456421844-5901-2-git-send-email-pablo@netfilter.org> (raw)
In-Reply-To: <1456421844-5901-1-git-send-email-pablo@netfilter.org>
Just a container structure, this change is required by follow up patches
that rework the u32 classifier offload on top of the generic
intermediate representation.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter•org>
---
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 51 +++++++++++++++------------
1 file changed, 28 insertions(+), 23 deletions(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
index cf4b729..252e9ff 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
@@ -8238,6 +8238,12 @@ static int ixgbe_configure_clsu32_del_hnode(struct ixgbe_adapter *adapter,
return 0;
}
+struct ixgbe_filter {
+ struct ixgbe_fdir_filter *input;
+ union ixgbe_atr_input mask;
+ u8 queue;
+};
+
static int ixgbe_configure_clsu32(struct ixgbe_adapter *adapter,
__be16 protocol,
struct tc_cls_u32_offload *cls)
@@ -8245,16 +8251,14 @@ static int ixgbe_configure_clsu32(struct ixgbe_adapter *adapter,
u32 loc = cls->knode.handle & 0xfffff;
struct ixgbe_hw *hw = &adapter->hw;
struct ixgbe_mat_field *field_ptr;
- struct ixgbe_fdir_filter *input;
- union ixgbe_atr_input mask;
+ struct ixgbe_filter f;
#ifdef CONFIG_NET_CLS_ACT
const struct tc_action *a;
#endif
int i, err = 0;
- u8 queue;
u32 handle;
- memset(&mask, 0, sizeof(union ixgbe_atr_input));
+ memset(&f.mask, 0, sizeof(union ixgbe_atr_input));
handle = cls->knode.handle;
/* At the moment cls_u32 jumps to transport layer and skips past
@@ -8320,8 +8324,8 @@ static int ixgbe_configure_clsu32(struct ixgbe_adapter *adapter,
if (!field_ptr)
return -EINVAL;
- input = kzalloc(sizeof(*input), GFP_KERNEL);
- if (!input)
+ f.input = kzalloc(sizeof(struct ixgbe_fdir_filter), GFP_KERNEL);
+ if (!f.input)
return -ENOMEM;
for (i = 0; i < cls->knode.sel->nkeys; i++) {
@@ -8334,8 +8338,8 @@ static int ixgbe_configure_clsu32(struct ixgbe_adapter *adapter,
for (j = 0; field_ptr[j].val; j++) {
if (field_ptr[j].off == off &&
field_ptr[j].mask == m) {
- field_ptr[j].val(input, &mask, val, m);
- input->filter.formatted.flow_type |=
+ field_ptr[j].val(f.input, &f.mask, val, m);
+ f.input->filter.formatted.flow_type |=
field_ptr[j].type;
found_entry = true;
break;
@@ -8346,11 +8350,11 @@ static int ixgbe_configure_clsu32(struct ixgbe_adapter *adapter,
goto err_out;
}
- mask.formatted.flow_type = IXGBE_ATR_L4TYPE_IPV6_MASK |
- IXGBE_ATR_L4TYPE_MASK;
+ f.mask.formatted.flow_type = IXGBE_ATR_L4TYPE_IPV6_MASK |
+ IXGBE_ATR_L4TYPE_MASK;
- if (input->filter.formatted.flow_type == IXGBE_ATR_FLOW_TYPE_IPV4)
- mask.formatted.flow_type &= IXGBE_ATR_L4TYPE_IPV6_MASK;
+ if (f.input->filter.formatted.flow_type == IXGBE_ATR_FLOW_TYPE_IPV4)
+ f.mask.formatted.flow_type &= IXGBE_ATR_L4TYPE_IPV6_MASK;
#ifdef CONFIG_NET_CLS_ACT
if (list_empty(&cls->knode.exts->actions))
@@ -8362,34 +8366,35 @@ static int ixgbe_configure_clsu32(struct ixgbe_adapter *adapter,
}
#endif
- input->action = IXGBE_FDIR_DROP_QUEUE;
- queue = IXGBE_FDIR_DROP_QUEUE;
- input->sw_idx = loc;
+ f.input->action = IXGBE_FDIR_DROP_QUEUE;
+ f.queue = IXGBE_FDIR_DROP_QUEUE;
+ f.input->sw_idx = loc;
spin_lock(&adapter->fdir_perfect_lock);
if (hlist_empty(&adapter->fdir_filter_list)) {
- memcpy(&adapter->fdir_mask, &mask, sizeof(mask));
- err = ixgbe_fdir_set_input_mask_82599(hw, &mask);
+ memcpy(&adapter->fdir_mask, &f.mask, sizeof(f.mask));
+ err = ixgbe_fdir_set_input_mask_82599(hw, &f.mask);
if (err)
goto err_out_w_lock;
- } else if (memcmp(&adapter->fdir_mask, &mask, sizeof(mask))) {
+ } else if (memcmp(&adapter->fdir_mask, &f.mask, sizeof(f.mask))) {
err = -EINVAL;
goto err_out_w_lock;
}
- ixgbe_atr_compute_perfect_hash_82599(&input->filter, &mask);
- err = ixgbe_fdir_write_perfect_filter_82599(hw, &input->filter,
- input->sw_idx, queue);
+ ixgbe_atr_compute_perfect_hash_82599(&f.input->filter, &f.mask);
+ err = ixgbe_fdir_write_perfect_filter_82599(hw, &f.input->filter,
+ f.input->sw_idx, f.queue);
if (!err)
- ixgbe_update_ethtool_fdir_entry(adapter, input, input->sw_idx);
+ ixgbe_update_ethtool_fdir_entry(adapter, f.input,
+ f.input->sw_idx);
spin_unlock(&adapter->fdir_perfect_lock);
return err;
err_out_w_lock:
spin_unlock(&adapter->fdir_perfect_lock);
err_out:
- kfree(input);
+ kfree(f.input);
return -EINVAL;
}
--
2.1.4
next prev parent reply other threads:[~2016-02-25 17:37 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-25 17:37 [PATCH RFC 0/3] intermediate representation for jit and cls_u32 conversion Pablo Neira Ayuso
2016-02-25 17:37 ` Pablo Neira Ayuso [this message]
2016-02-25 17:37 ` [PATCH RFC 2/3] net: intermediate representation for jit translation Pablo Neira Ayuso
2016-02-25 17:37 ` [PATCH RFC 3/3] net: convert tc_u32 to use the intermediate representation Pablo Neira Ayuso
2016-02-25 20:37 ` John Fastabend
2016-02-26 14:24 ` Pablo Neira Ayuso
2016-02-26 14:53 ` John Fastabend
2016-02-26 16:02 ` Pablo Neira Ayuso
2016-02-26 16:34 ` John Fastabend
2016-02-26 17:38 ` David Miller
2016-02-26 17:34 ` David Miller
2016-02-25 18:11 ` [PATCH RFC 0/3] intermediate representation for jit and cls_u32 conversion John Fastabend
2016-02-26 11:47 ` Pablo Neira Ayuso
2016-02-26 15:42 ` John Fastabend
2016-02-26 16:19 ` Pablo Neira Ayuso
2016-02-26 16:46 ` John Fastabend
2016-02-26 17:40 ` David Miller
2016-02-26 18:53 ` Alexei Starovoitov
2016-02-26 17:26 ` David Miller
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=1456421844-5901-2-git-send-email-pablo@netfilter.org \
--to=pablo@netfilter$(echo .)org \
--cc=davem@davemloft$(echo .)net \
--cc=horms@verge$(echo .)net.au \
--cc=jiri@resnulli$(echo .)us \
--cc=john.fastabend@gmail$(echo .)com \
--cc=netdev@vger$(echo .)kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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