issue链接
评审背景
背景介绍：当前使用串口名称判断使用的串口是否相同，但是串口名称不具有唯一性，所以使用具有唯一性的SrcSerial和DestSerial作为判断条件。
前置评审
https://discuss.openubmc.cn/t/topic/1170/2
评审点
1、当前SerialRecordConnect的配置
"SerialRecordConnect": {
"usage": [
"CSR"
],
"baseType": "Array",
"items": {
"baseType": "Struct",
"$ref": "types.json#/defs/Connection"
}
}
// Connetction内容
"Connection": {
"Source": {
"baseType": "String"
},
"Destination": {
"baseType": "String"
}
}
- 修改后的配置
// Connection增加SrcSerial和 DestSerial
"Connection": {
"Source": {
"baseType": "String"
},
"Destination": {
"baseType": "String"
},
"SrcSerial": {
"baseType": "U8"
},
"DestSerial": {
"baseType": "U8"
}
},
兼容性适配
- 新BMC搭配新CSR：问题解决
- 新BMC搭配老CSR：新BMC会内置新CSR，问题解决
- 老BMC搭配老CSR：问题未解决，符合预期
- 老BMC搭配新CSR：新CSR中Source和Destination字段还在，虽然问题未解决，但不会引入问题，符合预期
详细说明
资源对象：DPUCard
资源interface：私有属性不涉及
变化类型：新增私有属性
应用场景：优化极端场景下SDI/DPU卡串口稳定性
持久化类型： 不持久化
操作权限：NA
| 属性名称 | 变化类型 | 签名 | 读写&权限 | 持久化 | 变化通知 | 接口说明 | 接口约束 |
|---|---|---|---|---|---|---|---|
| SrcSerial | 新增属性 | y | NA | NA | NA | 源Uart或Port口，其中取值大于16是Port，否则是Uart | 无 |
| DestSerial | 新增属性 | y | NA | NA | NA | 目的Uart或Port口，其中取值大于16是Port，否则是Uart | 无 |
评审结论
通过
- 同意在私有属性SerialRecordConnect结构体中新增子属性SrcSerial，签名为y
- 同意在私有属性SerialRecordConnect结构体中新增子属性DestSerial，签名为y