netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* bonding: propagation of offload settings
@ 2010-10-30  2:54 Simon Horman
  2010-11-24 20:27 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: Simon Horman @ 2010-10-30  2:54 UTC (permalink / raw)
  To: netdev; +Cc: Jay Vosburgh

Hi,

I am wondering what the desired behaviour for the propagating
of offload settings between master and slaves when the settings
are modified using ethtool.

I have observed the following (using Linus' latest tree, 2.6.36+)

#1 Disabling gro on a slave device propagates to the master
   but not other slaves

   bond1: generic-receive-offload: on
   eth1: generic-receive-offload: on
   eth4: generic-receive-offload: on

   # ethtool -K eth1 gro off

   bond1: generic-receive-offload: off
   eth1: generic-receive-offload: on
   eth4: generic-receive-offload: off

   This seems to occur regardless of if the slave is the
   active slave or not.

#2 No other propagation of settings occurs


It seems to me that from a user point of view it may make more sense to:

a) propagate settings from the master to the slaves and;
b) possibly disallow setting the slaves directly


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: bonding: propagation of offload settings
  2010-10-30  2:54 bonding: propagation of offload settings Simon Horman
@ 2010-11-24 20:27 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2010-11-24 20:27 UTC (permalink / raw)
  To: horms; +Cc: netdev, fubar

From: Simon Horman <horms@verge.net.au>
Date: Sat, 30 Oct 2010 11:54:35 +0900

> It seems to me that from a user point of view it may make more sense to:
> 
> a) propagate settings from the master to the slaves and;
> b) possibly disallow setting the slaves directly

Yeah, good question.

Propagation from master to slave(s) would have difficult semantics.
If any of the slave changes fail (f.e. unsupported feature or memory
allocation failure) we'd have to unwind all of the slaves which did
accept the change without error.

What if the unwind operation fails, due to lack of resources?  A lot
of state would thus need to be tracked to support this reasonably.

However we pretty much have to respect whatever changes get made
directly to the slaves, since the master must at all times claim
support for only the lowest common denominator, feature wise, amongst
that master's slaves.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-11-24 20:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-30  2:54 bonding: propagation of offload settings Simon Horman
2010-11-24 20:27 ` David Miller

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).