feat: better error checking

MicroTCA-Tech-Lab · May 27, 2024 · 4b09af9 · 4b09af9
1 parent cbf9da2
commit 4b09af9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/DataHandlerAbstract.cpp b/src/DataHandlerAbstract.cpp
@@ -52,6 +52,9 @@ void DataHandlerAbstract::_handle_input(const boost::system::error_code& ec) {
         return;
     }
 
+    // Check for error flags and log them
+    _dma.check_for_errors();
+
     uint32_t irq_count = _dma.clear_interrupt();
     BOOST_LOG_SEV(_lg, bls::trace) << "irq count = " << irq_count;
 

diff --git a/src/UioMemSgdma.cpp b/src/UioMemSgdma.cpp
@@ -104,8 +104,18 @@ std::vector<UioRegion> UioMemSgdma::get_full_buffers() {
         desc.status.cmpit = 0;
         descriptors[_next_readable_buf].wr(desc);
 
+        if (desc.status.buffer_len != desc.control.buffer_len || desc.status.buffer_len == 0) {
+            BOOST_LOG_SEV(_lg, bls::error)
+                << "Descriptor #" << i << " size mismatch (expected " << desc.control.buffer_len
+                << ", received " << desc.status.buffer_len << "), skipping";
+
+            print_desc(desc);
+            continue;
+        }
+
         bufs.emplace_back(
             UioRegion{static_cast<uintptr_t>(desc.buffer_addr), desc.status.buffer_len});
+
         BOOST_LOG_SEV(_lg, bls::trace)
             << "save buf #" << _next_readable_buf << " @ 0x" << std::hex << desc.buffer_addr;