DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] net/mlx5: fix number of retries for UAR allocation
@ 2020-08-03  9:18 Dekel Peled
  2020-08-05 14:14 ` [dpdk-dev] [dpdk-stable] " Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Dekel Peled @ 2020-08-03  9:18 UTC (permalink / raw)
  To: matan, viacheslavo, rasland; +Cc: dev, stable

Previous patch added definition of number of retries for UAR allocation.
This value is adequate for x86 systems with 4K pages.
On power9 system with 64K pages the required value is 32.
This patch updates the defined value from 2 to 32.

Fixes: a0bfe9d56f74 ("net/mlx5: fix UAR memory mapping type")
Cc: stable@dpdk.org

Signed-off-by: Dekel Peled <dekelp@mellanox.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
---
 drivers/net/mlx5/mlx5_defs.h | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/net/mlx5/mlx5_defs.h b/drivers/net/mlx5/mlx5_defs.h
index e5f7acc..c26d5a2 100644
--- a/drivers/net/mlx5/mlx5_defs.h
+++ b/drivers/net/mlx5/mlx5_defs.h
@@ -202,9 +202,7 @@
  * UAR base address if UAR was not the first object in the UAR page.
  * It caused the PMD failure and we should try to get another UAR
  * till we get the first one with non-NULL base address returned.
- * Should follow the rdma_core internal (not exported) definition
- * MLX5_NUM_NON_FP_BFREGS_PER_UAR.
  */
-#define MLX5_ALLOC_UAR_RETRY 2
+#define MLX5_ALLOC_UAR_RETRY 32
 
 #endif /* RTE_PMD_MLX5_DEFS_H_ */
-- 
1.8.3.1


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

* Re: [dpdk-dev] [dpdk-stable] [PATCH] net/mlx5: fix number of retries for UAR allocation
  2020-08-03  9:18 [dpdk-dev] [PATCH] net/mlx5: fix number of retries for UAR allocation Dekel Peled
@ 2020-08-05 14:14 ` Thomas Monjalon
  2020-08-05 14:27   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2020-08-05 14:14 UTC (permalink / raw)
  To: Dekel Peled; +Cc: matan, viacheslavo, rasland, stable, dev

03/08/2020 11:18, Dekel Peled:
> Previous patch added definition of number of retries for UAR allocation.
> This value is adequate for x86 systems with 4K pages.
> On power9 system with 64K pages the required value is 32.
> This patch updates the defined value from 2 to 32.

What about bigger pages?

> Fixes: a0bfe9d56f74 ("net/mlx5: fix UAR memory mapping type")
> Cc: stable@dpdk.org

No need of stable@dpdk.org because it was introduced in this release.

> Signed-off-by: Dekel Peled <dekelp@mellanox.com>
> Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
> ---
>  drivers/net/mlx5/mlx5_defs.h | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/net/mlx5/mlx5_defs.h b/drivers/net/mlx5/mlx5_defs.h
> index e5f7acc..c26d5a2 100644
> --- a/drivers/net/mlx5/mlx5_defs.h
> +++ b/drivers/net/mlx5/mlx5_defs.h
> @@ -202,9 +202,7 @@
>   * UAR base address if UAR was not the first object in the UAR page.
>   * It caused the PMD failure and we should try to get another UAR
>   * till we get the first one with non-NULL base address returned.
> - * Should follow the rdma_core internal (not exported) definition
> - * MLX5_NUM_NON_FP_BFREGS_PER_UAR.
>   */
> -#define MLX5_ALLOC_UAR_RETRY 2
> +#define MLX5_ALLOC_UAR_RETRY 32

It is missing a comment to explain the calculation with 64K page.




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

* Re: [dpdk-dev] [dpdk-stable] [PATCH] net/mlx5: fix number of retries for UAR allocation
  2020-08-05 14:14 ` [dpdk-dev] [dpdk-stable] " Thomas Monjalon
@ 2020-08-05 14:27   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2020-08-05 14:27 UTC (permalink / raw)
  To: Dekel Peled; +Cc: dev, matan, viacheslavo, rasland

05/08/2020 16:14, Thomas Monjalon:
> 03/08/2020 11:18, Dekel Peled:
> > Previous patch added definition of number of retries for UAR allocation.
> > This value is adequate for x86 systems with 4K pages.
> > On power9 system with 64K pages the required value is 32.
> > This patch updates the defined value from 2 to 32.
> 
> What about bigger pages?
> 
> > Fixes: a0bfe9d56f74 ("net/mlx5: fix UAR memory mapping type")
> > Cc: stable@dpdk.org
> 
> No need of stable@dpdk.org because it was introduced in this release.
> 
> > Signed-off-by: Dekel Peled <dekelp@mellanox.com>
> > Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
> > ---
> >  drivers/net/mlx5/mlx5_defs.h | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> > 
> > diff --git a/drivers/net/mlx5/mlx5_defs.h b/drivers/net/mlx5/mlx5_defs.h
> > index e5f7acc..c26d5a2 100644
> > --- a/drivers/net/mlx5/mlx5_defs.h
> > +++ b/drivers/net/mlx5/mlx5_defs.h
> > @@ -202,9 +202,7 @@
> >   * UAR base address if UAR was not the first object in the UAR page.
> >   * It caused the PMD failure and we should try to get another UAR
> >   * till we get the first one with non-NULL base address returned.
> > - * Should follow the rdma_core internal (not exported) definition
> > - * MLX5_NUM_NON_FP_BFREGS_PER_UAR.
> >   */
> > -#define MLX5_ALLOC_UAR_RETRY 2
> > +#define MLX5_ALLOC_UAR_RETRY 32
> 
> It is missing a comment to explain the calculation with 64K page.

It seems getting reasons for this magic workaround is not trivial.

Applied, thanks



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

end of thread, other threads:[~2020-08-05 14:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-03  9:18 [dpdk-dev] [PATCH] net/mlx5: fix number of retries for UAR allocation Dekel Peled
2020-08-05 14:14 ` [dpdk-dev] [dpdk-stable] " Thomas Monjalon
2020-08-05 14:27   ` Thomas Monjalon

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git