diff options
| author | Moshe Lazer <moshel@mellanox.com> | 2013-07-10 07:31:03 -0400 |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2013-07-11 19:45:08 -0400 |
| commit | 288dde9f23b6726c1e8147bf635721372bf77b16 (patch) | |
| tree | 93867e6ea58c82b182c073b1b17e1e4a12b65762 /include/linux/mlx5 | |
| parent | 582c016e68dc5dfea4d3582512157f165a428149 (diff) | |
mlx5_core: Adjust hca_cap.uar_page_sz to conform to Connect-IB spec
Sparse reported an endianness bug in the assignment to hca_cap.uar_page_sz.
Fix the declaration of this field to be __be16 (which is what is in
the firmware spec), renaming the field to log_uar_pg_size to conform
to the spec, which fixes the endianness bug reported by sparse.
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Moshe Lazer <moshel@mellanox.com>
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'include/linux/mlx5')
| -rw-r--r-- | include/linux/mlx5/device.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/mlx5/device.h b/include/linux/mlx5/device.h index 51390915e538..8de8d8f22384 100644 --- a/include/linux/mlx5/device.h +++ b/include/linux/mlx5/device.h | |||
| @@ -317,8 +317,8 @@ struct mlx5_hca_cap { | |||
| 317 | u8 log_max_pd; | 317 | u8 log_max_pd; |
| 318 | u8 rsvd25; | 318 | u8 rsvd25; |
| 319 | u8 log_max_xrcd; | 319 | u8 log_max_xrcd; |
| 320 | u8 rsvd26[40]; | 320 | u8 rsvd26[42]; |
| 321 | __be32 uar_page_sz; | 321 | __be16 log_uar_page_sz; |
| 322 | u8 rsvd27[28]; | 322 | u8 rsvd27[28]; |
| 323 | u8 log_msx_atomic_size_qp; | 323 | u8 log_msx_atomic_size_qp; |
| 324 | u8 rsvd28[2]; | 324 | u8 rsvd28[2]; |