On Mon, 2007-10-08 at 10:29 -0700, Stephen Hemminger wrote:
>

quoted text
> Looks good.  Some minor stuff:

> * You can use network device stats in network device structure and

>   no longer need the copy in bp

We can't use net device stats in the bp because the elements are also

used by HW and we can't rearrange them.

struct bmac_stats and struct emac_stats are written to by the HW

struct nig_stats and struct bnx2x_eth_stats are read by HW

(I would have loved to get rid of at least one of these long structs.)
> * The MACRO's for 64 bit stats look like they could be done with

quoted text
>   u64 and/or turned into inline's.

The MACRO's modify some of their arguments, plus they need to work on 32

bit machines (are 64 bit counters always available on 32 bit machines?).

so using an inline would allow the inline to be more readable but

calling it would get ugly.

I'm open to suggestions.
> * RCS/CVS tags for date and version # are kind of ugly. They git system

quoted text
>   doesn't use them, perhaps you just want to track your internal version

>   control information, but what happens when changes come from outside?

We need some way to correlate cvs to opensource to bug tracking tool.

Note that driver and Microcode code are very tightly coupled on this

device, so if I want to debug a microcode related problem I would need

both CVS version and microcode version.
I agree that tags are ugly but can't think of a better way to do it.

Changes coming from the outside are a problem, but using the cvs version

as an anchor I can track them using git.

Again, I'm open to suggestions and if someone can think of an elegant

way of doing it, I will gladly use it.
thanks

Eliezer
-

To unsubscribe from this list: send the line "unsubscribe netdev" in

the body of a message to majordomo@vger.kernel.org

More majordomo info at  http://vger.kernel.org/majordomo-info.html