Hi,

I apologize for the late reply.

On Thu, Aug 20, 2026 at 04:48:08PM +0000, Banoth Saikumar wrote:
> Oracle Confidential
> 
> Hi Dariusz,
> 
> I noticed that this patch was later reverted by:
> 
> > net/mlx5: revert fallback to Verbs Tx allocation
> 
> The revert mentions that on Windows, DevX CQ creation was failing due to
> a different allocation fallback behavior.
> 
> Could you please clarify what specific behavior needs to be addressed
> for Windows?

The problem is that mlx5_devx_obj_ops_en() checks too much
for both platforms. Specifically, it returns true if:

- DevX is available (sh->cdev->config.devx) AND
- At least Direct Verbs steering is enabled (sh->config.dv_flow_en) AND
- mlx5dv_dr TIR flow action is available (sh->dev_cap.dest_tir;
  RSS action can be configured over Direct Rules with rdma-core).

On Windows, the last condition is not satisfied because
a different mechanism is used there to create flow rules.

mlx5_devx_obj_ops_en() function and its callers should be refactored
to only check the DevX state as the name suggests.
After this is done, the fix can reapplied.

We'll provide the patches which rework the above,
so that original fix can be reapplied, in this release cycle.

> 
> Should the original fallback be reworked to handle the Windows case,
> or should the fallback be avoided entirely when running on Windows?
> 
> I would like to understand the expected behavior so I can work on a
> correct fix.
> 
> Thanks,
> Banoth Saikumar

[snip]

Best regards,
Dariusz Sosnowski

Reply via email to