The code was attempting to pass int* instead of uint32_t* to umad_register2, which is not OK. Signed-off-by: Jason Gunthorpe <jgunthorpe@xxxxxxxxxxxxxxxxxxxx> --- libibumad/tests/umad_reg2_compat.c | 2 +- libibumad/tests/umad_register2.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libibumad/tests/umad_reg2_compat.c b/libibumad/tests/umad_reg2_compat.c index 413a3bec6d58..6dd4a48a59b2 100644 --- a/libibumad/tests/umad_reg2_compat.c +++ b/libibumad/tests/umad_reg2_compat.c @@ -154,7 +154,7 @@ void test_fall_back(void) { int rc = 0; struct umad_reg_attr reg_attr; - int agent_id; + uint32_t agent_id; int fd; fd = open_test_device(); diff --git a/libibumad/tests/umad_register2.c b/libibumad/tests/umad_register2.c index c2d7846038bd..ed7e816bb63c 100644 --- a/libibumad/tests/umad_register2.c +++ b/libibumad/tests/umad_register2.c @@ -92,8 +92,8 @@ void test_fail(void) { int rc = 0; struct umad_reg_attr reg_attr; - int agent_id; - int agent_id2; + uint32_t agent_id; + uint32_t agent_id2; int fd; printf("\n *****\nBegin invalid tests\n"); @@ -200,7 +200,7 @@ void test_oui(void) { int rc = 0; struct umad_reg_attr reg_attr; - int agent_id; + uint32_t agent_id; int fd; printf("\n *****\nStart valid oui tests\n"); -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html