be788d891d
Add a mutex in MapInfo, and a mutex in Elf. Lock the creation of an Elf file using the MapInfo mutex, and lock when calling Step, GetFunctionName, or GetSoname since they can modify information in the object. It might be beneficial to use a fine grained lock in the future. Change the Maps object to contain a vector of MapInfo pointers rather than the total objects. This avoids copying this data around. Add a test to libbacktrace to verify that sharing a map while doing unwinds in different threads works. Add concurrency tests in libunwindstack to verify the locking works. Add always inline to the RegsGetLocal arm and aarch64 functions. I had a case where clang did not inline the code, so make sure this is specified. Bug: 68813077 Test: New unit tests to cover the case. Passes all unit tests. Test: Ran a monkey test while dumping bugreports and verified that Test: no crashes in libunwind. Test: Remove the locking and verified that all of the concurrenty tests fail. Change-Id: I769e728c676f6bdae9e64ce4cdc03b6749beae03
113 lines
2.6 KiB
C++
113 lines
2.6 KiB
C++
/*
|
|
* Copyright (C) 2016 The Android Open Source Project
|
|
*
|
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
|
* you may not use this file except in compliance with the License.
|
|
* You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
* See the License for the specific language governing permissions and
|
|
* limitations under the License.
|
|
*/
|
|
|
|
#ifndef _LIBUNWINDSTACK_MAPS_H
|
|
#define _LIBUNWINDSTACK_MAPS_H
|
|
|
|
#include <sys/types.h>
|
|
#include <unistd.h>
|
|
|
|
#include <string>
|
|
#include <vector>
|
|
|
|
#include <unwindstack/MapInfo.h>
|
|
|
|
namespace unwindstack {
|
|
|
|
// Special flag to indicate a map is in /dev/. However, a map in
|
|
// /dev/ashmem/... does not set this flag.
|
|
static constexpr int MAPS_FLAGS_DEVICE_MAP = 0x8000;
|
|
|
|
class Maps {
|
|
public:
|
|
Maps() = default;
|
|
virtual ~Maps();
|
|
|
|
MapInfo* Find(uint64_t pc);
|
|
|
|
virtual bool Parse();
|
|
|
|
virtual const std::string GetMapsFile() const { return ""; }
|
|
|
|
typedef std::vector<MapInfo*>::iterator iterator;
|
|
iterator begin() { return maps_.begin(); }
|
|
iterator end() { return maps_.end(); }
|
|
|
|
typedef std::vector<MapInfo*>::const_iterator const_iterator;
|
|
const_iterator begin() const { return maps_.begin(); }
|
|
const_iterator end() const { return maps_.end(); }
|
|
|
|
size_t Total() { return maps_.size(); }
|
|
|
|
MapInfo* Get(size_t index) {
|
|
if (index >= maps_.size()) return nullptr;
|
|
return maps_[index];
|
|
}
|
|
|
|
protected:
|
|
std::vector<MapInfo*> maps_;
|
|
};
|
|
|
|
class RemoteMaps : public Maps {
|
|
public:
|
|
RemoteMaps(pid_t pid) : pid_(pid) {}
|
|
virtual ~RemoteMaps() = default;
|
|
|
|
virtual const std::string GetMapsFile() const override;
|
|
|
|
private:
|
|
pid_t pid_;
|
|
};
|
|
|
|
class LocalMaps : public RemoteMaps {
|
|
public:
|
|
LocalMaps() : RemoteMaps(getpid()) {}
|
|
virtual ~LocalMaps() = default;
|
|
};
|
|
|
|
class BufferMaps : public Maps {
|
|
public:
|
|
BufferMaps(const char* buffer) : buffer_(buffer) {}
|
|
virtual ~BufferMaps() = default;
|
|
|
|
bool Parse() override;
|
|
|
|
private:
|
|
const char* buffer_;
|
|
};
|
|
|
|
class FileMaps : public Maps {
|
|
public:
|
|
FileMaps(const std::string& file) : file_(file) {}
|
|
virtual ~FileMaps() = default;
|
|
|
|
const std::string GetMapsFile() const override { return file_; }
|
|
|
|
protected:
|
|
const std::string file_;
|
|
};
|
|
|
|
class OfflineMaps : public FileMaps {
|
|
public:
|
|
OfflineMaps(const std::string& file) : FileMaps(file) {}
|
|
virtual ~OfflineMaps() = default;
|
|
|
|
bool Parse() override;
|
|
};
|
|
|
|
} // namespace unwindstack
|
|
|
|
#endif // _LIBUNWINDSTACK_MAPS_H
|