labath created this revision. labath added reviewers: tberghammer, ovyalov. labath added a subscriber: lldb-commits. Herald added subscribers: srhines, danalbert, tberghammer.
Signalfd is not used in the code anymore, and given that the same functionality can be achieved with the new MainLoop class, it's unlikely we will need it in the future. Remove all traces of it. http://reviews.llvm.org/D17510 Files: include/lldb/Host/linux/Signalfd.h source/Host/android/LibcGlue.cpp source/Plugins/Process/Linux/NativeProcessLinux.cpp tools/lldb-server/lldb-gdbserver.cpp
Index: tools/lldb-server/lldb-gdbserver.cpp =================================================================== --- tools/lldb-server/lldb-gdbserver.cpp +++ tools/lldb-server/lldb-gdbserver.cpp @@ -414,13 +414,6 @@ signal (SIGPIPE, signal_handler); MainLoop::SignalHandleUP sighup_handle = mainloop.RegisterSignal(SIGHUP, sighup_handler, error); #endif -#ifdef __linux__ - // Block delivery of SIGCHLD on linux. NativeProcessLinux will read it using signalfd. - sigset_t set; - sigemptyset(&set); - sigaddset(&set, SIGCHLD); - pthread_sigmask(SIG_BLOCK, &set, NULL); -#endif const char *progname = argv[0]; const char *subcommand = argv[1]; Index: source/Plugins/Process/Linux/NativeProcessLinux.cpp =================================================================== --- source/Plugins/Process/Linux/NativeProcessLinux.cpp +++ source/Plugins/Process/Linux/NativeProcessLinux.cpp @@ -58,7 +58,6 @@ #include "lldb/Host/linux/Personality.h" #include "lldb/Host/linux/Ptrace.h" -#include "lldb/Host/linux/Signalfd.h" #include "lldb/Host/linux/Uio.h" #include "lldb/Host/android/Android.h" Index: source/Host/android/LibcGlue.cpp =================================================================== --- source/Host/android/LibcGlue.cpp +++ source/Host/android/LibcGlue.cpp @@ -27,11 +27,6 @@ return (time_t) timegm64(t); } -int signalfd (int fd, const sigset_t *mask, int flags) -{ - return syscall(__NR_signalfd4, fd, mask, _NSIG / 8, flags); -} - int posix_openpt(int flags) { return open("/dev/ptmx", flags); Index: include/lldb/Host/linux/Signalfd.h =================================================================== --- include/lldb/Host/linux/Signalfd.h +++ /dev/null @@ -1,54 +0,0 @@ -//===-- Signalfd.h ----------------------------------------------*- C++ -*-===// -// -// The LLVM Compiler Infrastructure -// -// This file is distributed under the University of Illinois Open Source -// License. See LICENSE.TXT for details. -// -//===----------------------------------------------------------------------===// - -// This file defines signalfd functions & structures - -#ifndef liblldb_Host_linux_Signalfd_h_ -#define liblldb_Host_linux_Signalfd_h_ - -#ifdef __ANDROID_NDK__ -#include <android/api-level.h> -#endif - -#if defined(__ANDROID_API__) && __ANDROID_API__ < 21 - -#include <linux/types.h> -#include <linux/fcntl.h> - -#define SFD_CLOEXEC O_CLOEXEC -#define SFD_NONBLOCK O_NONBLOCK - -struct signalfd_siginfo { - __u32 ssi_signo; - __s32 ssi_errno; - __s32 ssi_code; - __u32 ssi_pid; - __u32 ssi_uid; - __s32 ssi_fd; - __u32 ssi_tid; - __u32 ssi_band; - __u32 ssi_overrun; - __u32 ssi_trapno; - __s32 ssi_status; - __s32 ssi_int; - __u64 ssi_ptr; - __u64 ssi_utime; - __u64 ssi_stime; - __u64 ssi_addr; - __u16 ssi_addr_lsb; - __u8 __pad[46]; -}; - -int signalfd (int fd, const sigset_t *mask, int flags); - -#else -#include <sys/signalfd.h> -#endif - -#endif // liblldb_Host_linux_Signalfd_h_
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits