All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Liang, Cunming" <cunming.liang@intel.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: "Wang, Haiyue" <haiyue.wang@intel.com>,
	"Singhai, Anjali" <anjali.singhai@intel.com>,
	"Samudrala, Sridhar" <sridhar.samudrala@intel.com>,
	"Sarangam, Parthasarathy" <parthasarathy.sarangam@intel.com>,
	Jamal Hadi Salim <jhs@mojatatu.com>,
	"shm@cumulusnetworks.com" <shm@cumulusnetworks.com>,
	Tom Herbert <tom@herbertland.com>,
	"Venkataramanan, Anirudh" <anirudh.venkataramanan@intel.com>,
	"davem@davemloft.net" <davem@davemloft.net>,
	"nhorman@redhat.com" <nhorman@redhat.com>,
	"sassmann@redhat.com" <sassmann@redhat.com>,
	"Bowers, AndrewX" <andrewx.bowers@intel.com>,
	"Kirsher, Jeffrey T" <jeffrey.t.kirsher@intel.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"Nguyen, Anthony L" <anthony.l.nguyen@intel.com>,
	"Lu, Nannan" <nannan.lu@intel.com>,
	"Brandeburg, Jesse" <jesse.brandeburg@intel.com>,
	"Parikh, Neerav" <neerav.parikh@intel.com>
Subject: RE: [net-next 1/5] ice: add the virtchnl handler for AdminQ command
Date: Wed, 26 Aug 2020 02:53:51 +0000	[thread overview]
Message-ID: <DM6PR11MB25546FA7A35A763806CFE2FFF9540@DM6PR11MB2554.namprd11.prod.outlook.com> (raw)
In-Reply-To: <BN8PR11MB37951AC65BFBDF0E9BFBF86FF74B0@BN8PR11MB3795.namprd11.prod.outlook.com>



> -----Original Message-----
> From: Wang, Haiyue <haiyue.wang@intel.com>
> Sent: Wednesday, August 5, 2020 9:06 AM
> To: Jakub Kicinski <kuba@kernel.org>
> Cc: Tom Herbert <tom@herbertland.com>; Venkataramanan, Anirudh
> <anirudh.venkataramanan@intel.com>; davem@davemloft.net;
> nhorman@redhat.com; sassmann@redhat.com; Bowers, AndrewX
> <andrewx.bowers@intel.com>; Kirsher, Jeffrey T <jeffrey.t.kirsher@intel.com>;
> netdev@vger.kernel.org; Nguyen, Anthony L <anthony.l.nguyen@intel.com>; Lu,
> Nannan <nannan.lu@intel.com>; Liang, Cunming <cunming.liang@intel.com>
> Subject: RE: [net-next 1/5] ice: add the virtchnl handler for AdminQ command
> 
> > -----Original Message-----
> > From: Jakub Kicinski <kuba@kernel.org>
> > Sent: Tuesday, August 4, 2020 04:46
> > To: Wang, Haiyue <haiyue.wang@intel.com>
> > Cc: Tom Herbert <tom@herbertland.com>; Venkataramanan, Anirudh
> > <anirudh.venkataramanan@intel.com>;
> > davem@davemloft.net; nhorman@redhat.com; sassmann@redhat.com;
> Bowers,
> > AndrewX <andrewx.bowers@intel.com>; Kirsher, Jeffrey T
> > <jeffrey.t.kirsher@intel.com>; netdev@vger.kernel.org; Nguyen, Anthony
> > L <anthony.l.nguyen@intel.com>; Lu, Nannan <nannan.lu@intel.com>;
> > Liang, Cunming <cunming.liang@intel.com>
> > Subject: Re: [net-next 1/5] ice: add the virtchnl handler for AdminQ
> > command
> >
> > On Mon, 3 Aug 2020 10:39:52 +0000 Wang, Haiyue wrote:
> > > > In this case, I'm guessing, Intel can reuse RTE flow -> AQ code
> > > > written to run on PFs on the special VF.
> > > >
> > > > This community has selected switchdev + flower for programming flows.
> > > > I believe implementing flower offloads would solve your use case,
> > > > and at the same time be most beneficial to the netdev community.

Jakub, thanks for the feedback. We proposed the previous solution in our eagerness to satisfy customers who were using mature, and validated (for their data centers) host kernels and still enable rapid adaption to new network control planes.

When revisiting the real problems we were facing, basically we're looking for a rapid self-iteration control plane independent of a mature deployed host kernel. Definitely kernel is the most suitable path for a control plane and we need to enhance the kernel to add the missing piece required for this solution. Best way to achieve this is allow such use cases is to deploy control plane on latest kernel running as virtual machine. We shared some thoughts on netdev 0x14 workshop, attached link as https://github.com/seaturn/switchdev-trust-vf/blob/master/netconf-workshop.pdf.

As a follow-up, we'll continue work on tc-generic proposal and look for programming rate improvement. As an independent effort of enhancing tc-generic/switchdev on trusted VF, delegating device specific capabilities (e.g. eswitch) to an assignable trusted VF brings all the benefit of a separated kernel to upgrade up-to-date features in the pace of applications, and always prevent host stack from any connectivity (e.g. stable access) issues.

Will be happy to answer any queries...and thank you for guiding us in the right path.

> > >
> > > Jakub,
> > >
> > > Thanks, I deep into the switchdev, it is kernel software bridge for
> > > hardware offload, and each port is registered with register_netdev.
> > > So this solution is not suitable for current case: VF can be assigned to VMs.
> >
> > You may be missing the concept of a representor.
> >
> 
> I found the concept, thanks, missed it!

  parent reply	other threads:[~2020-08-26  2:53 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-13 17:43 [net-next 0/5][pull request] 100GbE Intel Wired LAN Driver Updates 2020-07-13 Tony Nguyen
2020-07-13 17:43 ` [net-next 1/5] ice: add the virtchnl handler for AdminQ command Tony Nguyen
2020-07-13 22:48   ` Jakub Kicinski
2020-07-14  1:29     ` Wang, Haiyue
2020-07-14 18:24       ` Jakub Kicinski
2020-07-15  1:17         ` Wang, Haiyue
2020-07-15 18:03           ` Jakub Kicinski
2020-07-15 18:18             ` Wang, Haiyue
2020-07-23  0:37             ` Venkataramanan, Anirudh
2020-07-23  1:07               ` Jakub Kicinski
2020-07-25 16:39                 ` Tom Herbert
2020-07-27 23:04                   ` Jakub Kicinski
2020-08-03 10:39                     ` Wang, Haiyue
2020-08-03 20:45                       ` Jakub Kicinski
2020-08-05  1:06                         ` Wang, Haiyue
2020-08-26  2:45                           ` Liang, Cunming
2020-08-26  2:53                           ` Liang, Cunming [this message]
2020-07-13 17:43 ` [net-next 2/5] ice: add DCF cap negotiation and state machine Tony Nguyen
2020-07-13 17:43 ` [net-next 3/5] ice: support to get the VSI mapping Tony Nguyen
2020-07-13 17:43 ` [net-next 4/5] ice: enable DDP package info querying Tony Nguyen
2020-07-13 17:43 ` [net-next 5/5] ice: add switch rule management for DCF Tony Nguyen

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=DM6PR11MB25546FA7A35A763806CFE2FFF9540@DM6PR11MB2554.namprd11.prod.outlook.com \
    --to=cunming.liang@intel.com \
    --cc=andrewx.bowers@intel.com \
    --cc=anirudh.venkataramanan@intel.com \
    --cc=anjali.singhai@intel.com \
    --cc=anthony.l.nguyen@intel.com \
    --cc=davem@davemloft.net \
    --cc=haiyue.wang@intel.com \
    --cc=jeffrey.t.kirsher@intel.com \
    --cc=jesse.brandeburg@intel.com \
    --cc=jhs@mojatatu.com \
    --cc=kuba@kernel.org \
    --cc=nannan.lu@intel.com \
    --cc=neerav.parikh@intel.com \
    --cc=netdev@vger.kernel.org \
    --cc=nhorman@redhat.com \
    --cc=parthasarathy.sarangam@intel.com \
    --cc=sassmann@redhat.com \
    --cc=shm@cumulusnetworks.com \
    --cc=sridhar.samudrala@intel.com \
    --cc=tom@herbertland.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.