svn commit: r258328 - head/sys/net

George Neville-Neil gnn at freebsd.org
Wed Nov 20 22:29:56 UTC 2013


On Nov 20, 2013, at 17:02 , Adrian Chadd <adrian at freebsd.org> wrote:

> Can we revert this and just quickly put together something else that
> won't potentially come back to bite us with weird side effects?
> 
> My suggestions (and I'm happy to do this work if needed):
> 
> * create a lightweight mbuf queue representation so an mbuf list isn't
> just the mbuf header pointer;
> * create a new ether input (say, ether_input_multi) that takes an mbuf
> list, so existing driver code does the right thing.
> 
> After that it'd be nice to write a set of mbuf list macros for
> abstract the whole queue, dequeue, concat, iterate, etc (like
> sys/queue.h, but for mbufs.)
> 
> What do people think?
> 
> (I've been doing it for m->next chained things, but not m->m_nextpkt things..)

I think the right way to do this is to move forwards and not backwards.
This change has the nice effect of not breaking anything else in the
tree while providing us with a feature that will be useful.  

If you want to add an ether_input_multi() that might make sense as an
adjunct to what’s there now.

Best,
Geoge
 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 203 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20131120/0cec9f0b/attachment.sig>


More information about the svn-src-head mailing list