Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752578AbaGSFqJ (ORCPT ); Sat, 19 Jul 2014 01:46:09 -0400 Received: from mail-pd0-f169.google.com ([209.85.192.169]:36335 "EHLO mail-pd0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751124AbaGSFqH convert rfc822-to-8bit (ORCPT ); Sat, 19 Jul 2014 01:46:07 -0400 Content-Type: text/plain; charset=gb2312 Mime-Version: 1.0 (1.0) Subject: Re: [PATCH V3] netxen: fix ethtool rx_dropped information in ethtool get_ethtool_stats() From: Ethan Zhao X-Mailer: iPad Mail (11D201) In-Reply-To: Date: Sat, 19 Jul 2014 13:45:49 +0800 Cc: Ethan Zhao , Manish Chopra , Sony Chacko , netdev , linux-kernel Content-Transfer-Encoding: 8BIT Message-Id: <049BCC6F-0A2E-421A-9873-0B7ED6FFB3AC@gmail.com> References: <1405655009-18975-1-git-send-email-ethan.zhao@oracle.com> To: Rajesh Borundia Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ?? 2014??7??19?գ?????2:21??Rajesh Borundia д???? >> -----Original Message----- >> From: Ethan Zhao [mailto:ethan.zhao@oracle.com] >> Sent: Friday, July 18, 2014 9:13 AM >> To: Manish Chopra; Sony Chacko; Rajesh Borundia; netdev >> Cc: linux-kernel; ethan.kernel@gmail.com; Ethan Zhao >> Subject: [PATCH V3] netxen: fix ethtool rx_dropped information in ethtool >> get_ethtool_stats() >> >> netxen driver has implemented netxen_nic_get_ethtool_stats() interface, >> but it doesn't collect stats.rxdropped in driver, so we will get different >> rx_dropped statistic information while using ifconfig and ethtool. >> this patch fills stats.rxdropped field with data from net core with >> dev_get_stats() just as ixgbe driver did for some of its stats. > > I think ethtool stats are adapter specific. Driver does not maintain > rx_dropped stats and we also don't get it from adapter. I am not > sure if ethtool stats should match ifconfig stats as ifconfig also > adds net core stats to adapter stats. I held the same point as yours before I read more drivers implementation under Ethernet dir, in fact there is no specification/convention that ethtool info is adapter Specific. All information output by ethtool is based on the information name itself, One fact we want this patch is netxen defined 'rx dropped' field but output nothing. > > If ethtool stats should report same values as ifconfig stats then > we may also need to fix tx_dropped statistics. > at least, tx dropped output information... Thanks, Ethan > Thanks, > Rajesh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/