From e271658ec1d7ef546d44c70f72baaa26d5903cec Mon Sep 17 00:00:00 2001 From: Chaoren Lin Date: Wed, 15 Jul 2015 19:22:12 +0000 Subject: [PATCH] Use accept instead of accept4 for Android. Summary: The accept4 syscall is missing on older ARM Android kernels, and the accept() call is implemented with the accept4 syscall, so we'll need to call the accept syscall directly. Reviewers: vharron, tberghammer, labath Subscribers: ovyalov, chaoren, labath, tberghammer, aemerson, lldb-commits Differential Revision: http://reviews.llvm.org/D10887 llvm-svn: 242319 --- lldb/cmake/platforms/Android.cmake | 4 ++++ lldb/source/Host/common/Socket.cpp | 21 +++++++++++++++++++-- 2 files changed, 23 insertions(+), 2 deletions(-) diff --git a/lldb/cmake/platforms/Android.cmake b/lldb/cmake/platforms/Android.cmake index 3374bcc7f0d4..adf947a7a9ce 100644 --- a/lldb/cmake/platforms/Android.cmake +++ b/lldb/cmake/platforms/Android.cmake @@ -102,6 +102,10 @@ elseif( ANDROID_ABI STREQUAL "armeabi" ) # 64 bit atomic operations used in c++ libraries require armv7-a instructions # armv5te and armv6 were tried but do not work. set( ANDROID_CXX_FLAGS "${ANDROID_CXX_FLAGS} -march=armv7-a" ) + if ( LLVM_BUILD_STATIC ) + # Temporary workaround for static linking with the latest API. + set( ANDROID_CXX_FLAGS "${ANDROID_CXX_FLAGS} -DANDROID_ARM_BUILD_STATIC" ) + endif() endif() # linker flags diff --git a/lldb/source/Host/common/Socket.cpp b/lldb/source/Host/common/Socket.cpp index f7e93c634a12..4be8b74658e2 100644 --- a/lldb/source/Host/common/Socket.cpp +++ b/lldb/source/Host/common/Socket.cpp @@ -24,7 +24,11 @@ #include #include #include -#endif +#if defined(ANDROID_ARM_BUILD_STATIC) +#include +#include +#endif // ANDROID_ARM_BUILD_STATIC +#endif // __ANDROID_NDK__ #ifndef LLDB_DISABLE_POSIX #include @@ -70,7 +74,20 @@ NativeSocket CreateSocket(const int domain, const int type, const int protocol, NativeSocket Accept(NativeSocket sockfd, struct sockaddr *addr, socklen_t *addrlen, bool child_processes_inherit) { -#ifdef SOCK_CLOEXEC +#if defined(ANDROID_ARM_BUILD_STATIC) + // Temporary workaround for statically linking Android lldb-server with the + // latest API. + int fd = syscall(__NR_accept, sockfd, addr, addrlen); + if (fd >= 0 && !child_processes_inherit) + { + int flags = ::fcntl(fd, F_GETFD); + if (flags == -1) + return -1; + if (::fcntl(fd, F_SETFD, flags | FD_CLOEXEC) == -1) + return -1; + } + return fd; +#elif defined(SOCK_CLOEXEC) int flags = 0; if (!child_processes_inherit) { flags |= SOCK_CLOEXEC;