From f312d827602fafa21625106dafe2f209e10a22b3 Mon Sep 17 00:00:00 2001 From: Ciro Nishiguchi Date: Wed, 11 Sep 2019 16:50:11 -0500 Subject: rfnoc: Fix transport buffer reservations Change transports to reserve the number of frame buffers they actually need from the I/O service. Previously some I/O service clients reserved 0 buffers since they shared frame buffers with other clients, as we know the two clients do not use the links simultaneously. This is possible with the inline_io_service but not with a multithreaded I/O service which queues buffer for clients before they are requested. --- host/lib/rfnoc/chdr_rx_data_xport.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'host/lib/rfnoc/chdr_rx_data_xport.cpp') diff --git a/host/lib/rfnoc/chdr_rx_data_xport.cpp b/host/lib/rfnoc/chdr_rx_data_xport.cpp index bcd9f7ea9..cdcd70393 100644 --- a/host/lib/rfnoc/chdr_rx_data_xport.cpp +++ b/host/lib/rfnoc/chdr_rx_data_xport.cpp @@ -143,10 +143,10 @@ chdr_rx_data_xport::fc_params_t chdr_rx_data_xport::configure_sep(io_service::sp // Create a temporary recv_io to receive the strc init auto recv_io = io_srv->make_recv_client(recv_link, - /* num_recv_frames*/ 1, + 1, // num_recv_frames recv_cb, send_link, - /* num_send_frames*/ 1, + 1, // num_send_frames fc_cb); // Create a control transport with the rx data links to send mgmt packets @@ -157,8 +157,8 @@ chdr_rx_data_xport::fc_params_t chdr_rx_data_xport::configure_sep(io_service::sp recv_link, pkt_factory, local_epid, - 0, // num_send_frames - 0); // num_recv_frames + 1, // num_send_frames + 1); // num_recv_frames // Setup a route to the EPID // Note that this may be gratuitous--The endpoint may already have been set up -- cgit v1.2.3