https://mp.weixin.qq.com/s/7h9Bdb0x4_clyigMU_0B7Q
讨论DebugCustomXbar中的几个问题。
1. sources/sourceParams
node.in.unzip解出来输入边中的BI, EI参数,分别是sources和sourceParams:
其中,sourceParams的类型为Seq[DebugCustomParams],其中包含了每一条输入边的地址列表(addrs:List[Int])和数据宽度;
2. decoded
decoded是对sourceParams中的每一个元素做map操作:
而sourceParams中的每一个元素代表了一个输入边的参数,这个输入边的参数中包含了多个地址,每一个地址都与sink.addr进行比较,最后输出一个结果(Bool)。
sourceParams中的所有元素,亦即所有边的比较结果(Bool)集合在一起,成为decoded(Seq[Bool])。也就是说,decoded中的每一个掩码表示了sink.addr与相应输入边中地址参数比对的结果,即sink.addr是否选择输入边连接的source。
3. sourceParams与dSeq
diplomacy node的sourceFn中的dSeq也是Seq[DebugCustomParams]:
这里的dSeq与sourceParams是什么关系?相同吗?
sourceFn是把当前节点即DebugCustomXbar节点看到的上游节点的参数,转换为下游节点看到的DebugCustomXbar节点的参数。
即dSeq是DebugCustomXbar节点看到的上游节点的参数,DebugCustomXbar节点根据这组参数生成lazy module。所以lazy module中的sourceParams,其实就是这sourceFn中的dSeq。
sourceFn把dSeq中全部的地址集合到一起进行了去重操作,然后传递给下游节点(调试模块)。所以这里默认dSeq中不同的source节点可能包含重复的地址。
综合这两点:decoded中可能存在两个或更多个为真的值。从这个结论出发,会导致很多问题。
4. source.addr & source.valid
可能存在多个source.valid为真,进而sink.addr输入到source.addr中被认为合法:
5. sink.ready
可能存在多个d & i.ready为真,进而向sink.ready中输入真:
6. sink.data
即便把i.ready更改为d & i.ready,也可能存在多个i.data输出相或在一起,从而导致sink.data错误:
7. 总结
这里存在两种情况:
a. 存在一个假设,即上游节点的地址不互相重复,那么sourceFn中去重的代码多余;
b. 上游节点的地址可以互相重复,那么decoded需要使用带优先级的掩码(参考仲裁器的实现);
8. 勘误及补充:
sourceFn中有限制各个source节点的地址不重复:require(all_addrs.size == all_addrs.distinct.size, "Different Custom sources can't use the same addresses.")。