All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
To: Stephen Hemminger
	<stephen-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>
Cc: dev-VfR2kkLFssw@public.gmane.org,
	Stephen Hemminger
	<shemming-43mecJUBy8ZBDgjK7y7TUQ@public.gmane.org>
Subject: Re: [PATCH 6/6] vmxnet3: replace Intel driver
Date: Thu, 15 May 2014 19:14:20 +0200	[thread overview]
Message-ID: <1812257.SL2iW8YYEL@xps13> (raw)
In-Reply-To: <20140514185752.176396147-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>

Hi Stephen,

2014-05-14 11:55, Stephen Hemminger:
> Replace the Intel VMXNET3 driver with the one developed by Brocade.
> The Intel code was a missing features, and had style and other issues.
> 
> Major changes:
>       - supports multi-segment
>       - supports vlan offload
>       - per queue stats
> 
> Fixes:
>       - link update reports link change correctly
>       - allows reconfiguration of # of queues
>         (original code would have too small memzone).
> 
> Signed-off-by: Stephen Hemminger <shemming-43mecJUBy8ZBDgjK7y7TUQ@public.gmane.org>

We cannot replace a driver by another one without comparing performances.
Actually there is no good reason to remove an implementation if some people 
are willing to maintain it.
Please send a patch to import your vmxnet3 PMD as extension in a separated 
repository like these ones:
	http://dpdk.org/browse
This way, we can have a first integration step and discuss/test this 
implementation during the next release cycle.

Thank you for contributing
-- 
Thomas

      parent reply	other threads:[~2014-05-15 17:14 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-14 18:55 [PATCH 0/6] Ethernet driver enhancements Stephen Hemminger
2014-05-14 18:55 ` [PATCH 1/6] Subjec: ethdev: add macro to cover all checksum flags Stephen Hemminger
2014-05-14 18:55 ` [PATCH 2/6] Subjet: ethdev: add common code to atomicly access link Stephen Hemminger
     [not found]   ` <20140514185743.414400480-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>
2014-05-15  8:43     ` Ivan Boule
2014-05-14 18:55 ` [PATCH 3/6] ixgbe: use eth_dev_{get,set}_link Stephen Hemminger
2014-05-14 18:55 ` [PATCH 4/6] e1000: " Stephen Hemminger
2014-05-14 18:55 ` [PATCH 5/6] ether: allow setting mac address Stephen Hemminger
     [not found]   ` <20140514185750.257523699-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>
2014-05-15  9:04     ` Ivan Boule
     [not found]       ` <CAOm6c7qumb3xHT4K0QXzyPBbfXsWTeZLp4pxAKoJT6Dh+VpiHA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-21 15:24         ` Ivan Boule
2014-05-14 18:55 ` [PATCH 6/6] vmxnet3: replace Intel driver Stephen Hemminger
     [not found]   ` <20140514185752.176396147-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>
2014-05-15 17:14     ` Thomas Monjalon [this message]

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=1812257.SL2iW8YYEL@xps13 \
    --to=thomas.monjalon-pdr9zngts4eavxtiumwx3w@public.gmane.org \
    --cc=dev-VfR2kkLFssw@public.gmane.org \
    --cc=shemming-43mecJUBy8ZBDgjK7y7TUQ@public.gmane.org \
    --cc=stephen-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.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.