Skip to content

Commit c44915d

Browse files
Muhammad SammarmuhamadSammar
authored andcommitted
mlx5: Fix definer id setting in rule member object
Definer id is taken from last matcher builder, but we missed checking if builder and rule member type are the same.
1 parent 8f944e2 commit c44915d

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

mlx_steering_dump_parser.py

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -179,8 +179,15 @@ def parse_domain(csv_reader, domain_obj=None):
179179
DR_DUMP_REC_TYPE_RULE_RX_ENTRY_V1,
180180
DR_DUMP_REC_TYPE_RULE_TX_ENTRY_V1]:
181181

182-
dr_obj.data["definer_id"] = \
183-
dump_ctx.matcher.builders[-1].data["definer_id"]
182+
definer_id = '-1'
183+
for builder in dump_ctx.matcher.builders[::-1]:
184+
if ((int(builder.data["is_rx"]) and dr_rec_type in [DR_DUMP_REC_TYPE_RULE_RX_ENTRY_V0, \
185+
DR_DUMP_REC_TYPE_RULE_RX_ENTRY_V1]) or \
186+
((not int(builder.data["is_rx"]) and dr_rec_type in [DR_DUMP_REC_TYPE_RULE_TX_ENTRY_V0, \
187+
DR_DUMP_REC_TYPE_RULE_TX_ENTRY_V1]))):
188+
definer_id = builder.data["definer_id"]
189+
190+
dr_obj.data["definer_id"] = definer_id
184191
dump_ctx.rule.add_rule_entry(dr_obj)
185192

186193
# update Action objects

0 commit comments

Comments
 (0)