netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Graf <tgraf@suug.ch>
To: Tom Herbert <therbert@google.com>
Cc: David Miller <davem@davemloft.net>,
	Linux Netdev List <netdev@vger.kernel.org>
Subject: Re: [PATCH net-next 2/2] vxlan: Eliminate dependency on UDP socket in transmit path
Date: Tue, 20 Jan 2015 18:13:35 +0000	[thread overview]
Message-ID: <20150120181335.GL20315@casper.infradead.org> (raw)
In-Reply-To: <CA+mtBx-QVcK2yP7hwr01X-KaXedzrgMGrHH503-i5y54BPnNPA@mail.gmail.com>

On 01/20/15 at 09:29am, Tom Herbert wrote:
> I didn't see any reason to preclude that, if it needs to be symmetric
> in that case it can be forced at the configuration. Being able to
> receive RCO but not have to send it to certain peers is important use
> case. You may want to consider this also for GBP if there are cases
> where we accept GBP from different peers, but only send it to certain
> ones.

I think asymmetric configurations are fine, in particular
receive-only. I was reluctant to the send-only scenario initially
as I would expect a VTEP sending RCO frames on UDP dport 8472 to
also always be able to accept RCO frames on that port. I can't
come up with any specific cases where this would lead to problems
though so I have no objections.

As for GBP, as processing of the policy group requires additional
iptables or OVS rules anyway, such behaviour would be implemented
in those rules by either ignoring the mark or dropping such frames.

  reply	other threads:[~2015-01-20 18:13 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-17 18:18 [PATCH net-next 0/2] vxlan: Don't use UDP socket for transmit Tom Herbert
2015-01-17 18:18 ` [PATCH net-next 1/2] udp: Do not require sock in udp_tunnel_xmit_skb Tom Herbert
2015-01-17 23:45   ` Jesse Gross
2015-01-18 22:43   ` Or Gerlitz
2015-01-20 17:36     ` Tom Herbert
2015-01-20 21:47       ` Or Gerlitz
2015-01-20 22:24         ` Tom Herbert
2015-01-21 20:57           ` Or Gerlitz
2015-01-21 22:24             ` Tom Herbert
2015-01-21 22:37               ` Or Gerlitz
2015-01-21 22:55                 ` Tom Herbert
2015-01-17 18:18 ` [PATCH net-next 2/2] vxlan: Eliminate dependency on UDP socket in transmit path Tom Herbert
2015-01-19  8:59   ` Thomas Graf
2015-01-20 17:29     ` Tom Herbert
2015-01-20 18:13       ` Thomas Graf [this message]
2015-01-20 22:53         ` 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=20150120181335.GL20315@casper.infradead.org \
    --to=tgraf@suug.ch \
    --cc=davem@davemloft.net \
    --cc=netdev@vger.kernel.org \
    --cc=therbert@google.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).