From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [GIT/PATCH v4] xen network backend driver Date: Thu, 10 Mar 2011 09:15:55 -0800 Message-ID: <20110310091555.0fa8764d@nehalam> References: <1299776554.17339.824.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "netdev@vger.kernel.org" , xen-devel , Ben Hutchings , Jeremy Fitzhardinge , Herbert Xu , Konrad Rzeszutek Wilk , Francois Romieu To: Ian Campbell Return-path: Received: from mail.vyatta.com ([76.74.103.46]:55887 "EHLO mail.vyatta.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752425Ab1CJRP6 (ORCPT ); Thu, 10 Mar 2011 12:15:58 -0500 In-Reply-To: <1299776554.17339.824.camel@zakaz.uk.xensource.com> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 10 Mar 2011 17:02:33 +0000 Ian Campbell wrote: > +struct xenvif { > + /* Unique identifier for this interface. */ ... > + struct net_device_stats stats; > + There is already a stats struct in net_device in current kernel versions, unless there is a compelling reason otherwise please use that. Also, you probably want to implement per-cpu and 64 bit stats. --