Stephen Hemminger wrote:
>> +struct netfront_info {
>> +    struct list_head list;
>> +    struct net_device *netdev;
>> +
>> +    struct net_device_stats stats;
>>     
>
> There is now a net_device_stats element inside net_device on
> 2.6.21 or later.
>   

Ah, OK.  Should I just do a s/stats/netdev->stats/?  Is there a generic
get_stats routine as well?

>> +
>> +    struct xen_netif_tx_front_ring tx;
>> +    struct xen_netif_rx_front_ring rx;
>> +
>> +    spinlock_t   tx_lock;
>> +    spinlock_t   rx_lock;
>>     
>
> It might be a performance advantage to reorder/align these
> structure elements to put transmit hot elements together, and
> put tx and rx on different cache lines?
>   

Oh, right.  I'd been meaning to look at that layout more closely.

Thanks,
    J
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to