netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Brenden Blanco <bblanco@plumgrid.com>
To: Or Gerlitz <gerlitz.or@gmail.com>
Cc: David Miller <davem@davemloft.net>,
	Linux Netdev List <netdev@vger.kernel.org>,
	Jamal Hadi Salim <jhs@mojatatu.com>,
	Saeed Mahameed <saeedm@dev.mellanox.co.il>,
	Martin KaFai Lau <kafai@fb.com>,
	Jesper Dangaard Brouer <brouer@redhat.com>,
	Ari Saha <as754m@att.com>,
	Alexei Starovoitov <alexei.starovoitov@gmail.com>,
	john fastabend <john.fastabend@gmail.com>,
	Hannes Frederic Sowa <hannes@stressinduktion.org>,
	Thomas Graf <tgraf@suug.ch>, Tom Herbert <tom@herbertland.com>,
	Daniel Borkmann <daniel@iogearbox.net>
Subject: Re: [PATCH v7 00/11] Add driver bpf hook for early packet drop and forwarding
Date: Mon, 11 Jul 2016 15:06:33 -0700	[thread overview]
Message-ID: <20160711220632.GD22729@gmail.com> (raw)
In-Reply-To: <CAJ3xEMgVtqETM8XHt5gedAmKoBGwggnKDr3iWUeRYFtvQRfQEw@mail.gmail.com>

On Tue, Jul 12, 2016 at 12:53:34AM +0300, Or Gerlitz wrote:
> On Tue, Jul 12, 2016 at 12:29 AM, Brenden Blanco <bblanco@plumgrid.com> wrote:
> 
> > v7:
> [...]
> >  TODOs:
> >  Add ethtool per-ring stats for aborted, default cases, maybe even drop
> >  and tx as well.
> 
> please no... lets stop and think if we can have something better vs
> every XDP enabled driver to have bunch of new ethtool based stats, was
> this somehow discussed over the threads so far?
This was in the context of the discussion in [01/12] relating to the
return codes and debuggability.

  reply	other threads:[~2016-07-11 22:06 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-11 21:29 [PATCH v7 00/11] Add driver bpf hook for early packet drop and forwarding Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 01/11] bpf: add XDP prog type for early driver filter Brenden Blanco
2016-07-12  6:02   ` David Miller
2016-07-12  6:26     ` Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 02/11] net: add ndo to setup/query xdp prog in adapter rx Brenden Blanco
2016-07-12  6:12   ` David Miller
2016-07-12  6:35     ` Brenden Blanco
2016-07-12  6:45       ` David Miller
2016-07-11 21:29 ` [PATCH v7 03/11] rtnl: add option for setting link xdp prog Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 04/11] net/mlx4_en: add support for fast rx drop bpf program Brenden Blanco
2016-07-12  6:17   ` David Miller
2016-07-12 16:31     ` Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 05/11] Add sample for adding simple drop program to link Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 06/11] net/mlx4_en: add page recycle to prepare rx ring for tx support Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 07/11] bpf: add XDP_TX xdp_action for direct forwarding Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 08/11] net/mlx4_en: break out tx_desc write into separate function Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 09/11] net/mlx4_en: add xdp forwarding and data write support Brenden Blanco
2016-07-13 15:25   ` Saeed Mahameed
2016-07-13 17:30     ` Brenden Blanco
2016-07-13 19:16       ` Saeed Mahameed
2016-07-11 21:29 ` [PATCH v7 10/11] bpf: enable direct packet data write for xdp progs Brenden Blanco
2016-07-11 21:29 ` [PATCH v7 11/11] bpf: add sample for xdp forwarding and rewrite Brenden Blanco
2016-07-11 21:53 ` [PATCH v7 00/11] Add driver bpf hook for early packet drop and forwarding Or Gerlitz
2016-07-11 22:06   ` Brenden Blanco [this message]
2016-07-11 23:26   ` Tom Herbert

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=20160711220632.GD22729@gmail.com \
    --to=bblanco@plumgrid.com \
    --cc=alexei.starovoitov@gmail.com \
    --cc=as754m@att.com \
    --cc=brouer@redhat.com \
    --cc=daniel@iogearbox.net \
    --cc=davem@davemloft.net \
    --cc=gerlitz.or@gmail.com \
    --cc=hannes@stressinduktion.org \
    --cc=jhs@mojatatu.com \
    --cc=john.fastabend@gmail.com \
    --cc=kafai@fb.com \
    --cc=netdev@vger.kernel.org \
    --cc=saeedm@dev.mellanox.co.il \
    --cc=tgraf@suug.ch \
    --cc=tom@herbertland.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).