tech-net archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

RSVP_ISI



There is a lot of dead code under #ifdef RSVP_ISI. I would like to remove it
entirely. My remarks are:

 * RSVP_ISI was added 22 years ago. The commit message says that it is a
   "working snapshot of work in progress". Since, it has almost not moved.

 * If you enabled RSVP_ISI in a config file, the option wouldn't build
   correctly, because there is no defflag.

 * RSVP_ISI makes the code in ip_mroute.c more complicated to understand. In
   fact, it pollutes the whole thing.

 * The functions are not called. Eg ip_rsvp_vif_done() is never called.

 * FreeBSD and OpenBSD both removed it, as general cleanup. The former 8
   years ago, the latter 4 years ago.

Maxime


Home | Main Index | Thread Index | Old Index