cvs commit: src/share/man/man4 miibus.4 polling.4 vlan.4 src/sys/config LINT src/sys/boot/forth loader-bootp.conf loader.conf

[Sepherosa Ziehau]

On 6/1/07, Christian Brueffer <chris at unixpages.org> wrote:
On Mon, May 28, 2007 at 06:50:44PM +0800, Sepherosa Ziehau wrote:
> On 5/28/07, Christian Brueffer <chris at unixpages.org> wrote:
> >Hi,
> >
> >have you actually tested bce(4) polling?  The polling code in the
> >FreeBSD driver is broken (made the mistake of adding bce(4) to
> >polling(4) as well ;-)
>
> It works and I have tested it (test polling in dfly is much easier...)
>
> I have adjusted following two things:
> 1) process POLL_AND_CHECK_STATUS in bce_poll(), which will affect link
> status.
> 2) in bce_rx_intr(), we can't just process 'count' packets and leave,
> we need to process until there are no remaining packets in the current
> round, but don't update RX consumer index again in as we do in RX intr
> processing.
>
> 1) was made before my first round polling testing, so I don't know
> whether it will have great impact on the bce(4).  But 2) does make
> things work.
>
Nice to hear, thanks.  Do you plan to port these changes back to FreeBSD
yourself?
I may contact davidch on that, he is the maintainer.

Best Regards,
sephe
--
Live Free or Die