All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas.monjalon@6wind.com>
To: Alejandro Lucero <alejandro.lucero@netronome.com>
Cc: dev@dpdk.org
Subject: Re: [PATCH v3 2/4] nfp-uio: new uio driver for netronome nfp6000 card
Date: Wed, 21 Oct 2015 17:25:14 +0200	[thread overview]
Message-ID: <2304404.uKjJZErY2S@xps13> (raw)
In-Reply-To: <CAD+H991+yGrX73qtBdBSKZpV2VryrKahVFXnzm6KPcFAMo70hg@mail.gmail.com>

2015-10-21 15:39, Alejandro Lucero:
> On Wed, Oct 21, 2015 at 6:24 AM, David Marchand <david.marchand@6wind.com>
> wrote:
> > Please, can you elaborate on the need for (yet another) uio driver, rather
> > than make igb_uio work with your hardware ?
[...]
> I have been looking at the possibility of getting rid of nfp_uio. The fact
> is our PMD can work without it, both for the PF and VF (not the PMD version
> already submitted but one under development).  The PF support requires not
> using UIO at all, because the device is attached to the BSP driver. The
> only problem with this approach is we do not have support for interrupts,
> what is not critical (I can see other PMDs not having support for Link
> Status Changes) but we do not like it as programs can register callbacks
> for these interrupts which would not work at all.
> 
> Interrupt support could be implemented in the BSP, doing the same UIO or
> VFIO do, but this will require (minor) changes to DPDK for having another
> intr_handle (not UIO, not VFIO). I do not know if other PMDs could also
> make use of such a change but I guess that would help to accept those
> changes.

We are going to have an external handler (used for mlx5):
	http://dpdk.org/ml/archives/dev/2015-October/024678.html
Problem solved :)

Is it possible to rework your PMD without nfp-uio?

Thanks

  reply	other threads:[~2015-10-21 15:26 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-16 10:45 [PATCH v3 0/4] support for netronome nfp-6xxx card Alejandro.Lucero
2015-10-16 10:45 ` [PATCH v3 1/4] nfp: new poll mode driver for netronome nfp6000 card Alejandro.Lucero
2015-10-16 10:45 ` [PATCH v3 2/4] nfp-uio: new uio " Alejandro.Lucero
2015-10-21  5:24   ` David Marchand
2015-10-21 14:39     ` Alejandro Lucero
2015-10-21 15:25       ` Thomas Monjalon [this message]
2015-10-21 15:57         ` Alejandro Lucero
2015-10-21 16:03           ` Thomas Monjalon
2015-10-21 19:40             ` Alejandro Lucero
2015-10-22 11:46               ` Alejandro Lucero
2015-10-16 10:45 ` [PATCH v3 3/4] doc: add netronome nfp6000 guide Alejandro.Lucero
2015-10-16 10:45 ` [PATCH v3 4/4] tools: add support for nfp_uio Alejandro.Lucero
2015-10-19 19:17 ` [PATCH v3 0/4] support for netronome nfp-6xxx card Mcnamara, John

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=2304404.uKjJZErY2S@xps13 \
    --to=thomas.monjalon@6wind.com \
    --cc=alejandro.lucero@netronome.com \
    --cc=dev@dpdk.org \
    /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.