DPDK patches and discussions
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas@monjalon.net>
To: Alejandro Lucero <alejandro.lucero@netronome.com>
Cc: Ferruh Yigit <ferruh.yigit@intel.com>, dev <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH] net/nfp: add CPP bridge as service
Date: Mon, 14 Jan 2019 19:54:54 +0100	[thread overview]
Message-ID: <11471688.z4v4DX61lF@xps> (raw)
In-Reply-To: <CAD+H990ciHqFrGD6zBF3jmtZWNqJQntN94uV1nDLDG-rYkxOjA@mail.gmail.com>

14/01/2019 19:29, Alejandro Lucero:
> On Mon, Jan 14, 2019 at 6:22 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> 
> > On 1/14/2019 6:00 PM, Alejandro Lucero wrote:
> > >
> > >
> > > On Mon, Jan 14, 2019 at 10:40 AM Ferruh Yigit <ferruh.yigit@intel.com
> > > <mailto:ferruh.yigit@intel.com>> wrote:
> > >
> > >     On 1/13/2019 9:41 PM, Thomas Monjalon wrote:
> > >     > 11/01/2019 17:42, Ferruh Yigit:
> > >     >> On 1/11/2019 1:25 PM, Alejandro Lucero wrote:
> > >     >>> The Netronome's Network Flow Processor chip is highly
> > programmable
> > >     >>> with the goal of processing packets at high speed. Processing
> > units
> > >     >>> and other chip components are available from the host through the
> > >     >>> PCIe CPP(Command Push Pull bus) interface. The NFP PF PMD
> > configures
> > >     >>> a CPP handler for setting up and working with vNICs, perform
> > actions
> > >     >>> like link up or down, or accessing extended stats from the MAC
> > component.
> > >     >>>
> > >     >>> There exist NFP host tools which access the NFP components for
> > >     >>> programming and debugging but they require the CPP interface.
> > When the
> > >     >>> PMD is bound to the PF, the DPDK app owns the CPP interface, so
> > these
> > >     >>> host tools can not access the NFP through other means like NFP
> > kernel
> > >     >>> drivers.
> > >     >>>
> > >     >>> This patch adds a CPP bridge using the rte_service API which can
> > be
> > >     >>> enabled by a DPDK app. Interestingly, DPDK clients like OVS will
> > not
> > >     >>> enable specific service cores, but this can be performed with a
> > >     >>> secondary process specifically enabling this CPP bridge service
> > and
> > >     >>> therefore giving access to the NFP to those host tools.
> > >     >>>
> > >     >>> v2:
> > >     >>>  - Avoid printfs for debugging
> > >     >>>  - fix compilation problems for powerpc
> > >     >>>
> > >     >>> Signed-off-by: Alejandro Lucero <alejandro.lucero@netronome.com
> > >     <mailto:alejandro.lucero@netronome.com>>
> > >     >>
> > >     >> Applied to dpdk-next-net/master, thanks.
> > >     >
> > >     > It does not compile with 32-bit toolchain.
> > >     >
> > >     > Please check the occurences of %lu, thanks.
> > >
> > >     Hi Thomas,
> > >
> > >     We aware the build error, but let it because nfp doesn't support
> > 32-bit.
> > >
> > >     But I just recognized that it is enabled by default on 32-bit
> > default configs,
> > >     we should disable them.
> > >
> > >
> > >     Hi Alejandro,
> > >
> > >     Can you please disable nfp driver explicitly on
> > >     'defconfig_i686-native-linuxapp-*' config files, perhaps also on
> > >     'defconfig_x86_x32-native-linuxapp-gcc' too?
> > >
> > >     I will drop the existing patch from next-net.
> > >
> > >
> > > Ok. I'll do asap.
> > >
> > >
> > >     And if it is possible to fix the build error, specially if it is
> > just for %lu of
> > >     the logging, I prefer the fix against the config update, but it is
> > up to you.
> > >
> > >
> > > I did not see any logging error/warning when compiling nor any when using
> > > checkpatch. I have used a gcc 7.3.1 (Ubuntu) and a 8.2.1 (RH). What are
> > you
> > > using for triggering such error?
> >
> > Using 'i686-native-linuxapp-gcc' config, which is for 32-bit, gives
> > following
> > build error [1] with this patch.
> >
> >
> OK. But after the patch I have just sent for removing NFP PMD from 32 bits
> builds, nothing is really needed then. Right?
> 
> If so, should I send the patch again about the CPP bridge or you can redo
> it?

I will re-apply it on master.

  reply	other threads:[~2019-01-14 18:54 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-11 13:25 Alejandro Lucero
2019-01-11 16:42 ` Ferruh Yigit
2019-01-13 21:41   ` Thomas Monjalon
2019-01-14 10:40     ` Ferruh Yigit
2019-01-14 18:00       ` Alejandro Lucero
2019-01-14 18:22         ` Ferruh Yigit
2019-01-14 18:29           ` Alejandro Lucero
2019-01-14 18:54             ` Thomas Monjalon [this message]
2019-01-14 19:26               ` Alejandro Lucero
  -- strict thread matches above, loose matches on Subject: below --
2019-01-03  8:56 Alejandro Lucero
2019-01-09 10:54 ` Ferruh Yigit
2019-01-09 14:20   ` Alejandro Lucero
2019-01-09 16:15     ` Ferruh Yigit
2019-01-10 11:55       ` Alejandro Lucero
2019-01-11 11:48         ` Ferruh Yigit
2019-01-11 12:15           ` Alejandro Lucero
2019-01-11 12:35             ` Ferruh Yigit

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=11471688.z4v4DX61lF@xps \
    --to=thomas@monjalon.net \
    --cc=alejandro.lucero@netronome.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.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 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).