Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp187060ybi; Tue, 16 Jul 2019 18:49:01 -0700 (PDT) X-Google-Smtp-Source: APXvYqwNOuRkBuvIUEpWA0K6wMQRIfRvfGKNxH4+fUgzpElUzkDjoAWmB/bA6FiXHB5pUpgM1dkV X-Received: by 2002:a17:902:2865:: with SMTP id e92mr697525plb.264.1563328141431; Tue, 16 Jul 2019 18:49:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1563328141; cv=none; d=google.com; s=arc-20160816; b=Iz37wes1MJDt5hHoCQOWCkPDYZOTA3qLs17yzT7hVBIimlb9e3N4oDCOs5MCn0RPiI Uubj9yUOXNlk4wQxf0EE/0T3DXeJXsFVdksjMEMHjQKv+wd3biG1i6NFdPCGNWGsmG86 q8MqZTa5/leJC7726rtSu29uQbs5V037CsDT9PE8QvAuATU7W01k/1Cysvpz4naGuMs6 lNmmfQIAItGycC4SYSHkcG5FY9bM7WG2PiJFm+U3X2256ueGzgm1lFQpbCvGhJevTY6J 9XP1iV7iTYz+Dbw8W+Vu9MH+egvte5AmKW+Rg4xigPcMcJO1Qn6QXhzg+2MQF+oLI/Gv 0P6g== 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; bh=r2m54zIUab1nKIOPE45gZZhdrlRQDqurELs1U1KRCyU=; b=Y6Quxj3cIIIXW/s0grmqwNvXdpgaXo3ppqvnJHaQpMmVnIwRPyMNpO+Qi4NuNslf2+ 2Z5aRMPiaoNaLiIt3IkMJTAVDu+WSJHDTn4A6zzf9kX8Rv4VWCrEO8pkhu6lSjzYIBgp PsQbmLh7zq9q6N5QQy+zd8w2YvaoSz5hI/3NBnpnC9Zy2T55TnSvbLih3+cB7GC+BHj+ bIep1C8y64AQ9AUm6z938phQ6aR16KooO5jJpsGwNVa5abIPX1fHiht3rRahq70N30QQ qtEevnkqrbkzVwsNxEsFw1zYCIfHjr4lgyO4Do4VFANWBvLBtl/Lu01kAlckFZ/JrX2o zdMA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id n18si19777693plp.215.2019.07.16.18.48.44; Tue, 16 Jul 2019 18:49:01 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726282AbfGQBs0 (ORCPT + 99 others); Tue, 16 Jul 2019 21:48:26 -0400 Received: from zeniv.linux.org.uk ([195.92.253.2]:51824 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725856AbfGQBsZ (ORCPT ); Tue, 16 Jul 2019 21:48:25 -0400 Received: from viro by ZenIV.linux.org.uk with local (Exim 4.92 #3 (Red Hat Linux)) id 1hnZ34-00075C-8b; Wed, 17 Jul 2019 01:48:02 +0000 Date: Wed, 17 Jul 2019 02:48:02 +0100 From: Al Viro To: Palmer Dabbelt Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-api@vger.kernel.org, Arnd Bergmann , rth@twiddle.net, ink@jurassic.park.msu.ru, mattst88@gmail.com, linux@armlinux.org.uk, catalin.marinas@arm.com, will@kernel.org, tony.luck@intel.com, fenghua.yu@intel.com, geert@linux-m68k.org, monstr@monstr.eu, ralf@linux-mips.org, paul.burton@mips.com, jhogan@kernel.org, James.Bottomley@hansenpartnership.com, deller@gmx.de, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, heiko.carstens@de.ibm.com, gor@linux.ibm.com, borntraeger@de.ibm.com, ysato@users.sourceforge.jp, dalias@libc.org, davem@davemloft.net, luto@kernel.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, hpa@zytor.com, x86@kernel.org, peterz@infradead.org, acme@kernel.org, alexander.shishkin@linux.intel.com, jolsa@redhat.com, namhyung@kernel.org, dhowells@redhat.com, firoz.khan@linaro.org, stefan@agner.ch, schwidefsky@de.ibm.com, axboe@kernel.dk, christian@brauner.io, hare@suse.com, deepa.kernel@gmail.com, tycho@tycho.ws, kim.phillips@arm.com, linux-alpha@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-ia64@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-mips@vger.kernel.org, linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-sh@vger.kernel.org, sparclinux@vger.kernel.org, linux-arch@vger.kernel.org Subject: Re: [PATCH v2 2/4] Add fchmodat4(), a new syscall Message-ID: <20190717014802.GS17978@ZenIV.linux.org.uk> References: <20190717012719.5524-1-palmer@sifive.com> <20190717012719.5524-3-palmer@sifive.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190717012719.5524-3-palmer@sifive.com> User-Agent: Mutt/1.11.3 (2019-02-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 16, 2019 at 06:27:17PM -0700, Palmer Dabbelt wrote: > -int do_fchmodat(int dfd, const char __user *filename, umode_t mode) > +int do_fchmodat4(int dfd, const char __user *filename, umode_t mode, int flags) > { > struct path path; > int error; > - unsigned int lookup_flags = LOOKUP_FOLLOW; > + unsigned int lookup_flags; > + > + if (unlikely(flags & ~AT_SYMLINK_NOFOLLOW)) > + return -EINVAL; > + > + lookup_flags = flags & AT_SYMLINK_NOFOLLOW ? 0 : LOOKUP_FOLLOW; > + Why not do that in sys_fchmodat4() itself, passing lookup_flags to do_fchmodat() and updating old callers to pass it 0 as extra argument?