From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: Hardware capabilities and bonding offload Date: Wed, 18 Nov 2015 15:29:23 +0100 Message-ID: <20151118142923.GA2192@nanopsycho.orion> References: <77EF4405DD4BB54AACCE7DB593DF6A9AA0D643@SJEXCHMB14.corp.ad.broadcom.com> <20151116153032.GC2217@nanopsycho.orion> <5649FFF3.9080401@gmail.com> <20151117220349.GA7762@penelope.isobedori.kobe.vergenet.net> <564BCD12.4010804@gmail.com> <20151118140512.GC6123@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: John Fastabend , Simon Horman , Premkumar Jonnala , "netdev@vger.kernel.org" To: Andrew Lunn Return-path: Received: from mail-wm0-f53.google.com ([74.125.82.53]:34871 "EHLO mail-wm0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932297AbbKRO31 (ORCPT ); Wed, 18 Nov 2015 09:29:27 -0500 Received: by wmdw130 with SMTP id w130so200979024wmd.0 for ; Wed, 18 Nov 2015 06:29:25 -0800 (PST) Content-Disposition: inline In-Reply-To: <20151118140512.GC6123@lunn.ch> Sender: netdev-owner@vger.kernel.org List-ID: Wed, Nov 18, 2015 at 03:05:12PM CET, andrew@lunn.ch wrote: >> To be honest though this is more of an argument in theory versus >> some existing management agent I know of today. If you need to do >> bonding type X in your network and the particular switch doesn't support >> it I'm not even sure what the mgmt layer is going to do. Maybe just >> put the switch offline for that network segment. >> >> If you leave the sw bit out in the first iteration I'm OK with that >> we can easily add it when we have software that needs it. > >Taking a step back... > >Have we defined a consistent way for signalling: > >1) Failed to offload to the hardware, because the hardware cannot do > what you requested. >2) Do this in software, rather than trying and failing to offload to > hardware. > >At least in DSA, we return EOPNOTSUP for 1). Well for example in case of bonding there is quite impossible to do things in software in case the hardware datapath simply cannot pass packets to kernel. Driver should know and should forbid such non-functional setup. > > Andrew > >-- >To unsubscribe from this list: send the line "unsubscribe netdev" in >the body of a message to majordomo@vger.kernel.org >More majordomo info at http://vger.kernel.org/majordomo-info.html