From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 158E1C433E0 for ; Wed, 10 Jun 2020 07:09:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id DA6AE20835 for ; Wed, 10 Jun 2020 07:09:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1591772993; bh=FL8sB9PMoiA3wZBbIbF3vDjHrhgweK5nKjNqzrG6KF4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=AK28qEcEu7CY5+KSaHziEqoMV3hn93z+SC7qfoeffq25HiiuAHFDMACjOVQKsfluk nfSNxDlQOXKNg96d8xqKbq+813zcm9ALTttmon1rwAJNU/MyrA7c1P2UUH9Q4CpQTz 1elxYq1rZfkmqoeKn3Ulsg1is0RyEcUQGK7bgHCw= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726260AbgFJHJw (ORCPT ); Wed, 10 Jun 2020 03:09:52 -0400 Received: from mail.kernel.org ([198.145.29.99]:51830 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726114AbgFJHJv (ORCPT ); Wed, 10 Jun 2020 03:09:51 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id DEE3A20760; Wed, 10 Jun 2020 07:09:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1591772991; bh=FL8sB9PMoiA3wZBbIbF3vDjHrhgweK5nKjNqzrG6KF4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UBFIbXiGP352wVQVe0VPRDSzfXCC6dQQiPXBLdUC5Ti9vO7UEiNmNsOsg3vYCe7Lu cVrV7rp7ISmv4ONraVrO4CZgsOzPxU7SdouxiFl05RpK3xyguCnxRHgHO6ZsDiWRtM eA2ECnnfTDzDvn5eQpoaUA6a77zy5Z6b2EnzgtmI= Date: Wed, 10 Jun 2020 09:09:49 +0200 From: Greg Kroah-Hartman To: Joe Perches Cc: Stanimir Varbanov , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-acpi@vger.kernel.org, netdev@vger.kernel.org, Jason Baron , Jonathan Corbet , Jim Cromie Subject: Re: [PATCH v3 1/7] Documentation: dynamic-debug: Add description of level bitmask Message-ID: <20200610070949.GB1923109@kroah.com> References: <20200609104604.1594-1-stanimir.varbanov@linaro.org> <20200609104604.1594-2-stanimir.varbanov@linaro.org> <20200609111615.GD780233@kroah.com> <20200610063103.GD1907120@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Tue, Jun 09, 2020 at 11:35:31PM -0700, Joe Perches wrote: > On Wed, 2020-06-10 at 08:31 +0200, Greg Kroah-Hartman wrote: > > On Tue, Jun 09, 2020 at 09:58:07AM -0700, Joe Perches wrote: > > > On Tue, 2020-06-09 at 13:16 +0200, Greg Kroah-Hartman wrote: > > > > What is wrong with the existing control of dynamic > > > > debug messages that you want to add another type of arbitrary grouping > > > > to it? > > > > > > There is no existing grouping mechanism. > > > > info/warn/err/dbg is what I am referring to. > > > > > Many drivers and some subsystems used an internal one > > > before dynamic debug. > > > > > > $ git grep "MODULE_PARM.*\bdebug\b"|wc -l > > > 501 > > > > Yes, and it's horrid and needs to be cleaned up, not added to. > > Or unified so driver authors have a standardized mechanism > rather than reinventing or doing things differently. But each "level" you all come up with will be intrepreted differently per driver, causing total confusion (like we have today.) Try to make it better by just removing that mess. > > In the beginning, yes, adding loads of different types of debugging > > options to a driver is needed by the author, but by the time it is added > > to the kernel, all of that should be able to be removed and only a > > single "enable debug" should be all that is needed. > > No one does that. We did that for USB drivers a decade ago, it can be done. greg k-h