DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues
@ 2015-06-19  6:35 Jingjing Wu
  2015-06-19 13:57 ` Gaetan Rivet
  2015-06-23  1:26 ` Lu, Wenzhuo
  0 siblings, 2 replies; 5+ messages in thread
From: Jingjing Wu @ 2015-06-19  6:35 UTC (permalink / raw)
  To: dev

The flexbytes offset can not be set, because the value is over written
when fdir is enabled.
This patch fixes this issue, and also removes some reduplicate lines.

Reported-by: David Marchand <david.marchand@6wind.com>
Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
---
 drivers/net/ixgbe/ixgbe_fdir.c | 24 ++++++++----------------
 1 file changed, 8 insertions(+), 16 deletions(-)

diff --git a/drivers/net/ixgbe/ixgbe_fdir.c b/drivers/net/ixgbe/ixgbe_fdir.c
index 40b144e..d294f85 100644
--- a/drivers/net/ixgbe/ixgbe_fdir.c
+++ b/drivers/net/ixgbe/ixgbe_fdir.c
@@ -109,7 +109,7 @@ static int fdir_erase_filter_82599(struct ixgbe_hw *hw, uint32_t fdirhash);
 static int fdir_set_input_mask_82599(struct rte_eth_dev *dev,
 		const struct rte_eth_fdir_masks *input_mask);
 static int ixgbe_set_fdir_flex_conf(struct rte_eth_dev *dev,
-		const struct rte_eth_fdir_flex_conf *conf);
+		const struct rte_eth_fdir_flex_conf *conf, uint32_t *fdirctrl);
 static int fdir_enable_82599(struct ixgbe_hw *hw, uint32_t fdirctrl);
 static int ixgbe_fdir_filter_to_atr_input(
 		const struct rte_eth_fdir_filter *fdir_filter,
@@ -247,13 +247,6 @@ configure_fdir_flags(const struct rte_fdir_conf *conf, uint32_t *fdirctrl)
 		*fdirctrl |= IXGBE_FDIRCTRL_PERFECT_MATCH;
 		*fdirctrl |= (conf->drop_queue << IXGBE_FDIRCTRL_DROP_Q_SHIFT);
 	}
-	/*
-	 * Continue setup of fdirctrl register bits:
-	 *  Set the maximum length per hash bucket to 0xA filters
-	 *  Send interrupt when 64 filters are left
-	 */
-	*fdirctrl |= (0xA << IXGBE_FDIRCTRL_MAX_LENGTH_SHIFT) |
-		    (4 << IXGBE_FDIRCTRL_FULL_THRESH_SHIFT);
 
 	return 0;
 }
@@ -370,18 +363,17 @@ fdir_set_input_mask_82599(struct rte_eth_dev *dev,
  */
 static int
 ixgbe_set_fdir_flex_conf(struct rte_eth_dev *dev,
-		const struct rte_eth_fdir_flex_conf *conf)
+		const struct rte_eth_fdir_flex_conf *conf, uint32_t *fdirctrl)
 {
 	struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 	struct ixgbe_hw_fdir_info *info =
 			IXGBE_DEV_PRIVATE_TO_FDIR_INFO(dev->data->dev_private);
 	const struct rte_eth_flex_payload_cfg *flex_cfg;
 	const struct rte_eth_fdir_flex_mask *flex_mask;
-	uint32_t fdirctrl, fdirm;
+	uint32_t fdirm;
 	uint16_t flexbytes = 0;
 	uint16_t i;
 
-	fdirctrl = IXGBE_READ_REG(hw, IXGBE_FDIRCTRL);
 	fdirm = IXGBE_READ_REG(hw, IXGBE_FDIRM);
 
 	if (conf == NULL) {
@@ -398,8 +390,9 @@ ixgbe_set_fdir_flex_conf(struct rte_eth_dev *dev,
 		if (((flex_cfg->src_offset[0] & 0x1) == 0) &&
 		    (flex_cfg->src_offset[1] == flex_cfg->src_offset[0] + 1) &&
 		    (flex_cfg->src_offset[0] <= IXGBE_MAX_FLX_SOURCE_OFF)) {
-			fdirctrl &= ~IXGBE_FDIRCTRL_FLEX_MASK;
-			fdirctrl |= (flex_cfg->src_offset[0] / sizeof(uint16_t)) <<
+			*fdirctrl &= ~IXGBE_FDIRCTRL_FLEX_MASK;
+			*fdirctrl |=
+				(flex_cfg->src_offset[0] / sizeof(uint16_t)) <<
 					IXGBE_FDIRCTRL_FLEX_SHIFT;
 		} else {
 			PMD_DRV_LOG(ERR, "invalid flexbytes arguments.");
@@ -423,10 +416,9 @@ ixgbe_set_fdir_flex_conf(struct rte_eth_dev *dev,
 			return -EINVAL;
 		}
 	}
-	IXGBE_WRITE_REG(hw, IXGBE_FDIRCTRL, fdirctrl);
 	IXGBE_WRITE_REG(hw, IXGBE_FDIRM, fdirm);
 	info->mask.flex_bytes_mask = flexbytes ? UINT16_MAX : 0;
-	info->flex_bytes_offset = (uint8_t)((fdirctrl &
+	info->flex_bytes_offset = (uint8_t)((*fdirctrl &
 					    IXGBE_FDIRCTRL_FLEX_MASK) >>
 					    IXGBE_FDIRCTRL_FLEX_SHIFT);
 	return 0;
@@ -476,7 +468,7 @@ ixgbe_fdir_configure(struct rte_eth_dev *dev)
 		return err;
 	}
 	err = ixgbe_set_fdir_flex_conf(dev,
-		&dev->data->dev_conf.fdir_conf.flex_conf);
+		&dev->data->dev_conf.fdir_conf.flex_conf, &fdirctrl);
 	if (err < 0) {
 		PMD_INIT_LOG(ERR, " Error on setting FD flexible arguments.");
 		return err;
-- 
1.9.3

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues
  2015-06-19  6:35 [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues Jingjing Wu
@ 2015-06-19 13:57 ` Gaetan Rivet
  2015-06-19 14:08   ` David Marchand
  2015-06-23  1:26 ` Lu, Wenzhuo
  1 sibling, 1 reply; 5+ messages in thread
From: Gaetan Rivet @ 2015-06-19 13:57 UTC (permalink / raw)
  To: Jingjing Wu; +Cc: dev

On Fri, Jun 19, 2015 at 8:35 AM, Jingjing Wu <jingjing.wu@intel.com> wrote:
> The flexbytes offset can not be set, because the value is over written
> when fdir is enabled.
> This patch fixes this issue, and also removes some reduplicate lines.
>
> Reported-by: David Marchand <david.marchand@6wind.com>
> Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>

Tested-by: Gaetan Rivet <gaetan.rivet@6wind.com>

-- 
Gaëtan Rivet
Software Engineer

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues
  2015-06-19 13:57 ` Gaetan Rivet
@ 2015-06-19 14:08   ` David Marchand
  2015-06-22 10:00     ` Thomas Monjalon
  0 siblings, 1 reply; 5+ messages in thread
From: David Marchand @ 2015-06-19 14:08 UTC (permalink / raw)
  To: Gaetan Rivet; +Cc: dev

On Fri, Jun 19, 2015 at 3:57 PM, Gaetan Rivet <gaetan.rivet@6wind.com>
wrote:

> On Fri, Jun 19, 2015 at 8:35 AM, Jingjing Wu <jingjing.wu@intel.com>
> wrote:
> > The flexbytes offset can not be set, because the value is over written
> > when fdir is enabled.
> > This patch fixes this issue, and also removes some reduplicate lines.
> >
> > Reported-by: David Marchand <david.marchand@6wind.com>
> > Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
>
> Tested-by: Gaetan Rivet <gaetan.rivet@6wind.com>
>

Thanks.
Acked-by: David Marchand <david.marchand@6wind.com>


-- 
David Marchand

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues
  2015-06-19 14:08   ` David Marchand
@ 2015-06-22 10:00     ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2015-06-22 10:00 UTC (permalink / raw)
  To: Jingjing Wu; +Cc: dev

> > > The flexbytes offset can not be set, because the value is over written
> > > when fdir is enabled.
> > > This patch fixes this issue, and also removes some reduplicate lines.
> > >
> > > Reported-by: David Marchand <david.marchand@6wind.com>
> > > Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
> >
> > Tested-by: Gaetan Rivet <gaetan.rivet@6wind.com>
> 
> Thanks.
> Acked-by: David Marchand <david.marchand@6wind.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues
  2015-06-19  6:35 [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues Jingjing Wu
  2015-06-19 13:57 ` Gaetan Rivet
@ 2015-06-23  1:26 ` Lu, Wenzhuo
  1 sibling, 0 replies; 5+ messages in thread
From: Lu, Wenzhuo @ 2015-06-23  1:26 UTC (permalink / raw)
  To: Wu, Jingjing, dev

> Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2015-06-23  1:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-19  6:35 [dpdk-dev] [PATCH] ixgbe: fix the flow director flexbytes offset issues Jingjing Wu
2015-06-19 13:57 ` Gaetan Rivet
2015-06-19 14:08   ` David Marchand
2015-06-22 10:00     ` Thomas Monjalon
2015-06-23  1:26 ` Lu, Wenzhuo

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).