K 10
svn:author
V 7
glebius
K 8
svn:date
V 27
2012-09-10T11:38:02.445275Z
K 7
svn:log
V 1422
Merge r238002, r238016, r238092 from head:

r238002 by tuexen@:
  Remove dead code (on FreeBSD) as suggested by glebius@.

r238016:
  Remove route caching from IP multicast routing code. There is no
  reason to do that, and also, cached route never got unreferenced,
  which meant a reference leak.

  Reviewed by:  bms

r238092:
  When ip_output()/ip6_output() is supplied a struct route *ro argument,
  it skips FLOWTABLE lookup. However, the non-NULL ro has dual meaning
  here: it may be supplied to provide route, and it may be supplied to
  store and return to caller the route that ip_output()/ip6_output()
  finds. In the latter case skipping FLOWTABLE lookup is pessimisation.

  The difference between struct route filled by FLOWTABLE and filled
  by rtalloc() family is that the former doesn't hold a reference on
  its rtentry. Reference is hold by flow entry, and it is about to
  be released in future. Thus, route filled by FLOWTABLE shouldn't
  be passed to RTFREE() macro.

  - Introduce new flag for struct route/route_in6, that marks route
    not holding a reference on rtentry.
  - Introduce new macro RO_RTFREE() that cleans up a struct route
    depending on its kind.
  - All callers to ip_output()/ip6_output() that do supply non-NULL
    but empty route should use RO_RTFREE() to free results of
    lookup.
  - ip_output()/ip6_output() now do FLOWTABLE lookup always when
    ro->ro_rt == NULL.

END