| To: | fubar@xxxxxxxxxx, bonding-devel@xxxxxxxxxxxxxxxxxxxxx |
|---|---|
| Subject: | [PATCH 2.6.13-rc1 9/17] bonding: get primary name from slave dev |
| From: | Radheka Godse <radheka.godse@xxxxxxxxx> |
| Date: | Fri, 1 Jul 2005 13:49:52 -0700 (PDT) |
| Cc: | netdev@xxxxxxxxxxx |
| Replyto: | "Radheka Godse" <radheka.godse@intel.com> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
|
This patch fixes a bug in the proc file handler in bonding. The name of the primary slave was taken from the command-line options, instead of from the dev structure of the primary slave itself. This caused an incorrect dispaly if t he primary is set or changed via sysfs. Signed-off-by: Radheka Godse <radheka.godse@xxxxxxxxx> Signed-off-by: Mitch Williams <mitch.a.williams@xxxxxxxxx> diff -urN -X dontdiff linux-2.6.12post/drivers/net/bonding/bond_main.c linux-2.6.12post-sysfs/drivers/net/bonding/bond_main.c --- linux-2.6.12post/drivers/net/bonding/bond_main.c 2005-06-28 18:18:03.000000000 -0700 +++ linux-2.6.12post-sysfs/drivers/net/bonding/bond_main.c 2005-06-30 13:53:55.000000000 -0700 @@ -3369,8 +3369,8 @@ if (USES_PRIMARY(bond->params.mode)) { seq_printf(seq, "Primary Slave: %s\n", - (bond->params.primary[0]) ? - bond->params.primary : "None"); + (bond->primary_slave) ? + bond->primary_slave->dev->name : "None"); seq_printf(seq, "Currently Active Slave: %s\n",
(curr) ? curr->dev->name : "None"); |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 2.6.13-rc1 8/17] bonding: SYSFS INTERFACE (large), Radheka Godse |
|---|---|
| Next by Date: | [PATCH 2.6.13-rc1 10/17] bonding: added missing mode descriptions to modinfo, Radheka Godse |
| Previous by Thread: | [PATCH 2.6.13-rc1 8/17] bonding: SYSFS INTERFACE (large), Radheka Godse |
| Next by Thread: | [PATCH 2.6.13-rc1 10/17] bonding: added missing mode descriptions to modinfo, Radheka Godse |
| Indexes: | [Date] [Thread] [Top] [All Lists] |