diff options
Diffstat (limited to 'host')
-rw-r--r-- | host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.cpp | 11 | ||||
-rw-r--r-- | host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.hpp | 4 |
2 files changed, 1 insertions, 14 deletions
diff --git a/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.cpp b/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.cpp index 625e56eba..578aa9640 100644 --- a/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.cpp +++ b/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.cpp @@ -733,7 +733,6 @@ std::string magnesium_radio_ctrl_impl::get_dboard_fe_from_chan( return std::to_string(chan); } -std::mutex magnesium_radio_ctrl_impl::_set_rpc_lock; // FIXME remove void magnesium_radio_ctrl_impl::set_rpc_client( uhd::rpc_client::sptr rpcc, @@ -779,15 +778,7 @@ void magnesium_radio_ctrl_impl::set_rpc_client( } UHD_LOG_DEBUG(unique_id(), "Master Clock Rate is: " << (_master_clock_rate / 1e6) << " MHz."); - { - // FIXME: Remove this lock. It's masking a bug that's probably, but not - // confirmed, to be in the receive packet demuxer. It'll pop up when - // running UHD over liberio without using serialize_init - std::lock_guard<std::mutex> l(magnesium_radio_ctrl_impl::_set_rpc_lock); - radio_ctrl_impl::set_rate(_master_clock_rate); - // Note: This lock needs to encompass all CHDR traffic. RPC traffic is - // OK from a thread-safety perspective. - } + radio_ctrl_impl::set_rate(_master_clock_rate); // EEPROM paths subject to change FIXME const size_t db_idx = get_block_id().get_block_count(); diff --git a/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.hpp b/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.hpp index 48414cd5b..c6fd1bee4 100644 --- a/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.hpp +++ b/host/lib/usrp/dboard/magnesium/magnesium_radio_ctrl_impl.hpp @@ -148,10 +148,6 @@ public: size_t get_chan_from_dboard_fe(const std::string &fe, const direction_t dir); std::string get_dboard_fe_from_chan(const size_t chan, const direction_t dir); - //! Disable concurrency in set_rpc_client(). This is a workaround for a - // thread-unsafety bug and should get deleted at some point in the future! - static std::mutex _set_rpc_lock; // FIXME remove this - void set_rpc_client( uhd::rpc_client::sptr rpcc, const uhd::device_addr_t &block_args |