diff options
| author | Ciro Nishiguchi <ciro.nishiguchi@ni.com> | 2019-09-11 16:50:11 -0500 |
|---|---|---|
| committer | Martin Braun <martin.braun@ettus.com> | 2019-11-26 11:49:45 -0800 |
| commit | f312d827602fafa21625106dafe2f209e10a22b3 (patch) | |
| tree | fc805bebde263da87da3d7ceeb133014156cba3f /host/lib/rfnoc/chdr_rx_data_xport.cpp | |
| parent | 1a6368331bf441290b0d08ac233c9e5050021493 (diff) | |
| download | uhd-f312d827602fafa21625106dafe2f209e10a22b3.tar.gz uhd-f312d827602fafa21625106dafe2f209e10a22b3.tar.bz2 uhd-f312d827602fafa21625106dafe2f209e10a22b3.zip | |
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.
Diffstat (limited to 'host/lib/rfnoc/chdr_rx_data_xport.cpp')
| -rw-r--r-- | host/lib/rfnoc/chdr_rx_data_xport.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
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 |