DPDK patches and discussions
 help / color / mirror / Atom feed
From: Ravi Kerur <rkerur@gmail.com>
To: "Mcnamara, John" <john.mcnamara@intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>,
	"Ananyev, Konstantin" <konstantin.ananyev@intel.com>,
	 "Richardson, Bruce" <bruce.richardson@intel.com>
Subject: Re: [dpdk-dev] [v4 0/3] Merge l3fwd-acl and l3fwd
Date: Wed, 8 Mar 2017 13:50:47 -0800	[thread overview]
Message-ID: <CAFb4SLAugjVxwskxE1duYo9PkOuBOeUhunXGxZejmoHT1W5Fmg@mail.gmail.com> (raw)
In-Reply-To: <CAFb4SLAmkAuLvnBJy9QSZkR56+HiGwydWdVw_h1dWdLtJr9OAQ@mail.gmail.com>

John, Konstantin,

Kindly let me know new 'v5' patch follows dpdk guidelines?

I just followed what's in the dpdk contributing code guidelines.


   - git format-patch -3 -o patch/ -v 5 --cover-letter
   - git send-email --to dev@dpdk.org --in-reply-to <v4-MSGID> patch/*

Thanks.


On Mon, Mar 6, 2017 at 3:20 PM, Ravi Kerur <rkerur@gmail.com> wrote:

> Hi John,
>
> Should I work with documentation team to update the document? If yes,
> please let me know the contact information.
>
> Thanks.
>
> On Mon, Mar 6, 2017 at 5:33 AM, Mcnamara, John <john.mcnamara@intel.com>
> wrote:
>
>> > -----Original Message-----
>> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ravi Kerur
>> > Sent: Sunday, March 5, 2017 7:47 PM
>> > To: dev@dpdk.org
>> > Cc: Ananyev, Konstantin <konstantin.ananyev@intel.com>; Richardson,
>> Bruce
>> > <bruce.richardson@intel.com>; Ravi Kerur <rkerur@gmail.com>
>> > Subject: [dpdk-dev] [v4 0/3] Merge l3fwd-acl and l3fwd
>> >
>> > This patchset merges l3fwd-acl and l3fwd code into common directory.
>> > Adds file read options to build LPM and EM tables.
>>
>> Hi Ravi,
>>
>> Thanks to this. It seems like a good change. There are probably too many
>> L2/L3 variants and some of them should be merged.
>>
>> Note, you will also have to merge the sample app guides in some sensible
>> way:
>>
>>     http://dpdk.org/doc/guides/sample_app_ug/l3_forward.html
>>     http://dpdk.org/doc/guides/sample_app_ug/l3_forward_access_ctrl.html
>>
>> John
>>
>>
>

  reply	other threads:[~2017-03-08 21:50 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-05 19:47 Ravi Kerur
2017-03-05 19:47 ` [dpdk-dev] [v4 1/3] Merge l3fwd-acl and l3fwd code Ravi Kerur
2017-03-08 21:32   ` [dpdk-dev] [PATCH v5 0/3] examples/l3fwd: merge l3fwd-acl code into l3fwd Ravi Kerur
2017-03-08 21:32     ` [dpdk-dev] [PATCH v5 1/3] " Ravi Kerur
2017-03-09 10:42       ` Mcnamara, John
2017-03-10 20:58       ` [dpdk-dev] [PATCH v6 0/3] " Ravi Kerur
2017-03-10 20:58         ` [dpdk-dev] [PATCH v6 1/3] " Ravi Kerur
2017-03-10 20:58         ` [dpdk-dev] [PATCH v6 2/3] examples/l3fwd: add config file support for lpm Ravi Kerur
2017-03-10 20:58         ` [dpdk-dev] [PATCH v6 3/3] examples/l3fwd: add config file support for exact Ravi Kerur
2017-06-04 11:14         ` [dpdk-dev] [PATCH v6 0/3] examples/l3fwd: merge l3fwd-acl code into l3fwd Thomas Monjalon
2019-01-23 16:32         ` Ferruh Yigit
2019-01-23 17:10           ` Thomas Monjalon
2019-01-23 17:20             ` Ananyev, Konstantin
2019-01-23 17:26               ` Thomas Monjalon
2019-01-23 19:36                 ` Ferruh Yigit
2019-01-23 20:27                   ` Thomas Monjalon
2019-01-24  9:14               ` Rami Rosen
2017-03-08 21:32     ` [dpdk-dev] [PATCH v5 2/3] examples/l3fwd: add config file support for LPM Ravi Kerur
2017-03-09 10:43       ` Mcnamara, John
2017-03-08 21:32     ` [dpdk-dev] [PATCH v5 3/3] examples/l3fwd: add config file support for EM Ravi Kerur
2017-03-09 10:45       ` Mcnamara, John
2017-03-10  0:00         ` Ravi Kerur
2017-03-05 19:47 ` [dpdk-dev] [v4 2/3] LPM config file read option Ravi Kerur
2017-03-06 13:57   ` Mcnamara, John
2017-03-06 14:10     ` Mcnamara, John
2017-03-05 19:47 ` [dpdk-dev] [v4 3/3] EM " Ravi Kerur
2017-03-06 13:42   ` Mcnamara, John
2017-03-06 13:33 ` [dpdk-dev] [v4 0/3] Merge l3fwd-acl and l3fwd Mcnamara, John
2017-03-06 23:20   ` Ravi Kerur
2017-03-08 21:50     ` Ravi Kerur [this message]
2017-03-09 10:37       ` Mcnamara, John
2017-03-09 10:23     ` Mcnamara, John
2017-03-10  0:07       ` Ravi Kerur
2017-03-14 21:14         ` Ravi Kerur

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=CAFb4SLAugjVxwskxE1duYo9PkOuBOeUhunXGxZejmoHT1W5Fmg@mail.gmail.com \
    --to=rkerur@gmail.com \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=john.mcnamara@intel.com \
    --cc=konstantin.ananyev@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).