From 6891b5fcfd29ea834b7969808487597021caf044 Mon Sep 17 00:00:00 2001 From: whitequark Date: Fri, 27 Jan 2017 13:11:38 +0000 Subject: [PATCH] firmware: accept sessions even when startup kernel is running. Fixes #625. Fixes #649. --- artiq/firmware/runtime/session.rs | 38 ++++++++++++++++++------------- 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/artiq/firmware/runtime/session.rs b/artiq/firmware/runtime/session.rs index ce8316543..0a7f57d31 100644 --- a/artiq/firmware/runtime/session.rs +++ b/artiq/firmware/runtime/session.rs @@ -639,27 +639,33 @@ fn respawn(io: &Io, handle: &mut Option, f: F) } pub fn thread(io: Io) { - let congress = Urc::new(RefCell::new(Congress::new())); - - info!("running startup kernel"); - match flash_kernel_worker(&io, &mut *borrow_mut!(congress), "startup_kernel") { - Ok(()) => info!("startup kernel finished"), - Err(err) => { - if err.kind() == io::ErrorKind::NotFound { - info!("no startup kernel found") - } else { - error!("startup kernel aborted: {}", err); - } - } - } - - BufferLogger::with_instance(|logger| logger.disable_trace_to_uart()); - let listener = TcpListener::new(&io, 65535); listener.listen(1381).expect("session: cannot listen"); info!("accepting network sessions"); + BufferLogger::with_instance(|logger| logger.disable_trace_to_uart()); + + let congress = Urc::new(RefCell::new(Congress::new())); + let mut kernel_thread = None; + { + let congress = congress.clone(); + respawn(&io, &mut kernel_thread, move |io| { + let mut congress = borrow_mut!(congress); + info!("running startup kernel"); + match flash_kernel_worker(&io, &mut congress, "startup_kernel") { + Ok(()) => info!("startup kernel finished"), + Err(err) => { + if err.kind() == io::ErrorKind::NotFound { + info!("no startup kernel found") + } else { + error!("startup kernel aborted: {}", err); + } + } + } + }) + } + loop { if listener.can_accept() { let mut stream = listener.accept().expect("session: cannot accept");