diff options
author | Martin Braun <martin.braun@ettus.com> | 2021-12-02 17:12:34 +0100 |
---|---|---|
committer | Aaron Rossetto <aaron.rossetto@ni.com> | 2021-12-09 13:10:00 -0800 |
commit | f308a453c4e615143a467869accf550ab36493c7 (patch) | |
tree | de873cd23da6a641263a9b478f4cb8269c3ca411 /host/tests/actions_test.cpp | |
parent | 4fd1cb317fbbce6facac1cc1bce65e46aa72dcc2 (diff) | |
download | uhd-f308a453c4e615143a467869accf550ab36493c7.tar.gz uhd-f308a453c4e615143a467869accf550ab36493c7.tar.bz2 uhd-f308a453c4e615143a467869accf550ab36493c7.zip |
rfnoc: Fix back-edge consistency check
On back-edges, no properties are forwarded, but properties must be
consistent after property resolution. This breaks when the source edge
on a back-edge has an edge property which the destination block does
not. Consider the following graph:
DDC -> Replay -> DDC
where both instances of 'DDC' refer to the same block. Now, assume the
first edge is declared a back edge (in principle, it shouldn't matter).
The DDC block has an edge property `samp_rate` which the Replay block
does not. Therefore, it can't forward this edge property to the Replay
block's input edge property list.
In the consistency check code, we don't check for the existence of edge
nodes, because it is assumed edge properties where either forwarded, or
aligned through some other manner. This leads to a property lookup
failure.
With this fix, we skip the consistency check for edge properties which
don't exist on the destination node. This is safe because the
destination block can not have a property resolver defined for undefined
properties. This means the destination block can either:
- Drop the property. In this case, there is no value in checking
consistency. Even if we could forward edge properties on back-edges,
they would always have the same value.
- Forward the property. In that case, the consistency check would happen
elsewhere in the graph where there's no back-edge.
Diffstat (limited to 'host/tests/actions_test.cpp')
0 files changed, 0 insertions, 0 deletions