| 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
| ||
|
Message-ID: <4F836364.6070406@intel.com> Date: Mon, 09 Apr 2012 15:32:04 -0700 From: John Fastabend <john.r.fastabend@...el.com> To: Stephen Hemminger <shemminger@...tta.com> CC: roprabhu@...co.com, mst@...hat.com, stephen.hemminger@...tta.com, davem@...emloft.net, hadi@...erus.ca, bhutchings@...arflare.com, jeffrey.t.kirsher@...el.com, netdev@...r.kernel.org, gregory.v.rose@...el.com, krkumar2@...ibm.com, sri@...ibm.com Subject: Re: [net-next PATCH v1 0/7] Managing the forwarding database(FDB) On 4/9/2012 3:15 PM, Stephen Hemminger wrote: > On Mon, 09 Apr 2012 15:00:11 -0700 > John Fastabend <john.r.fastabend@...el.com> wrote: > >> The following series is a submission for net-next to allow >> embedded switches and other stacked devices other then the >> Linux bridge to manage a forwarding database. >> >> This was previously posted here (where it was deferred for >> more review and testing) >> >> http://lists.openwall.net/netdev/2012/03/19/26 >> >> This series adds macvlan support per discussions with Roopa >> and Michael. Also ixgbe was updated to support adding multicast >> addresses per request from Greg Rose. >> >> Finally cleanups in the generic dump routines were added >> for multicast to support ixgbe and macvlan use cases. >> >> Thanks to everyone for the helpful review and comments. As >> always any comments/feedback welcome. >> >> .John >> >> --- >> >> Greg Rose (1): >> ixgbe: UTA table incorrectly programmed >> >> John Fastabend (6): >> macvlan: add FDB bridge ops and new macvlan mode >> ixgbe: allow RAR table to be updated in promisc mode >> ixgbe: enable FDB netdevice ops >> net: add fdb generic dump routine >> net: addr_list: add exclusive dev_uc_add and dev_mc_add >> net: add generic PF_BRIDGE:RTM_ FDB hooks >> >> >> drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 121 ++++++++++---- >> drivers/net/macvlan.c | 60 ++++++- >> include/linux/if_link.h | 1 >> include/linux/neighbour.h | 3 >> include/linux/netdevice.h | 28 +++ >> include/linux/rtnetlink.h | 4 >> net/bridge/br_device.c | 3 >> net/bridge/br_fdb.c | 128 ++++----------- >> net/bridge/br_netlink.c | 12 - >> net/bridge/br_private.h | 15 +- >> net/core/dev_addr_lists.c | 97 ++++++++++-- >> net/core/rtnetlink.c | 209 +++++++++++++++++++++++++ >> 12 files changed, 508 insertions(+), 173 deletions(-) >> > > How do statistics work in this case? What if you wanted to do BRIDGE MIB? I expect we will need to do the same sort of thing for PF_BRIDGE:RTM_GETLINK to dump the statistics for an embedded or stacked bridge. This series only gets the forwarding database setup. All the other entries in the MIB still need to be populated. What I would like is to be able to write a BRIDGE MIB implementation and have it work with any of the bridging components in the linux kernel. But I don't think that should block this series. Make sense? Based on a quick scan I'm not sure the existing net/bridge code has one place to pull all the stats from anyways looks like you might have to read some sysfs entries AND do some netlink commands. I would have to check that though. It is on my todo list to create a bridge mib for some of the EVB cases. Of course if someone else got to it that would be great. .John -- 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
Powered by blists - more mailing lists