diff options
| author | Leon Romanovsky <leonro@mellanox.com> | 2018-10-11 17:30:05 +0300 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2018-10-16 13:34:01 -0400 |
| commit | dbace111e5b320682eee63d7173959a2b2bd9ccb (patch) | |
| tree | 5f64c5d7ea0fb522b70f30e418f5576491b57109 /drivers/infiniband/core/addr.c | |
| parent | 9549c2bd094f0f54b8827d64886f5b1de370dff3 (diff) | |
| download | lwn-dbace111e5b320682eee63d7173959a2b2bd9ccb.tar.gz lwn-dbace111e5b320682eee63d7173959a2b2bd9ccb.zip | |
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 'drivers/infiniband/core/addr.c')
| -rw-r--r-- | drivers/infiniband/core/addr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/core/addr.c b/drivers/infiniband/core/addr.c index b6f7cde36c2d..0dce94e3c495 100644 --- a/drivers/infiniband/core/addr.c +++ b/drivers/infiniband/core/addr.c @@ -659,7 +659,7 @@ static void process_one_req(struct work_struct *_work) } int rdma_resolve_ip(struct sockaddr *src_addr, const struct sockaddr *dst_addr, - struct rdma_dev_addr *addr, int timeout_ms, + struct rdma_dev_addr *addr, unsigned long timeout_ms, void (*callback)(int status, struct sockaddr *src_addr, struct rdma_dev_addr *addr, void *context), bool resolve_by_gid_attr, void *context) |