Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp5492835pxj; Wed, 26 May 2021 11:48:49 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyT+vkH0YvU5BsmTD6p1ZCY9SKmxtLIWwVAV0LE+fE9Fp1i9uz976cLIVSmAFkIiRcGwnam X-Received: by 2002:aa7:d490:: with SMTP id b16mr38845062edr.39.1622054929368; Wed, 26 May 2021 11:48:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1622054929; cv=none; d=google.com; s=arc-20160816; b=NNY4Yd+L2nVw+8Oz5JozFZCI/CC7EnYju7nnLSyV4eq7NbZo9UtrPS1urv71PafjbD Ru8Bmn6ajX56Efit9Yk+XHqyc6qLc60lvLmm1D0qmJ6jHcFoq0s6rK263/YxKnKkvaDh 1jglSKILO0BIiSkoxfU/bY6JnSXjVxgf4TgvUpbam+ezXO2DdXre3xa2QZda9UlUe8Ov EcHEK5jjnXIEzcXlEM1HUFZUjFwJpNZIBiZa6nelcl/ve7sEYzDrCi6HiDx8Pdj5sBKT zLNllJg7414F6UqTvRQnQK1s+FHaNBaghEpRnWcK8lg/AgILtPGlrl2Dq57YIWjqxm2t 6pwQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=d1KiEETbdzs5kCt6ni0hp8RfLQ2hML7W4tsclFu6uvU=; b=QoO2NLLaYiDf+REwQknJ4JT8y880TZ0YbKUzstYGJI2Mw80br/7d5MXu6efqLMSG7j mUPf9LrhyYC1o+wCURjgaljbyd25ifPNnkvSzE8H4SD4zLzhTX0XHrjXREGWgGidG4wp bB8yjUEsI9ZuBYs7wGWSh6AnOyGDvAHW4QzCn8s93wL1pNncHLb5EOyBUYWXfY5jr3RF enP/C98ja2FhgJRBf29JSaPlB6tn5ICX95DvfliRWprkrldUAsosUgRXlb7CEyvQ9EaF UiDA0NywJ42WMQn6TbxuUe04smWsaMe4VWYfoQ8uMavgmKNUMBwHuIeNrTPIUWxbr2QS 8riQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=casper.20170209 header.b=JccLGrV4; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-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 gv3si155045ejc.220.2021.05.26.11.48.25; Wed, 26 May 2021 11:48:49 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-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=@infradead.org header.s=casper.20170209 header.b=JccLGrV4; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234611AbhEZOZH (ORCPT + 99 others); Wed, 26 May 2021 10:25:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59802 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233416AbhEZOZG (ORCPT ); Wed, 26 May 2021 10:25:06 -0400 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 691C6C061574 for ; Wed, 26 May 2021 07:23:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=d1KiEETbdzs5kCt6ni0hp8RfLQ2hML7W4tsclFu6uvU=; b=JccLGrV4mRWengejtPg+BTJGg4 HRNT003TarySCw7ZoeXHn+aRH4qMF52iB5hVgj4bsIdMABMkXVUU7V8V/oAHY5xR3Wpwqk+5f42Ev 76jzrkAglOM2gG2vI4Sun+nmILMx1kZkZozVm3p2qY4oopFIY3qzLa3s1+5DPczQfoFzOsyE3VLYQ d+nr7bCqhFpqpjZH88iKN7W9qKqgb37En+T68S6zkLlCAhVc45JRs+iwIAyE05HecCgJkWgB189Tz azLyfPa7Ve21da/ponzEhW/uhoVZV3o5bCFxTHqleBF13dWGuYZMBRPBlxeumD1JENqUim6tNSzyG 8wsp3Ocg==; Received: from j217100.upc-j.chello.nl ([24.132.217.100] helo=noisy.programming.kicks-ass.net) by casper.infradead.org with esmtpsa (Exim 4.94 #2 (Red Hat Linux)) id 1lluOA-004b5w-NV; Wed, 26 May 2021 14:20:11 +0000 Received: from hirez.programming.kicks-ass.net (hirez.programming.kicks-ass.net [192.168.1.225]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (Client did not present a certificate) by noisy.programming.kicks-ass.net (Postfix) with ESMTPS id 1232F30029E; Wed, 26 May 2021 16:19:59 +0200 (CEST) Received: by hirez.programming.kicks-ass.net (Postfix, from userid 1000) id EAE59201CF1A9; Wed, 26 May 2021 16:19:58 +0200 (CEST) Date: Wed, 26 May 2021 16:19:58 +0200 From: Peter Zijlstra To: Junio C Hamano Cc: Mark Rutland , "Linux-Kernel@Vger. Kernel. Org" , will@kernel.org, boqun.feng@gmail.com, aou@eecs.berkeley.edu, arnd@arndb.de, bcain@codeaurora.org, benh@kernel.crashing.org, chris@zankel.net, dalias@libc.org, davem@davemloft.net, deanbo422@gmail.com, deller@gmx.de, geert@linux-m68k.org, gerg@linux-m68k.org, green.hu@gmail.com, guoren@kernel.org, ink@jurassic.park.msu.ru, James.Bottomley@hansenpartnership.com, jcmvbkbc@gmail.com, jonas@southpole.se, ley.foon.tan@intel.com, linux@armlinux.org.uk, mattst88@gmail.com, monstr@monstr.eu, mpe@ellerman.id.au, nickhu@andestech.com, palmerdabbelt@google.com, paulus@samba.org, paul.walmsley@sifive.com, rth@twiddle.net, shorne@gmail.com, stefan.kristiansson@saunalahti.fi, tsbogend@alpha.franken.de, vgupta@synopsys.com, ysato@users.sourceforge.jp Subject: Re: [PATCH v2 00/33] locking/atomic: convert all architectures to ARCH_ATOMIC Message-ID: References: <20210525140232.53872-1-mark.rutland@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 26, 2021 at 09:29:12PM +0900, Junio C Hamano wrote: > > On Tue, May 25, 2021 at 03:01:59PM +0100, Mark Rutland wrote: > > > Note: I've generated the patches with: > > > > > > git format-patch -C -M -D > > > > > > ... so the preimage of include/linux/atomic-fallback.h is not included > > > in the diff when it is deleted. > > ... > > Junio; can we get something like the below sorted? > > Funny, I recently looked at "git format-patch --help" and read that "-D" > is explicitly described as "The resulting patch is not meant to be applied > with patch or git apply; this is solely for people who want to just > concentrate on reviewing the text after the change." Yeah, I read that too; but like I said, I feel strongly that anything generated by git-format-patch should be accepted by git-apply (albeit with a --do-what-I-want argument, I agree that -D output should never be applied without express permission). > So, no, I think it is anti-social to send such a patch generated with -D > unless it is purely for discussion. The side that generates the patch > should be told to drop "-D" when running the format-patch command, > or perhaps format-patch should be taught to error out when "-D" is > given, perhaps? Removing -D from git-format-patch works too, although I rather like not seeing the '-' for the entire files. I really don't care much for the reversible thing, I already have the content, I can zap the commit to get it back. I'll leave it up to you, either remove -D from format-patch, or add an option to apply to force accept it.