public inbox for netdev@vger.kernel.org 
 help / color / mirror / Atom feed
From: Ben Pfaff <blp-l0M0P4e3n4LQT0dZR+AlfA@public•gmane.org>
To: Simon Horman <horms-/R6kz+dDXgpPR4JQBCEnsQ@public•gmane.org>
Cc: dev-yBygre7rU0TnMu66kgdUjQ@public•gmane.org,
	netdev-u79uwXL29TY76Z2rM5mHXA@public•gmane.org
Subject: Re: [PATCH 05/21] vswitchd: Add add_tunnel_ports()
Date: Fri, 25 May 2012 10:18:47 -0700	[thread overview]
Message-ID: <20120525171847.GH25106@nicira.com> (raw)
In-Reply-To: <1337850554-10339-6-git-send-email-horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>

On Thu, May 24, 2012 at 06:08:58PM +0900, Simon Horman wrote:
> Add tunnel tundevs for tunnel realdevs as needed.
> 
> In general the notion is that realdevs may be configured by users
> and from an end-user point of view are compatible with the existing
> port-based tunneling code. And that tundevs exist in the datapath
> arnd are actually used to send and recieve packets, based on flows.
> 
> Cc: Kyle Mestery <kmestery-FYB4Gu1CFyUAvxtiuMwx3w@public•gmane.org>
> Signed-off-by: Simon Horman <horms-/R6kz+dDXgpPR4JQBCEnsQ@public•gmane.org>

This seems reasonable at a glance.  There are bits I might quibble
with as this gets closer, but the structure seems reasonable.

  parent reply	other threads:[~2012-05-25 17:18 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-24  9:08 [RFC v4 00/21] Flow Based Tunneling for Open vSwitch Simon Horman
2012-05-24  9:08 ` [PATCH 02/21] datapath: Use tun_key on transmit Simon Horman
2012-05-24  9:08 ` [PATCH 03/21] odp-util: Add tun_key to parse_odp_key_attr() Simon Horman
     [not found]   ` <1337850554-10339-4-git-send-email-horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2012-05-24 16:29     ` Ben Pfaff
     [not found]       ` <20120524162911.GD26173-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2012-05-25  0:01         ` Simon Horman
2012-05-24  9:09 ` [PATCH 08/21] ofproto: Add realdev_to_txdev() Simon Horman
     [not found] ` <1337850554-10339-1-git-send-email-horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2012-05-24  9:08   ` [PATCH 01/21] datapath: tunnelling: Replace tun_id with tun_key Simon Horman
     [not found]     ` <1337850554-10339-2-git-send-email-horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2012-06-03  9:01       ` Jesse Gross
2012-06-03  9:15     ` [ovs-dev] " Jesse Gross
     [not found]       ` <CAEP_g=9hkP-7fuFK3zSJcR=2BTK0feq7qUa8LHs3dbGQBy+suw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-04 22:34         ` Simon Horman
2012-06-05  3:33           ` [ovs-dev] " Jesse Gross
2012-06-05  8:12             ` Simon Horman
2012-05-24  9:08   ` [PATCH 04/21] vswitchd: Add iface_parse_tunnel Simon Horman
     [not found]     ` <1337850554-10339-5-git-send-email-horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2012-05-24 16:47       ` Ben Pfaff
2012-05-24 23:59         ` [ovs-dev] " Simon Horman
2012-05-24  9:08   ` [PATCH 05/21] vswitchd: Add add_tunnel_ports() Simon Horman
     [not found]     ` <1337850554-10339-6-git-send-email-horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2012-05-25 17:18       ` Ben Pfaff [this message]
2012-05-24  9:08   ` [PATCH 06/21] ofproto: Add set_tunnelling() Simon Horman
2012-05-24  9:09   ` [PATCH 07/21] vswitchd: Configure tunnel interfaces Simon Horman
2012-05-24  9:09   ` [PATCH 09/21] ofproto: Add tundev_to_realdev() Simon Horman
2012-05-24  9:09   ` [PATCH 10/21] classifier: Convert struct flow flow_metadata to use tun_key Simon Horman
2012-05-24  9:09   ` [PATCH 11/21] datapath, vport: Provide tunnel realdev and tundev classes and vports Simon Horman
2012-05-24  9:09   ` [PATCH 12/21] lib: Replace commit_set_tun_id_action() with commit_set_tunnel_action() Simon Horman
2012-05-24  9:09   ` [PATCH 13/21] global: Remove OVS_KEY_ATTR_TUN_ID Simon Horman
2012-05-24  9:09   ` [PATCH 14/21] ofproto: Set flow tun_key in compose_output_action() Simon Horman
2012-05-24  9:09   ` [PATCH 15/21] datapath: Remove mlink element from tnl_mutable_config Simon Horman
2012-05-24  9:09   ` [PATCH 18/21] dataptah: remove ttl and tos " Simon Horman
2012-05-24  9:09 ` [PATCH 16/21] datapath: remove tunnel cache Simon Horman
2012-05-24  9:09 ` [PATCH 17/21] datapath: Always use tun_key addresses for route lookup Simon Horman
2012-05-24  9:09 ` [PATCH 19/21] datapath: Simplify vport lookup Simon Horman
2012-05-24  9:09 ` [PATCH 20/21] datapath: Use tun_key flags for id and csum settings on transmit Simon Horman
2012-05-24  9:09 ` [PATCH 21/21] datapath: Always use tun_key flags Simon Horman

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=20120525171847.GH25106@nicira.com \
    --to=blp-l0m0p4e3n4lqt0dzr+alfa@public$(echo .)gmane.org \
    --cc=dev-yBygre7rU0TnMu66kgdUjQ@public$(echo .)gmane.org \
    --cc=horms-/R6kz+dDXgpPR4JQBCEnsQ@public$(echo .)gmane.org \
    --cc=netdev-u79uwXL29TY76Z2rM5mHXA@public$(echo .)gmane.org \
    /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