From: Eliezer Tamir <eliezer.tamir@linux•intel.com>
To: David Miller <davem@davemloft•net>
Cc: linux-kernel@vger•kernel.org, netdev@vger•kernel.org,
Jesse Brandeburg <jesse.brandeburg@intel•com>,
Don Skidmore <donald.c.skidmore@intel•com>,
e1000-devel@lists•sourceforge.net,
Willem de Bruijn <willemb@google•com>,
Eric Dumazet <erdnetdev@gmail•com>,
Ben Hutchings <bhutchings@solarflare•com>,
Andi Kleen <andi@firstfloor•org>, HPA <hpa@zytor•com>,
Eilon Greenstien <eilong@broadcom•com>,
Or Gerlitz <or.gerlitz@gmail•com>,
Alex Rosenbaum <alexr@mellanox•com>,
Eliezer Tamir <eliezer@tamir•org.il>
Subject: [PATCH v7 net-next 0/5] net: low latency Ethernet device polling
Date: Thu, 30 May 2013 14:40:45 +0300 [thread overview]
Message-ID: <20130530114045.12653.79183.stgit@ladj378.jer.intel.com> (raw)
more fixes.
Thank you all for your input.
-Eliezer
change log:
v7
- suggested by Ben Hutchings and Eric Dumazet:
type fixes, static for globals in net/core.c,
avoid napi_id collisions in napi_hash_add()
v6
- many small fixes suggested by Eric Dumazet:
data locality, typos, documentation
protect napi_hash insert/delete with a spinlock (napi_gen_id is no
longer atomic_t since it's only accessed with the spinlock held.)
- added IPv6 TCP and UDP support (only minimally tested)
v5
- corrections suggested by Ben Hutchings:
fixed typos, moved the config option and sysctl value from IPv4 to net
- moved sk_mark_ll() to the protocol handlers
- removed global id mechanism, replaced with a hashed napi_id.
based on code sample from Eric Dumazet
Note that ixgbe_free_q_vector() already waits an rcu grace period
before freeing the q_vector, so nothing additional needs to be done
when adding a call to napi_hash_del().
- simple poll/select support
v4
- removed separate config option for TCP as suggested Eric Dumazet.
- added linux mib counter for packets received through the low latency path,
as suggested by Andi Kleen.
- re-allow module unloading, remove module param, use a global generation id
instead to prevent the use of a stale napi pointer, as suggested
by Eric Dumazet
- updated Documentation/networking/ip-sysctl.txt text
v3
- coding style changes suggested by Dave Miller
v2
- the sysctl knob is now in microseconds. The default value is now 0 (off).
- for now the code depends at configure time on CONFIG_I86_TSC
- the napi reference in struct skb is now a union with the dma cookie
since the former is only used on RX and the latter on TX,
as suggested by Eric Dumazet.
- we do a better job at honoring non-blocking operations.
- removed busy-polling support for tcp_read_sock()
- remove dynamic disabling of GRO
- coding style fixes
- disallow unloading the device module after the feature has been used
Credit:
Jesse Brandeburg, Arun Chekhov Ilango, Julie Cummings,
Alexander Duyck, Eric Geisler, Jason Neighbors, Yadong Li,
Mike Polehn, Anil Vasudevan, Don Wood
Special thanks for finding bugs in earlier versions:
Willem de Bruijn and Andi Kleen
next reply other threads:[~2013-05-30 11:40 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-30 11:40 Eliezer Tamir [this message]
2013-05-30 11:40 ` [PATCH v7 net-next 1/5] net: add napi_id and hash Eliezer Tamir
2013-05-30 11:41 ` [PATCH v7 net-next 2/5] net: implement support for low latency socket polling Eliezer Tamir
2013-05-30 15:07 ` Amir Vadai
2013-05-30 20:01 ` Eliezer Tamir
2013-05-30 11:41 ` [PATCH v7 net-next 3/5] tcp: add TCP support for low latency receive poll Eliezer Tamir
2013-05-30 11:41 ` [PATCH v7 net-next 4/5] ixgbe: Add support for ndo_ll_poll Eliezer Tamir
2013-05-31 3:32 ` Cong Wang
2013-05-30 11:41 ` [PATCH v7 net-next 5/5] ixgbe: add extra stats " Eliezer Tamir
2013-06-02 14:02 ` [PATCH v7 net-next 0/5] net: low latency Ethernet device polling Eliezer Tamir
2013-06-02 20:11 ` David Miller
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=20130530114045.12653.79183.stgit@ladj378.jer.intel.com \
--to=eliezer.tamir@linux$(echo .)intel.com \
--cc=alexr@mellanox$(echo .)com \
--cc=andi@firstfloor$(echo .)org \
--cc=bhutchings@solarflare$(echo .)com \
--cc=davem@davemloft$(echo .)net \
--cc=donald.c.skidmore@intel$(echo .)com \
--cc=e1000-devel@lists$(echo .)sourceforge.net \
--cc=eilong@broadcom$(echo .)com \
--cc=eliezer@tamir$(echo .)org.il \
--cc=erdnetdev@gmail$(echo .)com \
--cc=hpa@zytor$(echo .)com \
--cc=jesse.brandeburg@intel$(echo .)com \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=netdev@vger$(echo .)kernel.org \
--cc=or.gerlitz@gmail$(echo .)com \
--cc=willemb@google$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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