All of lore.kernel.org
 help / color / mirror / Atom feed
* [pull-request] mlx5-next branch 2021-12-15
@ 2021-12-15 18:49 Saeed Mahameed
  2021-12-15 19:23 ` Jakub Kicinski
  0 siblings, 1 reply; 8+ messages in thread
From: Saeed Mahameed @ 2021-12-15 18:49 UTC (permalink / raw)
  To: Jakub Kicinski, David S. Miller, Jason Gunthorpe
  Cc: Leon Romanovsky, linux-rdma, netdev, Saeed Mahameed

From: Saeed Mahameed <saeedm@nvidia.com>

Hi Dave, Jakub, Jason

This pulls mlx5-next branch into net-next and rdma branches.
All patches already reviewed on both rdma and netdev mailing lists.

Please pull and let me know if there's any problem.

1) Add multiple FDB steering priorities [1]
2) Introduce HW bits needed to configure MAC list size of VF/SF.
   Required for ("net/mlx5: Memory optimizations") upcoming series [2].

[1] https://lore.kernel.org/netdev/20211201193621.9129-1-saeed@kernel.org/
[2] https://lore.kernel.org/lkml/20211208141722.13646-1-shayd@nvidia.com/

Thanks,
Saeed.

The following changes since commit fa55b7dcdc43c1aa1ba12bca9d2dd4318c2a0dbf:

  Linux 5.16-rc1 (2021-11-14 13:56:52 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mellanox/linux.git mlx5-next

for you to fetch changes up to 685b1afd7911676691c4167f420e16a957f5a38e:

  net/mlx5: Introduce log_max_current_uc_list_wr_supported bit (2021-12-15 10:21:50 -0800)

----------------------------------------------------------------
Maor Gottlieb (4):
      net/mlx5: Separate FDB namespace
      net/mlx5: Refactor mlx5_get_flow_namespace
      net/mlx5: Create more priorities for FDB bypass namespace
      RDMA/mlx5: Add support to multiple priorities for FDB rules

Shay Drory (1):
      net/mlx5: Introduce log_max_current_uc_list_wr_supported bit

 drivers/infiniband/hw/mlx5/fs.c                   | 18 +++++++++---------
 drivers/infiniband/hw/mlx5/mlx5_ib.h              |  3 ++-
 drivers/net/ethernet/mellanox/mlx5/core/fs_cmd.c  |  4 +++-
 drivers/net/ethernet/mellanox/mlx5/core/fs_core.c | 76 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------
 include/linux/mlx5/fs.h                           |  1 +
 include/linux/mlx5/mlx5_ifc.h                     |  2 +-
 6 files changed, 76 insertions(+), 28 deletions(-)


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

* Re: [pull-request] mlx5-next branch 2021-12-15
  2021-12-15 18:49 [pull-request] mlx5-next branch 2021-12-15 Saeed Mahameed
@ 2021-12-15 19:23 ` Jakub Kicinski
  2021-12-15 22:14   ` Saeed Mahameed
  2021-12-15 22:41   ` Saeed Mahameed
  0 siblings, 2 replies; 8+ messages in thread
From: Jakub Kicinski @ 2021-12-15 19:23 UTC (permalink / raw)
  To: Saeed Mahameed
  Cc: David S. Miller, Jason Gunthorpe, Leon Romanovsky, linux-rdma,
	netdev, Saeed Mahameed

On Wed, 15 Dec 2021 10:49:45 -0800 Saeed Mahameed wrote:
> This pulls mlx5-next branch into net-next and rdma branches.
> All patches already reviewed on both rdma and netdev mailing lists.
> 
> Please pull and let me know if there's any problem.
> 
> 1) Add multiple FDB steering priorities [1]
> 2) Introduce HW bits needed to configure MAC list size of VF/SF.
>    Required for ("net/mlx5: Memory optimizations") upcoming series [2].

Why are you not posting the patches?

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

* Re: [pull-request] mlx5-next branch 2021-12-15
  2021-12-15 19:23 ` Jakub Kicinski
@ 2021-12-15 22:14   ` Saeed Mahameed
  2021-12-15 22:58     ` Jakub Kicinski
  2021-12-15 22:41   ` Saeed Mahameed
  1 sibling, 1 reply; 8+ messages in thread
From: Saeed Mahameed @ 2021-12-15 22:14 UTC (permalink / raw)
  To: kuba; +Cc: Leon Romanovsky, Jason Gunthorpe, davem, linux-rdma, netdev

On Wed, 2021-12-15 at 11:23 -0800, Jakub Kicinski wrote:
> On Wed, 15 Dec 2021 10:49:45 -0800 Saeed Mahameed wrote:
> > This pulls mlx5-next branch into net-next and rdma branches.
> > All patches already reviewed on both rdma and netdev mailing lists.
> > 
> > Please pull and let me know if there's any problem.
> > 
> > 1) Add multiple FDB steering priorities [1]
> > 2) Introduce HW bits needed to configure MAC list size of VF/SF.
> >    Required for ("net/mlx5: Memory optimizations") upcoming series
> > [2].
> 
> Why are you not posting the patches?

already posted before :
[1]
https://lore.kernel.org/netdev/20211201193621.9129-1-saeed@kernel.org/
[2]
https://lore.kernel.org/lkml/20211208141722.13646-1-shayd@nvidia.com/



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

* Re: [pull-request] mlx5-next branch 2021-12-15
  2021-12-15 19:23 ` Jakub Kicinski
  2021-12-15 22:14   ` Saeed Mahameed
@ 2021-12-15 22:41   ` Saeed Mahameed
  1 sibling, 0 replies; 8+ messages in thread
From: Saeed Mahameed @ 2021-12-15 22:41 UTC (permalink / raw)
  To: kuba; +Cc: Leon Romanovsky, Jason Gunthorpe, davem, linux-rdma, netdev

On Wed, 2021-12-15 at 11:23 -0800, Jakub Kicinski wrote:
> On Wed, 15 Dec 2021 10:49:45 -0800 Saeed Mahameed wrote:
> > This pulls mlx5-next branch into net-next and rdma branches.
> > All patches already reviewed on both rdma and netdev mailing lists.
> > 
> > Please pull and let me know if there's any problem.
> > 
> > 1) Add multiple FDB steering priorities [1]
> > 2) Introduce HW bits needed to configure MAC list size of VF/SF.
> >    Required for ("net/mlx5: Memory optimizations") upcoming series
> > [2].
> 
> Why are you not posting the patches?

already posted before :
[1]
https://lore.kernel.org/netdev/20211201193621.9129-1-saeed@kernel.org/
[2]
https://lore.kernel.org/lkml/20211208141722.13646-1-shayd@nvidia.com/


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

* Re: [pull-request] mlx5-next branch 2021-12-15
  2021-12-15 22:14   ` Saeed Mahameed
@ 2021-12-15 22:58     ` Jakub Kicinski
  2021-12-15 23:35       ` Saeed Mahameed
  0 siblings, 1 reply; 8+ messages in thread
From: Jakub Kicinski @ 2021-12-15 22:58 UTC (permalink / raw)
  To: Saeed Mahameed
  Cc: Leon Romanovsky, Jason Gunthorpe, davem, linux-rdma, netdev

On Wed, 15 Dec 2021 22:14:11 +0000 Saeed Mahameed wrote:
> On Wed, 2021-12-15 at 11:23 -0800, Jakub Kicinski wrote:
> > On Wed, 15 Dec 2021 10:49:45 -0800 Saeed Mahameed wrote:  
> > > This pulls mlx5-next branch into net-next and rdma branches.
> > > All patches already reviewed on both rdma and netdev mailing lists.
> > > 
> > > Please pull and let me know if there's any problem.
> > > 
> > > 1) Add multiple FDB steering priorities [1]
> > > 2) Introduce HW bits needed to configure MAC list size of VF/SF.
> > >    Required for ("net/mlx5: Memory optimizations") upcoming series
> > > [2].  
> > 
> > Why are you not posting the patches?  
> 
> already posted before :
> [1] https://lore.kernel.org/netdev/20211201193621.9129-1-saeed@kernel.org/
> [2] https://lore.kernel.org/lkml/20211208141722.13646-1-shayd@nvidia.com/

Post them as a reply to the pull request like you usually do, please.

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

* Re: [pull-request] mlx5-next branch 2021-12-15
  2021-12-15 22:58     ` Jakub Kicinski
@ 2021-12-15 23:35       ` Saeed Mahameed
  2021-12-15 23:44         ` Jakub Kicinski
  0 siblings, 1 reply; 8+ messages in thread
From: Saeed Mahameed @ 2021-12-15 23:35 UTC (permalink / raw)
  To: Jakub Kicinski
  Cc: Leon Romanovsky, Jason Gunthorpe, davem, linux-rdma, netdev

On Wed, 2021-12-15 at 14:58 -0800, Jakub Kicinski wrote:
> On Wed, 15 Dec 2021 22:14:11 +0000 Saeed Mahameed wrote:
> > On Wed, 2021-12-15 at 11:23 -0800, Jakub Kicinski wrote:
> > > On Wed, 15 Dec 2021 10:49:45 -0800 Saeed Mahameed wrote:  
> > > > This pulls mlx5-next branch into net-next and rdma branches.
> > > > All patches already reviewed on both rdma and netdev mailing
> > > > lists.
> > > > 
> > > > Please pull and let me know if there's any problem.
> > > > 
> > > > 1) Add multiple FDB steering priorities [1]
> > > > 2) Introduce HW bits needed to configure MAC list size of
> > > > VF/SF.
> > > >    Required for ("net/mlx5: Memory optimizations") upcoming
> > > > series
> > > > [2].  
> > > 
> > > Why are you not posting the patches?  
> > 
> > already posted before :
> > [1]
> > https://lore.kernel.org/netdev/20211201193621.9129-1-saeed@kernel.org/
> > [2]
> > https://lore.kernel.org/lkml/20211208141722.13646-1-shayd@nvidia.com/
> 
> Post them as a reply to the pull request like you usually do, please.

They were already posted, and reviewed, and all maintainers were CC'ed,
including you, why do you want me to spam the mailing list over and
over again with the same patches ?  This is not how genuine pull
requests are supposed to work.




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

* Re: [pull-request] mlx5-next branch 2021-12-15
  2021-12-15 23:35       ` Saeed Mahameed
@ 2021-12-15 23:44         ` Jakub Kicinski
  2021-12-16  0:55           ` Saeed Mahameed
  0 siblings, 1 reply; 8+ messages in thread
From: Jakub Kicinski @ 2021-12-15 23:44 UTC (permalink / raw)
  To: Saeed Mahameed
  Cc: Leon Romanovsky, Jason Gunthorpe, davem, linux-rdma, netdev

On Wed, 15 Dec 2021 15:35:09 -0800 Saeed Mahameed wrote:
> On Wed, 2021-12-15 at 14:58 -0800, Jakub Kicinski wrote:
> > On Wed, 15 Dec 2021 22:14:11 +0000 Saeed Mahameed wrote:  
> > > already posted before :
> > > [1] https://lore.kernel.org/netdev/20211201193621.9129-1-saeed@kernel.org/
> > > [2] https://lore.kernel.org/lkml/20211208141722.13646-1-shayd@nvidia.com/  
> > 
> > Post them as a reply to the pull request like you usually do, please.  
> 
> They were already posted, and reviewed, and all maintainers were CC'ed,
> including you, why do you want me to spam the mailing list over and
> over again with the same patches ?

You reposting measly 5 patches is hardly spamming. I have scripts which
check if the contents of the PR match the posting, which obviously
can't follow random links from the email.

> This is not how genuine pull requests are supposed to work.

I'm not comfortable with blindly pulling patches from vendor trees.

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

* Re: [pull-request] mlx5-next branch 2021-12-15
  2021-12-15 23:44         ` Jakub Kicinski
@ 2021-12-16  0:55           ` Saeed Mahameed
  0 siblings, 0 replies; 8+ messages in thread
From: Saeed Mahameed @ 2021-12-16  0:55 UTC (permalink / raw)
  To: Jakub Kicinski
  Cc: Leon Romanovsky, Jason Gunthorpe, davem, linux-rdma, netdev

On Wed, 2021-12-15 at 15:44 -0800, Jakub Kicinski wrote:
> On Wed, 15 Dec 2021 15:35:09 -0800 Saeed Mahameed wrote:
> > On Wed, 2021-12-15 at 14:58 -0800, Jakub Kicinski wrote:
> > > On Wed, 15 Dec 2021 22:14:11 +0000 Saeed Mahameed wrote:  
> > > > already posted before :
> > > > [1] https://lore.kernel.org/netdev/20211201193621.9129-1-
> > > > saeed@kernel.org/
> > > > [2] https://lore.kernel.org/lkml/20211208141722.13646-1-
> > > > shayd@nvidia.com/ 
> > > 
> > > Post them as a reply to the pull request like you usually do,
> > > please.  
> > 
> > They were already posted, and reviewed, and all maintainers were
> > CC'ed,
> > including you, why do you want me to spam the mailing list over and
> > over again with the same patches ?
> 
> You reposting measly 5 patches is hardly spamming. I have scripts
> which

That's not the point, sometimes it is less or much more than 5.
For mlx5-next (Shared branch between RDMA and netdev), I would like to
submit standard clean PRs like all other maintainers.

> check if the contents of the PR match the posting, which obviously
> can't follow random links from the email.
> 

The script is great to validate pseudo-PRs (netdev only) where we post
patches for the first time in the PR.

but this is different, this is a true clean PR where all patches
already posted and reviewed. the tree is already written genuinely in
ink!

> > This is not how genuine pull requests are supposed to work.
> 
> I'm not comfortable with blindly pulling patches from vendor trees.

Good to know and very disappointing, is there anything i could do to
make you better trust my mlx5-next PRs ? other than RE-posting ?

Again I don't see any point of RE-posting the patches together with a
clean PR, redundant and unnecessary and against all standards, it just
cancels the whole process and duplicates my work, and will confuse
other maintainers who might have already pulled part or all of the
patches in the branch.




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

end of thread, other threads:[~2021-12-16  0:55 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-15 18:49 [pull-request] mlx5-next branch 2021-12-15 Saeed Mahameed
2021-12-15 19:23 ` Jakub Kicinski
2021-12-15 22:14   ` Saeed Mahameed
2021-12-15 22:58     ` Jakub Kicinski
2021-12-15 23:35       ` Saeed Mahameed
2021-12-15 23:44         ` Jakub Kicinski
2021-12-16  0:55           ` Saeed Mahameed
2021-12-15 22:41   ` Saeed Mahameed

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.