dev.dpdk.org archive mirror
 help / color / mirror / Atom feed
From: Eric Kinzie <ehkinzie-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Declan Doherty <declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: dev-VfR2kkLFssw@public.gmane.org
Subject: Re: [PATCH v2 1/4] Link Bonding Library
Date: Mon, 9 Jun 2014 17:11:03 -0400	[thread overview]
Message-ID: <20140609211054.GA4853@roosta.home> (raw)
In-Reply-To: <e6e8ecba5e2ba9d1a0e5299e042e7c54757e8644.1401891670.git.declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

On Wed Jun 04 16:18:53 +0100 2014, Declan Doherty wrote:
> - Broadcast TX burst broadcast bug fix
> - Add/remove slave behavior fix
> - Checkpatch fixes

Declan, would you consider the following change to rte_bond.c?  The two
header files from librte_cmdline don't seem to be necessary.

 Eric


--- a/lib/librte_bond/rte_bond.c
+++ b/lib/librte_bond/rte_bond.c
@@ -44,9 +44,6 @@
 #include <rte_ip.h>
 #include <rte_udp.h>
 
-#include <cmdline_parse.h>
-#include <cmdline_parse_etheraddr.h>
-
 #include "rte_bond.h"
 
 static const char *driver_name = "Link Bonding PMD";

  parent reply	other threads:[~2014-06-09 21:11 UTC|newest]

Thread overview: 127+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-28 15:32 [PATCH 0/4] Link Bonding Library declan.doherty-ral2JQCrhuEAvxtiuMwx3w
     [not found] ` <cover.1401287412.git.declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-05-28 15:32   ` [PATCH 1/4] " declan.doherty-ral2JQCrhuEAvxtiuMwx3w
     [not found]     ` <4d8e6bc2665fbaac641f0577714d7be9b0415d3c.1401287412.git.declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-05-28 16:54       ` Shaw, Jeffrey B
     [not found]         ` <4032A54B6BB5F04B8C08B6CFF08C59285543C357-AtyAts71sc9Qxe9IK+vIArfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-05-29 13:32           ` Doherty, Declan
2014-05-28 15:32   ` [PATCH 2/4] Link bonding unit tests declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-05-28 15:32   ` [PATCH 3/4] Link bonding integration into testpmd declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-05-28 15:32   ` [PATCH 4/4] Add Link Bonding Library to Doxygen declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-05-28 17:49   ` [PATCH 0/4] Link Bonding Library Neil Horman
     [not found]     ` <20140528174908.GB2648-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2014-05-29 10:33       ` Doherty, Declan
     [not found]         ` <345C63BAECC1AD42A2EC8C63AFFC3ADC13D3327C-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-05-29 11:33           ` Neil Horman
2014-05-29  3:23   ` Cao, Waterman
     [not found]     ` <AA3F441F262C58498CD6D0C1801DE7EB0AA89A63-0J0gbvR4kTggGBtAFL8yw7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-05-29 10:35       ` Doherty, Declan
2014-06-04 15:18   ` [PATCH v2 " declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-06-04 15:18   ` declan.doherty-ral2JQCrhuEAvxtiuMwx3w
     [not found]   ` <cover.1401891670.git.declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-04 15:18     ` [PATCH v2 1/4] " declan.doherty-ral2JQCrhuEAvxtiuMwx3w
     [not found]       ` <e6e8ecba5e2ba9d1a0e5299e042e7c54757e8644.1401891670.git.declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-05 15:15         ` Stephen Hemminger
     [not found]           ` <20140605081557.42a797e8-We1ePj4FEcvRI77zikRAJc56i+j3xesD0e7PPNI6Mm0@public.gmane.org>
2014-06-06  9:07             ` Doherty, Declan
     [not found]               ` <345C63BAECC1AD42A2EC8C63AFFC3ADC13D3737A-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-06 15:13                 ` Stephen Hemminger
2014-06-09 21:11         ` Eric Kinzie [this message]
     [not found]           ` <20140609211054.GA4853-5G/Vjf02Nsf/9pzu0YdTqQ@public.gmane.org>
2014-06-13 14:03             ` Doherty, Declan
2014-06-04 15:18     ` [PATCH v2 2/4] Link bonding unit tests, including: - code to generate packet bursts for testing rx and tx functionality of bonded device - virtual/stubbed out ethdev for use as slave ethdev in testing - checkpack fixes declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-06-04 15:18     ` [PATCH v2 1/4] Link Bonding Library declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-06-04 15:18     ` [PATCH v2 2/4] Link bonding unit tests, including: - code to generate packet bursts for testing rx and tx functionality of bonded device - virtual/stubbed out ethdev for use as slave ethdev in testing - checkpack fixes declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-06-04 15:18     ` [PATCH v2 3/4] Adding link bonding support to testpmd. - Includes the ability to create new bonded devices. - Add /remove bonding slave devices. - Interogate bonded device stats/configuration - Change bonding modes and select balance transmit polices declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-06-04 15:18     ` [PATCH v2 4/4] Add Link Bonding Library to Doxygen declan.doherty-ral2JQCrhuEAvxtiuMwx3w
2014-06-04 16:10     ` [PATCH v2 0/4] Link Bonding Library Doherty, Declan
2014-06-05  8:03     ` De Lara Guarch, Pablo
2014-06-05 11:03     ` Neil Horman
     [not found]       ` <20140605110340.GB20841-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-06-06  8:23         ` Doherty, Declan
     [not found]           ` <345C63BAECC1AD42A2EC8C63AFFC3ADC13D37331-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-06 14:54             ` Neil Horman
     [not found]               ` <20140606145426.GA2543-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-06-13 14:56                 ` Doherty, Declan
     [not found]                   ` <345C63BAECC1AD42A2EC8C63AFFC3ADC13D38DBF-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-13 15:11                     ` Neil Horman
2014-06-06  3:26     ` Cao, Waterman
2014-06-11 16:33     ` Thomas Monjalon
2014-06-13 14:08       ` Doherty, Declan
     [not found]         ` <345C63BAECC1AD42A2EC8C63AFFC3ADC13D38CDE-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-13 15:15           ` Thomas Monjalon
2014-06-13 14:41   ` [PATCH v3 0/5] Link Bonding PMD Library Declan Doherty
2014-06-13 14:41   ` [PATCH v3 1/5] " Declan Doherty
2014-06-13 14:41   ` [PATCH v3 2/5] Link Bonding PMD Library (librte_eal/librte_ether link bonding support changes) Declan Doherty
     [not found]     ` <258914f35917ae07dddc991ac9726542964dce44.1402662300.git.declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-13 16:08       ` Neil Horman
     [not found]         ` <20140613160807.GD22451-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-06-13 18:34           ` Doherty, Declan
     [not found]             ` <345C63BAECC1AD42A2EC8C63AFFC3ADC13D38EE9-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-13 19:38               ` Neil Horman
     [not found]                 ` <20140613193815.GE22451-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-06-16  8:59                   ` Doherty, Declan
     [not found]                     ` <345C63BAECC1AD42A2EC8C63AFFC3ADC13D39383-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-16 11:07                       ` Neil Horman
     [not found]                         ` <20140616110758.GA15165-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-06-16 16:17                           ` Richardson, Bruce
     [not found]                             ` <59AF69C657FD0841A61C55336867B5B01AA368CE-kPTMFJFq+rELt2AQoY/u9bfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-16 17:47                               ` Neil Horman
     [not found]                                 ` <20140616174746.GC15165-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-06-16 18:07                                   ` Richardson, Bruce
2014-06-16 18:09                                   ` Thomas Monjalon
2014-06-13 21:59       ` Stephen Hemminger
     [not found]         ` <20140613145918.5faebfde-We1ePj4FEcvRI77zikRAJc56i+j3xesD0e7PPNI6Mm0@public.gmane.org>
2014-06-16  7:59           ` Doherty, Declan
2014-06-13 14:42   ` [PATCH v3 3/5] Link Bonding PMD Library (Unit Test Suite) Declan Doherty
2014-06-13 14:42   ` [PATCH v3 4/5] Link Bonding PMD Library (testpmd link bonding API support) Declan Doherty
2014-06-13 14:42   ` [PATCH v3 5/5] Link Bonding PMD Library (Doxygen Additions) Declan Doherty
     [not found] ` <cover.1402662300.git.declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-13 15:20   ` [PATCH v3 0/5] Link Bonding PMD Library Neil Horman
2014-06-16 11:18   ` [PATCH v4 0/6] Link Bonding Library Declan Doherty
     [not found]     ` <1402917513-19495-1-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-18 16:14       ` [PATCH v5 " Declan Doherty
     [not found]         ` <1403108063-27169-1-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-18 16:18           ` Neil Horman
2014-06-24 14:52           ` [PATCH v6 " Declan Doherty
     [not found]             ` <1403621531-30487-1-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-24 16:03               ` [PATCH v7 " Declan Doherty
     [not found]                 ` <1403625828-20956-1-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-25 20:07                   ` [PATCH v8 " Declan Doherty
     [not found]                     ` <1403726868-8161-1-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-26 16:02                       ` De Lara Guarch, Pablo
2014-06-26 23:57                       ` [PATCH v9 0/5] link bonding Thomas Monjalon
     [not found]                         ` <1403827075-9192-1-git-send-email-thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
2014-06-26 23:57                           ` [PATCH v9 1/5] bond: new link bonding library Thomas Monjalon
     [not found]                             ` <1403827075-9192-2-git-send-email-thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
2014-06-27  0:45                               ` Thomas Monjalon
2014-06-26 23:57                           ` [PATCH v9 2/5] ethdev: add unique name to devices Thomas Monjalon
2014-06-26 23:57                           ` [PATCH v9 3/5] eal: support link bonding device initialization Thomas Monjalon
2014-06-26 23:57                           ` [PATCH v9 4/5] bond: unit tests Thomas Monjalon
2014-06-26 23:57                           ` [PATCH v9 5/5] bond: testpmd support Thomas Monjalon
2014-06-27 10:18                           ` [PATCH v10 0/5] link bonding Declan Doherty
     [not found]                             ` <1403864324-12022-1-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-27 20:58                               ` Thomas Monjalon
2014-06-29 17:49                               ` [PATCH v11 0/5] link bonding library Declan Doherty
     [not found]                                 ` <1404064161-26370-1-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-30  9:21                                   ` Thomas Monjalon
2014-06-30  9:28                                     ` Doherty, Declan
     [not found]                                       ` <345C63BAECC1AD42A2EC8C63AFFC3ADC2730631F-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-07-01 22:01                                         ` Thomas Monjalon
2014-06-29 17:49                               ` [PATCH v11 1/5] bond: new " Declan Doherty
     [not found]                                 ` <1404064161-26370-2-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-30  9:13                                   ` Thomas Monjalon
2014-06-30 22:29                                   ` Robert Sanford
     [not found]                                     ` <CA+cr1cojs0sx5s1ohwVcdT6ojQ_7PEwEAqmtCca0LAz_jkvVdw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-07-01 14:16                                       ` Thomas Monjalon
2014-07-01 14:19                                       ` Doherty, Declan
     [not found]                                         ` <345C63BAECC1AD42A2EC8C63AFFC3ADC273077A5-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-07-01 14:26                                           ` Thomas Monjalon
2014-06-29 17:49                               ` [PATCH v11 2/5] ethdev: add unique name to devices Declan Doherty
2014-06-29 17:49                               ` [PATCH v11 3/5] eal: support link bonding device initialization Declan Doherty
2014-06-29 17:49                               ` [PATCH v11 4/5] bond: unit tests Declan Doherty
     [not found]                                 ` <1404064161-26370-5-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-30  8:56                                   ` Thomas Monjalon
2014-06-29 17:49                               ` [PATCH v11 5/5] bond: testpmd support Declan Doherty
2014-06-27 10:18                           ` [PATCH v10 1/5] bond: new link bonding library Declan Doherty
2014-06-27 10:18                           ` [PATCH v10 2/5] ethdev: add unique name to devices Declan Doherty
2014-06-27 10:18                           ` [PATCH v10 3/5] eal: support link bonding device initialization Declan Doherty
2014-06-27 10:18                           ` [PATCH v10 4/5] bond: unit tests Declan Doherty
2014-06-27 10:18                           ` [PATCH v10 5/5] bond: testpmd support Declan Doherty
2014-06-25 20:07                   ` [PATCH v8 1/6] Link Bonding Library (lib/librte_pmd_bond) Declan Doherty
2014-06-25 20:07                   ` [PATCH v8 2/6] Support for unique interface naming of pmds Declan Doherty
2014-06-25 20:07                   ` [PATCH v8 3/6] EAL support for link bonding device initialization Declan Doherty
2014-06-25 20:07                   ` [PATCH v8 4/6] Link bonding Unit Tests Declan Doherty
2014-06-25 20:07                   ` [PATCH v8 5/6] testpmd link bonding additions Declan Doherty
2014-06-25 20:07                   ` [PATCH v8 6/6] Link Bonding Library doxygen additions Declan Doherty
2014-06-24 16:03               ` [PATCH v7 1/6] Link Bonding Library (lib/librte_pmd_bond) Declan Doherty
2014-06-24 16:03               ` [PATCH v7 2/6] Support for unique interface naming of pmds Declan Doherty
2014-06-24 16:03               ` [PATCH v7 3/6] EAL support for link bonding device initialization Declan Doherty
     [not found]                 ` <1403625828-20956-4-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-25 13:54                   ` Thomas Monjalon
2014-06-25 14:41                     ` Doherty, Declan
     [not found]                       ` <345C63BAECC1AD42A2EC8C63AFFC3ADC272FB41C-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-25 16:00                         ` Thomas Monjalon
2014-06-25 16:15                           ` Richardson, Bruce
2014-06-24 16:03               ` [PATCH v7 4/6] Link bonding Unit Tests Declan Doherty
2014-06-24 16:03               ` [PATCH v7 5/6] testpmd link bonding additions Declan Doherty
2014-06-24 16:03               ` [PATCH v7 6/6] Link Bonding Library doxygen additions Declan Doherty
     [not found]                 ` <1403625828-20956-7-git-send-email-declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-06-25 13:43                   ` Thomas Monjalon
2014-06-25 14:19                     ` Doherty, Declan
     [not found]                       ` <345C63BAECC1AD42A2EC8C63AFFC3ADC272FA3EA-kPTMFJFq+rF9qrmMLTLiibfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-06-25 14:23                         ` Thomas Monjalon
2014-06-24 14:52           ` [PATCH v6 1/6] Link Bonding Library (lib/librte_pmd_bond) Declan Doherty
2014-06-24 14:52           ` [PATCH v6 2/6] Support for unique interface naming of pmds Declan Doherty
2014-06-24 14:52           ` [PATCH v6 3/6] EAL support for link bonding device initialization Declan Doherty
2014-06-24 14:52           ` [PATCH v6 4/6] Link bonding Unit Tests Declan Doherty
2014-06-24 14:52           ` [PATCH v6 5/6] testpmd link bonding additions Declan Doherty
2014-06-24 14:52           ` [PATCH v6 6/6] Link Bonding Library doxygen additions Declan Doherty
2014-06-18 16:14       ` [PATCH v5 1/6] Link Bonding Library (lib/librte_pmd_bond) Declan Doherty
2014-06-18 16:14       ` [PATCH v5 2/6] Support for unique interface naming of pmds Declan Doherty
2014-06-18 16:14       ` [PATCH v5 3/6] EAL support for link bonding device initialization Declan Doherty
2014-06-18 16:14       ` [PATCH v5 4/6] Link bonding Unit Tests Declan Doherty
2014-06-18 16:14       ` [PATCH v5 5/6] testpmd link bonding additions Declan Doherty
2014-06-18 16:14       ` [PATCH v5 6/6] Link Bonding Library doxygen additions Declan Doherty
2014-06-16 11:18   ` [PATCH v4 1/6] Link Bonding Library (lib/librte_pmd_bond) initial release with support for Mode 0 - Round Robin Mode 1 - Active Backup Mode 2 - Balance -> Supports 3 transmit polices (layer 2, layer 2+3, la Mode 3 - Broadcast Declan Doherty
2014-06-16 11:18   ` [PATCH v4 2/6] Support for unique interface naming of pmds Declan Doherty
2014-06-16 11:18   ` [PATCH v4 3/6] EAL support for link bonding device initialization Declan Doherty
2014-06-16 11:18   ` [PATCH v4 4/6] Link bonding Unit Tests Declan Doherty
2014-06-16 11:18   ` [PATCH v4 5/6] testpmd link bonding additions Declan Doherty
2014-06-16 11:18   ` [PATCH v4 6/6] Link Bonding Library doxygen additions Declan Doherty

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=20140609211054.GA4853@roosta.home \
    --to=ehkinzie-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=declan.doherty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=dev-VfR2kkLFssw@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 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).