netdev - Re: [PATCH iproute2] route: label externally offloaded routes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150309175338.GB22009@tuxdriver.com>
Date:	Mon, 9 Mar 2015 13:53:38 -0400
From:	"John W. Linville" <linville@...driver.com>
To:	jeffrey.t.kirsher@...el.com
Cc:	Scott Feldman <sfeldma@...il.com>,
	Andy Gospodarek <gospo@...ulusnetworks.com>,
	Netdev <netdev@...r.kernel.org>,
	"stephen@...workplumber.org" <stephen@...workplumber.org>,
	Jiří Pírko <jiri@...nulli.us>
Subject: Re: [PATCH iproute2] route: label externally offloaded routes

On Mon, Mar 09, 2015 at 09:51:35AM -0700, Jeff Kirsher wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA512
> 
> On 03/09/2015 09:16 AM, Scott Feldman wrote:
> > On Mon, Mar 9, 2015 at 7:02 AM, Andy Gospodarek 
> > <gospo@...ulusnetworks.com> wrote:
> >> On Sat, Mar 07, 2015 at 10:15:35PM -0800, sfeldma@...il.com
> >> wrote:
> >>> From: Scott Feldman <sfeldma@...il.com>
> >>> 
> >>> On ip route print dump, label externally offloaded routes with
> >>> "external". Offloaded routes are flagged with RTNH_F_EXTERNAL,
> >>> a recent additon to net-next.  For example:
> >>> 
> >>> $ ip route default via 192.168.0.2 dev eth0 11.0.0.0/30 dev
> >>> swp1  proto kernel  scope link  src 11.0.0.2 external 
> >>> 11.0.0.4/30 via 11.0.0.1 dev swp1  proto zebra  metric 20
> >>> external 11.0.0.8/30 dev swp2  proto kernel  scope link  src
> >>> 11.0.0.10 external 11.0.0.12/30 via 11.0.0.9 dev swp2  proto
> >>> zebra  metric 20 external 12.0.0.2  proto zebra  metric 30
> >>> external nexthop via 11.0.0.1  dev swp1 weight 1 nexthop via
> >>> 11.0.0.9  dev swp2 weight 1 12.0.0.3 via 11.0.0.1 dev swp1
> >>> proto zebra  metric 20 external 12.0.0.4 via 11.0.0.9 dev swp2
> >>> proto zebra  metric 20 external 192.168.0.0/24 dev eth0  proto
> >>> kernel  scope link  src 192.168.0.15
> >>> 
> >>> Signed-off-by: Scott Feldman <sfeldma@...il.com>
> >> 
> >> You know we cannot have a thread about nomenclature without a
> >> comment from me.  ;)
> >> 
> >> My only concern about 'external' is whether or not people will
> >> gloss over it since many of the routes listed will be to networks
> >> that actually are external to the system.  That namespace
> >> collision could seem awkward if you are not a traditional
> >> follower of this list. Listing an FDB entry as 'external' does
> >> not have the same issue since all FDB entries are local to the
> >> system, so I didn't even think about the potential when that set
> >> was posted.
> >> 
> >> I would like to just call this 'hardware' since that is what we
> >> appear to be using for offload and would make it clear to the
> >> user that this route was in hardware as well as in the kernel.
> >> I'd say the same for FDB entries, too.
> >> 
> >> Scott, you have lots of experience in both traditional and Linux
> >> worlds, do you think my concern about confusion is unnecessary?
> > 
> > Votes so far: 1 for "external", 1 for "hardware", 1 for "offload"
> > or "hw".
> > 
> > Good point about confusion wrt routes to non-local networks.
> 
> My $0.02, "hardware" seems most appropriate to me.

Ditto...

-- 
John W. Linville		Someday the world will need a hero, and you
linville@...driver.com			might be all we have.  Be ready.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html