Index: include/lldb/Target/Platform.h =================================================================== --- include/lldb/Target/Platform.h +++ include/lldb/Target/Platform.h @@ -1137,6 +1137,9 @@ const uint64_t src_size, const FileSpec& dst_file_spec); + virtual const char * + GetCacheHostname (); + private: typedef std::function ModuleResolver; Index: source/Plugins/Platform/Android/AdbClient.h =================================================================== --- source/Plugins/Platform/Android/AdbClient.h +++ source/Plugins/Platform/Android/AdbClient.h @@ -21,6 +21,7 @@ // Project includes #include "lldb/Core/Error.h" +#include "lldb/Host/ConnectionFileDescriptor.h" namespace lldb_private { @@ -29,11 +30,14 @@ public: using DeviceIDList = std::list; + static Error + CreateByDeviceID (const char* device_id, AdbClient &adb); + AdbClient () = default; explicit AdbClient (const std::string &device_id); - void - SetDeviceID (const std::string& device_id); + const std::string& + GetDeviceID() const; Error GetDevices (DeviceIDList &device_list); @@ -48,6 +52,9 @@ Error Connect (); + void + SetDeviceID (const std::string& device_id); + Error SendMessage (const std::string &packet); Index: source/Plugins/Platform/Android/AdbClient.cpp =================================================================== --- source/Plugins/Platform/Android/AdbClient.cpp +++ source/Plugins/Platform/Android/AdbClient.cpp @@ -8,7 +8,6 @@ //===----------------------------------------------------------------------===// // Other libraries and framework includes -#include "lldb/Host/ConnectionFileDescriptor.h" #include "llvm/ADT/SmallVector.h" #include "llvm/ADT/StringRef.h" #include "llvm/ADT/STLExtras.h" @@ -16,6 +15,7 @@ // Project includes #include "AdbClient.h" +#include #include using namespace lldb; @@ -29,6 +29,32 @@ } // namespace +Error +AdbClient::CreateByDeviceID (const char* device_id, AdbClient &adb) +{ + DeviceIDList connect_devices; + auto error = adb.GetDevices (connect_devices); + if (error.Fail ()) + return error; + + if (device_id) + { + auto find_it = std::find(connect_devices.begin (), connect_devices.end (), device_id); + if (find_it == connect_devices.end ()) + return Error ("Device \"%s\" not found", device_id); + + adb.SetDeviceID (*find_it); + } + else + { + if (connect_devices.size () != 1) + return Error ("Expected a single connected device, got instead %zu", connect_devices.size ()); + + adb.SetDeviceID (connect_devices.front ()); + } + return error; +} + AdbClient::AdbClient (const std::string &device_id) : m_device_id (device_id) { @@ -40,6 +66,12 @@ m_device_id = device_id; } +const std::string& +AdbClient::GetDeviceID() const +{ + return m_device_id; +} + Error AdbClient::Connect () { Index: source/Plugins/Platform/Android/PlatformAndroid.h =================================================================== --- source/Plugins/Platform/Android/PlatformAndroid.h +++ source/Plugins/Platform/Android/PlatformAndroid.h @@ -12,6 +12,9 @@ // C Includes // C++ Includes + +#include + // Other libraries and framework includes // Project includes #include "Plugins/Platform/Linux/PlatformLinux.h" @@ -60,7 +63,12 @@ lldb_private::Error ConnectRemote (lldb_private::Args& args) override; + protected: + const char * + GetCacheHostname () override; + private: + std::string m_device_id; DISALLOW_COPY_AND_ASSIGN (PlatformAndroid); }; } // namespace lldb_private Index: source/Plugins/Platform/Android/PlatformAndroid.cpp =================================================================== --- source/Plugins/Platform/Android/PlatformAndroid.cpp +++ source/Plugins/Platform/Android/PlatformAndroid.cpp @@ -15,6 +15,7 @@ #include "lldb/Host/HostInfo.h" // Project includes +#include "AdbClient.h" #include "PlatformAndroid.h" #include "PlatformAndroidRemoteGDBServer.h" @@ -171,6 +172,8 @@ Error PlatformAndroid::ConnectRemote (Args& args) { + m_device_id.clear (); + if (IsHost()) { return Error ("can't connect to the host platform '%s', always connected", GetPluginName().GetCString()); @@ -178,5 +181,24 @@ if (!m_remote_platform_sp) m_remote_platform_sp = PlatformSP(new PlatformAndroidRemoteGDBServer()); - return PlatformLinux::ConnectRemote (args); + + auto error = PlatformLinux::ConnectRemote (args); + if (error.Success ()) + { + // Fetch the device list from ADB and if only 1 device found then use that device + // TODO: Handle the case when more device is available + AdbClient adb; + error = AdbClient::CreateByDeviceID (nullptr, adb); + if (error.Fail ()) + return error; + + m_device_id = adb.GetDeviceID (); + } + return error; +} + +const char * +PlatformAndroid::GetCacheHostname () +{ + return m_device_id.c_str (); } Index: source/Plugins/Platform/Android/PlatformAndroidRemoteGDBServer.cpp =================================================================== --- source/Plugins/Platform/Android/PlatformAndroidRemoteGDBServer.cpp +++ source/Plugins/Platform/Android/PlatformAndroidRemoteGDBServer.cpp @@ -10,8 +10,6 @@ // Other libraries and framework includes #include "lldb/Core/Error.h" #include "lldb/Core/Log.h" -#include "lldb/Host/ConnectionFileDescriptor.h" -#include "llvm/ADT/StringRef.h" // Project includes #include "AdbClient.h" @@ -31,20 +29,13 @@ // Fetch the device list from ADB and if only 1 device found then use that device // TODO: Handle the case when more device is available AdbClient adb; - - AdbClient::DeviceIDList connect_devices; - auto error = adb.GetDevices (connect_devices); + auto error = AdbClient::CreateByDeviceID (nullptr, adb); if (error.Fail ()) return error; - if (connect_devices.size () != 1) - return Error ("Expected a single connected device, got instead %zu", connect_devices.size ()); - - device_id = connect_devices.front (); if (log) - log->Printf("Connected to Android device \"%s\"", device_id.c_str ()); + log->Printf("Connected to Android device \"%s\"", adb.GetDeviceID ().c_str ()); - adb.SetDeviceID (device_id); return adb.SetPortForwarding (port); } Index: source/Target/Platform.cpp =================================================================== --- source/Target/Platform.cpp +++ source/Target/Platform.cpp @@ -1838,7 +1838,7 @@ // Check local cache for a module. auto error = m_module_cache->Get (GetModuleCacheRoot (), - GetHostname (), + GetCacheHostname (), module_spec, module_sp, did_create_ptr); @@ -1880,7 +1880,7 @@ // Put downloaded file into local module cache. error = m_module_cache->Put (GetModuleCacheRoot (), - GetHostname (), + GetCacheHostname (), module_spec, tmp_download_file_spec); if (error.Fail ()) @@ -1893,7 +1893,7 @@ } error = m_module_cache->Get (GetModuleCacheRoot (), - GetHostname (), + GetCacheHostname (), module_spec, module_sp, did_create_ptr); @@ -1958,3 +1958,9 @@ dir_spec.AppendPathComponent (GetName ().AsCString ()); return dir_spec; } + +const char * +Platform::GetCacheHostname () +{ + return GetHostname (); +}