Skip to content

Commit

Permalink
tests: Drop unintended t10dif flags
Browse files Browse the repository at this point in the history
Using both flags simultaneously contradicts the man page, and the tests
do not check their behavior. So we can safely drop the flags.

Fixes: 6375906 ("tests: Add mkey signature tests")
Signed-off-by: Sergey Gorenko <sergeygo@nvidia.com>
Signed-off-by: Edward Srouji <edwards@nvidia.com>
  • Loading branch information
SergeyGorenko authored and EdwardSro committed May 21, 2024
1 parent c4f1885 commit d108cda
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions tests/test_mlx5_mkey.py
Original file line number Diff line number Diff line change
Expand Up @@ -160,9 +160,7 @@ def reg_mr_sig_t10dif(self):
player.qp.wr_set_mkey_access_flags(e.IBV_ACCESS_LOCAL_WRITE)
player.qp.wr_set_mkey_layout_list([sge])

t10dif_flags = (dve.MLX5DV_SIG_T10DIF_FLAG_REF_REMAP |
dve.MLX5DV_SIG_T10DIF_FLAG_APP_ESCAPE |
dve.MLX5DV_SIG_T10DIF_FLAG_APP_REF_ESCAPE)
t10dif_flags = dve.MLX5DV_SIG_T10DIF_FLAG_REF_REMAP
sig_t10dif = Mlx5SigT10Dif(bg_type=dve.MLX5DV_SIG_T10DIF_CRC,
bg=0xFFFF, app_tag=0xABCD,
ref_tag=0x01234567, flags=t10dif_flags)
Expand Down

0 comments on commit d108cda

Please sign in to comment.