From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754579AbbA2GfG (ORCPT ); Thu, 29 Jan 2015 01:35:06 -0500 Received: from ni.piap.pl ([195.187.100.4]:42287 "EHLO ni.piap.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752222AbbA2GfE (ORCPT ); Thu, 29 Jan 2015 01:35:04 -0500 From: khalasa@piap.pl (Krzysztof =?utf-8?Q?Ha=C5=82asa?=) To: Nicholas Krause Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org References: <1422143181-4479-1-git-send-email-xerofoify@gmail.com> Date: Thu, 29 Jan 2015 07:35:02 +0100 In-Reply-To: <1422143181-4479-1-git-send-email-xerofoify@gmail.com> (Nicholas Krause's message of "Sat, 24 Jan 2015 18:46:21 -0500") MIME-Version: 1.0 Message-ID: Content-Type: text/plain Subject: Re: [PATCH] net:wan:Change location of debug printk statement in the function,hss_hdlc_poll X-Anti-Virus: Kaspersky Anti-Virus for Linux Mail Server 5.6.44/RELEASE, bases: 20140401 #7726142, check: 20150129 clean Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Nicholas Krause writes: > This changes the location of the printk for montioring if the stucture pointer > desc of type structure desc has a error count due to failing in the switch > statement for checking it's status from the default switch case in this switch > to the area of the function for checking if there is a error count. Due to this > this part of the function can now be uncommented as it's now needed for printing > out important information if the structure pointer,desc has a error count for > debugging purposes. This error count seems to be secondary to the desc->status. It's isn't normally logged (RX errors on serial links are quite normal) and only the "default" case, normally not seen, would be logged (to indicate unhandled condition). In the "normal" error path, the driver ignores error count, because there could be several errors counted for a single erroneous frame (or non-frame). Also the patch seems to remove the beginning of a preprocessor directive (#if) without removing its end (#endif). I'd be surprised if it compiles. > --- a/drivers/net/wan/ixp4xx_hss.c > +++ b/drivers/net/wan/ixp4xx_hss.c > @@ -697,7 +697,6 @@ static int hss_hdlc_poll(struct napi_struct *napi, int budget) > } > > desc = rx_desc_ptr(port, n); > -#if 0 /* FIXME - error_count counts modulo 256, perhaps we should use it */ > if (desc->error_count) > printk(KERN_DEBUG "%s: hss_hdlc_poll status 0x%02X" > " errors %u\n", dev->name, desc->status, > @@ -735,9 +734,7 @@ static int hss_hdlc_poll(struct napi_struct *napi, int budget) > dev->stats.rx_length_errors++; > dev->stats.rx_errors++; > break; > - default: /* FIXME - remove printk */ > - netdev_err(dev, "hss_hdlc_poll: status 0x%02X errors %u\n", > - desc->status, desc->error_count); > + default: > dev->stats.rx_errors++; > } -- Krzysztof Halasa Research Institute for Automation and Measurements PIAP Al. Jerozolimskie 202, 02-486 Warsaw, Poland