Received: by 2002:a25:6193:0:0:0:0:0 with SMTP id v141csp4500258ybb; Tue, 14 Apr 2020 08:29:15 -0700 (PDT) X-Google-Smtp-Source: APiQypIdpGETEMm9oRHjs56DQIyPLzzskjGSnHs2TcC5RtfnaEc4N/D/adx4ZnusiRWuayMkj8/P X-Received: by 2002:a17:906:4c1:: with SMTP id g1mr671731eja.296.1586878155090; Tue, 14 Apr 2020 08:29:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1586878155; cv=none; d=google.com; s=arc-20160816; b=sGDCxlNubAr0vfhxttQyRmwBKD2ZpLuk/H57PdweU4D7WnWXwy7g/g3BtxfLUCRbre rKbTwq7uKpupORl7pI256pN2FtPgDPwutzuebZH/ukrIk6p52mF2uiU0VJyQc3sfJ9Te Juwoa2jDq3o06PYVmXSq6XKRfcYQHY/Ji2wgSMifr7ZahCSwAVrTnfKJPyvcOf7qO1E7 t0kzmlgmd9wDjjWUW4OaKpsnC/C1xNrd/9nHfqBDLLiTlZht9f+Q/bpHn1W0b8z7dV0B nrkFR8qAS5DtJPHnQI2ELfc7JXBXeEKgQSXkvvewe2L0/CUf3QII9yRHC2ovZKHctIIj Ov7A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=COIbcevwLYkzrhP2CU190PF2JhFzcK8tcNLYHrnm44U=; b=JMelFWlAltYNWJn2RnRvAQg+hs2jGGmQF3Kr/ZQKhkAMcIuZYoVVNAuAWuvjAXMdsJ bydNmZ/4f1lFn51Q3hcgJwHqjWBt6l7O7+k/Zv4fbf847NrJ9ZGim9BoZp2wxWTJU6xX Hy7oLY03PkV64UFvIJRa01Yojx7l+O5PUDxTDZFNEgxhIa5QfsBJt2hSbLqJAES9gasc nLPcJ23KMfvXaBSIzlClFWBGFe4Qz2uRSwaro86dstRcRBiuJlaoFjZu/+Jzer9HQ0Qn 8YQ8SSz+fTM4j5GleRsymC6Bk/6cqbziYUYecwuIh3yczQpEqFnkhsBXJxAPTR9TZtCw VZGw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ziepe.ca header.s=google header.b="ZRER/ysH"; spf=pass (google.com: best guess record for domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id la2si8915061ejb.371.2020.04.14.08.28.43; Tue, 14 Apr 2020 08:29:15 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@ziepe.ca header.s=google header.b="ZRER/ysH"; spf=pass (google.com: best guess record for domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2440020AbgDNMTi (ORCPT + 99 others); Tue, 14 Apr 2020 08:19:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46368 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S2440011AbgDNMTd (ORCPT ); Tue, 14 Apr 2020 08:19:33 -0400 Received: from mail-qk1-x743.google.com (mail-qk1-x743.google.com [IPv6:2607:f8b0:4864:20::743]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4B46EC061A0C for ; Tue, 14 Apr 2020 05:19:33 -0700 (PDT) Received: by mail-qk1-x743.google.com with SMTP id t3so9051969qkg.1 for ; Tue, 14 Apr 2020 05:19:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=COIbcevwLYkzrhP2CU190PF2JhFzcK8tcNLYHrnm44U=; b=ZRER/ysHClBeCuSTucMariAEakIOsNBLRQomZpMIWd39mCAZVNA3TIcDSP/kruVs3p 9BvKuSi9zEruLi/xJu+fjlrkF6ljfJscGeoIecRAxyAa6RLpqRg26mNY6PiOFIkgNO1p VQZ6WysBhueSLv9B6PByvqGbIEX+knuOQw8uu/e5CMF2FPKKRGoYdgGs6nJbqQ0k1far NtQLnvDRWSqpFtQ1NiPn2wv2w62idaPTrrqJziHhQNwCj5mv/Yva31xwSmCJYNWL/hcg n8U3psolduGgBJMSJSvzqggkXBkZKoUgq4Gf5vrqUPJya0v2tPpzL36T4axCa2B7TC7E lklQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=COIbcevwLYkzrhP2CU190PF2JhFzcK8tcNLYHrnm44U=; b=ZIOGgfJEn/pP1LH/TE16gTvjwu3rD/HAjCuvG1uFCCEufTj0AwqDflAK0vuc3kb1eP Oem83XStsAJFkPiFQiDki+SpIozLAVxp5uKj9PpnWADktuHAEHNCeVO5O+AyCuIzGtHx L+iGxZQYdeev/8w2BTFttO9ECzQz14Wiu8MKbxUxQKyg+QUuLJ8VPkQ0vdLTBPuhbkuq kEl+vuyR9dLruutwOSmwDsLUQVTB6EaEDKq8Vzp5JaHrkCCWzMCTd5ViCGrQdTd5gL6Q bomfGCHGdEeY4SrfZ0bn2Gg6v5BPV2B/qUAZ5nkBJrUjjLBQWTcJUAc/9IMsGOIgBXDr +k0g== X-Gm-Message-State: AGi0PuacgAVg4ZwH1Cw10zTjV8tMed98NiFTCzyANtYdwn3zQOTZDw0j nT6aNwckDzdnvr7ng92VR+I09w== X-Received: by 2002:a37:b887:: with SMTP id i129mr13233238qkf.28.1586866772522; Tue, 14 Apr 2020 05:19:32 -0700 (PDT) Received: from ziepe.ca (hlfxns017vw-142-68-57-212.dhcp-dynamic.fibreop.ns.bellaliant.net. [142.68.57.212]) by smtp.gmail.com with ESMTPSA id d4sm10806832qtc.48.2020.04.14.05.19.31 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 14 Apr 2020 05:19:31 -0700 (PDT) Received: from jgg by mlx.ziepe.ca with local (Exim 4.90_1) (envelope-from ) id 1jOKXL-0003F8-C7; Tue, 14 Apr 2020 09:19:31 -0300 Date: Tue, 14 Apr 2020 09:19:31 -0300 From: Jason Gunthorpe To: "J. Bruce Fields" Cc: Chuck Lever , Leon Romanovsky , Linux NFS Mailing List , linux-rdma@vger.kernel.org Subject: Re: [PATCH v1 3/3] svcrdma: Fix leak of svc_rdma_recv_ctxt objects Message-ID: <20200414121931.GA5100@ziepe.ca> References: <20200407190938.24045.64947.stgit@klimt.1015granger.net> <20200407191106.24045.88035.stgit@klimt.1015granger.net> <20200408060242.GB3310@unreal> <20200409174750.GK11886@ziepe.ca> <20200413192907.GA23596@fieldses.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200413192907.GA23596@fieldses.org> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org On Mon, Apr 13, 2020 at 03:29:07PM -0400, J. Bruce Fields wrote: > On Thu, Apr 09, 2020 at 02:47:50PM -0300, Jason Gunthorpe wrote: > > On Thu, Apr 09, 2020 at 10:33:32AM -0400, Chuck Lever wrote: > > > The commit ID is what automation should key off of. The short > > > description is only for human consumption. > > > > Right, so if the actual commit message isn't included so humans can > > read it then what was the point of including anything? > > Personally as a human reading commits in a terminal window I prefer the > abbreviated form. Frankly, I think they are useless, picking one of yours at random: Fixes: 4e48f1cccab3 "NFSD: allow inter server COPY to have... " And sadly the '4e48f1cccab3' commit doesn't appear in Linus's tree so now we are just totally lost, with a bad commit ID and a mangled subject line. > I haven't been doing the redundant parentheses and quotes either. Was > that dreamt up by an Arlo Guthrie fan? ("KID, HAVE YOU REHABILITATED > YOURSELF?") Well it seems like you are just aren't following the standard style at all. :( Jason