Re: Add fchmodat2() - or add a more general syscall?
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: David Howells <dhowells@xxxxxxxxxx>
- Subject: Re: Add fchmodat2() - or add a more general syscall?
- From: Eric Biggers <ebiggers@xxxxxxxxxx>
- Date: Wed, 26 Jul 2023 20:57:10 -0700
- Cc: Alexey Gladkov <legion@xxxxxxxxxx>, James.Bottomley@xxxxxxxxxxxxxxxxxxxxx, acme@xxxxxxxxxx, alexander.shishkin@xxxxxxxxxxxxxxx, axboe@xxxxxxxxx, benh@xxxxxxxxxxxxxxxxxxx, borntraeger@xxxxxxxxxx, bp@xxxxxxxxx, catalin.marinas@xxxxxxx, christian@xxxxxxxxxx, dalias@xxxxxxxx, davem@xxxxxxxxxxxxx, deepa.kernel@xxxxxxxxx, deller@xxxxxx, fenghua.yu@xxxxxxxxx, fweimer@xxxxxxxxxx, geert@xxxxxxxxxxxxxx, glebfm@xxxxxxxxxxxx, gor@xxxxxxxxxxxxx, hare@xxxxxxxx, hpa@xxxxxxxxx, ink@xxxxxxxxxxxxxxxxxxxx, jhogan@xxxxxxxxxx, kim.phillips@xxxxxxx, ldv@xxxxxxxxxxxx, linux-alpha@xxxxxxxxxxxxxxx, linux-arch@xxxxxxxxxxxxxxx, linux-ia64@xxxxxxxxxxxxxxx, linux-m68k@xxxxxxxxxxxxxxx, linux-mips@xxxxxxxxxxxxxxx, linux-parisc@xxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, linux-sh@xxxxxxxxxxxxxxx, linux@xxxxxxxxxxxxxxx, linuxppc-dev@xxxxxxxxxxxxxxxx, luto@xxxxxxxxxx, mattst88@xxxxxxxxx, mingo@xxxxxxxxxx, monstr@xxxxxxxxx, mpe@xxxxxxxxxxxxxx, namhyung@xxxxxxxxxx, paulus@xxxxxxxxx, peterz@xxxxxxxxxxxxx, ralf@xxxxxxxxxxxxxx, sparclinux@xxxxxxxxxxxxxxx, stefan@xxxxxxxx, tglx@xxxxxxxxxxxxx, tony.luck@xxxxxxxxx, tycho@xxxxxxxx, will@xxxxxxxxxx, x86@xxxxxxxxxx, ysato@xxxxxxxxxxxxx, LKML <linux-kernel@xxxxxxxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, linux-api@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, viro@xxxxxxxxxxxxxxxxxx
- In-reply-to: <104971.1690300714@warthog.procyon.org.uk>
- References: <cover.1689092120.git.legion@kernel.org> <cover.1689074739.git.legion@kernel.org> <104971.1690300714@warthog.procyon.org.uk>
On Tue, Jul 25, 2023 at 04:58:34PM +0100, David Howells wrote:
> Rather than adding a fchmodat2() syscall, should we add a "set_file_attrs()"
> syscall that takes a mask and allows you to set a bunch of stuff all in one
> go? Basically, an interface to notify_change() in the kernel that would allow
> several stats to be set atomically. This might be of particular interest to
> network filesystems.
>
> David
>
fchmodat2() is a simple addition that fits well with the existing syscalls.
It fixes an oversight in fchmodat().
IMO we should just add fchmodat2(), and not get sidetracked by trying to add
some super-generalized syscall instead. That can always be done later.
- Eric
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]