On Mon, 18 Jan 2021 19:20:23 +0800 Huazhong Tan wrote:
On 2021/1/17 10:23, Jakub Kicinski wrote:
On Fri, 15 Jan 2021 15:09:29 +0800 Huazhong Tan wrote:
From: Guangbin Huang huangguangbin2@huawei.com
To increase methods to dump more tm info, adds three debugfs commands to dump tm info of nodes, priority and qset. And a new tm file of debugfs is created for only dumping tm info.
Unlike previous debugfs commands, to dump each tm information, user needs to enter two commands now. The first command writes parameters to tm and the second command reads info from tm. For examples, to dump tm info of priority 0, user needs to enter follow two commands:
- echo dump priority 0 > tm
- cat tm
The reason for adding new tm file is because we accepted Jakub Kicinski's opinion as link https://lkml.org/lkml/2020/9/29/2101. And in order to avoid generating too many files, we implement write ops to allow user to input parameters.
Why are you trying to avoid generating too many files? How many files would it be? What's the size of each dump/file?
The maximum number of tm node, priority and qset are 8, 256, 1280, if we create a file for each one, then there are 8 node files, 256 priority files, 1280 qset files. It seems a little bit hard for using as well.
Would the information not fit in one file per type with multiple rows? Can you show example outputs?
For example if I'm reading right the Qset only has 5 attributes:
"qset id: %u\n" qset_id "QS map pri id: %u\n" map->priority "QS map link_vld: %u\n" map->link_vld); "QS schedule mode: %s\n" (qs_sch_mode->sch_mode & HCLGE_TM_TX_SCHD_DWRR_MSK) ? "dwrr" : "sp"); "QS dwrr: %u\n" qs_weight->dwrr