From: Parav Pandit <email@example.com> To: Jakub Kicinski <firstname.lastname@example.org>, Saeed Mahameed <email@example.com> Cc: "firstname.lastname@example.org" <email@example.com>, "firstname.lastname@example.org" <email@example.com>, "firstname.lastname@example.org" <email@example.com>, Jiri Pirko <firstname.lastname@example.org>, Jason Gunthorpe <email@example.com>, "firstname.lastname@example.org" <email@example.com>, Leon Romanovsky <firstname.lastname@example.org>, "email@example.com" <firstname.lastname@example.org> Subject: RE: [PATCH net-next 00/13] Add mlx5 subfunction support Date: Tue, 17 Nov 2020 04:08:57 +0000 [thread overview] Message-ID: <BY5PR12MB43229F23C101AFBCD2971534DCE20@BY5PR12MB4322.namprd12.prod.outlook.com> (raw) In-Reply-To: <email@example.com> > From: Jakub Kicinski <firstname.lastname@example.org> > Sent: Tuesday, November 17, 2020 7:28 AM > > On Mon, 16 Nov 2020 16:06:02 -0800 Saeed Mahameed wrote: > > > > Subfunction support is discussed in detail in RFC  and . > > > > RFC  and extension  describes requirements, design, and > > > > proposed plumbing using devlink, auxiliary bus and sysfs for > > > > systemd/udev support. > > > > > > So we're going to have two ways of adding subdevs? Via devlink and > > > via the new vdpa netlink thing? Nop. Subfunctions (subdevs) are added only one way, i.e. devlink port as settled in RFC . Just to refresh all our memory, we discussed and settled on the flow in ; RFC  followed this discussion. vdpa tool of  can add one or more vdpa device(s) on top of already spawned PF, VF, SF device. > > > > Via devlink you add the Sub-function bus device - think of it as > > spawning a new VF - but has no actual characteristics > > (netdev/vpda/rdma) "yet" until user admin decides to load an interface > > on it via aux sysfs. > > By which you mean it doesn't get probed or the device type is not set (IOW it can > still become a block device or netdev depending on the vdpa request)? > > > Basically devlink adds a new eswitch port (the SF port) and loading > > the drivers and the interfaces is done via the auxbus subsystem only > > after the SF is spawned by FW. > > But why? > > Is this for the SmartNIC / bare metal case? The flow for spawning on the local > host gets highly convoluted. > The flow of spawning for (a) local host or (b) for external host controller from smartnic is same. $ devlink port add.. [..] Followed by $ devlink port function set state... Only change would be to specify the destination where to spawn it. (controller number, pf, sf num etc) Please refer to the detailed examples in individual patch. Patch 12 and 13 mostly covers the complete view. > > > Also could you please wrap your code at 80 chars? > > > > I prefer no to do this in mlx5, in mlx5 we follow a 95 chars rule. > > But if you insist :) .. > > Oh yeah, I meant the devlink patches! May I ask why? Past few devlink patches  followed 100 chars rule. When did we revert back to 80? If so, any pointers to the thread for 80? checkpatch.pl with --strict mode didn't complain me when I prepared the patches.  https://lore.kernel.org/netdev/20200519092258.GF4655@nanopsycho/  https://lore.kernel.org/netdev/20200324132044.GI20941@ziepe.ca/  https://lists.linuxfoundation.org/pipermail/virtualization/2020-November/050623.html  commits dc64cc7c6310, 77069ba2e3ad, a1e8ae907c8d, 2a916ecc4056, ba356c90985d
next prev parent reply other threads:[~2020-11-17 4:09 UTC|newest] Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-12 19:24 Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 01/13] devlink: Prepare code to fill multiple port function attributes Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 02/13] devlink: Introduce PCI SF port flavour and port attribute Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 03/13] devlink: Support add and delete devlink port Parav Pandit 2020-11-18 16:21 ` David Ahern 2020-11-18 17:02 ` Parav Pandit 2020-11-18 18:03 ` David Ahern 2020-11-18 18:38 ` Jason Gunthorpe 2020-11-18 19:36 ` David Ahern 2020-11-18 20:42 ` Jason Gunthorpe 2020-11-18 19:22 ` Parav Pandit 2020-11-19 0:41 ` Jacob Keller 2020-11-19 1:17 ` David Ahern 2020-11-19 1:56 ` Samudrala, Sridhar 2020-11-19 0:52 ` Jacob Keller 2020-11-12 19:24 ` [PATCH net-next 04/13] devlink: Support get and set state of port function Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 05/13] devlink: Avoid global devlink mutex, use per instance reload lock Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 06/13] devlink: Introduce devlink refcount to reduce scope of global devlink_mutex Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 07/13] net/mlx5: SF, Add auxiliary device support Parav Pandit 2020-12-07 2:48 ` David Ahern 2020-12-07 4:53 ` Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 08/13] net/mlx5: SF, Add auxiliary device driver Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 09/13] net/mlx5: E-switch, Prepare eswitch to handle SF vport Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 10/13] net/mlx5: E-switch, Add eswitch helpers for " Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 11/13] net/mlx5: SF, Add SF configuration hardware commands Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 12/13] net/mlx5: SF, Add port add delete functionality Parav Pandit 2020-11-12 19:24 ` [PATCH net-next 13/13] net/mlx5: SF, Port function state change support Parav Pandit 2020-11-16 22:52 ` [PATCH net-next 00/13] Add mlx5 subfunction support Jakub Kicinski 2020-11-17 0:06 ` Saeed Mahameed 2020-11-17 1:58 ` Jakub Kicinski 2020-11-17 4:08 ` Parav Pandit [this message] 2020-11-17 17:11 ` Jakub Kicinski 2020-11-17 18:49 ` Jason Gunthorpe 2020-11-19 2:14 ` Jakub Kicinski 2020-11-19 4:35 ` David Ahern 2020-11-19 5:57 ` Saeed Mahameed 2020-11-20 1:31 ` Jakub Kicinski 2020-11-25 5:33 ` David Ahern 2020-11-25 6:00 ` Parav Pandit 2020-11-25 14:37 ` David Ahern 2020-11-20 1:29 ` Jakub Kicinski 2020-11-20 17:58 ` Alexander Duyck 2020-11-20 19:04 ` Samudrala, Sridhar 2020-11-23 21:51 ` Saeed Mahameed 2020-11-24 7:01 ` Jason Wang 2020-11-24 7:05 ` Jason Wang 2020-11-19 6:12 ` Saeed Mahameed 2020-11-19 8:25 ` Parav Pandit 2020-11-20 1:35 ` Jakub Kicinski 2020-11-20 3:34 ` Parav Pandit 2020-11-17 18:50 ` Parav Pandit 2020-11-19 2:23 ` Jakub Kicinski 2020-11-19 6:22 ` Saeed Mahameed 2020-11-19 14:00 ` Jason Gunthorpe 2020-11-20 3:35 ` Jakub Kicinski 2020-11-20 3:50 ` Parav Pandit 2020-11-20 16:16 ` Jason Gunthorpe
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=BY5PR12MB43229F23C101AFBCD2971534DCE20@BY5PR12MB4322.namprd12.prod.outlook.com \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='RE: [PATCH net-next 00/13] Add mlx5 subfunction support' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
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).