Re: kern/110720: [net] [patch] support for interface descriptions
- From: Eugene Grosbein <eugen@xxxxxxxxxx>
- Date: Sat, 24 Mar 2007 19:37:55 +0700
On Sat, Mar 24, 2007 at 11:30:44AM +0100, Andre Oppermann wrote:
Harti Brandt wrote:
Nice feature, although it would be nice to align the maximum length with
IF-MIB::ifDescr (255 byte + \0). Also I suppose that the field more
naturally fits into struct if_data (see net/if.h) given the comment for
that struct:
/*
* Structure describing information about an interface
* which may be of interest to management entities.
*/
The string array should most likely not be part of struct ifnet as that
would bloat it quite a bit. If it is in there it should be at the end
of the struct to avoid cache line busting effects.
Also vote for this. And is it possible to make it a pointer instead
of array? The bloat would be minimal for system with tons of interfaces,
think about large pptp or pppoe server or similar that never would
utilize arrays.
Eugene Grosbein
_______________________________________________
freebsd-net@xxxxxxxxxxx mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "freebsd-net-unsubscribe@xxxxxxxxxxx"
- Follow-Ups:
- Re: kern/110720: [net] [patch] support for interface descriptions
- From: Hartmut Brandt
- Re: kern/110720: [net] [patch] support for interface descriptions
- References:
- Re: kern/110720: [net] [patch] support for interface descriptions
- From: Mark Linimon
- Re: kern/110720: [net] [patch] support for interface descriptions
- From: Harti Brandt
- Re: kern/110720: [net] [patch] support for interface descriptions
- From: Andre Oppermann
- Re: kern/110720: [net] [patch] support for interface descriptions
- Prev by Date: Re: kern/110720: [net] [patch] support for interface descriptions
- Next by Date: Re: kern/110720: [net] [patch] support for interface descriptions
- Previous by thread: Re: kern/110720: [net] [patch] support for interface descriptions
- Next by thread: Re: kern/110720: [net] [patch] support for interface descriptions
- Index(es):
Relevant Pages
|
|