Skip to content

Commit

Permalink
rfnoc: tx streamer: Remove warning regarding alignment with CHDR width
Browse files Browse the repository at this point in the history
This warning turned out to be scientifically inaccurate. The actual
auto-coercion of the spp value to the CHDR width is left in place, as it
improves streaming under certain conditions, but we no longer warn if
a user-selected spp value is not aligned with the CHDR width.
  • Loading branch information
mbr0wn authored and joergho committed Jan 14, 2025
1 parent ee4c15c commit 544997d
Showing 1 changed file with 2 additions and 10 deletions.
12 changes: 2 additions & 10 deletions host/lib/rfnoc/rfnoc_tx_streamer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -176,23 +176,15 @@ void rfnoc_tx_streamer::connect_channel(
const size_t misalignment = spp % chdr_w_items;
if (!_stream_args.args.has_key("spp")) {
// By default, find an spp value that is an integer multiple of the CHDR
// width. This can improve streaming performance under some conditions.
// width. This has proven useful under some corner conditions, but still
// requires a root-cause analysis.
if (misalignment != 0) {
RFNOC_LOG_DEBUG("Reducing spp from "
<< spp << " to " << spp - misalignment
<< " to align with CHDR width of " << chdr_w_bytes
<< " bytes (= " << chdr_w_items << " samples).");
tx_streamer_impl<chdr_tx_data_xport>::set_max_num_samps(spp - misalignment);
}
} else {
// If the user has provided a custom spp value, check if it is a multiple
// of the CHDR width. Generate a warning otherwise.
if (misalignment != 0) {
RFNOC_LOG_WARNING("Samples per packet (spp="
<< spp << ") is not a multiple of CHDR width which is "
<< chdr_w_bytes << " bytes (= " << chdr_w_items
<< " samples). This may cause performance issues.");
}
}

// Update MTU property based on xport limits. We need to do this after
Expand Down

0 comments on commit 544997d

Please sign in to comment.