- 
                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 16 commits
889f32f
              2a047c3
              3d11208
              874fb85
              9bdb79c
              d7c0df2
              346cf9f
              609a590
              777fa06
              10c8c06
              ae46678
              a6a02dc
              be37853
              73fd7d5
              78e14aa
              3c6be4e
              90df737
              0cba670
              eda00f8
              290becd
              5ab3a13
              8c8bafd
              275a9f9
              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.
               
              
                Outdated
          
            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; | ||||||
                
       | 
||||||
| 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 | ||||||||||||||||||||||||||||||||||||||||||||
                
       | 
||||||||||||||||||||||||||||||||||||||||||||
| 
     | 
||||||||||||||||||||||||||||||||||||||||||||
| 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.