From: Ido Schimmel <idosch@idosch.org>
To: Ilan Tayari <ilant@mellanox.com>
Cc: Jiri Pirko <jiri@resnulli.us>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"davem@davemloft.net" <davem@davemloft.net>,
Ido Schimmel <idosch@mellanox.com>, Elad Raz <eladr@mellanox.com>,
Yotam Gigi <yotamg@mellanox.com>,
Nogah Frankel <nogahf@mellanox.com>,
Or Gerlitz <ogerlitz@mellanox.com>
Subject: Re: [patch net 5/9] mlxsw: spectrum: Add missing packet traps
Date: Mon, 15 Aug 2016 22:56:31 +0300 [thread overview]
Message-ID: <20160815195631.GA29013@splinter> (raw)
In-Reply-To: <AM4PR0501MB1940ACE440494A1D8018FC46DB120@AM4PR0501MB1940.eurprd05.prod.outlook.com>
On Mon, Aug 15, 2016 at 02:51:20PM +0000, Ilan Tayari wrote:
> > Add the following traps:
> >
> > 1) MTU Error: Trap packets whose size is bigger than the egress RIF's MTU. If DF
> > bit isn't set, traffic will continue to be routed in slow path.
> >
> > 2) TTL Error: Trap packets whose TTL expired. This allows traceroute to work
> > properly.
> >
> > 3) OSPF packets.
>
> Don't you need LBERROR as well?
Yep. Will send v2 tomorrow morning.
Thanks!
next prev parent reply other threads:[~2016-08-15 19:56 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-15 14:37 [patch net 0/9] mlxsw: couple of fixes Jiri Pirko
2016-08-15 14:37 ` [patch net 1/9] mlxsw: spectrum: Don't return upon error in removal path Jiri Pirko
2016-08-15 14:37 ` [patch net 2/9] mlxsw: spectrum: Remove redundant errors from the code Jiri Pirko
2016-08-15 14:37 ` [patch net 3/9] mlxsw: spectrum: Create PVID vPort before registering netdevice Jiri Pirko
2016-08-15 14:37 ` [patch net 4/9] mlxsw: spectrum: Mark port as active before registering it Jiri Pirko
2016-08-15 14:37 ` [patch net 5/9] mlxsw: spectrum: Add missing packet traps Jiri Pirko
2016-08-15 14:51 ` Ilan Tayari
2016-08-15 19:56 ` Ido Schimmel [this message]
2016-08-15 14:37 ` [patch net 6/9] mlxsw: spectrum: Add missing rollbacks in error path Jiri Pirko
2016-08-15 14:37 ` [patch net 7/9] mlxsw: spectrum: Unmap 802.1Q FID before destroying it Jiri Pirko
2016-08-15 14:37 ` [patch net 8/9] mlxsw: spectrum: Allow packets to be trapped from any PG Jiri Pirko
2016-08-15 14:37 ` [patch net 9/9] mlxsw: reg: Fix missing op field fill-up Jiri Pirko
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=20160815195631.GA29013@splinter \
--to=idosch@idosch.org \
--cc=davem@davemloft.net \
--cc=eladr@mellanox.com \
--cc=idosch@mellanox.com \
--cc=ilant@mellanox.com \
--cc=jiri@resnulli.us \
--cc=netdev@vger.kernel.org \
--cc=nogahf@mellanox.com \
--cc=ogerlitz@mellanox.com \
--cc=yotamg@mellanox.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 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.