<div dir="ltr">Recheck-request: iol-intel-Performance<br><br>Looks like this patch was affected by an infra failure - putting in a retest request for the CI testing system.</div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Sun, Apr 27, 2025 at 7:28 AM Maayan Kashani <<a href="mailto:mkashani@nvidia.com">mkashani@nvidia.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Queue sync operation was skipped on rule destroy.<br>
Unlike on fw wqe rule create in which both fence and notify_hw<br>
are set to true, on destroy fence was set to false causing<br>
previous queue operation to be stuck in the queue forever.<br>
Example:<br>
rule_a - HW rule, rule_b - FW WQE rule.<br>
Sequence:<br>
rule_a destroy, burst=1 (HW rule put to queue but no DB)<br>
rule_b destroy, burst=0 (FW WQE rule cmd but no queue sync)<br>
Outcome:<br>
rule_a is stuck forever in the queue - no completion.<br>
<br>
Fixes: 338aaf911665 ("net/mlx5/hws: add send FW match STE using gen WQE")<br>
Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
<br>
Signed-off-by: Alex Vesker <<a href="mailto:valex@nvidia.com" target="_blank">valex@nvidia.com</a>><br>
Signed-off-by: Maayan Kashani <<a href="mailto:mkashani@nvidia.com" target="_blank">mkashani@nvidia.com</a>><br>
---<br>
drivers/net/mlx5/hws/mlx5dr_send.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/net/mlx5/hws/mlx5dr_send.c b/drivers/net/mlx5/hws/mlx5dr_send.c<br>
index e121c7f7ed5..d01fc7ef2ca 100644<br>
--- a/drivers/net/mlx5/hws/mlx5dr_send.c<br>
+++ b/drivers/net/mlx5/hws/mlx5dr_send.c<br>
@@ -339,7 +339,7 @@ void mlx5dr_send_stes_fw(struct mlx5dr_send_engine *queue,<br>
pdn = ctx->pd_num;<br>
<br>
/* Writing through FW can't HW fence, therefore we drain the queue */<br>
- if (send_attr->fence)<br>
+ if (send_attr->fence || send_attr->notify_hw)<br>
mlx5dr_send_queue_action(ctx,<br>
queue_id,<br>
MLX5DR_SEND_QUEUE_ACTION_DRAIN_SYNC);<br>
-- <br>
2.21.0<br>
<br>
</blockquote></div>