* [PATCH] linux-user/syscall: add support for CLONE_PIDFD
@ 2021-09-21 18:48 Andreas Schwab
2021-09-27 13:42 ` Peter Maydell
0 siblings, 1 reply; 3+ messages in thread
From: Andreas Schwab @ 2021-09-21 18:48 UTC (permalink / raw)
To: Laurent Vivier; +Cc: qemu-devel
Add basic support for CLONE_PIDFD, only fork-like clone without additional
flags. This is enough to make Qt/forkfd working.
Signed-off-by: Andreas Schwab <schwab@suse.de>
---
linux-user/syscall.c | 52 ++++++++++++++++++++++++++++++++++++++++++--
1 file changed, 50 insertions(+), 2 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 544f5b662f..8b40064e75 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -143,6 +143,9 @@
#ifndef CLONE_IO
#define CLONE_IO 0x80000000 /* Clone io context */
#endif
+#ifndef CLONE_PIDFD
+#define CLONE_PIDFD 0x00001000 /* set if a pidfd should be placed in parent */
+#endif
/* We can't directly call the host clone syscall, because this will
* badly confuse libc (breaking mutexes, for example). So we must
@@ -169,7 +172,8 @@
/* Flags for fork which we can implement within QEMU itself */
#define CLONE_OPTIONAL_FORK_FLAGS \
(CLONE_SETTLS | CLONE_PARENT_SETTID | \
- CLONE_CHILD_CLEARTID | CLONE_CHILD_SETTID)
+ CLONE_CHILD_CLEARTID | CLONE_CHILD_SETTID | \
+ CLONE_PIDFD)
/* Flags for thread creation which we can implement within QEMU itself */
#define CLONE_OPTIONAL_THREAD_FLAGS \
@@ -494,6 +498,39 @@ _syscall4(int, sys_prlimit64, pid_t, pid, int, resource,
struct host_rlimit64 *, old_limit)
#endif
+#if defined __NR_clone2
+#define __NR_sys_clone2 __NR_clone2
+_syscall6(int, sys_clone2, int, flags, void *, child_stack, size_t, stack_size,
+ int *, ptid, int *, ctid, void *, newtls);
+#else
+#define __NR_sys_clone __NR_clone
+#if defined __cris__ || defined __s390x__
+_syscall5(int, sys_clone, void *, child_stack, int, flags, int *, ptid,
+ void *, newtls, int *, ctid);
+#elif defined __microblaze__
+_syscall6(int, sys_clone, int, flags, void *, child_stack, size_t, stack_size,
+ int *, ptid, void *, newtls, int *, ctid);
+#else
+/*
+ * Note: ctid and newtls are swapped on some architectures, but both are
+ * passed as NULL only for now.
+ */
+_syscall5(int, sys_clone, int, flags, void *, child_stack, int *, ptid,
+ int *, ctid, void *, newtls);
+#endif
+#endif
+static int sys_clone_pidfd(int flags, int *pidfd)
+{
+#ifdef __NR_clone2
+ return sys_clone2(flags, NULL, 0, pidfd, NULL, NULL);
+#elif defined __cris__ || defined __s390x__
+ return sys_clone(NULL, flags, pidfd, NULL, NULL);
+#elif defined __microblaze__
+ return sys_clone(flags, NULL, 0, pidfd, NULL, NULL);
+#else
+ return sys_clone(flags, NULL, pidfd, NULL, NULL);
+#endif
+}
#if defined(TARGET_NR_timer_create)
/* Maximum of 32 active POSIX timers allowed at any one time. */
@@ -6355,6 +6392,7 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
CPUState *new_cpu;
CPUArchState *new_env;
sigset_t sigmask;
+ int pidfd;
flags &= ~CLONE_IGNORED_FLAGS;
@@ -6362,6 +6400,10 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
if (flags & CLONE_VFORK)
flags &= ~(CLONE_VFORK | CLONE_VM);
+ /* Only basic fork-like clone is supported with CLONE_PIDFD for now. */
+ if (flags & CLONE_PIDFD && flags & ~(CLONE_PIDFD|CSIGNAL))
+ return -TARGET_EINVAL;
+
if (flags & CLONE_VM) {
TaskState *parent_ts = (TaskState *)cpu->opaque;
new_thread_info info;
@@ -6460,7 +6502,11 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
}
fork_start();
- ret = fork();
+ if (flags & CLONE_PIDFD) {
+ ret = sys_clone_pidfd(flags, &pidfd);
+ } else {
+ ret = fork();
+ }
if (ret == 0) {
/* Child Process. */
cpu_clone_regs_child(env, newsp, flags);
@@ -6483,6 +6529,8 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
} else {
cpu_clone_regs_parent(env, flags);
fork_end(0);
+ if (flags & CLONE_PIDFD)
+ put_user_u32(pidfd, parent_tidptr);
}
}
return ret;
--
2.33.0
--
Andreas Schwab, SUSE Labs, schwab@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] linux-user/syscall: add support for CLONE_PIDFD
2021-09-21 18:48 [PATCH] linux-user/syscall: add support for CLONE_PIDFD Andreas Schwab
@ 2021-09-27 13:42 ` Peter Maydell
0 siblings, 0 replies; 3+ messages in thread
From: Peter Maydell @ 2021-09-27 13:42 UTC (permalink / raw)
To: Andreas Schwab; +Cc: Laurent Vivier, QEMU Developers
On Tue, 21 Sept 2021 at 19:50, Andreas Schwab <schwab@suse.de> wrote:
>
> Add basic support for CLONE_PIDFD, only fork-like clone without additional
> flags. This is enough to make Qt/forkfd working.
>
> Signed-off-by: Andreas Schwab <schwab@suse.de>
> ---
> linux-user/syscall.c | 52 ++++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 50 insertions(+), 2 deletions(-)
> /* We can't directly call the host clone syscall, because this will
> * badly confuse libc (breaking mutexes, for example). So we must
So, this pre-existing comment says "we can't directly call the
host clone syscall"...
> +#if defined __NR_clone2
> +#define __NR_sys_clone2 __NR_clone2
> +_syscall6(int, sys_clone2, int, flags, void *, child_stack, size_t, stack_size,
> + int *, ptid, int *, ctid, void *, newtls);
> +#else
> +#define __NR_sys_clone __NR_clone
> +#if defined __cris__ || defined __s390x__
> +_syscall5(int, sys_clone, void *, child_stack, int, flags, int *, ptid,
> + void *, newtls, int *, ctid);
> +#elif defined __microblaze__
> +_syscall6(int, sys_clone, int, flags, void *, child_stack, size_t, stack_size,
> + int *, ptid, void *, newtls, int *, ctid);
> +#else
> +/*
> + * Note: ctid and newtls are swapped on some architectures, but both are
> + * passed as NULL only for now.
> + */
> +_syscall5(int, sys_clone, int, flags, void *, child_stack, int *, ptid,
> + int *, ctid, void *, newtls);
> +#endif
> +#endif
> +static int sys_clone_pidfd(int flags, int *pidfd)
> +{
> +#ifdef __NR_clone2
> + return sys_clone2(flags, NULL, 0, pidfd, NULL, NULL);
> +#elif defined __cris__ || defined __s390x__
> + return sys_clone(NULL, flags, pidfd, NULL, NULL);
> +#elif defined __microblaze__
> + return sys_clone(flags, NULL, 0, pidfd, NULL, NULL);
> +#else
> + return sys_clone(flags, NULL, pidfd, NULL, NULL);
> +#endif
> +}
...but this patch introduces code which directly calls the host
clone syscall.
I think this ought to have a bit more explanation and updating
of the existing comments to explain why this is OK.
thanks
-- PMM
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] linux-user/syscall: add support for CLONE_PIDFD
@ 2021-08-02 12:55 Andreas Schwab
0 siblings, 0 replies; 3+ messages in thread
From: Andreas Schwab @ 2021-08-02 12:55 UTC (permalink / raw)
To: qemu-devel; +Cc: Laurent Vivier
Add basic support for CLONE_PIDFD, only fork-like clone without additional
flags. This is enough to make Qt/forkfd working.
Signed-off-by: Andreas Schwab <schwab@suse.de>
---
linux-user/syscall.c | 52 ++++++++++++++++++++++++++++++++++++++++++--
1 file changed, 50 insertions(+), 2 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index ccd3892b2d..ad0de26dd7 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -137,6 +137,9 @@
#ifndef CLONE_IO
#define CLONE_IO 0x80000000 /* Clone io context */
#endif
+#ifndef CLONE_PIDFD
+#define CLONE_PIDFD 0x00001000 /* set if a pidfd should be placed in parent */
+#endif
/* We can't directly call the host clone syscall, because this will
* badly confuse libc (breaking mutexes, for example). So we must
@@ -163,7 +166,8 @@
/* Flags for fork which we can implement within QEMU itself */
#define CLONE_OPTIONAL_FORK_FLAGS \
(CLONE_SETTLS | CLONE_PARENT_SETTID | \
- CLONE_CHILD_CLEARTID | CLONE_CHILD_SETTID)
+ CLONE_CHILD_CLEARTID | CLONE_CHILD_SETTID | \
+ CLONE_PIDFD)
/* Flags for thread creation which we can implement within QEMU itself */
#define CLONE_OPTIONAL_THREAD_FLAGS \
@@ -488,6 +492,39 @@ _syscall4(int, sys_prlimit64, pid_t, pid, int, resource,
struct host_rlimit64 *, old_limit)
#endif
+#if defined __NR_clone2
+#define __NR_sys_clone2 __NR_clone2
+_syscall6(int, sys_clone2, int, flags, void *, child_stack, size_t, stack_size,
+ int *, ptid, int *, ctid, void *, newtls);
+#else
+#define __NR_sys_clone __NR_clone
+#if defined __cris__ || defined __s390x__
+_syscall5(int, sys_clone, void *, child_stack, int, flags, int *, ptid,
+ void *, newtls, int *, ctid);
+#elif defined __microblaze__
+_syscall6(int, sys_clone, int, flags, void *, child_stack, size_t, stack_size,
+ int *, ptid, void *, newtls, int *, ctid);
+#else
+/*
+ * Note: ctid and newtls are swapped on some architectures, but both are
+ * passed as NULL only for now.
+ */
+_syscall5(int, sys_clone, int, flags, void *, child_stack, int *, ptid,
+ int *, ctid, void *, newtls);
+#endif
+#endif
+static int sys_clone_pidfd(int flags, int *pidfd)
+{
+#ifdef __NR_clone2
+ return sys_clone2(flags, NULL, 0, pidfd, NULL, NULL);
+#elif defined __cris__ || defined __s390x__
+ return sys_clone(NULL, flags, pidfd, NULL, NULL);
+#elif defined __microblaze__
+ return sys_clone(flags, NULL, 0, pidfd, NULL, NULL);
+#else
+ return sys_clone(flags, NULL, pidfd, NULL, NULL);
+#endif
+}
#if defined(TARGET_NR_timer_create)
/* Maximum of 32 active POSIX timers allowed at any one time. */
@@ -6346,6 +6383,7 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
CPUState *new_cpu;
CPUArchState *new_env;
sigset_t sigmask;
+ int pidfd;
flags &= ~CLONE_IGNORED_FLAGS;
@@ -6353,6 +6391,10 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
if (flags & CLONE_VFORK)
flags &= ~(CLONE_VFORK | CLONE_VM);
+ /* Only basic fork-like clone is supported with CLONE_PIDFD for now. */
+ if (flags & CLONE_PIDFD && flags & ~(CLONE_PIDFD|CSIGNAL))
+ return -TARGET_EINVAL;
+
if (flags & CLONE_VM) {
TaskState *parent_ts = (TaskState *)cpu->opaque;
new_thread_info info;
@@ -6451,7 +6493,11 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
}
fork_start();
- ret = fork();
+ if (flags & CLONE_PIDFD) {
+ ret = sys_clone_pidfd(flags, &pidfd);
+ } else {
+ ret = fork();
+ }
if (ret == 0) {
/* Child Process. */
cpu_clone_regs_child(env, newsp, flags);
@@ -6474,6 +6520,8 @@ static int do_fork(CPUArchState *env, unsigned int flags, abi_ulong newsp,
} else {
cpu_clone_regs_parent(env, flags);
fork_end(0);
+ if (flags & CLONE_PIDFD)
+ put_user_u32(pidfd, parent_tidptr);
}
}
return ret;
--
2.32.0
--
Andreas Schwab, SUSE Labs, schwab@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-09-27 14:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-21 18:48 [PATCH] linux-user/syscall: add support for CLONE_PIDFD Andreas Schwab
2021-09-27 13:42 ` Peter Maydell
-- strict thread matches above, loose matches on Subject: below --
2021-08-02 12:55 Andreas Schwab
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).