diff options
| author | Leon Romanovsky <leonro@mellanox.com> | 2018-10-11 17:30:05 +0300 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2018-10-16 20:34:01 +0300 |
| commit | dbace111e5b320682eee63d7173959a2b2bd9ccb (patch) | |
| tree | 5f64c5d7ea0fb522b70f30e418f5576491b57109 /include/rdma/ib_cm.h | |
| parent | 9549c2bd094f0f54b8827d64886f5b1de370dff3 (diff) | |
| download | linux-dbace111e5b320682eee63d7173959a2b2bd9ccb.tar.xz | |
RDMA/core: Annotate timeout as unsigned long
The ucma users supply timeout in u32 format, it means that any number
with most significant bit set will be converted to negative value
by various rdma_*, cma_* and sa_query functions, which treat timeout
as int.
In the lowest level, the timeout is converted back to be unsigned long.
Remove this ambiguous conversion by updating all function signatures to
receive unsigned long.
Reported-by: Noa Osherovich <noaos@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'include/rdma/ib_cm.h')
| -rw-r--r-- | include/rdma/ib_cm.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/rdma/ib_cm.h b/include/rdma/ib_cm.h index c10f4b5ea8ab..49f4f75499b3 100644 --- a/include/rdma/ib_cm.h +++ b/include/rdma/ib_cm.h @@ -583,7 +583,7 @@ struct ib_cm_sidr_req_param { struct sa_path_rec *path; const struct ib_gid_attr *sgid_attr; __be64 service_id; - int timeout_ms; + unsigned long timeout_ms; const void *private_data; u8 private_data_len; u8 max_cm_retries; |