User: Password:
|
|
Subscribe / Log in / New account

Re: [ANNOUNCE] Experimental Driver for Neterion/S2io 10GbE Adapters

From:  "David S. Miller" <davem-AT-davemloft.net>
To:  <alex-AT-neterion.com>
Subject:  Re: [ANNOUNCE] Experimental Driver for Neterion/S2io 10GbE Adapters
Date:  Mon, 14 Mar 2005 12:38:15 -0800
Cc:  netdev-AT-oss.sgi.com, leonid-AT-neterion.com, jgarzik-AT-pobox.com
Archive-link:  Article, Thread

On Mon, 14 Mar 2005 12:22:51 -0800
"Alex Aizman" <alex@neterion.com> wrote:

> For these reviewers who consider this a minus, we hope you will find the HAL
> code in full compliance with Linux guidelines (in fact, it was written by
> our Linux team). Performance-wise, there was no negative impact discovered
> either. Testing-wise, this HAL has undergone numerous stress, functional,
> and performance tests "under" different drivers on a variety of platforms.

So you wrote a non-HAL version of this driver and compared the
results?  Simply comparing against the existins s2io driver
does not count.

If you're simply comparing against s2io, and your driver is faster
than s2io is already, imagine how much faster it might be without
that HAL layer.

I totally reject this driver, HAL is unacceptable for in-tree drivers.
We've been over this a thousand times.




(Log in to post comments)


Copyright © 2005, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds