From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [Bonding-devel] [PATCH 4/4] bonding: add sysfs files to display tlb and alb hash table contents Date: Mon, 28 Sep 2009 17:34:20 -0700 Message-ID: <20090928173420.07e9dfad@nehalam> References: <20090911211317.GT8515@gospo.rdu.redhat.com> <20090928162237.6db0c9f5@nehalam> <20090929001203.GE4436@gospo.rdu.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Andy Gospodarek , netdev@vger.kernel.org, fubar@us.ibm.com, bonding-devel@lists.sourceforge.net To: Andy Gospodarek Return-path: Received: from mail.vyatta.com ([76.74.103.46]:52916 "EHLO mail.vyatta.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753839AbZI2AeW (ORCPT ); Mon, 28 Sep 2009 20:34:22 -0400 In-Reply-To: <20090929001203.GE4436@gospo.rdu.redhat.com> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, 28 Sep 2009 20:12:03 -0400 Andy Gospodarek wrote: > On Mon, Sep 28, 2009 at 04:22:37PM -0700, Stephen Hemminger wrote: > > On Fri, 11 Sep 2009 17:13:17 -0400 > > Andy Gospodarek wrote: > > > > > > > > bonding: add sysfs files to display tlb and alb hash table contents > > > > > > While debugging some problems with alb (mode 6) bonding I realized that > > > being able to output the contents of both hash tables would be helpful. > > > This is what the output looks like for the two files: > > > > > > device load > > > eth1 491 > > > eth2 491 > > > hash device last device tx bytes load next previous > > > 2 eth1 eth1 2254 491 0 0 > > > 3 eth2 eth2 2744 491 0 0 > > > 6 eth2 0 488 0 0 > > > 8 eth2 0 461698 0 0 > > > 1b eth2 0 249 0 0 > > > eb eth2 0 21 0 0 > > > ff eth2 0 22 0 0 > > > > > > hash ip_src ip_dst mac_dst slave assign ntt > > > 2 10.0.3.2 10.0.3.11 00:e0:81:71:ee:a9 eth1 1 0 > > > 3 10.0.3.2 10.0.3.10 00:e0:81:71:ee:a9 eth2 1 0 > > > 8 10.0.3.2 10.0.3.1 00:e0:81:71:ee:a9 eth2 1 0 > > > > > > These were a great help debugging the fixes I have just posted and they > > > might be helpful for others, so I decided to include them in my > > > patchset. > > > > > > Signed-off-by: Andy Gospodarek > > > > No. > > > > Please don't put formatted output in sysfs, it is not meant to be > > used like proc, there is supposed to be only one value per file. > > Then based on the over 300 files in /sys/ that are more than 1 line on > my currently running kernel, it seems there is significant work to do. > > Seemingly arbitrary requests like this are extremely annoying when the > current kernel violates them all over the place. > The rules are documented in Documentation/sysfs-rules.txt. If you want to change the rules, submit a change to the rules. --