X-Git-Url: https://git.lttng.org/?p=ust.git;a=blobdiff_plain;f=libinterfork%2Finterfork.c;h=ecc39e38ca78c3e413de26da20c99367b2c50554;hp=26a8bd6ad9f314525b6cc9b079d0fb03fd295ef5;hb=761547672693a192995b72ad69b41d079636de3a;hpb=df793c559d17d5b3c39df6973b2b3d85ed6328b9 diff --git a/libinterfork/interfork.c b/libinterfork/interfork.c index 26a8bd6..ecc39e3 100644 --- a/libinterfork/interfork.c +++ b/libinterfork/interfork.c @@ -20,21 +20,21 @@ #include #include #include -#include "share/usterr.h" +#include +#include +#include +#include +#include "usterr.h" -extern void ust_fork(void); -extern void ust_potential_exec(void); +struct user_desc; pid_t fork(void) { static pid_t (*plibc_func)(void) = NULL; + ust_fork_info_t fork_info; pid_t retval; - int result; - sigset_t all_sigs; - sigset_t orig_sigs; - if(plibc_func == NULL) { plibc_func = dlsym(RTLD_NEXT, "fork"); if(plibc_func == NULL) { @@ -43,37 +43,17 @@ pid_t fork(void) } } - /* Disable interrupts. This is to avoid that the child - * intervenes before it is properly setup for tracing. It is - * safer to disable all signals, because then we know we are not - * breaking anything by restoring the original mask. - */ - - /* FIXME: - - only do this if tracing is active - */ - - /* Disable signals */ - sigfillset(&all_sigs); - result = sigprocmask(SIG_BLOCK, &all_sigs, &orig_sigs); - if(result == -1) { - PERROR("sigprocmask"); - return -1; - } + ust_before_fork(&fork_info); /* Do the real fork */ retval = plibc_func(); if(retval == 0) { /* child */ - ust_fork(); + ust_after_fork_child(&fork_info); } - - /* Restore signals */ - result = sigprocmask(SIG_BLOCK, &orig_sigs, NULL); - if(result == -1) { - PERROR("sigprocmask"); - return -1; + else { + ust_after_fork_parent(&fork_info); } return retval; @@ -88,7 +68,7 @@ int execve(const char *filename, char *const argv[], char *const envp[]) if(plibc_func == NULL) { plibc_func = dlsym(RTLD_NEXT, "execve"); if(plibc_func == NULL) { - fprintf(stderr, "libcwrap: unable to find execve\n"); + fprintf(stderr, "libinterfork: unable to find execve\n"); return -1; } } @@ -99,3 +79,65 @@ int execve(const char *filename, char *const argv[], char *const envp[]) return retval; } + +struct interfork_clone_info { + int (*fn)(void *); + void *arg; + ust_fork_info_t fork_info; +}; + +static int clone_fn(void *arg) +{ + struct interfork_clone_info *info = (struct interfork_clone_info *)arg; + + /* clone is now done and we are in child */ + ust_after_fork_child(&info->fork_info); + + return info->fn(info->arg); +} + +int clone(int (*fn)(void *), void *child_stack, int flags, void *arg, ...) +{ + static int (*plibc_func)(int (*fn)(void *), void *child_stack, int flags, void *arg, pid_t *ptid, struct user_desc *tls, pid_t *ctid) = NULL; + + /* varargs */ + pid_t *ptid; + struct user_desc *tls; + pid_t *ctid; + + int retval; + + va_list ap; + + va_start(ap, arg); + ptid = va_arg(ap, pid_t *); + tls = va_arg(ap, struct user_desc *); + ctid = va_arg(ap, pid_t *); + va_end(ap); + + if(plibc_func == NULL) { + plibc_func = dlsym(RTLD_NEXT, "clone"); + if(plibc_func == NULL) { + fprintf(stderr, "libinterfork: unable to find clone\n"); + return -1; + } + } + + if(flags & CLONE_VM) { + /* creating a thread, no need to intervene, just pass on the arguments */ + retval = plibc_func(fn, child_stack, flags, arg, ptid, tls, ctid); + } + else { + /* creating a real process, we need to intervene */ + struct interfork_clone_info info = { fn: fn, arg: arg }; + + ust_before_fork(&info.fork_info); + + retval = plibc_func(clone_fn, child_stack, flags, &info, ptid, tls, ctid); + + /* The child doesn't get here */ + ust_after_fork_parent(&info.fork_info); + } + + return retval; +}