show
netns exclusive
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
rdma/Makefile | 2 +-
rdma/rdma.c | 3 +-
rdma/rdma.h | 1 +
rdma/sys.c| 143 ++
rdma/utils.c | 1 +
5 files changed, 148 insertions(+), 2 deletions
Add man page to describe additional set netns command
for rdma device.
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
man/man8/rdma-dev.8 | 18 --
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/man/man8/rdma-dev.8 b/man/man8/rdma-dev.8
index
network namespace and assign RDMA device to this
network namespace.
$ ip netns add foo
$ rdma dev set mlx5_1 netns foo
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
rdma/dev.c | 37 +
1 file changed, 37 insertions(+)
diff --git a/rdma/dev.c b
device to a specific newly created net namespace
$ ip netns add foo
$ rdma dev set mlx5_1 netns foo
Parav Pandit (4):
rdma: Add an option to query,set net namespace sharing sys parameter
rdma: Add man pages for rdma system commands
rdma: Add an option to set net namespace of rdma device
rdma
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
man/man8/rdma-system.8 | 82 ++
man/man8/rdma.8| 7 +++-
2 files changed, 88 insertions(+), 1 deletion(-)
create mode 100644 man/man8/rdma-system.8
diff --git a/man/man8/rdma-system.8
Hi David,
> -Original Message-
> From: David Ahern
> Sent: Wednesday, May 29, 2019 9:43 PM
> To: Parav Pandit
> Cc: netdev@vger.kernel.org; linux-r...@vger.kernel.org;
> step...@networkplumber.org; Leon Romanovsky
> Subject: Re: [PATCH iproute2-next 1/4] rdma: A
_no_args()
Parav Pandit (4):
rdma: Add an option to query,set net namespace sharing sys parameter
rdma: Add man pages for rdma system commands
rdma: Add an option to set net namespace of rdma device
rdma: Add man page for rdma dev set netns command
man/man8/rdma-dev.8| 18 +-
man/m
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
man/man8/rdma-system.8 | 82 ++
man/man8/rdma.8| 7 +++-
2 files changed, 88 insertions(+), 1 deletion(-)
create mode 100644 man/man8/rdma-system.8
diff --git a/man/man8/rdma-system.8
show
netns exclusive
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
rdma/Makefile | 2 +-
rdma/rdma.c | 3 +-
rdma/rdma.h | 1 +
rdma/sys.c| 138 ++
rdma/utils.c | 1 +
5 files changed, 143 insertions(+), 2 deletions
Add man page to describe additional set netns command
for rdma device.
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
man/man8/rdma-dev.8 | 18 --
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/man/man8/rdma-dev.8 b/man/man8/rdma-dev.8
index
network namespace and assign RDMA device to this
network namespace.
$ ip netns add foo
$ rdma dev set mlx5_1 netns foo
Reviewed-by: Leon Romanovsky
Signed-off-by: Parav Pandit
---
rdma/dev.c | 37 +
1 file changed, 37 insertions(+)
diff --git a/rdma/dev.c b
Device name on mdev bus is 36 characters long which follow standard uuid
RFC 4122.
This is probably the longest name that a kernel will return for a
device.
Hence increase the buffer size to 64 bytes.
Acked-by: Jiri Pirko
Signed-off-by: Parav Pandit
---
devlink/devlink.c | 4 ++--
1 file
> -Original Message-
> From: linux-rdma-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Leon Romanovsky
> Sent: Thursday, June 6, 2019 4:36 PM
> To: Doug Ledford ; Jason Gunthorpe
>
> Cc: Leon Romanovsky ; RDMA mailing list r...@vger.kernel.org>; Maor Gottlieb ; Mark Bloch
>
> -Original Message-
> From: Stephen Hemminger
> Sent: Thursday, June 6, 2019 8:49 PM
> To: Parav Pandit
> Cc: dsah...@gmail.com; netdev@vger.kernel.org; Jiri Pirko
>
> Subject: Re: [PATCH iproute2-next] devlink: Increase bus,device buffer size to
> 64 bytes
> From: Jason Wang
> Sent: Thursday, January 14, 2021 9:48 AM
>
> On 2021/1/7 上午11:48, Parav Pandit wrote:
> >
> >> From: Michael S. Tsirkin
> >> Sent: Tuesday, January 5, 2021 6:53 PM
> >>
> >> On Tue, Jan 05, 2021 at 12:30:15PM +
> From: Jakub Kicinski
> Sent: Thursday, January 14, 2021 11:13 PM
>
> On Wed, 13 Jan 2021 11:27:18 -0800 Saeed Mahameed wrote:
> > /**
> > * struct devlink_port_attrs - devlink port object
> > * @flavour: flavour of the port
> > @@ -114,6 +126,7 @@ struct devlink_port_attrs {
> >
> From: Jakub Kicinski
> Sent: Thursday, January 14, 2021 11:52 PM
>
> On Thu, 14 Jan 2021 17:53:09 + Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Thursday, January 14, 2021 11:13 PM
> > >
> > > On Wed,
> From: Jason Wang
> Sent: Friday, January 15, 2021 11:09 AM
>
>
> On 2021/1/14 下午3:58, Parav Pandit wrote:
> >
> >> From: Jason Wang
> >> Sent: Thursday, January 14, 2021 9:48 AM
> >>
> >> On 2021/1/7 上午11:48, Parav Pandit wro
Hi Michael, Jason,
> From: Jason Wang
> Sent: Friday, January 15, 2021 11:09 AM
>
>
> Thanks for the clarification. I think we'd better document the above in the
> patch that introduces the mac setting from management API.
Can we proceed with this patchset?
We like to progress next to iproute2
Hi David,
> From: David Ahern
> Sent: Wednesday, December 9, 2020 4:17 AM
>
> On 11/26/20 8:53 PM, Jason Wang wrote:
> > 1. Where does userspace vdpa tool reside which users can use?
> > Ans: vdpa tool can possibly reside in iproute2 [1] as it enables user
> > to create vdpa net devices.
>
> ip
> From: Jason Wang
> Sent: Tuesday, January 19, 2021 4:39 PM
> To: Parav Pandit ; Michael S. Tsirkin
> Cc: virtualizat...@lists.linux-foundation.org; Eli Cohen ;
> netdev@vger.kernel.org; Sean Mooney
> Subject: Re: [PATCH linux-next v3 6/6] vdpa_sim_net: Add support for user
&
> From: Jakub Kicinski
> Sent: Wednesday, January 20, 2021 8:58 AM
>
> On Mon, 18 Jan 2021 12:12:20 -0800 Saeed Mahameed wrote:
> > From: Parav Pandit
>
> Saeed, this is closed to being merged - when you post the next version please
> make sure to CC appropriate
> From: Virtualization On
> Behalf Of Parav Pandit
>
> > From: Jason Wang
> > Sent: Tuesday, January 19, 2021 4:39 PM
> > To: Parav Pandit ; Michael S. Tsirkin
> >
> > Cc: virtualizat...@lists.linux-foundation.org; Eli Cohen
> > ; netdev@vger
> From: Samudrala, Sridhar
> Sent: Friday, January 22, 2021 2:21 AM
>
> > $ devlink port show
> > pci/:06:00.0/65535: type eth netdev ens2f0np0 flavour physical
> > port 0 splittable false
> >
> > $ devlink port add pci/:06:00.0 flavour pcisf pfnum 0 sfnum 88
>
> Do we need to specify
> From: Samudrala, Sridhar
> Sent: Friday, January 22, 2021 2:23 AM
>
> On 1/21/2021 12:52 AM, Saeed Mahameed wrote:
> > From: Parav Pandit
> >
> > devlink port function can be in active or inactive state.
> > Allow users to get and set port function's
Extended devlink-port man page for synopsis, description and
example for setting devlink port function attribute.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
---
man/man8/devlink-port.8 | 21 +
1 file changed, 21 insertions(+)
diff --git a/man/man8/devlink-port.8 b
Add kernel headers to commit from kernel tree [1].
79991caf5202c7 ("vdpa_sim_net: Add support for user supported devices")
[1] https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git branch:
linux-next
Signed-off-by: Parav Pandit
---
include/uapi/linux/vdpa.h
mary:
Patch-1 adds kernel headers for vdpa subsystem
Patch-2 adds vdpa tool and its man pages
Kernel headers are from the vhost kernel tree [1] from branch linux-next.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git
Parav Pandit (2):
Add kernel headers
vdpa: Add vdpa tool
"max_vqs": 2,
"max_vq_size": 256
}
}
}
Delete the vdpa device after its use:
$ vdpa dev del foo2
Signed-off-by: Parav Pandit
---
Makefile| 2 +-
man/man8/vdpa-dev.8 | 96 +
man/man8/vdpa-mgmtdev.8 | 53 +++
man/man8/
> From: Samudrala, Sridhar
> Sent: Friday, December 11, 2020 9:42 AM
>
> On 12/8/2020 11:29 PM, sa...@kernel.org wrote:
> > From: Parav Pandit
> > Subfunction provide the same functionality as SRIOV but in a very
> > lightweight manner, please see the thoro
> From: Leon Romanovsky
> Sent: Sunday, December 13, 2020 5:39 PM
>
> On Sat, Dec 12, 2020 at 12:25:18PM -0800, Jakub Kicinski wrote:
> > On Fri, 11 Dec 2020 22:12:11 -0800 Saeed Mahameed wrote:
> > > Hi Dave, Jakub, Jason,
> > >
> > > This series form Parav was the theme of this mlx5 release c
> From: Alexander Duyck
> Sent: Tuesday, December 15, 2020 7:24 AM
>
> On Mon, Dec 14, 2020 at 1:49 PM Saeed Mahameed
> wrote:
> >
> > Hi Dave, Jakub, Jason,
> >
>
> Just to clarify a few things for myself. You mention virtualization and SR-IOV
> in your patch description but you cannot suppor
> From: Alexander Duyck
> Sent: Tuesday, December 15, 2020 9:47 PM
>
> On Mon, Dec 14, 2020 at 6:44 PM David Ahern wrote:
> >
> > On 12/14/20 6:53 PM, Alexander Duyck wrote:
> > >> example subfunction usage sequence:
> > >> ---
> > >> Change device to switchdev
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 4:58 AM
>
> On Tue, 15 Dec 2020 01:03:46 -0800 Saeed Mahameed wrote:
> > + * devlink_port_attrs_pci_sf_set - Set PCI SF port attributes
> > + *
> > + * @devlink_port: devlink port
> > + * @controller: associated controller number for
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 6:31 AM
>
> On Tue, 15 Dec 2020 01:03:57 -0800 Saeed Mahameed wrote:
> > +Subfunctions are lightweight functions that has parent PCI function
> > +on which it is deployed. Subfunctions are created and deployed in
> > +unit of 1. Unlik
> From: Alexander Duyck
> Sent: Wednesday, December 16, 2020 9:43 AM
>
> >
> > That is goal here. This is not about creating just a netdev, this is
> > about the whole kit: rdma, netdev, vdpa virtio-net, virtio-mdev.
>
> One issue is right now we are only seeing the rdma and netdev. It is kind
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 5:59 AM
>
> > +struct devlink_port_new_attrs {
> > + enum devlink_port_flavour flavour;
> > + unsigned int port_index;
> > + u32 controller;
> > + u32 sfnum;
> > + u16 pfnum;
>
> Oh. So you had the structure which actually g
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 6:08 AM
>
> On Tue, 15 Dec 2020 01:03:48 -0800 Saeed Mahameed wrote:
> > From: Parav Pandit
> >
> > devlink port function can be in active or inactive state.
> > Allow users to get and set por
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 6:14 AM
>
> On Tue, 15 Dec 2020 01:03:50 -0800 Saeed Mahameed wrote:
> > +static ssize_t sfnum_show(struct device *dev, struct device_attribute
> > +*attr, char *buf) {
> > + struct auxiliary_device *adev = container_of(dev, struct
>
> > Change functions to map SF's vport numbers to indices when accessing
> > the vports or representors arrays, and vice versa.
> >
> > Signed-off-by: Vu Pham
> > Signed-off-by: Parav Pandit
> > Reviewed-by: Roi Dayan
> > Signed-off-by: Saeed Mahame
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 6:21 AM
>
> On Tue, 15 Dec 2020 01:03:54 -0800 Saeed Mahameed wrote:
> > To handle SF port management outside of the eswitch as independent
> > software layer, introduce eswitch notifier APIs so that upper layer
> > who wish to suppor
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 6:28 AM
>
> On Tue, 15 Dec 2020 01:03:56 -0800 Saeed Mahameed wrote:
> > +PCI controllers
> > +---
> > +In most cases a PCI device has only one controller. A controller
> > +consists of potentially multiple physical and vir
> From: Jakub Kicinski
> Sent: Wednesday, December 16, 2020 9:36 PM
>
> On Wed, 16 Dec 2020 04:13:51 -0500 Michael S. Tsirkin wrote:
> > > > > 3. Why not use ioctl() interface?
> > > >
> > > > Obviously I'm gonna ask you - why can't you use devlink?
> > > >
> > > This was considered.
> > > Howeve
> From: Jakub Kicinski
> Sent: Thursday, December 17, 2020 5:42 AM
>
> On Wed, 16 Dec 2020 05:19:15 + Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Wednesday, December 16, 2020 6:14 AM
> > >
> > > On Tue, 15 Dec 2020 01:0
> From: Jakub Kicinski
> Sent: Thursday, December 17, 2020 5:39 AM
>
> On Wed, 16 Dec 2020 05:15:04 + Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Wednesday, December 16, 2020 6:08 AM
> > >
> > > On Tue, 15 Dec 2020 01:03:48
> From: Michael S. Tsirkin
> Sent: Thursday, December 17, 2020 1:28 AM
>
> On Wed, Dec 16, 2020 at 04:54:37PM +, Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Wednesday, December 16, 2020 9:36 PM
> > >
> > > On Wed, 16
> From: Alexander Duyck
> Sent: Friday, December 18, 2020 8:41 AM
>
> On Thu, Dec 17, 2020 at 5:30 PM David Ahern wrote:
> >
> > On 12/16/20 3:53 PM, Alexander Duyck wrote:
> The problem is PCIe DMA wasn't designed to function as a network switch
> fabric and when we start talking about a 400Gb
> From: Parav Pandit
> Sent: Friday, December 18, 2020 10:51 AM
>
> > From: Alexander Duyck
> > Sent: Friday, December 18, 2020 8:41 AM
> >
> > On Thu, Dec 17, 2020 at 5:30 PM David Ahern wrote:
> > >
> > > On 12/16/20 3:53 PM, Alexan
> From: Alexander Duyck
> Sent: Friday, December 18, 2020 9:31 PM
>
> On Thu, Dec 17, 2020 at 9:20 PM Parav Pandit wrote:
> >
> >
> > > From: Alexander Duyck
> > > Sent: Friday, December 18, 2020 8:41 AM
> > >
> > > On Thu, Dec 17,
> From: Jakub Kicinski
> Sent: Saturday, December 19, 2020 1:18 AM
> So the SF is always created from the eswitch controller side?
> How does the host side look?
>
Host side creates the auxiliary device for the SF.
$ ls -l /sys/bus/auxiliary/devices/
mlx5_core.sf.4 ->
../../../devices/pci
> From: Jakub Kicinski
> Sent: Saturday, December 19, 2020 1:29 AM
>
> On Thu, 17 Dec 2020 05:23:10 + Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Thursday, December 17, 2020 5:42 AM
> > >
> > > On Wed, 16 Dec 2020 05:19:15
> From: Jakub Kicinski
> Sent: Saturday, December 19, 2020 1:21 AM
>
> On Thu, 17 Dec 2020 05:46:45 + Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Thursday, December 17, 2020 5:39 AM
> > >
> > > On Wed, 16 Dec 2020 05:15:04
Added documentation for devlink port and port function related commands.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
---
.../networking/devlink/devlink-port.rst | 102 ++
Documentation/networking/devlink/index.rst| 1 +
2 files changed, 103 insertions
Extended devlink-port man page for synopsis, description and
example for setting devlink port function attribute.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
---
man/man8/devlink-port.8 | 21 +
1 file changed, 21 insertions(+)
diff --git a/man/man8/devlink-port.8 b
> From: Jacob Keller
> Sent: Tuesday, December 1, 2020 12:59 AM
>
>
>
> On 11/30/2020 8:41 AM, Parav Pandit wrote:
> > Added documentation for devlink port and port function related commands.
> >
> > Signed-off-by: Parav Pandit
> > Reviewed-by:
Added documentation for devlink port and port function related commands.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
---
Changelog:
v1->v2:
- Removed duplicate table entries for DEVLINK_PORT_FLAVOUR_VIRTUAL.
- replaced 'consist of' to 'consisting'
- changed
> From: Yongji Xie
> Sent: Tuesday, December 1, 2020 3:26 PM
>
> On Tue, Dec 1, 2020 at 2:25 PM Jason Wang wrote:
> >
> >
> > On 2020/11/30 下午3:07, Yongji Xie wrote:
> > >>> Thanks for adding me, Jason!
> > >>>
> > >>> Now I'm working on a v2 patchset for VDUSE (vDPA Device in
> > >>> Userspac
> From: Yongji Xie
> Sent: Tuesday, December 1, 2020 7:49 PM
>
> On Tue, Dec 1, 2020 at 7:32 PM Parav Pandit wrote:
> >
> >
> >
> > > From: Yongji Xie
> > > Sent: Tuesday, December 1, 2020 3:26 PM
> > >
> > > On Tue, Dec 1, 20
> From: Jakub Kicinski
> Sent: Wednesday, December 2, 2020 7:05 AM
>
> On Mon, 30 Nov 2020 22:00:25 +0200 Parav Pandit wrote:
> > Added documentation for devlink port and port function related commands.
> >
> > Signed-off-by: Parav Pandi
> From: Yongji Xie
> Sent: Wednesday, December 2, 2020 9:00 AM
>
> On Tue, Dec 1, 2020 at 11:59 PM Parav Pandit wrote:
> >
> >
> >
> > > From: Yongji Xie
> > > Sent: Tuesday, December 1, 2020 7:49 PM
> > >
> From: Jason Wang
> Sent: Wednesday, December 2, 2020 11:21 AM
>
> On 2020/12/2 下午12:53, Parav Pandit wrote:
> >
> >> From: Yongji Xie
> >> Sent: Wednesday, December 2, 2020 9:00 AM
> >>
> >> On Tue, Dec 1, 2020 at 11:59 PM Pa
> From: Yongji Xie
> Sent: Wednesday, December 2, 2020 2:52 PM
>
> On Wed, Dec 2, 2020 at 12:53 PM Parav Pandit wrote:
> >
> >
> >
> > > From: Yongji Xie
> > > Sent: Wednesday, December 2, 2020 9:00 AM
> > >
Added documentation for devlink port and port function related commands.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
Reviewed-by: Jacob Keller
---
Changelog:
v2->v3:
- rephased many lines
- first paragraph now describe devlink port
- instead of saying PCI device/function, using
> From: Randy Dunlap
> Sent: Thursday, December 3, 2020 7:57 AM
>
> Hi--
>
> On 12/2/20 5:53 AM, Parav Pandit wrote:
> > Added documentation for devlink port and port function related commands.
> >
> > Signed-off-by: Parav Pandit
> > Reviewed-by:
> From: David Ahern
> Sent: Thursday, December 3, 2020 9:12 AM
>
> On 11/30/20 9:47 AM, Parav Pandit wrote:
> > Extended devlink-port man page for synopsis, description and example
> > for setting devlink port function attribute.
> >
> > Signed-off-by: Parav
Added documentation for devlink port and port function related commands.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
Reviewed-by: Jacob Keller
---
Changelog:
v3->v4:
- changed 'exist' to 'exists'
- added 'an' eswitch
- changed 'can have one
Oh
> > > Co-developed-by: Leon Romanovsky
> > > Signed-off-by: Kiran Patil
> > > Signed-off-by: Ranjani Sridharan
> > > Signed-off-by: Fred Oh
> > > Signed-off-by: Leon Romanovsky
> > > Signed-off-by: Dave Ertman
> > > Reviewed
> From: Jakub Kicinski
> Sent: Sunday, December 6, 2020 1:57 AM
>
> On Thu, 3 Dec 2020 20:02:55 +0200 Parav Pandit wrote:
> > Added documentation for devlink port and port function related commands.
> >
> > Signed-off-by: Parav Pandit
> > Reviewed-by
> From: David Ahern
> Sent: Monday, December 7, 2020 8:19 AM
>
> On 11/12/20 12:24 PM, Parav Pandit wrote:
> > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/Kconfig
> > b/drivers/net/ethernet/mellanox/mlx5/core/Kconfig
> > index 485478979b1a..10dfaf671c9
> From: Jakub Kicinski
> Sent: Monday, December 7, 2020 11:10 PM
>
> On Mon, 7 Dec 2020 04:46:14 + Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Sunday, December 6, 2020 1:57 AM
> > >
> > > On Thu, 3 Dec 2020 20:02:55 +0200 Par
> From: Jakub Kicinski
> Sent: Tuesday, December 8, 2020 1:43 AM
>
> On Mon, 7 Dec 2020 20:00:53 + Parav Pandit wrote:
> > > > > Each port is a logically separate ingress/egress point of the device.
> > > > >
> > > > > ?
> > &
Added documentation for devlink port and port function related commands.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
Reviewed-by: Jacob Keller
---
Changelog:
v4->v5:
- described logically ingress and egress point of devlink port
- removed networking from devlink port descript
Hi Randy,
> From: Randy Dunlap
> Sent: Tuesday, December 8, 2020 6:45 AM
>
> > +An eswitch on the PCI device may suppport ports of multiple controllers.
>
> support
>
> > +
> > +An example view of two controller systems::
>
> I think that this is
>
> An
> From: Randy Dunlap
> Sent: Tuesday, December 8, 2020 6:45 AM
>
> Hi--
>
> On 12/7/20 2:13 PM, Parav Pandit wrote:
> > Added documentation for devlink port and port function related
> commands.
> >
> > Signed-off-by: Parav Pandit
> > Reviewed-
> From: David Ahern
> Sent: Wednesday, November 18, 2020 9:51 PM
>
> On 11/12/20 12:24 PM, Parav Pandit wrote:
> > Extended devlink interface for the user to add and delete port.
> > Extend devlink to connect user requests to driver to add/delete such
> > port in t
> From: David Ahern
> Sent: Wednesday, November 18, 2020 11:33 PM
>
>
> With Connectx-4 Lx for example the netdev can have at most 63 queues
> leaving 96 cpu servers a bit short - as an example of the limited number of
> queues that a nic can handle (or currently exposes to the OS not sure whi
> From: Saeed Mahameed
> Sent: Thursday, November 19, 2020 11:42 AM
>
> From how this discussion is going, i think you are right, we need to clarify
> what we are doing in a more high level simplified and generic documentation
> to give some initial context, Parav, let's add the missing documen
> From: Jakub Kicinski
> Sent: Friday, November 20, 2020 7:05 AM
>
> On Wed, 18 Nov 2020 22:12:22 -0800 Saeed Mahameed wrote:
> > > Right, devices of other subsystems are fine, I don't care.
> >
> > But a netdev will be loaded on SF automatically just through the
> > current driver design and m
> From: Jakub Kicinski
> Sent: Friday, November 20, 2020 9:05 AM
>
> On Thu, 19 Nov 2020 10:00:17 -0400 Jason Gunthorpe wrote:
> > Finally, in the mlx5 model VDPA is just an "application". It asks the
> > device to create a 'RDMA' raw ethernet packet QP that is uses rings
> > formed in the virtio
same net namespace
Parav Pandit (2):
devlink: Hold rtnl lock while reading netdev attributes
devlink: Make sure devlink instance and port are in same net namespace
net/core/devlink.c | 33 +++--
1 file changed, 27 insertions(+), 6 deletions(-)
--
2.26.2
use case, devlink port query may read stale netdev
attributes.
Fix it by reading them under rtnl lock.
Fixes: bfcd3a466172 ("Introduce devlink infrastructure")
Signed-off-by: Parav Pandit
---
net/core/devlink.c | 30 --
1 file changed, 24 insertions(+), 6
vlink: allow to change namespaces during reload")
Signed-off-by: Parav Pandit
---
net/core/devlink.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/net/core/devlink.c b/net/core/devlink.c
index 6135ef5972ce..2e1b0e82a102 100644
--- a/net/core/devlink.c
+++ b/net/core
Hi David,
> From: David Ahern
> Sent: Wednesday, November 25, 2020 11:04 AM
>
> On 11/18/20 10:57 PM, Saeed Mahameed wrote:
> >
> > We are not slicing up any queues, from our HW and FW perspective SF ==
> > VF literally, a full blown HW slice (Function), with isolated control
> > and data plane
> From: Jakub Kicinski
> Sent: Wednesday, November 25, 2020 3:59 AM
>
> On Sun, 22 Nov 2020 08:12:56 +0200 Parav Pandit wrote:
> > A netdevice of a devlink port can be moved to different net namespace
> > than its parent devlink instance.
> > This scenario occ
same net namespace
---
Changelog:
v1->v2:
- avoided refactoring the port fill attribute routine
Parav Pandit (2):
devlink: Hold rtnl lock while reading netdev attributes
devlink: Make sure devlink instance and port are in same net namespace
net/core/devlink.c | 7 ++-
1 file ch
stale netdev attributes.
Fix it by reading them under rtnl lock.
Fixes: bfcd3a466172 ("Introduce devlink infrastructure")
Signed-off-by: Parav Pandit
---
Changelog:
v1->v2:
- avoided refactoring the code
- updated commit log
---
net/core/devlink.c | 4
1 file changed, 4 insert
vlink: allow to change namespaces during reload")
Signed-off-by: Parav Pandit
---
Changelog:
v1->v2:
- avoided refactor code from previous patch
---
net/core/devlink.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/net/core/devlink.c b/net/core/devlink.c
index
> From: Jakub Kicinski
> Sent: Wednesday, November 25, 2020 10:00 PM
>
> On Wed, 25 Nov 2020 07:13:40 + Parav Pandit wrote:
> > > Maybe even add a check that drivers
> > > which support reload set namespace local on their netdevs.
> > This will
> From: Jakub Kicinski
> Sent: Wednesday, November 25, 2020 11:12 PM
>
> On Wed, 25 Nov 2020 17:21:41 + Parav Pandit wrote:
> > > From: Jakub Kicinski
> > > Sent: Wednesday, November 25, 2020 10:00 PM
> > >
> > > On Wed, 25 Nov 2020 07:13:4
Cited commit in fixes tag overwrites the port attributes for the
registered port.
Avoid such error by checking registered flag before setting attributes.
Signed-off-by: Parav Pandit
Reviewed-by: Jiri Pirko
---
net/core/devlink.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions
DKIM-Signature: v a
Cited commit in fixes tag overwrites the port attributes for the
registered port.
Avoid such error by checking registered flag before setting attributes.
Fixes: 71ad8d55f8e5 ("devlink: Replace devlink_port_attrs_set parameters with a
struct")
Signed-off-by: Parav Pandit
Reviewed-by:
JSON format.
$ vdpa parentdev show -jp
{
"show": {
"vdpasim": {
"supported_classes": [ "net" ]
}
}
}
Signed-off-by: Parav Pandit
Reviewed-by: Eli Cohen
Reviewed-by: Jason Wang
---
drivers/vdpa/Kconfig
Add missing comment for number of virtqueue.
Signed-off-by: Parav Pandit
Reviewed-by: Eli Cohen
Acked-by: Jason Wang
---
include/linux/vdpa.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h
index 30bc7a7223bb..0fefeb976877 100644
--- a/include
quot;max_vqs": 2,
"max_vq_size": 256
}
}
}
Delete the vdpa device after its use:
$ vdpa dev del foo2
Signed-off-by: Parav Pandit
Reviewed-by: Eli Cohen
Acked-by: Jason Wang
---
drivers/vdpa/vdpa_sim/vdpa_sim.c | 81 +++
Add the ability to add and delete a vdpa device.
Examples:
Create a vdpa device of type network named "foo2" from
the parent device vdpasim:
$ vdpa dev add parentdev vdpasim type net name foo2
Delete the vdpa device after its use:
$ vdpa dev del foo2
Signed-off-by: Parav Pandit
R
device can be removed from the list after its deleted.
Signed-off-by: Parav Pandit
Reviewed-by: Eli Cohen
Acked-by: Jason Wang
---
drivers/vdpa/ifcvf/ifcvf_main.c | 2 +-
drivers/vdpa/mlx5/net/mlx5_vnet.c | 2 +-
drivers/vdpa/vdpa.c | 36 +++
drivers
vdpa doesn't have any specific need to define start and end range of the
device index.
Hence use the simper version of the ida allocator.
Signed-off-by: Parav Pandit
Reviewed-by: Eli Cohen
Acked-by: Jason Wang
---
drivers/vdpa/vdpa.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
"foo2": {
"type": "network",
"parentdev": "vdpasim",
"vendor_id": 0,
"max_vqs": 2,
"max_vq_size": 256
}
}
}
Signed-off-by: Parav Pan
removed that belong to a parent.
[1] git://git.kernel.org/pub/scm/network/iproute2/iproute2-next.git
Next steps:
---
(a) Post this patchset and iproute2/vdpa inclusion, remaining two drivers
will be coverted to support vdpa tool instead of creating unmanaged default
device on driver lo
1 - 100 of 530 matches
Mail list logo
