Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758550AbbGQPER (ORCPT ); Fri, 17 Jul 2015 11:04:17 -0400 Received: from mx1.redhat.com ([209.132.183.28]:45509 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758527AbbGQPEP (ORCPT ); Fri, 17 Jul 2015 11:04:15 -0400 From: Vitaly Kuznetsov To: Dexuan Cui Cc: David Miller , "olaf\@aepfle.de" , "gregkh\@linuxfoundation.org" , "jasowang\@redhat.com" , "driverdev-devel\@linuxdriverproject.org" , "linux-kernel\@vger.kernel.org" , "stephen\@networkplumber.org" , "netdev\@vger.kernel.org" , "apw\@canonical.com" , "pebolle\@tiscali.nl" Subject: Re: [V2 6/7] hvsock: introduce Hyper-V VM Sockets feature References: <1436868048-17875-1-git-send-email-decui@microsoft.com> <20150715.211916.1258342954251964451.davem@davemloft.net> Date: Fri, 17 Jul 2015 17:04:10 +0200 In-Reply-To: (Dexuan Cui's message of "Thu, 16 Jul 2015 11:19:01 +0000") Message-ID: <87fv4mn7v9.fsf@vitty.brq.redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1072 Lines: 37 Dexuan Cui writes: >> From: David Miller >> Sent: Thursday, July 16, 2015 12:19 >> >> From: Dexuan Cui >> Date: Tue, 14 Jul 2015 03:00:48 -0700 >> >> > + pr_debug("hvsock_sk_destruct: called\n"); >> >> Debug logging just to state that a function is called is not appropriate, >> we have very sophisticated tracing facilities in the kernel that can do >> that transparently, and more. >> >> Please remove this. > OK. > >> > + if (hvsk->channel) { >> > + pr_debug("hvsock_sk_destruct: calling vmbus_close()\n"); >> >> Likewise, these kinds of debug logs are totally inappropriate. > OK, I'll remove all the pr_debug() in the patch. > I'd suggest we rather use something like net_dbg_ratelimited() intead. The driver is new so issues are expected. Some debugging may be useful) [...] -- Vitaly -- 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/