DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v1] net/failsafe: add an RSS hash update callback
@ 2018-05-10 14:38 Ophir Munk
  2018-05-11  8:31 ` Gaëtan Rivet
  0 siblings, 1 reply; 3+ messages in thread
From: Ophir Munk @ 2018-05-10 14:38 UTC (permalink / raw)
  To: dev, Gaetan Rivet; +Cc: Thomas Monjalon, Olga Shern, Ophir Munk, Shahaf Shuler

Add an RSS hash update callback to eth_dev_ops.

Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
---
 drivers/net/failsafe/failsafe_ops.c | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/drivers/net/failsafe/failsafe_ops.c b/drivers/net/failsafe/failsafe_ops.c
index 089f114..d04277b 100644
--- a/drivers/net/failsafe/failsafe_ops.c
+++ b/drivers/net/failsafe/failsafe_ops.c
@@ -13,6 +13,7 @@
 #include <rte_malloc.h>
 #include <rte_flow.h>
 #include <rte_cycles.h>
+#include <rte_ethdev.h>
 
 #include "failsafe_private.h"
 
@@ -947,6 +948,31 @@ fs_mac_addr_set(struct rte_eth_dev *dev, struct ether_addr *mac_addr)
 }
 
 static int
+fs_rss_hash_update(struct rte_eth_dev *dev,
+			struct rte_eth_rss_conf *rss_conf)
+{
+	struct sub_device *sdev;
+	uint8_t i;
+	int ret;
+
+	fs_lock(dev, 0);
+	FOREACH_SUBDEV_STATE(sdev, i, dev, DEV_ACTIVE) {
+		ret = rte_eth_dev_rss_hash_update(PORT_ID(sdev), rss_conf);
+		ret = fs_err(sdev, ret);
+		if (ret) {
+			ERROR("Operation rte_eth_dev_rss_hash_update"
+				" failed for sub_device %d with error %d",
+				i, ret);
+			fs_unlock(dev, 0);
+			return ret;
+		}
+	}
+	fs_unlock(dev, 0);
+
+	return 0;
+}
+
+static int
 fs_filter_ctrl(struct rte_eth_dev *dev,
 		enum rte_filter_type type,
 		enum rte_filter_op op,
@@ -1005,5 +1031,6 @@ const struct eth_dev_ops failsafe_ops = {
 	.mac_addr_remove = fs_mac_addr_remove,
 	.mac_addr_add = fs_mac_addr_add,
 	.mac_addr_set = fs_mac_addr_set,
+	.rss_hash_update = fs_rss_hash_update,
 	.filter_ctrl = fs_filter_ctrl,
 };
-- 
2.7.4

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

* Re: [dpdk-dev] [PATCH v1] net/failsafe: add an RSS hash update callback
  2018-05-10 14:38 [dpdk-dev] [PATCH v1] net/failsafe: add an RSS hash update callback Ophir Munk
@ 2018-05-11  8:31 ` Gaëtan Rivet
  2018-05-11 16:18   ` Ferruh Yigit
  0 siblings, 1 reply; 3+ messages in thread
From: Gaëtan Rivet @ 2018-05-11  8:31 UTC (permalink / raw)
  To: Ophir Munk; +Cc: dev, Thomas Monjalon, Olga Shern, Shahaf Shuler

Hi Ophir,

On Thu, May 10, 2018 at 02:38:10PM +0000, Ophir Munk wrote:
> Add an RSS hash update callback to eth_dev_ops.
> 
> Signed-off-by: Ophir Munk <ophirmu@mellanox.com>

Thank you for submitting this, the patch is clean.

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

> ---
>  drivers/net/failsafe/failsafe_ops.c | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> diff --git a/drivers/net/failsafe/failsafe_ops.c b/drivers/net/failsafe/failsafe_ops.c
> index 089f114..d04277b 100644
> --- a/drivers/net/failsafe/failsafe_ops.c
> +++ b/drivers/net/failsafe/failsafe_ops.c
> @@ -13,6 +13,7 @@
>  #include <rte_malloc.h>
>  #include <rte_flow.h>
>  #include <rte_cycles.h>
> +#include <rte_ethdev.h>
>  
>  #include "failsafe_private.h"
>  
> @@ -947,6 +948,31 @@ fs_mac_addr_set(struct rte_eth_dev *dev, struct ether_addr *mac_addr)
>  }
>  
>  static int
> +fs_rss_hash_update(struct rte_eth_dev *dev,
> +			struct rte_eth_rss_conf *rss_conf)
> +{
> +	struct sub_device *sdev;
> +	uint8_t i;
> +	int ret;
> +
> +	fs_lock(dev, 0);
> +	FOREACH_SUBDEV_STATE(sdev, i, dev, DEV_ACTIVE) {
> +		ret = rte_eth_dev_rss_hash_update(PORT_ID(sdev), rss_conf);
> +		ret = fs_err(sdev, ret);
> +		if (ret) {
> +			ERROR("Operation rte_eth_dev_rss_hash_update"
> +				" failed for sub_device %d with error %d",
> +				i, ret);
> +			fs_unlock(dev, 0);
> +			return ret;
> +		}
> +	}
> +	fs_unlock(dev, 0);
> +
> +	return 0;
> +}
> +
> +static int
>  fs_filter_ctrl(struct rte_eth_dev *dev,
>  		enum rte_filter_type type,
>  		enum rte_filter_op op,
> @@ -1005,5 +1031,6 @@ const struct eth_dev_ops failsafe_ops = {
>  	.mac_addr_remove = fs_mac_addr_remove,
>  	.mac_addr_add = fs_mac_addr_add,
>  	.mac_addr_set = fs_mac_addr_set,
> +	.rss_hash_update = fs_rss_hash_update,
>  	.filter_ctrl = fs_filter_ctrl,
>  };
> -- 
> 2.7.4
> 

-- 
Gaëtan Rivet
6WIND

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

* Re: [dpdk-dev] [PATCH v1] net/failsafe: add an RSS hash update callback
  2018-05-11  8:31 ` Gaëtan Rivet
@ 2018-05-11 16:18   ` Ferruh Yigit
  0 siblings, 0 replies; 3+ messages in thread
From: Ferruh Yigit @ 2018-05-11 16:18 UTC (permalink / raw)
  To: Gaëtan Rivet, Ophir Munk
  Cc: dev, Thomas Monjalon, Olga Shern, Shahaf Shuler

On 5/11/2018 9:31 AM, Gaëtan Rivet wrote:
> Hi Ophir,
> 
> On Thu, May 10, 2018 at 02:38:10PM +0000, Ophir Munk wrote:
>> Add an RSS hash update callback to eth_dev_ops.
>>
>> Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
> 
> Thank you for submitting this, the patch is clean.
> 
> Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>

Applied to dpdk-next-net/master, thanks.

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

end of thread, other threads:[~2018-05-11 16:18 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-10 14:38 [dpdk-dev] [PATCH v1] net/failsafe: add an RSS hash update callback Ophir Munk
2018-05-11  8:31 ` Gaëtan Rivet
2018-05-11 16:18   ` Ferruh Yigit

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