-
Notifications
You must be signed in to change notification settings - Fork 36
[wip] Implementing debugger in xeus-cpp(Prototype) #354
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 15 commits
889f32f
2a047c3
3d11208
874fb85
9bdb79c
d7c0df2
346cf9f
609a590
777fa06
10c8c06
ae46678
a6a02dc
be37853
73fd7d5
78e14aa
3c6be4e
90df737
0cba670
eda00f8
290becd
5ab3a13
8c8bafd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,87 @@ | ||||||
/************************************************************************************ | ||||||
* Copyright (c) 2023, xeus-cpp contributors * | ||||||
* Copyright (c) 2023, Johan Mabille, Loic Gouarin, Sylvain Corlay, Wolf Vollprecht * | ||||||
* * | ||||||
* Distributed under the terms of the BSD 3-Clause License. * | ||||||
* * | ||||||
* The full license is in the file LICENSE, distributed with this software. * | ||||||
************************************************************************************/ | ||||||
|
||||||
#ifndef XEUS_CPP_DEBUGGER_HPP | ||||||
#define XEUS_CPP_DEBUGGER_HPP | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
#ifdef __GNUC__ | ||||||
#pragma GCC diagnostic push | ||||||
#pragma GCC diagnostic ignored "-Wattributes" | ||||||
#endif | ||||||
|
||||||
#include <map> | ||||||
#include <mutex> | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
#include <set> | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
#include <string> | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
#include "nlohmann/json.hpp" | ||||||
#include "xeus-zmq/xdebugger_base.hpp" | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
#include "xeus_cpp_config.hpp" | ||||||
|
||||||
namespace xcpp | ||||||
{ | ||||||
class xdebuglldb_client; | ||||||
|
||||||
class XEUS_CPP_API debugger : public xeus::xdebugger_base | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
{ | ||||||
public: | ||||||
|
||||||
using base_type = xeus::xdebugger_base; | ||||||
|
||||||
debugger(xeus::xcontext& context, | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
const xeus::xconfiguration& config, | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
const std::string& user_name, | ||||||
const std::string& session_id, | ||||||
const nl::json& debugger_config); | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
virtual ~debugger(); | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
private: | ||||||
|
||||||
nl::json inspect_variables_request(const nl::json& message); | ||||||
nl::json stack_trace_request(const nl::json& message); | ||||||
nl::json attach_request(const nl::json& message); | ||||||
nl::json configuration_done_request(const nl::json& message); | ||||||
|
||||||
nl::json variables_request_impl(const nl::json& message) override; | ||||||
|
||||||
bool start_lldb(); | ||||||
bool start() override; | ||||||
void stop() override; | ||||||
xeus::xdebugger_info get_debugger_info() const override; | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
std::string get_cell_temporary_file(const std::string& code) const override; | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
bool connect_to_lldb_tcp(); | ||||||
std::string send_dap_message(const nl::json& message); | ||||||
std::string receive_dap_response(); | ||||||
|
||||||
xdebuglldb_client* p_debuglldb_client; | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
std::string m_lldb_host; | ||||||
std::string m_lldb_port; | ||||||
std::string m_lldbdap_port; | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the difference between There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looks like |
||||||
nl::json m_debugger_config; | ||||||
bool m_is_running; | ||||||
int m_tcp_socket; | ||||||
bool m_tcp_connected; | ||||||
pid_t jit_process_pid; | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
}; | ||||||
|
||||||
XEUS_CPP_API | ||||||
std::unique_ptr<xeus::xdebugger> make_cpp_debugger(xeus::xcontext& context, | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
const xeus::xconfiguration& config, | ||||||
const std::string& user_name, | ||||||
const std::string& session_id, | ||||||
const nl::json& debugger_config); | ||||||
} | ||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. warning: namespace 'xcpp' not terminated with a closing comment [llvm-namespace-comment]
Suggested change
Additional contextinclude/xeus-cpp/xdebugger.hpp:27: namespace 'xcpp' starts here namespace xcpp
^ |
||||||
|
||||||
#ifdef __GNUC__ | ||||||
#pragma GCC diagnostic pop | ||||||
#endif | ||||||
|
||||||
#endif |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
{ | ||
"display_name": "C++17-Debugger", | ||
"env": { | ||
"PATH":"@XEUS_CPP_PATH@", | ||
"LD_LIBRARY_PATH":"@XEUS_CPP_LD_LIBRARY_PATH@" | ||
}, | ||
"argv": [ | ||
"@XEUS_CPP_KERNELSPEC_PATH@xcpp", | ||
"-f", | ||
"{connection_file}", | ||
"-gdwarf-4", | ||
"-O0", | ||
"-resource-dir", "@XEUS_CPP_RESOURCE_DIR@", | ||
"-I", "@XEUS_CPP_INCLUDE_DIR@", | ||
"-std=c++17" | ||
], | ||
"language": "cpp", | ||
"metadata": {"debugger": true | ||
} | ||
} |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -19,16 +19,18 @@ | |||||||||||||||||||||||||||||||||||||||||||
#include <unistd.h> | ||||||||||||||||||||||||||||||||||||||||||||
#endif | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
#include "xeus/xhelper.hpp" | ||||||||||||||||||||||||||||||||||||||||||||
#include "nlohmann/json.hpp" | ||||||||||||||||||||||||||||||||||||||||||||
#include <xeus/xhelper.hpp> | ||||||||||||||||||||||||||||||||||||||||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||||||
#include <xeus/xkernel.hpp> | ||||||||||||||||||||||||||||||||||||||||||||
#include <xeus/xkernel_configuration.hpp> | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
#include "xeus-zmq/xzmq_context.hpp" | ||||||||||||||||||||||||||||||||||||||||||||
#include <xeus-zmq/xserver_zmq.hpp> | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
#include "xeus-cpp/xeus_cpp_config.hpp" | ||||||||||||||||||||||||||||||||||||||||||||
#include "xeus-cpp/xinterpreter.hpp" | ||||||||||||||||||||||||||||||||||||||||||||
#include "xeus-cpp/xutils.hpp" | ||||||||||||||||||||||||||||||||||||||||||||
#include "xeus-cpp/xdebugger.hpp" | ||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. warning: #includes are not sorted properly [llvm-include-order]
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. warning: #includes are not sorted properly [llvm-include-order]
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||||||
namespace nl = nlohmann; | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||||||
int main(int argc, char* argv[]) | ||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||
|
@@ -58,6 +60,13 @@ int main(int argc, char* argv[]) | |||||||||||||||||||||||||||||||||||||||||||
#endif | ||||||||||||||||||||||||||||||||||||||||||||
signal(SIGINT, xcpp::stop_handler); | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
#ifdef XEUS_CPP_USE_DEBUGGER | ||||||||||||||||||||||||||||||||||||||||||||
nl::json debugger_config; | ||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. warning: no header providing "nlohmann::json" is directly included [misc-include-cleaner] src/main.cpp:9: - #include <string>
+ #include <nlohmann/json_fwd.hpp>
+ #include <string> |
||||||||||||||||||||||||||||||||||||||||||||
debugger_config["lldb"]["initCommands"] = { | ||||||||||||||||||||||||||||||||||||||||||||
"settings set plugin.jit-loader.gdb.enable on" | ||||||||||||||||||||||||||||||||||||||||||||
}; | ||||||||||||||||||||||||||||||||||||||||||||
#endif | ||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This does not look correct. The use of |
||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
std::string file_name = xeus::extract_filename(argc, argv); | ||||||||||||||||||||||||||||||||||||||||||||
auto interpreter = std::make_unique<xcpp::interpreter>(argc, argv); | ||||||||||||||||||||||||||||||||||||||||||||
std::unique_ptr<xeus::xcontext> context = xeus::make_zmq_context(); | ||||||||||||||||||||||||||||||||||||||||||||
|
@@ -77,13 +86,18 @@ int main(int argc, char* argv[]) | |||||||||||||||||||||||||||||||||||||||||||
xeus::make_console_logger( | ||||||||||||||||||||||||||||||||||||||||||||
xeus::xlogger::msg_type, | ||||||||||||||||||||||||||||||||||||||||||||
xeus::make_file_logger(xeus::xlogger::content, "xeus.log") | ||||||||||||||||||||||||||||||||||||||||||||
#ifdef XEUS_CPP_USE_DEBUGGER | ||||||||||||||||||||||||||||||||||||||||||||
), | ||||||||||||||||||||||||||||||||||||||||||||
xcpp::make_cpp_debugger, | ||||||||||||||||||||||||||||||||||||||||||||
debugger_config | ||||||||||||||||||||||||||||||||||||||||||||
#else | ||||||||||||||||||||||||||||||||||||||||||||
) | ||||||||||||||||||||||||||||||||||||||||||||
#endif | ||||||||||||||||||||||||||||||||||||||||||||
); | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
std::clog << "Starting xcpp kernel...\n\n" | ||||||||||||||||||||||||||||||||||||||||||||
"If you want to connect to this kernel from an other client, you can use" | ||||||||||||||||||||||||||||||||||||||||||||
" the " | ||||||||||||||||||||||||||||||||||||||||||||
+ file_name + " file." | ||||||||||||||||||||||||||||||||||||||||||||
" the " + file_name + " file." | ||||||||||||||||||||||||||||||||||||||||||||
<< std::endl; | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
kernel.start(); | ||||||||||||||||||||||||||||||||||||||||||||
|
@@ -99,7 +113,13 @@ int main(int argc, char* argv[]) | |||||||||||||||||||||||||||||||||||||||||||
xeus::make_console_logger( | ||||||||||||||||||||||||||||||||||||||||||||
xeus::xlogger::msg_type, | ||||||||||||||||||||||||||||||||||||||||||||
xeus::make_file_logger(xeus::xlogger::content, "xeus.log") | ||||||||||||||||||||||||||||||||||||||||||||
#ifdef XEUS_CPP_USE_DEBUGGER | ||||||||||||||||||||||||||||||||||||||||||||
), | ||||||||||||||||||||||||||||||||||||||||||||
xcpp::make_cpp_debugger, | ||||||||||||||||||||||||||||||||||||||||||||
debugger_config | ||||||||||||||||||||||||||||||||||||||||||||
#else | ||||||||||||||||||||||||||||||||||||||||||||
) | ||||||||||||||||||||||||||||||||||||||||||||
#endif | ||||||||||||||||||||||||||||||||||||||||||||
); | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
std::cout << "Getting config" << std::endl; | ||||||||||||||||||||||||||||||||||||||||||||
|
@@ -109,37 +129,20 @@ int main(int argc, char* argv[]) | |||||||||||||||||||||||||||||||||||||||||||
" and paste the following content inside of a `kernel.json` file. And then run for example:\n\n" | ||||||||||||||||||||||||||||||||||||||||||||
"# jupyter console --existing kernel.json\n\n" | ||||||||||||||||||||||||||||||||||||||||||||
"kernel.json\n```\n{\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"transport\": \"" | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_transport | ||||||||||||||||||||||||||||||||||||||||||||
+ "\",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"ip\": \"" | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_ip | ||||||||||||||||||||||||||||||||||||||||||||
+ "\",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"control_port\": " | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_control_port | ||||||||||||||||||||||||||||||||||||||||||||
+ ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"shell_port\": " | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_shell_port | ||||||||||||||||||||||||||||||||||||||||||||
+ ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"stdin_port\": " | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_stdin_port | ||||||||||||||||||||||||||||||||||||||||||||
+ ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"iopub_port\": " | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_iopub_port | ||||||||||||||||||||||||||||||||||||||||||||
+ ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"hb_port\": " | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_hb_port | ||||||||||||||||||||||||||||||||||||||||||||
+ ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"signature_scheme\": \"" | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_signature_scheme | ||||||||||||||||||||||||||||||||||||||||||||
+ "\",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"key\": \"" | ||||||||||||||||||||||||||||||||||||||||||||
+ config.m_key | ||||||||||||||||||||||||||||||||||||||||||||
+ "\"\n" | ||||||||||||||||||||||||||||||||||||||||||||
"}\n```\n"; | ||||||||||||||||||||||||||||||||||||||||||||
" \"transport\": \"" + config.m_transport + "\",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"ip\": \"" + config.m_ip + "\",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"control_port\": " + config.m_control_port + ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"shell_port\": " + config.m_shell_port + ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"stdin_port\": " + config.m_stdin_port + ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"iopub_port\": " + config.m_iopub_port + ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"hb_port\": " + config.m_hb_port + ",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"signature_scheme\": \"" + config.m_signature_scheme + "\",\n" | ||||||||||||||||||||||||||||||||||||||||||||
" \"key\": \"" + config.m_key + "\"\n" | ||||||||||||||||||||||||||||||||||||||||||||
"}\n```" | ||||||||||||||||||||||||||||||||||||||||||||
<< std::endl; | ||||||||||||||||||||||||||||||||||||||||||||
kr-2003 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
kernel.start(); | ||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
return 0; | ||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this is a good idea. I need to know the purpose.