All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Graf <tgraf@suug.ch>
To: jesse@nicira.com
Cc: dev@openvswitch.org, eric.dumazet@gmail.com, fleitner@redhat.com,
	ffusco@redhat.com, dborkmann@redhat.com,
	bhutchings@solarflare.com, netdev@vger.kernel.org,
	davem@davemloft.net
Subject: [PATCH net-next 0/6 v9] Open vSwitch zercopy upcall optimiziation
Date: Fri, 13 Dec 2013 15:22:16 +0100	[thread overview]
Message-ID: <cover.1386943635.git.tgraf@suug.ch> (raw)

Series implementing a zerocopy method for OVS upcall messages. Based
on top of commit: (''openvswitch: Enable memory mapped Netlink i/o'')

Thomas Graf (6):
  net: Export skb_zerocopy() to zerocopy from one skb to another
  openvswitch: Allow user space to announce ability to accept unaligned
    Netlink messages
  openvswitch: Drop user features if old user space attempted to create
    datapath
  openvswitch: Pass datapath into userspace queue functions
  openvswitch: Use skb_zerocopy() for upcall
  openvswitch: Compute checksum in skb_gso_segment() if needed

 include/linux/skbuff.h               |   3 +
 include/uapi/linux/openvswitch.h     |  14 ++++-
 net/core/skbuff.c                    |  85 ++++++++++++++++++++++++++++
 net/netfilter/nfnetlink_queue_core.c |  59 ++-----------------
 net/openvswitch/datapath.c           | 106 ++++++++++++++++++++++++++---------
 net/openvswitch/datapath.h           |   2 +
 6 files changed, 185 insertions(+), 84 deletions(-)

---
V9: - Dropped patches 1-3 (merged)
    - Dropped skb argument from upcall_msg_size()
    - Dropped dp_ifindex and net argument from all queue functions
    - Added patch to remove NETIF_F_HW_CSUM from __skb_gso_segment()
V8: - Dropped patch adding NLM_F_REPLACE support, I'll pursue this in a
      separate patch series. Addresses Jesse's comments.
    - Improved comment describing OVS_DATAPATH_VERSION bump.
V7: - removed unintential kernel-doc comment
    - WARN_ONCE() -> WARN(), message on single line, added \n
V6: - Added memory mapped netlink i/o support
    - Drop user_features if old user space not aware of user features
      reuses an existing datapath
V5: - Removed padding requirement in user space
    - Added OVS_DP_F_UNALIGNED flag let user space signal ability to
      receive unaligned Netlink messages, fall back to linear copy
      otherwise.
V4: - Daniel Borkmann pointed out that the style in skbuff.h has changed
      in net-next, adapted to no longer using extern in headers.
V3: - Removed unneeded alignment of nlmsg_len after padding
V2: - Added skb_zerocopy_headlen() to calculate headroom of destination
      buffer. This also takes care of the from->head_frag issue.
    - Attribute alignment for frag_list case
    - API documentation
    - performance data for CHECKSUM_PARTIAL tx case

             reply	other threads:[~2013-12-13 14:22 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-13 14:22 Thomas Graf [this message]
2013-12-13 14:22 ` [PATCH net-next 1/6] net: Export skb_zerocopy() to zerocopy from one skb to another Thomas Graf
2013-12-13 14:22 ` [PATCH net-next 2/6] openvswitch: Allow user space to announce ability to accept unaligned Netlink messages Thomas Graf
2013-12-13 14:22 ` [PATCH net-next 3/6] openvswitch: Drop user features if old user space attempted to create datapath Thomas Graf
2013-12-13 14:22 ` [PATCH net-next 4/6] openvswitch: Pass datapath into userspace queue functions Thomas Graf
2013-12-13 14:22 ` [PATCH net-next 5/6] openvswitch: Use skb_zerocopy() for upcall Thomas Graf
2013-12-13 14:22 ` [PATCH net-next 6/6] openvswitch: Compute checksum in skb_gso_segment() if needed Thomas Graf
2013-12-17  1:22 ` [PATCH net-next 0/6 v9] Open vSwitch zercopy upcall optimiziation Jesse Gross

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=cover.1386943635.git.tgraf@suug.ch \
    --to=tgraf@suug.ch \
    --cc=bhutchings@solarflare.com \
    --cc=davem@davemloft.net \
    --cc=dborkmann@redhat.com \
    --cc=dev@openvswitch.org \
    --cc=eric.dumazet@gmail.com \
    --cc=ffusco@redhat.com \
    --cc=fleitner@redhat.com \
    --cc=jesse@nicira.com \
    --cc=netdev@vger.kernel.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.