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=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED 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 BE006C4CEC8 for ; Fri, 13 Sep 2019 12:18:38 +0000 (UTC) Received: from mail.linuxfoundation.org (mail.linuxfoundation.org [140.211.169.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 930642089F for ; Fri, 13 Sep 2019 12:18:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=intel-com.20150623.gappssmtp.com header.i=@intel-com.20150623.gappssmtp.com header.b="XOYIJFsa" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 930642089F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=ksummit-discuss-bounces@lists.linuxfoundation.org Received: from mail.linux-foundation.org (localhost [127.0.0.1]) by mail.linuxfoundation.org (Postfix) with ESMTP id E55CFF9E; Fri, 13 Sep 2019 12:18:37 +0000 (UTC) Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org [172.17.192.35]) by mail.linuxfoundation.org (Postfix) with ESMTPS id 0CDB1F9B for ; Fri, 13 Sep 2019 12:18:37 +0000 (UTC) X-Greylist: whitelisted by SQLgrey-1.7.6 Received: from mail-ot1-f68.google.com (mail-ot1-f68.google.com [209.85.210.68]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id C36677DB for ; Fri, 13 Sep 2019 12:18:35 +0000 (UTC) Received: by mail-ot1-f68.google.com with SMTP id n7so29191855otk.6 for ; Fri, 13 Sep 2019 05:18:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=A/c97/nFme+ivPwJjzTzfClgEa0OIy/3njro+D9o78o=; b=XOYIJFsaopl13XxNMqGXKTMtqthGHFDixCdxfm/BRkbm8C7nqft8bOMYav+KkL38Qt QBPmqIznV8C75U0erZcJeN+FS4ORrdfnOjlRb9HcFlpbAXpaDUwEMEFfgca6deU40mKQ dAOsSZkAX1fiHcUyIlZI0MpYcLKZkJiURtJujVfDoxC7cHZxhUrtdZawEtZRorybqoKp Du310WWUJuQYwvmI+m/z4o8nvQoTbQdB9+0Eb9svR+kK5/cpRBeRAWUoHJP1VOu6fTY3 9DYzNQKc+KVnxHkBABtxBwWcts/Vcw2KusB5j9MY08YSFdZXJcIUkmnHlOhvEmYtyq/g +xmA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=A/c97/nFme+ivPwJjzTzfClgEa0OIy/3njro+D9o78o=; b=HZxc4JcujSe3H1hoUCchI+nejOEtnAlDqjc4EVD+svM21/Qix8bLz0jjXcSSUSaJTW IKhTF42yeVobsQ3ctY0l1mPNSln++0/MJppPQy7VJBJceOEeFdWDEtDcoI7mNpn14roG zVW1JtShAhPmwzgVfsO8644XopblAcHHFUh2ykRQh0mdx1PSetO0ot+gIWxxy82xrxAK N11JX0nuBMh4agpQxYysnnR3TEka1drJUrAbTUwnZPm+h29YwEEII7uvCfUbSDuAtgWC 0+aKc+lt7niTseTyP3rmaZKa5UeHNEk/4eKPs+hbtRVKtPWXSl9COajz8lkNswEhRJRy J+lA== X-Gm-Message-State: APjAAAWgjjIlw2sVwaKI2SRDA6TWcn6qA6bHxbQJxFTJQ3NLKvcn+gh+ Vpxyuh7CPHU6qbXz2l3swxlUfTh1H1rbkzT3T7Kb+w== X-Google-Smtp-Source: APXvYqyUP3UinjilPMLX5L/ce/vPrHjlyiR9Mzwouxkvl/+9B3J3lEInbbUjdvdAF29WZpR8yBBtHGAMbtEPFqknxsQ= X-Received: by 2002:a9d:2642:: with SMTP id a60mr8278939otb.247.1568377114876; Fri, 13 Sep 2019 05:18:34 -0700 (PDT) MIME-Version: 1.0 References: <156821692280.2951081.18036584954940423225.stgit@dwillia2-desk3.amr.corp.intel.com> <156821693963.2951081.11214256396118531359.stgit@dwillia2-desk3.amr.corp.intel.com> <20190911184332.GL20699@kadam> <9132e214-9b57-07dc-7ee2-f6bc52e960c5@kernel.dk> <20190913010937.7fc20d93@lwn.net> <20190913114849.GP20699@kadam> In-Reply-To: <20190913114849.GP20699@kadam> From: Dan Williams Date: Fri, 13 Sep 2019 05:18:22 -0700 Message-ID: To: Dan Carpenter Cc: Dave Jiang , ksummit , linux-nvdimm , Vishal Verma , Linux Kernel Mailing List , bpf@vger.kernel.org Subject: Re: [Ksummit-discuss] [PATCH v2 3/3] libnvdimm, MAINTAINERS: Maintainer Entry Profile X-BeenThere: ksummit-discuss@lists.linuxfoundation.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: ksummit-discuss-bounces@lists.linuxfoundation.org Errors-To: ksummit-discuss-bounces@lists.linuxfoundation.org On Fri, Sep 13, 2019 at 4:49 AM Dan Carpenter wrote: > > On Fri, Sep 13, 2019 at 01:09:37AM -0600, Jonathan Corbet wrote: > > On Wed, 11 Sep 2019 16:11:29 -0600 > > Jens Axboe wrote: > > > > > On 9/11/19 12:43 PM, Dan Carpenter wrote: > > > > > > > > I kind of hate all this extra documentation because now everyone thinks > > > > they can invent new hoops to jump through. > > > > > > FWIW, I completely agree with Dan (Carpenter) here. I absolutely > > > dislike having these kinds of files, and with subsystems imposing weird > > > restrictions on style (like the quoted example, yuck). > > > > > > Additionally, it would seem saner to standardize rules around when > > > code is expected to hit the maintainers hands for kernel releases. Both > > > yours and Martins deals with that, there really shouldn't be the need > > > to have this specified in detail per sub-system. > > > > This sort of objection came up at the maintainers summit yesterday; the > > consensus was that, while we might not like subsystem-specific rules, they > > do currently exist and we're just documenting reality. To paraphrase > > Phillip K. Dick, reality is that which, when you refuse to document it, > > doesn't go away. > > There aren't that many subsystem rules. The big exception is > networking, with the comment style and reverse Chrismas tree > declarations. Also you have to label which git tree the patch applies > to like [net] or [net-next]. > > It used to be that infiniband used "sizeof foo" instead of sizeof(foo) > but now there is a new maintainer. > > There is one subsystem which where the maintainer will capitalize your > patch prefix and complain. There are others where they will silently > change it to lower case. (Maybe that has changed in recent years). > > There is one subsystem where the maintainer is super strict rules that > you can't use "I" or "we" in the commit message. So you can't say "I > noticed a bug while reviewing", you have to say "The code has a bug". > > Some maintainers have rules about what you can put in the declaration > block. No kmalloc() in the declarations is a common rule. > "struct foo *p = kmalloc();". > > Some people (I do) have strict rules for error handling, but most won't > complain unless the error handling has bugs. > > The bpf people want you to put [bpf] or [bpf-next] in the subject. > Everyone just guesses, and uneducated guesses are worse than leaving it > blank, but that's just my opinion. > > > So I'm expecting to take this kind of stuff into Documentation/. My own > > personal hope is that it can maybe serve to shame some of these "local > > quirks" out of existence. The evidence from this brief discussion suggests > > that this might indeed happen. > > I don't think it's shaming, I think it's validating. Everyone just > insists that since it's written in the Book of Rules then it's our fault > for not reading it. It's like those EULA things where there is more > text than anyone can physically read in a life time. > > And the documentation doesn't help. For example, I knew people's rules > about capitalizing the subject but I'd just forget. I say that if you > can't be bothered to add it to checkpatch then it means you don't really > care that strongly. True, can someone with better perl skills than me take a shot at a rule for checkpatch to catch the capitalization preference based on the subsystem being touched, or otherwise agree that if a maintainer has a changelog capitalization preference they just silently fix it up at application time and not waste time pointing out something so trivial? For example, I notice Linus likes "-" instead of "*" for bullet lists in changelogs he just fixes it up silently if I forget. _______________________________________________ Ksummit-discuss mailing list Ksummit-discuss@lists.linuxfoundation.org https://lists.linuxfoundation.org/mailman/listinfo/ksummit-discuss