Skip to content

Commit 02a325c

Browse files
committed
cleanup: cleanup a bit old sender/response code
virtualization layers is now broken for rr-backend: TODO discuss/adapt for DAP?
1 parent 5e45269 commit 02a325c

File tree

11 files changed

+107
-574
lines changed

11 files changed

+107
-574
lines changed

src/common/ct_event.nim

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,8 @@ type
6161
CtRunToEntry,
6262
CtRunTracepoints,
6363
CtSetupTraceSession,
64+
CtLoadAsmFunction,
65+
CtLoadAsmFunctionResponse,
6466
InternalLastCompleteMove,
6567
InternalAddToScratchpad,
6668
InternalAddToScratchpadFromExpression,

src/db-backend/src/bin/virtualization-layers.rs

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,14 @@ use std::error::Error;
1515
use std::fs;
1616
use std::io::Write;
1717
use std::path::PathBuf;
18-
use std::sync::mpsc;
18+
// use std::sync::mpsc;
1919

2020
extern crate db_backend;
2121
use db_backend::core::Core;
2222
use db_backend::db::Db;
2323
use db_backend::handler::Handler;
24-
use db_backend::receiver::Receiver;
25-
use db_backend::response::Response;
24+
// use db_backend::receiver::Receiver;
25+
// use db_backend::response::Response;
2626

2727
/// virtualization layers for event log/trace/calltrace(maybe others)
2828
#[derive(Parser, Debug)]
@@ -68,18 +68,19 @@ fn main() -> Result<(), Box<dyn Error>> {
6868
.filter(None, log::LevelFilter::Info)
6969
.init();
7070

71-
let mut receiver = Receiver::new();
72-
let (tx, _rx): (mpsc::Sender<Response>, mpsc::Receiver<Response>) = mpsc::channel();
71+
// TODO: DAP
72+
// let mut receiver = Receiver::new();
73+
// let (tx, _rx): (mpsc::Sender<Response>, mpsc::Receiver<Response>) = mpsc::channel();
7374

7475
// let socket_path = cli.socket_path.clone();
7576

7677
// a placeholder, we won't use it
7778
let db = Db::new(&PathBuf::from(""));
78-
receiver.setup_for_virtualization_layers(&cli.socket_path, cli.caller_process_pid)?;
79+
// receiver.setup_for_virtualization_layers(&cli.socket_path, cli.caller_process_pid)?;
7980

80-
let mut handler = Handler::construct(Box::new(db), tx.clone(), true);
81+
let mut _handler = Handler::construct(Box::new(db), true);
8182

82-
receiver.receive_loop(&mut handler)?;
83+
// receiver.receive_loop(&mut handler)?;
8384

8485
Ok(())
8586
}

src/db-backend/src/dap_server.rs

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ use crate::handler::Handler;
77
use crate::task::{
88
gen_task_id, Action, CallSearchArg, CalltraceLoadArgs, CollapseCallsArgs, LoadHistoryArg, LocalStepJump, Location,
99
ProgramEvent, RunTracepointsArg, SourceCallJumpTarget, SourceLocation, StepArg, Task, TaskKind, TracepointId,
10-
UpdateTableArgs,
10+
UpdateTableArgs, FunctionLocation,
1111
};
1212
use crate::trace_processor::{load_trace_data, load_trace_metadata, TraceProcessor};
1313
use log::{error, info, warn};
@@ -18,7 +18,6 @@ use std::fmt;
1818
use std::io::{BufRead, BufReader, Write};
1919
use std::os::unix::net::UnixListener;
2020
use std::path::{Path, PathBuf};
21-
use std::sync::mpsc;
2221
use std::time::Instant;
2322

2423
// fn forward_raw_events<W: Write>(
@@ -67,7 +66,6 @@ fn launch(
6766
trace_folder: &Path,
6867
trace_file: &Path,
6968
seq: i64,
70-
tx: mpsc::Sender<crate::response::Response>,
7169
) -> Result<Handler, Box<dyn Error>> {
7270
info!("run launch() for {:?}", trace_folder);
7371
let trace_file_format = if trace_file.extension() == Some(std::ffi::OsStr::new("json")) {
@@ -96,7 +94,7 @@ fn launch(
9694
info!("postprocessing trace: duration: {:?}", duration2);
9795

9896
// eprintln!("TRACE METADATA: {:?}", meta);
99-
let mut handler = Handler::new(Box::new(db), tx.clone());
97+
let mut handler = Handler::new(Box::new(db));
10098
handler.dap_client.seq = seq;
10199
handler.run_to_entry(dap::Request::default())?;
102100
Ok(handler)
@@ -188,6 +186,7 @@ fn handle_request<W: Write>(
188186
info!("load_calltrace_section");
189187
handler.load_calltrace_section(req.clone(), req.load_args::<CalltraceLoadArgs>()?)?
190188
}
189+
"ct/load-asm-function" => handler.load_asm_function(req.clone(), req.load_args::<FunctionLocation>()?)?,
191190
_ => {
192191
match dap_command_to_step_action(&req.command) {
193192
Ok((action, is_reverse)) => {
@@ -212,7 +211,7 @@ fn handle_request<W: Write>(
212211
fn handle_client<R: BufRead, W: Write>(reader: &mut R, writer: &mut W) -> Result<(), Box<dyn Error>> {
213212
let mut seq = 1i64;
214213
let mut breakpoints: HashMap<String, HashSet<i64>> = HashMap::new();
215-
let (tx, _rx) = mpsc::channel();
214+
// let (tx, _rx) = mpsc::channel();
216215
let mut handler: Option<Handler> = None;
217216
let mut received_launch = false;
218217
let mut launch_trace_folder = PathBuf::from("");
@@ -337,7 +336,7 @@ fn handle_client<R: BufRead, W: Write>(reader: &mut R, writer: &mut W) -> Result
337336
}
338337
info!("stored launch trace folder: {launch_trace_folder:?}");
339338
if received_configuration_done {
340-
handler = Some(launch(&launch_trace_folder, &launch_trace_file, seq, tx.clone())?);
339+
handler = Some(launch(&launch_trace_folder, &launch_trace_file, seq)?);
341340
if let Some(h) = handler.as_mut() {
342341
write_dap_messages(writer, h, &mut seq)?;
343342
}
@@ -380,7 +379,7 @@ fn handle_client<R: BufRead, W: Write>(reader: &mut R, writer: &mut W) -> Result
380379

381380
info!("configuration done sent response; received_launch: {received_launch}");
382381
if received_launch {
383-
handler = Some(launch(&launch_trace_folder, &launch_trace_file, seq, tx.clone())?);
382+
handler = Some(launch(&launch_trace_folder, &launch_trace_file, seq)?);
384383
if let Some(h) = handler.as_mut() {
385384
write_dap_messages(writer, h, &mut seq)?;
386385
}

0 commit comments

Comments
 (0)