fixes for 4.3.3 GCC warnings/errors

After fetching current git code and compiling with gcc 4.3.3 got errors
related to Werror (2 functions was ignoring return value), and ulimits.h was
not declared, but INT_MAX used
Here is fix, so rstp compile fine with gcc 4.3.3

Signed-off-by: Denys Fedoryschenko <denys@visp.net.lb>

Remove file "bridge" which was incorrectly included.

   The "bridge" script was incorrectly included into this tree and isn't
   needed here anyway. Remove it.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Remove unused RSTPLIB and related files.

   Remove all the RSTPLIB files. They aren't used anymore.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Use new RSTP library.

   This is a somewhat big commit which replaces the use of RSTPLIB in the
   RSTP daemon by the new RSTP library. This makes the daemon much more
   compliant with the 802.1D-2004 standard. The control program, rstpctl,
   has been modified so its output closer to the output of brctl showstp.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Add in new RSTP implementation to replace RSTPLIB.

   We have a new implementation of RSTP, which is meant to implement the
   Spanning Tree Protocol in IEEE 802.1D-2004. This implementation is
   written as a library consisting of one .c file and one .h file.
   It is written from scratch directly from the 802.1D-2004 document.
   This commit adds the files rstp.h and rstp.c to the directory, in
   preparation for modifying other code to use this instead of rstplib.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Removed portion of brmon.c which received BPDUs through netlink.

   At one time, BPDUs were being received through netlink using a kernel
   patch. We now receive and send BPDUs through PF_PACKET sockets, so this
   code is not needed anymore.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Put in Ethernet and LLC headers ourselves while sending BPDUs

   Take BPDU contents from RSTPLIB but create the Ethernet and LLC
   headers ourselves. The packet send interface has been modified to
   take an iovec array so we can pass the headers and data to it without
   making copies into another flat buffer.
   RSTPLIB then doesn't even need to know the port MAC addresses if we
   add an interface to configure the bridge address.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Track and cache bridge and port MAC addresses.

   Check for MAC address change when we get notifications.
   If we get a port notification and find that the port MAC address
   changed, we check if the bridge MAC address changed as well.
   This will be useful for modifying RSTPLIB to allow setting bridge
   address instead of trying to replicate the kernel bridge address
   computation.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Improve and fix validation of received BPDUs.

   Validate ethernet and LLC headers: Check destination address, length,
   and SAP fields. Also fix length checks for Config and RST BPDUs.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

Change notification processing to account for kernel changes

   We now always get NEWLINK and DELLINK notifications for bridge port
   additions and deletions. And DELLINK is only for port deletions.
   So we can always remove a bridge port on a DELLINK notification,
   and not make uncertain checks in fear of missing this.

Signed-off-by: Srinivas Aji <Aji_Srinivas@emc.com>

RSTP testing - PATCH: Typo in libnetlink.c

 Typo in libnetlink.c

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>

RSTP testing - PATCH: BPDU length calculation

   The bpdu_len sent to STP_OUT_tx_bpdu() excludes the ethernet and
   and LLC headers, though the bpdu pointer points to the start of the
   ethernet header. Given the types in rstplib/stp_bpdu.h, we need to
   be sending a packet with (base, len) given by
   (bpdu, bpdu_len + sizeof(MAC_HEADER_T) + sizeof(ETH_HEADER_T))
   MAC_HEADER_T is dest and source ethernet addresses and ETH_HEADER_T
   is the 802.2 part.

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>

RSTP testing - PATCH: source MAC address of BPDU

   BPDU's were being sent out with an all zero source ethernet
   address. These packets don't make it far in the receive path on
   another bridge since bridge_handle_frame() checks for valid source
   ethernet address and drops them if that isn't the case.

   I needed to undo a change I made in rstplib/transmit.c, where I was
   not setting source MAC address because the LLC layer was doing it.

   TODO: We get notified when a port MAC address changes, so we should
   cache it. Right now it uses SIOCGIFHWADDR each time (for each BPDU
   sent) using a cached name, which is not good.

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>

rstpd: working version

This is the first version that works with modifications to bridge
that are going into 2.6.22

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>

Simple usermode helper script

Initialize packet socket

Build brmon again

Sorry, that code was needed.

Revised packet SOCK_RAW code

Still not complete.

remove ifdef'd code

The code is no longer needed.

netlink shouldn't kill daemon

A netlink error shouldn't cause silent death of daemon.
It doesn't work well if library calls exit().

Convert to using AF_PACKET for send and receive

Use AF_PACKET to send and receive spanning tree bpdu's
UNTESTED at this point.

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>

reindent source

Use kernel indentation style. Existing style was too odd.

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>

update ignore file

fix build

Initial commit

sources from  Aji_Srinivas@emc.com