OK, no need to keep unused functions.
> On 14.10.2016, at 18:47, Rafael Zalamena <rzalam...@gmail.com> wrote:
>
> The switch(4) device has a function called switch_forward_flooder()
> which doesn't seem to be used anywhere.
>
> In switchofp.c we have the swofp_action_output() which would be the place
> where it would be likely called, however it already has the code that
> does it.
>
> Since it doesn't seem to fit anywhere I think we should just removed it,
> and that's what this diff does.
>
> ok?
>
> Index: net/if_switch.c
> ===================================================================
> RCS file: /home/obsdcvs/src/sys/net/if_switch.c,v
> retrieving revision 1.9
> diff -u -p -r1.9 if_switch.c
> --- net/if_switch.c 8 Oct 2016 23:36:10 -0000 1.9
> +++ net/if_switch.c 14 Oct 2016 13:28:18 -0000
> @@ -85,8 +85,6 @@ int switch_stop(struct ifnet *, int);
> struct mbuf
> *switch_port_ingress(struct switch_softc *, struct ifnet *,
> struct mbuf *);
> -void switch_forward_flooder(struct switch_softc *,
> - struct switch_flow_classify *, struct mbuf *);
> void switch_port_egress(struct switch_softc *, struct switch_fwdp_queue *,
> struct mbuf *);
> int switch_ifenqueue(struct switch_softc *, struct ifnet *,
> @@ -725,25 +720,6 @@ switch_port_ingress(struct switch_softc
> #endif /* NPF */
>
> return (m);
> -}
> -
> -void
> -switch_forward_flooder(struct switch_softc *sc,
> - struct switch_flow_classify *swfcl, struct mbuf *m)
> -{
> - struct switch_port *swpo;
> - struct switch_fwdp_queue fwdp_q;
> - uint32_t src_port_no;
> -
> - src_port_no = swfcl->swfcl_in_port;
> - TAILQ_INIT(&fwdp_q);
> - TAILQ_FOREACH(swpo, &sc->sc_swpo_list, swpo_list_next) {
> - if (swpo->swpo_port_no == src_port_no)
> - continue;
> - TAILQ_INSERT_HEAD(&fwdp_q, swpo, swpo_fwdp_next);
> - }
> -
> - switch_port_egress(sc, &fwdp_q, m);
> }
>
> void
>