Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 25 Jun 2009 09:18:47 -0400
From:      George Neville-Neil <gnn@neville-neil.com>
To:        Navdeep Parhar <np@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r194918 - head/sys/net
Message-ID:  <29388187-3617-40D4-ACDA-DECD97F8274F@neville-neil.com>
In-Reply-To: <200906242153.n5OLrPEV030916@svn.freebsd.org>
References:  <200906242153.n5OLrPEV030916@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On Jun 24, 2009, at 17:53 , Navdeep Parhar wrote:

> Author: np
> Date: Wed Jun 24 21:53:25 2009
> New Revision: 194918
> URL: http://svn.freebsd.org/changeset/base/194918
>
> Log:
>  Add 10Gbase-T to known ethernet media types.
>
>  Approved by:	gnn (mentor)
>  MFC after:	1 week.
>
> Modified:
>  head/sys/net/if_media.h
>
> Modified: head/sys/net/if_media.h
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- head/sys/net/if_media.h	Wed Jun 24 21:51:42 2009	(r194917)
> +++ head/sys/net/if_media.h	Wed Jun 24 21:53:25 2009	(r194918)
> @@ -149,6 +149,7 @@ uint64_t	ifmedia_baudrate(int);
> #define IFM_10G_TWINAX_LONG	23	/* 10GBase Twinax Long copper */
> #define IFM_10G_LRM	24		/* 10GBase-LRM 850nm Multi-mode */
> #define IFM_UNKNOWN	25		/* media types not defined yet */
> +#define IFM_10G_T	26		/* 10GBase-T - RJ45 */
>
>
> /* note 31 is the max! */
> @@ -358,6 +359,7 @@ struct ifmedia_description {
> 	{ IFM_10G_TWINAX,	"10Gbase-Twinax" },			\
> 	{ IFM_10G_TWINAX_LONG,	"10Gbase-Twinax-Long" },		\
> 	{ IFM_UNKNOWN,	"Unknown" },					\
> +	{ IFM_10G_T,	"10Gbase-T" },					\
> 	{ 0, NULL },							\
> }
>
> @@ -615,6 +617,7 @@ struct ifmedia_baudrate {
> 	{ IFM_ETHER | IFM_10G_TWINAX,	IF_Gbps(10ULL) },		\
> 	{ IFM_ETHER | IFM_10G_TWINAX_LONG,	IF_Gbps(10ULL) },	\
> 	{ IFM_ETHER | IFM_10G_LRM,	IF_Gbps(10ULL) },		\
> +	{ IFM_ETHER | IFM_10G_T,	IF_Gbps(10ULL) },		\
> 									\
> 	{ IFM_TOKEN | IFM_TOK_STP4,	IF_Mbps(4) },			\
> 	{ IFM_TOKEN | IFM_TOK_STP16,	IF_Mbps(16) },			\


Howdy,

John Baldwin pointed out to me that UNKNOWN really should have been  
pushed up to the end.

Can you make a small patch and send it to me?  Increase the number and  
move its string to the end
of the array?

Best,
George




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?29388187-3617-40D4-ACDA-DECD97F8274F>