Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 12 Mar 2010 10:21:47 -0700 (MST)
From:      "M. Warner Losh" <imp@bsdimp.com>
To:        jmallett@freebsd.org
Cc:        svn-src-head@freebsd.org, qingli@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, rwatson@freebsd.org
Subject:   Re: svn commit: r205024 - head/sys/net
Message-ID:  <20100312.102147.1139860627944844739.imp@bsdimp.com>
In-Reply-To: <eaa228be1003111535s12febe62x91124b0a015fd0ff@mail.gmail.com>
References:  <alpine.BSF.2.00.1003112128020.97017@fledge.watson.org> <9ace436c1003111530s3bd0de9cq451671909fb6aa64@mail.gmail.com> <eaa228be1003111535s12febe62x91124b0a015fd0ff@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
In message: <eaa228be1003111535s12febe62x91124b0a015fd0ff@mail.gmail.co=
m>
            Juli Mallett <jmallett@FreeBSD.org> writes:
: On Thu, Mar 11, 2010 at 15:30, Qing Li <qingli@freebsd.org> wrote:
: >>
: >> A couple of questions:
: >>
: >> (1) It used to be the case that quite a few interface drivers and =
types
: >> didn't have a notion of "link up" -- especially older ethernet dev=
ices. =A0Do
: >> those all have the same problem? =A0It was probably a design overs=
ight that
: >>=A0devices don't declare an explicit capability for "can report lin=
k state".
: >>
: >
: > =A0What you raised is definitely a possibility and these fixes take=
 the
: > =A0similar approach. I am going to try and go through each of these=

: > =A0drivers in /sys/dev/ and converting them, very soon.
: =

: Go through drivers in the embedded port directories, too.  The Octeon=

: port's Ethernet driver was broken by this, and it looks like the
: Atheros if_arge is probably broken, too.  I would even suggest going
: back to the old behavior briefly while the port maintainers are given=

: an opportunity to update their drivers.  Actually, it looks like only=

: MIPS has Ethernet drivers outside of dev/ at a quick glance, but I'd
: be surprised if there weren't other broken examples.

arm has them as well.  sys/arm/at91/if_ate.c.  Powerpc does as well.

Warner



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100312.102147.1139860627944844739.imp>