cvs commit: src/sys/dev/netif/iwi if_iwi.c

Mon Mar 7 08:59:23 PST 2005

On Mon, Mar 07, 2005 at 04:42:47PM +0000, Andrew Atrens wrote:
> I wasn't sure about that. In general then we want to deprecate
> the use of the IF_PREPEND macro ?

You can't use IF_PREPEND on if_snd and activate ALTQ.
> $ find . | xargs grep IF_PREPEND
>> This one's okay, but maybe there's another way to do the same thing
> without prepending ? -
>> ./awi/awi.c: IF_PREPEND(&sc->sc_mgtq, m0);

It's an internal queue, that's fine.
> These ones need to be re-worked too I guess in preparation for ALTQ -
>> ./de/if_de.c: IF_PREPEND(&sc->tulip_if.if_snd, m);

I haven't touched this yet, I was looking for the right way to tackle
this beast.
> ./le/if_le.c: IF_PREPEND(&sc->le_if.if_snd, di->di_mbuf);
> ./le/if_le.c: IF_PREPEND(&ifp->if_snd, m);
> ./le/if_le.c: IF_PREPEND(&ifp->if_snd, m);

Just the same.
> ./vr/if_vr.c: IF_PREPEND(&ifp->if_snd, m_head);

/me scratches his head
This was on the DONE list. Must have got lost somewhere.
Joerg