cvs commit: src/sys/net if.c if_ethersubr.c src/sys/netgraph/ether ng_ether.c src/sys/netgraph/fec ng_fec.c src/sys/netinet if_ether.c ip_carp.c

[Sepherosa Ziehau <sephe@xxxxxxxxxxxxxxxxxxxxxxx>]

sephe       2008/05/28 05:11:13 PDT

DragonFly src repository

  Modified files:
    sys/net              if.c if_ethersubr.c 
    sys/netgraph/ether   ng_ether.c 
    sys/netgraph/fec     ng_fec.c 
    sys/netinet          if_ether.c ip_carp.c 
  Log:
  - ifnet.if_output() should be called without ifnet.if_serializer being
    held.  Add assertion about it in ether_output().
  - ether_output_frame() should be called without the ifnet.if_serializer
    being held.  Add assertion in it.
  - arp_ifinit() will be called with ifnet.if_serializer being held.  To
    prevent serializer from recursion, ifnet.if_serializer is released
    before calling arprequest(), which supposes caller does not hold output
    iface's serializer.
  - ifnet.if_serializer can't be held when calling arp_ifinit2().
  
  Reported-by: dillon@
  
  Revision  Changes    Path
  1.67      +14 -17    src/sys/net/if.c
  1.62      +4 -0      src/sys/net/if_ethersubr.c
  1.16      +0 -3      src/sys/netgraph/ether/ng_ether.c
  1.26      +3 -2      src/sys/netgraph/fec/ng_fec.c
  1.47      +8 -1      src/sys/netinet/if_ether.c
  1.8       +0 -2      src/sys/netinet/ip_carp.c


http://www.dragonflybsd.org/cvsweb/src/sys/net/if.c.diff?r1=1.66&r2=1.67&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/net/if_ethersubr.c.diff?r1=1.61&r2=1.62&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/netgraph/ether/ng_ether.c.diff?r1=1.15&r2=1.16&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/netgraph/fec/ng_fec.c.diff?r1=1.25&r2=1.26&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/netinet/if_ether.c.diff?r1=1.46&r2=1.47&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/netinet/ip_carp.c.diff?r1=1.7&r2=1.8&f=u