Skip to content

Commit

Permalink
tests: Drop unintended t10dif flags
Browse files Browse the repository at this point in the history
[ Upstream commit d108cda ]

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>
Signed-off-by: Nicolas Morey <nmorey@suse.com>
  • Loading branch information
SergeyGorenko authored and nmorey committed May 30, 2024
1 parent ddabfaa commit a366cf9
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 @@ -173,9 +173,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 a366cf9

Please sign in to comment.