Re: [PATCH v5 0/4] man2: udpate mm/userfaultfd manpages to latest

2021-04-05 Thread Michael Kerrisk (man-pages)
ng patches from others? Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

man-pages-5.11 released

2021-03-22 Thread Michael Kerrisk (man-pages)
below. Cheers, Michael Changes in man-pages-5.11 Released: 2021-03-21, Munich New and rewritten pages --- close_range.2 Stephen Kitt, Michael Kerrisk [Christian Brauner] New page documenting close_range(2) process_madvise

Re: [PATCH v6] close_range.2: new page documenting close_range(2)

2021-03-21 Thread Michael Kerrisk (man-pages)
On 3/9/21 8:53 PM, Stephen Kitt wrote: > Hi Michael, > > On Thu, 28 Jan 2021 21:50:23 +0100, "Michael Kerrisk (man-pages)" > wrote: >> Thanks for your patch revision. I've merged it, and have >> done some light editing, but I still have a question: > > D

Re: [PATCH v6] close_range.2: new page documenting close_range(2)

2021-03-21 Thread Michael Kerrisk (man-pages)
Hello Stephen and Christian, Late follow-up, I'm afraid... On 1/29/21 11:00 AM, Christian Brauner wrote: > On Thu, Jan 28, 2021 at 11:10:40PM +0100, Stephen Kitt wrote: >> Hello Michael, >> >> On Thu, 28 Jan 2021 21:50:23 +0100, "Michael Kerrisk (man-pages)" >&

Re: [RFC v2] execve.2: SYNOPSIS: Document both glibc wrapper and kernel sycalls

2021-02-19 Thread Michael Kerrisk (man-pages)
uot;, int " flags ); > +.BI "int syscall(SYS_membarrier, int " cmd ", int " flags ); > .EE > .in > .SH CONFORMING TO > @@ -322,9 +321,6 @@ Examples where > .BR membarrier () > can be useful include implementations > of Read-Copy-Update libraries and garbage collectors. > -.PP > -Glibc does not provide a wrapper for this system call; call it using > -.BR syscall (2). > .SH EXAMPLES > Assuming a multithreaded application where "fast_path()" is executed > very frequently, and where "slow_path()" is executed infrequently, the > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [RFC] execve.2: SYNOPSIS: Document both glibc wrapper and kernel sycalls

2021-02-18 Thread Michael Kerrisk (man-pages)
; .PP > -.BI "int execve(const char *" pathname ", char *const " argv [], > -.BI " char *const " envp []); > +.BI "int execve(const char *" pathname ", > +.BI " char *const " argv "[], char *const " e

Re: [PATCH v3 1/1] process_madvise.2: Add process_madvise man page

2021-02-18 Thread Michael Kerrisk (man-pages)
de a wrapper for this system call; call it us‐ ing syscall(2). SEE ALSO madvise(2), pidfd_open(2), process_vm_readv(2), process_vm_write(2) -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Tr

Re: [PATCH v3 1/1] process_madvise.2: Add process_madvise man page

2021-02-13 Thread Michael Kerrisk (man-pages)
Hello Suren, On 2/2/21 11:12 PM, Suren Baghdasaryan wrote: > Hi Michael, > > On Tue, Feb 2, 2021 at 2:45 AM Michael Kerrisk (man-pages) > wrote: >> >> Hello Suren (and Minchan and Michal) >> >> Thank you for the revisions! >> >> I've applied thi

Re: [PATCH v2] ipc.2: Fix prototype parameter types

2021-02-09 Thread Michael Kerrisk (man-pages)
2 100644 > --- a/man2/ipc.2 > +++ b/man2/ipc.2 > @@ -27,9 +27,8 @@ > ipc \- System V IPC system calls > .SH SYNOPSIS > .nf > -.BI "int ipc(unsigned int " call ", int " first ", int " second \ > -", int " third , > -.BI "

Re: [PATCH v3 1/1] process_madvise.2: Add process_madvise man page

2021-02-02 Thread Michael Kerrisk (man-pages)
gt; changes in v2: > - Changed description of MADV_COLD per Michal Hocko's suggestion > - Applied fixes suggested by Michael Kerrisk > changes in v3: > - Added Michal's Reviewed-by > - Applied additional fixes suggested by Michael Kerrisk > > NAME > process_madvise - give ad

Re: [PATCH v2 1/1] process_madvise.2: Add process_madvise man page

2021-01-30 Thread Michael Kerrisk (man-pages)
ghdasaryan > --- > changes in v2: > - Changed description of MADV_COLD per Michal Hocko's suggestion > - Appled fixes suggested by Michael Kerrisk > > NAME > process_madvise - give advice about use of memory to a process s/-/\-/ > > SYNOPSI

Re: [PATCH v6] close_range.2: new page documenting close_range(2)

2021-01-28 Thread Michael Kerrisk (man-pages)
the limit on the number of open file descriptors has been reached? I mean: no new FDs are being opened, so how can we go over the limit. I think the cause of this error is something else, but what is it? Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-

Re: [PATCH 1/1] process_madvise.2: Add process_madvise man page

2021-01-28 Thread Michael Kerrisk (man-pages)
Hello Suren, On 1/28/21 7:40 PM, Suren Baghdasaryan wrote: > On Thu, Jan 28, 2021 at 4:24 AM Michael Kerrisk (man-pages) > wrote: >> >> Hello Suren, >> >> Thank you for writing this page! Some comments below. > > Thanks for the review! > Couple question

Re: [PATCH 1/1] process_madvise.2: Add process_madvise man page

2021-01-28 Thread Michael Kerrisk (man-pages)
> +.I iovec > +is outside the accessible address space of the process pid. s/process pid./ the process referred to by .IR pidfd . / > +.TP > +.B EINVAL > +.I flags > +is not 0. > +.TP > +.B EINVAL > +The sum of the > +.I iov_len > +values of > +.I iovec > +overflows a ssize_t value. .I ssize_t > +.TP > +.B EINVAL > +.I vlen > +is too large. > +.TP > +.B ENOMEM > +Could not allocate memory for internal copies of the > +.I iovec > +structures. > +.TP > +.B EPERM > +The caller does not have permission to access the address space of the > process > +.I pidfd. .IR pidfd . > +.TP > +.B ESRCH > +No process with ID > +.I pidfd > +exists. Should this maybe be: [[ The target process does not exist (i.e., it has terminated and been waited on). ]] See pidfd_send_signal(2). Also, is an EBADF error possible? Again, see pidfd_send_signal(2). > +.SH VERSIONS > +Since Linux 5.10, Better: This system call first appeared in Linux 5.10. > +.\" commit ecb8ac8b1f146915aa6b96449b66dd48984caacc > +support for this system call is optional, s/support/Support/ > +depending on the setting of the > +.B CONFIG_ADVISE_SYSCALLS > +configuration option. > +.SH SEE ALSO > +.BR madvise (2), > +.BR pidofd_open(2), > +.BR process_vm_readv (2), > +.BR process_vm_write (2) Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 5/5] Add manpage for fsconfig(2)

2021-01-22 Thread Michael Kerrisk (man-pages)
> +.TP > +.B EINVAL > +One of the source parameters referred to an invalid superblock. > +.TP > +.B ELOOP > +Too many links encountered during pathname resolution. > +.TP > +.B ENAMETOOLONG > +A path name was longer than > +.BR MAXPATHLEN . > +.TP > +.B ENOENT > +

Re: [PATCH 3/5] Add manpage for fspick(2)

2021-01-22 Thread Michael Kerrisk (man-pages)
NG TO > +These functions are Linux-specific and should not be used in programs > intended > +to be portable. > +.SH VERSIONS > +.BR fsopen "(), " fsmount "() and " fspick () > +were added to Linux in kernel 5.2. > +.SH EXAMPLES > +To illustrate the process, here's an example whereby this can be used to > +reconfigure a filesystem: > +.PP > +.in +4n > +.nf > +sfd = fspick(AT_FDCWD, "/mnt", FSPICK_NO_AUTOMOUNT | FSPICK_CLOEXEC); > +fsconfig(sfd, FSCONFIG_SET_FLAG, "ro", NULL, 0); > +fsconfig(sfd, FSCONFIG_SET_STRING, "user_xattr", "false", 0); > +fsconfig(sfd, FSCONFIG_CMD_RECONFIGURE, NULL, NULL, 0); > +.fi > +.in > +.PP > +.SH NOTES > +Glibc does not (yet) provide a wrapper for the > +.BR fspick "()" > +system call; call it using > +.BR syscall (2). > +.SH SEE ALSO > +.BR mountpoint (1), > +.BR fsconfig (2), > +.BR fsopen (2), > +.BR path_resolution (7), > +.BR mount (8) > > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 4/5] Add manpage for fsopen(2) and fsmount(2)

2021-01-22 Thread Michael Kerrisk (man-pages)
Hello David, Ping! Thanks, Michael On Fri, 16 Oct 2020 at 08:50, Michael Kerrisk (man-pages) wrote: > > Hi David, > > Another ping for these five patches please! > > Cheers, > > Michael > > On Fri, 11 Sep 2020 at 14:44, Michael Kerrisk (man-pages) > wrote:

Re: [PATCH 2/5] Add manpages for move_mount(2)

2021-01-22 Thread Michael Kerrisk (man-pages)
he object mounted on "/a" to "/b". It can also be used in > +conjunction with > +.BR open_tree "(2) or " open "(2) with " O_PATH : > +.PP > +.RS > +.nf > +fd = open_tree(AT_FDCWD, "/mnt", 0); > +move_mount(fd, "", AT_FDCWD, "/mnt2", MOVE_MOUNT_F_EMPTY_PATH); > +move_mount(fd, "", AT_FDCWD, "/mnt3", MOVE_MOUNT_F_EMPTY_PATH); > +move_mount(fd, "", AT_FDCWD, "/mnt4", MOVE_MOUNT_F_EMPTY_PATH); > +.fi > +.RE > +.PP > +This would attach the path point for "/mnt" to fd, then it would move the > +mount to "/mnt2", then move it to "/mnt3" and finally to "/mnt4". > +.PP > +It can also be used to attach new mounts: > +.PP > +.RS > +.nf > +sfd = fsopen("ext4", FSOPEN_CLOEXEC); > +fsconfig(sfd, FSCONFIG_SET_STRING, "source", "/dev/sda1", 0); > +fsconfig(sfd, FSCONFIG_SET_FLAG, "user_xattr", NULL, 0); > +fsconfig(sfd, FSCONFIG_CMD_CREATE, NULL, NULL, 0); > +mfd = fsmount(sfd, FSMOUNT_CLOEXEC, MOUNT_ATTR_NODEV); > +move_mount(mfd, "", AT_FDCWD, "/home", MOVE_MOUNT_F_EMPTY_PATH); > +.fi > +.RE > +.PP > +Which would open the Ext4 filesystem mounted on "/dev/sda1", turn on user > +extended attribute support and create a mount object for it. Finally, the > new > +mount object would be attached with > +.BR move_mount () > +to "/home". > +.SH SEE ALSO > +.BR fsmount (2), > +.BR fsopen (2), > +.BR open_tree (2) > > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 1/5] Add manpage for open_tree(2)

2021-01-22 Thread Michael Kerrisk (man-pages)
Hello David, Ping! Thanks, Michael On Thu, 27 Aug 2020 at 13:01, Michael Kerrisk (man-pages) wrote: > > Hello David, > > Can I ask that you please reply to each of my mails, rather than > just sending out a new patch series (which of course I would also > like you to do).

Re: [PATCH v27 12/12] landlock: Add user and kernel documentation

2021-01-22 Thread Michael Kerrisk (man-pages)
e kind of > special > +paths are then always allowed. Future Landlock evolutions will enable to > +restrict such paths with dedicated ruleset flags. > + > +Questions and answers > += > + > +What about user space sandbox managers? > +---------

Re: [PATCH] entry: Use different define for selector variable in SUD

2021-01-20 Thread Michael Kerrisk (man-pages)
Hello all, On Sat, 2 Jan 2021 at 00:55, Gabriel Krisman Bertazi wrote: > > Michael Kerrisk suggested that, from an API perspective, it is a bad > idea to share the PR_SYS_DISPATCH_ defines between the prctl operation > and the selector variable. Therefore, define two new constants

Re: [PATCH] getcpu.2: Document glibc wrapper instead of kernel syscall

2021-01-02 Thread Michael Kerrisk (man-pages)
sys_getcpu() tcache parameter -In earlier kernels, -if this argument was non-NULL, +In Linux 2.6.23 and earlier, if the +.I tcache +argument was non-NULL, then it specified a pointer to a caller-allocated buffer in thread-local storage that was used to provide a caching mechanism for .BR get

Re: [PATCH v5] close_range.2: new page documenting close_range(2)

2020-12-22 Thread Michael Kerrisk (man-pages)
t(EXIT_FAILURE); > +} > + > +for (int i = 2; i < argc; i++) { > +if (open(argv[i], O_RDONLY) == -1) { > +perror(argv[i]); > +exit(EXIT_FAILURE); > +} > +} > + > +if (syscall(__NR_close_range, 3, ~0U, CLOSE_RANGE_UNSHARE) == -1) { Line too long. Alternatively, what about s/CLOSE_RANGE_UNSHARE/0/? Or it considered best practice to always use CLOSE_RANGE_UNSHARE? > +perror("close_range"); > +exit(EXIT_FAILURE); > +} > + > +execve(argv[1], newargv, newenviron); > +perror("execve"); > +exit(EXIT_FAILURE); > +} > +.EE > +.in > +.PP > +We can use the second program to exec the first as follows: > +.PP > +.in +4n > +.EX > +.RB "$" " make listopen close_range" Perhaps we don't really need the preceding line? > +.RB "$" " ./close_range ./listopen /dev/null /dev/zero" > +FD 0 is open. > +FD 1 is open. > +FD 2 is open. > +.EE > +.in > +.PP > +Removing the call to > +.BR close_range () > +will show different output, > +with the file descriptors for the named files still open. > +.SH SEE ALSO > +.BR close (2) > > base-commit: b5dae3959625f5ff378e9edf9139057d1c06bb55 Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

man-pages-5.10 is released

2020-12-22 Thread Michael Kerrisk (man-pages)
Changes in man-pages-5.10 Newly documented interfaces in existing pages - access.2 Michael Kerrisk Document faccessat2() faccessat2() was added in Linux 5.8 and enables a fix

Re: [PATCH v2] close_range.2: new page documenting close_range(2)

2020-12-10 Thread Michael Kerrisk (man-pages)
a/man2/close_range.2 b/man2/close_range.2 >> new file mode 100644 >> index 0..403142b33 >> --- /dev/null >> +++ b/man2/close_range.2 [...] >> +.SH USE CASES > > This section is unconventional. Please move that text to one of the > traditional sect

Re: [patch] close_range.2: new page documenting close_range(2)

2020-12-10 Thread Michael Kerrisk (man-pages)
.BR close_range () >>> +can take care of this without requiring >>> +.B /proc > > By reading proc.5, I think this should s/.B/.I/, right mtk? Yes to both. Pathnames are formatted with .I. -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [patch] close_range.2: new page documenting close_range(2)

2020-12-09 Thread Michael Kerrisk (man-pages)
for (fd = 0; fd < maxfd; fd++) close(fd); I think it's fine to mention the use of a /proc as an (inferior and) alternative way of doing this. I'm just not sure that "commonly" is correct. > +descriptors in > +.B /proc/self/fd/ > +and calling > +.BR close (2) > +on each one. > +.BR close_range () > +can take care of this without requiring > +.B /proc > +and with a single system call, which provides significant performance > +benefits. > +.SH SEE ALSO > +.BR close (2) > > base-commit: b5dae3959625f5ff378e9edf9139057d1c06bb55 > -- > 2.20.1 Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Linux man-pages maintainership adjustments

2020-12-05 Thread Michael Kerrisk (man-pages)
will return to being just a contributor and no longer the maintainer. Cheers, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH] subpage_prot.2: SYNOPSIS: Fix return type: s/long/int/

2020-11-28 Thread Michael Kerrisk (man-pages)
00644 > --- a/man2/subpage_prot.2 > +++ b/man2/subpage_prot.2 > @@ -32,7 +32,7 @@ > subpage_prot \- define a subpage protection for an address range > .SH SYNOPSIS > .nf > -.BI "long subpage_prot(unsigned long " addr ", unsigned long " len , > +.BI &q

Re: [PATCH] spu_create.2: Clarify that one of the prototypes is the current one

2020-11-27 Thread Michael Kerrisk (man-pages)
uot; .BI " int " neighbor_fd ");" .fi @@ -89,6 +88,12 @@ for a full list of the possible values. .PP The +.I neighbor_fd +is used only when the +.B SPU_CREATE_AFFINITY_SPU +flag is specified; see below. +.PP +The .I flags argument can be zero or any bitwise OR-ed combination of the following constants: @@ -264,6 +269,14 @@ See .UR http://www.bsc.es\:/projects\:/deepcomputing\:/linuxoncell/ .UE for the recommended libraries. +.PP +Prior to the addition of the +.B SPU_CREATE_AFFINITY_SPU +flag in Linux 2.6.23, the +.BR spu_create () +system call took only three arguments (i.e., there was no +.I neighbor_fd +argument). .SH EXAMPLES See .BR spu_run (2) -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH v2 2/4] x86/elf: Support a new ELF aux vector AT_MINSIGSTKSZ

2020-11-27 Thread Michael Kerrisk (man-pages)
h reason to document it. > > Adding Michael. Commit 94b07c1f8c39 was your, Dave. Might I convince you to write a patch for getauxval(3)? Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: set_thread_area.2: csky architecture undocumented

2020-11-24 Thread Michael Kerrisk (man-pages)
fined __mips__ > + #elif defined(__mips__ || __csky__) > > and then change the rest of the text to add csky when appropriate. > Am I correct? AFAICT, you are correct. I think the reason that csky is missing is that the architecture was added after this manual pages was added.

Re: [PATCH] set_tid_address.2: SYNOPSIS: Fix set_tid_address() return type

2020-11-24 Thread Michael Kerrisk (man-pages)
set_tid_address.2 > +++ b/man2/set_tid_address.2 > @@ -29,7 +29,7 @@ set_tid_address \- set pointer to thread ID > .nf > .B #include > .PP > -.BI "long set_tid_address(int *" tidptr ); > +.BI "pid_t set_tid_address(int *" tidptr ); > .fi > .PP > .IR

Re: [PATCH] restart_syscall.2: SYNOPSIS: Fix restart_syscall() return type

2020-11-23 Thread Michael Kerrisk (man-pages)
ter interruption by a stop signal > .SH SYNOPSIS > -.B int restart_syscall(void); > +.B long restart_syscall(void); > .PP > .IR Note : > There is no glibc wrapper for this system call; see NOTES. > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: set_thread_area.2: csky architecture undocumented

2020-11-23 Thread Michael Kerrisk (man-pages)
f defined __mips__ > + #elif defined(__mips__ || __csky__) > > and then change the rest of the text to add csky when appropriate. > Am I correct? As far as I can tell, you are correct. Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH] lseek.2: SYNOPSIS: Use correct types

2020-11-22 Thread Michael Kerrisk (man-pages)
cially, have a look at lseek64(3), > which I suspect needs the same changes I propose in that patch. I think that no changes to the types are needed in lseek64(3). But maybe some of the info in this mail should be captured in that manual page. Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH] lseek.2: SYNOPSIS: Use correct types

2020-11-22 Thread Michael Kerrisk (man-pages)
hat the code of many wrapper functions is autogenerated from "syscalls.list" files in the glibc source, for example, sysdeps/unix/sysv/linux/syscalls.list. This isn't the case for lseek(), though, as far as I can see; I think the wrapper function is defined in sysdeps/unix/sysv/linux/lseek.c.

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-11-02 Thread Michael Kerrisk (man-pages)
Hello Sargun, Thanks for your reply! On 11/2/20 9:07 AM, Sargun Dhillon wrote: > On Sat, Oct 31, 2020 at 9:27 AM Michael Kerrisk (man-pages) > wrote: >> >> Hello Sargun, >> >> Thanks for your reply. >> >> On 10/30/20 9:27 PM, Sargun Dhillon wrote: &g

man-pages-5.09 is released

2020-11-01 Thread Michael Kerrisk (man-pages)
Alejandro Colomar, Michael Kerrisk A new page documenting a wide range of system data types. kernel_lockdown.7 David Howells, Heinrich Schuchardt [Michael Kerrisk] New page documenting the Kernel Lockdown feature Newly documented interfaces in existing pages

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-31 Thread Michael Kerrisk (man-pages)
Hello Sargun, Thanks for your reply. On 10/30/20 9:27 PM, Sargun Dhillon wrote: > On Thu, Oct 29, 2020 at 09:37:21PM +0100, Michael Kerrisk (man-pages) > wrote: [...] >>> I think I commented in another thread somewhere that the >>> supervisor is not notified if

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-31 Thread Michael Kerrisk (man-pages)
On 10/30/20 8:20 PM, Jann Horn wrote: > On Thu, Oct 29, 2020 at 8:14 PM Michael Kerrisk (man-pages) > wrote: >> On 10/29/20 2:42 AM, Jann Horn wrote: >>> As discussed at >>> <https://lore.kernel.org/r/CAG48ez0m4Y24ZBZCh+Tf4ORMm9_q4n7VOzpGjwGF7_Fe8EQH=q...@mail.gm

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-31 Thread Michael Kerrisk (man-pages)
On 10/30/20 8:14 PM, Jann Horn wrote: > On Thu, Oct 29, 2020 at 3:19 PM Michael Kerrisk (man-pages) > wrote: >> On 10/29/20 2:42 AM, Jann Horn wrote: >>> On Mon, Oct 26, 2020 at 10:55 AM Michael Kerrisk (man-pages) >>> wrote: >>>>static

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-30 Thread Michael Kerrisk (man-pages)
On 10/30/20 8:24 PM, Jann Horn wrote: > On Thu, Oct 29, 2020 at 8:53 PM Michael Kerrisk (man-pages) > wrote: >> On 10/29/20 4:26 PM, Christian Brauner wrote: >>> I like this manpage. I think this is the most comprehensive explanation >>> of any seccomp feature >

Re: [PATCH 2/2] futex.2: Use appropriate types

2020-10-30 Thread Michael Kerrisk (man-pages)
4,7 +1795,7 @@ fwait(int *futexp) >> while (1) { >> >> /* Is the futex available? */ >> -const int one = 1; >> +const uint32_t one = 1; >> if (atomic_compare_exchange_strong(futexp, , 0)) >> break; /

Re: [PATCH 1/2] futex.2: srcfix

2020-10-30 Thread Michael Kerrisk (man-pages)
SIS > .nf > .PP > -.B "#include " > -.B "#include " > +.B #include > +.B #include > .PP > .BI "int futex(int *" uaddr ", int " futex_op ", int " val , > .BI " const struct timespec *" timeout , \

Re: [PATCH 2/2] futex.2: Use appropriate types

2020-10-30 Thread Michael Kerrisk (man-pages)
gt; +const uint32_t one = 1; > if (atomic_compare_exchange_strong(futexp, , 0)) > break; /* Yes */ > > @@ -1811,13 +1812,13 @@ fwait(int *futexp) > so that if the peer is blocked in fpost(), it can proceed. */ > > static void > -fpost(int *futexp) >

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-29 Thread Michael Kerrisk (man-pages)
Hello Sargun,, On 10/29/20 9:53 AM, Sargun Dhillon wrote: > On Mon, Oct 26, 2020 at 10:55:04AM +0100, Michael Kerrisk (man-pages) wrote: [...] >>ioctl(2) operations >>The following ioctl(2) operations are provided to support seccomp >>user-space no

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-29 Thread Michael Kerrisk (man-pages)
Hello Christian Thanks for taking a look at the page. On 10/29/20 4:26 PM, Christian Brauner wrote: > On Mon, Oct 26, 2020 at 10:55:04AM +0100, Michael Kerrisk (man-pages) wrote: >> Hi all (and especially Tycho and Sargun), >> >> Following review comments on the first draft

Re: [PATCH v3] getdents.2: Use appropriate types

2020-10-29 Thread Michael Kerrisk (man-pages)
UF_SIZE]; > struct linux_dirent *d; > char d_type; > @@ -301,7 +301,7 @@ main(int argc, char *argv[]) > > printf("\-\-\-\-\-\-\-\-\-\-\-\-\-\-\- nread=%d > \-\-\-\-\-\-\-\-\-\-\-\-\-\-\-\en", nread); > printf("inode#file type d_reclen d_

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-29 Thread Michael Kerrisk (man-pages)
Hello Jann, On 10/29/20 2:42 AM, Jann Horn wrote: > On Mon, Oct 26, 2020 at 10:55 AM Michael Kerrisk (man-pages) > wrote: >>static bool >>getTargetPathname(struct seccomp_notif *req, int notifyFd, >> char *path, size_t len) >&g

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-29 Thread Michael Kerrisk (man-pages)
Hello Jann, On 10/29/20 2:42 AM, Jann Horn wrote: > On Mon, Oct 26, 2020 at 10:55 AM Michael Kerrisk (man-pages) > wrote: >>static bool >>getTargetPathname(struct seccomp_notif *req, int notifyFd, >> char *path, size_t len) >&g

Re: [PATCH v2] getdents.2: Use appropriate types

2020-10-29 Thread Michael Kerrisk (man-pages)
_reclen d_off d_name\en"); > -for (int bpos = 0; bpos < nread;) { > +for (long bpos = 0; bpos < nread;) { > d = (struct linux_dirent *) (buf + bpos); > printf("%8ld ", d\->d_ino); > d_type = *(buf + bpos + d\->d_reclen \- 1); > -- > 2.28.0 > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH v2] perf_event_open.2: update the man page with CAP_PERFMON related information

2020-10-27 Thread Michael Kerrisk (man-pages)
On Tue, 27 Oct 2020 at 18:10, Alexey Budankov wrote: > > > On 27.10.2020 19:57, Michael Kerrisk (man-pages) wrote: > > Hello Alexey, > > > > On 10/27/20 5:48 PM, Alexey Budankov wrote: > >> > >> Extend perf_event_open 2 man page with the informati

Re: [PATCH v2] perf_event_open.2: update the man page with CAP_PERFMON related information

2020-10-27 Thread Michael Kerrisk (man-pages)
N > +rather than the much more powerful > +.B CAP_SYS_ADMIN > +excludes chances to misuse credentials and makes operations more secure. > +.B CAP_SYS_ADMIN > +usage for secure system performance monitoring and observability > +is discouraged with respect to > +.B CAP_PERFMON > +capability. Thank you for adding the above piece. That point of course really needs to be emphasized! Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 1/2] system_data_types.7: Add 'off_t'

2020-10-27 Thread Michael Kerrisk (man-pages)
Hi Alex, On Tue, 27 Oct 2020 at 16:25, Alejandro Colomar wrote: > > > > On 2020-10-27 14:47, Michael Kerrisk (man-pages) wrote: > > On 10/27/20 11:23 AM, Alejandro Colomar wrote: > >> Hi Michael, > >> > >> On 2020-10-07 08:53, Michael Kerrisk

Re: [PATCH 1/2] system_data_types.7: Add 'off_t'

2020-10-27 Thread Michael Kerrisk (man-pages)
On 10/27/20 11:23 AM, Alejandro Colomar wrote: > Hi Michael, > > On 2020-10-07 08:53, Michael Kerrisk (man-pages) wrote: >> On 10/6/20 12:12 AM, Alejandro Colomar wrote: >>> Signed-off-by: Alejandro Colomar >> >> Hi Alex, >> >> Thanks, patch applie

Inconsistent capability requirements for prctl_set_mm_exe_file()

2020-10-27 Thread Michael Kerrisk (man-pages)
and surprising. Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH v1] perf_event_open.2: update the man page with CAP_PERFMON related information

2020-10-27 Thread Michael Kerrisk (man-pages)
Hi Alexei, Would you be able to refresh this patch and resend please? Thnks, Michael On Mon, 24 Aug 2020 at 22:17, Alexey Budankov wrote: > > Hi Michael, > > On 23.08.2020 20:28, Michael Kerrisk (man-pages) wrote: > > Hello Alexey, > > > > Could you look at

Re: For review: seccomp_user_notif(2) manual page

2020-10-27 Thread Michael Kerrisk (man-pages)
On 10/26/20 4:54 PM, Jann Horn wrote: > On Sun, Oct 25, 2020 at 5:32 PM Michael Kerrisk (man-pages) > wrote: [...] >> I tried applying the patch below to vanilla 5.9.0. >> (There's one typo: s/ENOTCON/ENOTCONN). >> >> It seems not to work though; when I send a signa

Re: For review: seccomp_user_notif(2) manual page [v2]

2020-10-26 Thread Michael Kerrisk (man-pages)
Hi Tycho, Thanks for getting back to me. On Mon, 26 Oct 2020 at 14:54, Tycho Andersen wrote: > > On Mon, Oct 26, 2020 at 10:55:04AM +0100, Michael Kerrisk (man-pages) wrote: > > Hi all (and especially Tycho and Sargun), > > > > Following review comments on the first dra

For review: seccomp_user_notif(2) manual page [v2]

2020-10-26 Thread Michael Kerrisk (man-pages)
t even arm64 │ │compat agrees[3] with arm32 in this regard. :( │ └─┘ ┌─┐ │FIXME│ ├───────

Re: For review: seccomp_user_notif(2) manual page

2020-10-26 Thread Michael Kerrisk (man-pages)
Hi Jann, On 10/26/20 10:32 AM, Jann Horn wrote: > On Sat, Oct 24, 2020 at 2:53 PM Michael Kerrisk (man-pages) > wrote: >> On 10/17/20 2:25 AM, Jann Horn wrote: >>> On Fri, Oct 16, 2020 at 8:29 PM Michael Kerrisk (man-pages) >>> wrote: > [...] >>>>

Re: For review: seccomp_user_notif(2) manual page

2020-10-26 Thread Michael Kerrisk (man-pages)
Hello Kees, On 10/26/20 1:19 AM, Kees Cook wrote: > On Thu, Oct 15, 2020 at 01:24:03PM +0200, Michael Kerrisk (man-pages) wrote: >> On 10/1/20 1:39 AM, Kees Cook wrote: >>> I'll comment more later, but I've run out of time today and I didn't see >>> anyone mention this

Re: For review: seccomp_user_notif(2) manual page

2020-10-25 Thread Michael Kerrisk (man-pages)
n wrote: >>>>> On Wed, Sep 30, 2020 at 10:34:51PM +0200, Michael Kerrisk (man-pages) >>>>> wrote: >>>>>> On 9/30/20 5:03 PM, Tycho Andersen wrote: >>>&g

Re: For review: seccomp_user_notif(2) manual page

2020-10-24 Thread Michael Kerrisk (man-pages)
Hello Jann, On 10/17/20 2:25 AM, Jann Horn wrote: > On Fri, Oct 16, 2020 at 8:29 PM Michael Kerrisk (man-pages) > wrote: >> On 10/15/20 10:32 PM, Jann Horn wrote: >>> On Thu, Oct 15, 2020 at 1:24 PM Michael Kerrisk (man-pages) >>> wrote: >>>> On 9/30

Re: For review: seccomp_user_notif(2) manual page

2020-10-16 Thread Michael Kerrisk (man-pages)
Hello Jann, Thanks for your reply! On 10/15/20 10:32 PM, Jann Horn wrote: > On Thu, Oct 15, 2020 at 1:24 PM Michael Kerrisk (man-pages) > wrote: >> On 9/30/20 5:53 PM, Jann Horn wrote: >>> On Wed, Sep 30, 2020 at 1:07 PM Michael Kerrisk (man-pages) >>> wrote: >

Re: [PATCH 4/5] Add manpage for fsopen(2) and fsmount(2)

2020-10-16 Thread Michael Kerrisk (man-pages)
Hi David, Another ping for these five patches please! Cheers, Michael On Fri, 11 Sep 2020 at 14:44, Michael Kerrisk (man-pages) wrote: > > Hi David, > > A ping for these five patches please! > > Cheers, > > Michael > > On Wed, 2 Sep 2020 at 22:14, Michae

Re: For review: seccomp_user_notif(2) manual page

2020-10-15 Thread Michael Kerrisk (man-pages)
Hello Christian, On 10/1/20 2:36 PM, Christian Brauner wrote: > [I'm on vacation so I'll just give this a quick glance for now.] > > On Wed, Sep 30, 2020 at 01:07:38PM +0200, Michael Kerrisk (man-pages) wrote: >> Hi Tycho, Sargun (and all), >> >> I knew it would be a

Re: For review: seccomp_user_notif(2) manual page

2020-10-15 Thread Michael Kerrisk (man-pages)
Hi Jann, So, first off, thank you for the detailed review. I really appreciate it! I've changed various pieces, and still have a few questions below. On 9/30/20 5:53 PM, Jann Horn wrote: > On Wed, Sep 30, 2020 at 1:07 PM Michael Kerrisk (man-pages) > wrote: >> I knew it would

Re: For review: seccomp_user_notif(2) manual page

2020-10-15 Thread Michael Kerrisk (man-pages)
Hello Kees, On 10/1/20 1:39 AM, Kees Cook wrote: > On Wed, Sep 30, 2020 at 01:07:38PM +0200, Michael Kerrisk (man-pages) wrote: >> [...] I did :-) > > Yay! Thank you! You're welcome :-) >> [...] >>Overview >>In conventional usage of a secco

Re: For review: seccomp_user_notif(2) manual page

2020-10-14 Thread Michael Kerrisk (man-pages)
n Wed, Sep 30, 2020 at 05:53:46PM +0200, Jann Horn via Containers wrote: >>>>> On Wed, Sep 30, 2020 at 1:07 PM Michael Kerrisk (man-pages) >>>>> wrote: >>>>>> NOTES >>>>>>The file descriptor returned when seccom

Re: For review: seccomp_user_notif(2) manual page

2020-10-14 Thread Michael Kerrisk (man-pages)
Hi Tycho, Ping on the question below! Thanks, Michael On 10/1/20 9:45 AM, Michael Kerrisk (man-pages) wrote: > On 10/1/20 1:03 AM, Tycho Andersen wrote: >> On Wed, Sep 30, 2020 at 10:34:51PM +0200, Michael Kerrisk (man-pages) wrote: >>> Hi Tycho, >>> >>

Re: Regression: epoll edge-triggered (EPOLLET) for pipes/FIFOs

2020-10-13 Thread Michael Kerrisk (man-pages)
Hello Linus, On 10/13/20 12:30 AM, Linus Torvalds wrote: > On Mon, Oct 12, 2020 at 1:30 PM Michael Kerrisk (man-pages) > wrote: >> >> I don't think this is correct. The epoll(7) manual page >> sill carries the text written long ago by Davide Libenzi, >> the cre

Re: Regression: epoll edge-triggered (EPOLLET) for pipes/FIFOs

2020-10-12 Thread Michael Kerrisk (man-pages)
On 10/12/20 10:52 PM, Linus Torvalds wrote: > On Mon, Oct 12, 2020 at 1:30 PM Michael Kerrisk (man-pages) > wrote: >> >> [CC += Davide] > > I'm not sure how active Davide is any more.. Yep, I know. But just in case. >> I don't think this is correct. The epoll(

Re: Regression: epoll edge-triggered (EPOLLET) for pipes/FIFOs

2020-10-12 Thread Michael Kerrisk (man-pages)
[CC += Davide] Hello Linus, Thanks for your quick reply. On 10/12/20 9:25 PM, Linus Torvalds wrote: > On Mon, Oct 12, 2020 at 11:40 AM Michael Kerrisk (man-pages) > wrote: >> >> Between Linux 5.4 and 5.5 a regression was introduced in the operation >> of the epoll

Regression: epoll edge-triggered (EPOLLET) for pipes/FIFOs

2020-10-12 Thread Michael Kerrisk (man-pages)
e to pipe() 3: FAIL: ret = 0; EXPECTED: ret = 1, events = [ EPOLLIN ] Closing write end of pipe() 4: OK: ret = 1, events = [ EPOLLIN EPOLLHUP ] Thanks, Michael = /* pipe_epollet_test.c Copyright (c) 2020, Michael Kerrisk Licensed under GNU GPLv2

Re: [PATCH 2/2] off_t.3: New link to system_data_types(7)

2020-10-07 Thread Michael Kerrisk (man-pages)
gt; new file mode 100644 > index 0..db50c0f09 > --- /dev/null > +++ b/man3/off_t.3 > @@ -0,0 +1 @@ > +.so man7/system_data_types.7 > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 1/2] system_data_types.7: Add 'off_t'

2020-10-07 Thread Michael Kerrisk (man-pages)
, > +.BR truncate (2), > +.BR fseeko (3), > +.BR getdirentries (3), > +.BR lockf (3), > +.BR posix_fallocate (3) > +.\".PP TODO: loff_t, off64_t > +.\"See also the > +.\".I loff_t > +.\"and > +.\".I off64_t > +.\"types in t

Re: Navigational corrections

2020-10-07 Thread Michael Kerrisk (man-pages)
On 10/6/20 12:08 AM, Alejandro Colomar wrote: > Hi Michael, > > On 2020-10-03 13:39, Michael Kerrisk (man-pages) wrote: >> Hi Alex, > [...] >> >> off_t would be great. >> >> In case you are looking for some other candidates, some others >> that I w

Re: [PATCH v4 1/2] system_data_types.7: Add 'void *'

2020-10-03 Thread Michael Kerrisk (man-pages)
On 10/3/20 9:48 AM, G. Branden Robinson wrote: > At 2020-10-03T09:10:14+0200, Michael Kerrisk (man-pages) wrote: >> On 10/2/20 10:27 PM, Alejandro Colomar wrote: >>> On 2020-10-02 22:14, Paul Eggert wrote: >>> > On 10/2/20 11:38 AM, Alejandro Colomar

Re: Navigational corrections

2020-10-03 Thread Michael Kerrisk (man-pages)
ted types, > which were the ones that made me want this page. off_t would be great. In case you are looking for some other candidates, some others that I would be interested to see go into the page would be fd_set clock_t clockid_t and probably dev_t Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH v5 2/2] void.3: New link to system_data_types(7)

2020-10-03 Thread Michael Kerrisk (man-pages)
--git a/man3/void.3 b/man3/void.3 > new file mode 100644 > index 0..db50c0f09 > --- /dev/null > +++ b/man3/void.3 > @@ -0,0 +1 @@ > +.so man7/system_data_types.7 > -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX Sy

Re: [PATCH v5 1/2] system_data_types.7: Add 'void *'

2020-10-03 Thread Michael Kerrisk (man-pages)
t; Compatibility between function pointers and void * hasn't always been so. > Document when that was added to POSIX. > > Reported-by: Michael Kerrisk > Signed-off-by: Alejandro Colomar > > system_data_types.7: void *: wfix > > Reported-by: Jonathan Wakely > Reported-by: Pa

Navigational corrections (was: Re: [PATCH v2 1/2] system_data_types.7: Add 'void *')

2020-10-03 Thread Michael Kerrisk (man-pages)
e a patch to remove these from the page? And again, my apologies for not being focused enough on the big picture sooner. I don't think 'void' belongs in this page. Nor basic types such as int, long, etc. The question of 'void *' is an interesting one. It is something like a fundamental C type, and

Re: [PATCH v4 1/2] system_data_types.7: Add 'void *'

2020-10-03 Thread Michael Kerrisk (man-pages)
comes from. However, for clarity I would normally also use quotes in this case. > Michael, you might want to have a look at it. > > I'll also add Branden, who might have something to say about it. Yes, maybe Branden can add some insight. Thanks, Michael -- Michael Kerrisk Linux man-

Re: [PATCH v2 1/2] system_data_types.7: Add 'void *'

2020-10-02 Thread Michael Kerrisk (man-pages)
would rather prefer one patch series, rather than patches on patches please. It also makes review of the overall 'void *' text easier if it's all one patch. So, If you could squash the patches together and resubmit, that would be helful. Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH v2 1/2] system_data_types.7: Add 'void *'

2020-10-02 Thread Michael Kerrisk (man-pages)
ver the line already very many years ago with operators(7), because who ever remembers all of the C operator precedences. -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: For review: seccomp_user_notif(2) manual page

2020-10-01 Thread Michael Kerrisk (man-pages)
the trouble of saying that (again). Cheers, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: For review: seccomp_user_notif(2) manual page

2020-10-01 Thread Michael Kerrisk (man-pages)
On 10/1/20 1:03 AM, Tycho Andersen wrote: > On Wed, Sep 30, 2020 at 10:34:51PM +0200, Michael Kerrisk (man-pages) wrote: >> Hi Tycho, >> >> Thanks for taking time to look at the page! >> >> On 9/30/20 5:03 PM, Tycho Andersen wrote: >>> On Wed, Sep 30, 2020

Re: For review: seccomp_user_notif(2) manual page

2020-09-30 Thread Michael Kerrisk (man-pages)
Hi Tycho, Thanks for taking time to look at the page! On 9/30/20 5:03 PM, Tycho Andersen wrote: > On Wed, Sep 30, 2020 at 01:07:38PM +0200, Michael Kerrisk (man-pages) wrote: >>2. In order that the supervisor process can obtain notifications >> using the

For review: seccomp_user_notif(2) manual page

2020-09-30 Thread Michael Kerrisk (man-pages)
SOCK_STREAM, 0, sockPair) == -1) errExit("socketpair"); /* Create a child process--the "target"--that installs seccomp filtering. The target process writes the seccomp notification file descriptor onto 'sockPair[0]' and then calls mkdir(2) for each directory in the command-line arguments. */ (void) targetProcess(sockPair, [optind]); /* Catch SIGCHLD when the target terminates, so that the supervisor can also terminate. */ struct sigaction sa; sa.sa_handler = sigchldHandler; sa.sa_flags = 0; sigemptyset(_mask); if (sigaction(SIGCHLD, , NULL) == -1) errExit("sigaction"); supervisor(sockPair); exit(EXIT_SUCCESS); } SEE ALSO ioctl(2), seccomp(2) -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 12/24] getgrent_r.3: Use sizeof() to get buffer size (instead of hardcoding macro name)

2020-09-29 Thread Michael Kerrisk (man-pages)
> 2.- Use sizeof() everywhere, and the macro for the initializer. > > pros: > - It is valid as long as the buffer is an array. > cons: > - If the code gets into a function, and the buffer is then a pointer, >it will definitively produce a silent bug. Sigh! I just did exactly the last

Re: [PATCH 22/24] membarrier.2: Note that glibc does not provide a wrapper

2020-09-29 Thread Michael Kerrisk (man-pages)
ep :) > I agree it's visually somewhat harder, but grepping is way easier. I'd say number 2 also. But, visually, it's the least difficult for me. Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 22/24] membarrier.2: Note that glibc does not provide a wrapper

2020-09-29 Thread Michael Kerrisk (man-pages)
Hi Branden, On 9/27/20 7:46 AM, G. Branden Robinson wrote: > At 2020-09-24T10:06:23+0200, Michael Kerrisk (man-pages) wrote: >> Thanks for the interesting history, Branden! > > Hi, Michael. And you're welcome! I often wonder if I test people's > patience with my info dumps bu

Re: [PATCH] man/statx: Add STATX_ATTR_DAX

2020-09-29 Thread Michael Kerrisk (man-pages)
ogether. >> +.PP >> +A DAX file may support being mapped with the MAP_SYNC flag, which enables a >> +program to use CPU cache flush instructions to persist CPU store operations >> +without an explicit >> +.BR fsync(2). >> +See >> +.BR mmap(2) >> +for more information. >> .SH RETURN VALUE >> On success, zero is returned. >> On error, \-1 is returned, and >> -- >> 2.25.1 >> -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 12/24] getgrent_r.3: Use sizeof() to get buffer size (instead of hardcoding macro name)

2020-09-24 Thread Michael Kerrisk (man-pages)
On 9/24/20 11:35 AM, Alejandro Colomar wrote: > Hi, > > On 2020-09-23 22:35, Michael Kerrisk (man-pages) wrote: >> On 9/15/20 12:03 PM, Stefan Puiu wrote: >>> Hi, >>> >>> On Fri, Sep 11, 2020 at 6:28 PM Alejandro Colomar >>> wrote: >>>

Re: [PATCH 12/24] getgrent_r.3: Use sizeof() to get buffer size (instead of hardcoding macro name)

2020-09-24 Thread Michael Kerrisk (man-pages)
Hi Alex, [..] > I was reverting the 3 patches I introduced (they changed from solution 1 > to solution 2), and also was grepping for already existing solution 2 in > the pages (it seems that solution 2 was a bit more extended than > solution 1). Just so I can refresh my cache, which commits

Re: [PATCH 22/24] membarrier.2: Note that glibc does not provide a wrapper

2020-09-24 Thread Michael Kerrisk (man-pages)
ve models like the IBM > Executive even featured true proportional type. Thanks for the interesting history, Branden! >From time toi time I wonder if the function prototypes in the SYNOPSIS should also be inside .EX/.EE. Your thoughts? Thanks, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH 12/24] getgrent_r.3: Use sizeof() to get buffer size (instead of hardcoding macro name)

2020-09-23 Thread Michael Kerrisk (man-pages)
enough, not perfect or completely foolproof. If you need to look up > obscure gcc features to understand an example, that's not very > helpful. So I'd be more inclined to prefer version 1 above. But let's > see Michael's opinion on this. > > Just my 2c, So, the fundamental

Re: [PATCH v5 1/3] open: add close_range()

2020-09-17 Thread Michael Kerrisk (man-pages)
Hey Christian, Could we please have a manual page for the close_range(2) syscall that's about to land in 5.9? Thanks, Michael On Wed, 3 Jun 2020 at 12:24, Michael Kerrisk (man-pages) wrote: > > Hi Christian, > > Could we have a manual page for this API (best before it's merged)

  1   2   3   4   5   6   7   8   9   10   >