From mboxrd@z Thu Jan 1 00:00:00 1970 From: Amir Vadai Subject: Re: [RFC net-next 0/2] net/sched: cls_flower, act_mirred: VXLAN redirect using TC Date: Mon, 15 Aug 2016 15:59:06 +0300 Message-ID: <20160815125906.GA30031@office.localdomain> References: <20160814140652.15985-1-amir@vadai.me> <20160815071121.GA2024@nanopsycho> <20160815081740.GA19058@office.localdomain> <20160815090804.GB19058@office.localdomain> <20160815123400.GH2024@nanopsycho> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jamal Hadi Salim , Cong Wang , John Fastabend , Jiri Pirko , Linux Kernel Network Developers , Or Gerlitz , Hadar Har-Zion , Oded Shanoon , Amir Vadai To: Jiri Pirko Return-path: Received: from mail-wm0-f68.google.com ([74.125.82.68]:35263 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752785AbcHOM7L (ORCPT ); Mon, 15 Aug 2016 08:59:11 -0400 Received: by mail-wm0-f68.google.com with SMTP id i5so11003425wmg.2 for ; Mon, 15 Aug 2016 05:59:10 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20160815123400.GH2024@nanopsycho> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, Aug 15, 2016 at 02:34:00PM +0200, Jiri Pirko wrote: > Mon, Aug 15, 2016 at 12:08:10PM CEST, jhs@mojatatu.com wrote: > >On 16-08-15 05:08 AM, Amir Vadai wrote: > >> On Mon, Aug 15, 2016 at 11:17:40AM +0300, Amir Vadai wrote: > >> > On Mon, Aug 15, 2016 at 09:11:22AM +0200, Jiri Pirko wrote: > >> > > Sun, Aug 14, 2016 at 07:53:30PM CEST, xiyou.wangcong@gmail.com wrote: > > > >> > > >> > Thanks, > >> > Amir > >> > >> Any objection to the following? > >> > >> # ENCAP rule > >> tc filter add dev $ETH protocol ip parent ffff: prio 10 \ > >> flower ip_proto 1 \ > >> action set_tunnel_key src_ip 11.11.0.1 dst_ip 11.11.0.2 key_id 11 dst_port 4789 \ > >> action mirred egress redirect dev $VXLAN > > > >Assuming $VXLAN is actually not a linux netdev of type vxlan? > >then the action does vxlan encap redirect sends it to the $VXLAN > >dev with encapsulation in place. > >Sounds to me like a name like "vxlan" would be more usable. Example: > > I believe those are generic tunelling data > > > > > >tc filter add dev $ETH protocol ip parent ffff: prio 10 .. > >action vxlan encap src_ip 11.11.0.1 dst_ip 11.11.0.2 key_id 11 .... > >action mirred egress redirect dev eth0 > > > >> > >> # DECAP rule > >> tc filter add dev $VXLAN protocol ip parent ffff: prio 10 \ > >> flower \ > >> enc_src_ip 11.11.0.2 enc_dst_ip 11.11.0.1 enc_key_id 11 \ > >> ip_proto 1 \ > >> action mirred egress redirect dev $ETH > >> > > > >And a decap would be of the form: > >tc filter add dev $ETH protocol ip parent ffff: prio 10 .. > >action vxlan decap > > That's right. Amir, don't you need decap here to drop the tunnel > metadata? Right. will add a decap that will release it. > > > > > >i.e there is no redirect needed here, no? > > > >cheers, > >jamal