From 91ef8e6336063c962190801ed0c47c4b48b52b18 Mon Sep 17 00:00:00 2001 From: ftillier Date: Thu, 22 Sep 2005 04:42:47 +0000 Subject: [PATCH] [HCA] query MR must return RKey in network byte order. [ALTS] Check that RKey and LKey for MR query match those returned by MR create. Signed-off-by: Yossi Leybovich (sleybo@mellanox.co.il) git-svn-id: svn://openib.tc.cornell.edu/gen1@79 ad392aa1-c5ef-ae45-8dd8-e69d62a5ef86 --- trunk/hw/mt23108/kernel/hca_data.c | 2 +- trunk/tests/alts/registermemregion.c | 20 ++++++++++++++++++++ trunk/tests/alts/user/alts_main.c | 4 +--- 3 files changed, 22 insertions(+), 4 deletions(-) diff --git a/trunk/hw/mt23108/kernel/hca_data.c b/trunk/hw/mt23108/kernel/hca_data.c index 6d1075ce..313c3202 100644 --- a/trunk/hw/mt23108/kernel/hca_data.c +++ b/trunk/hw/mt23108/kernel/hca_data.c @@ -1449,7 +1449,7 @@ mlnx_conv_vapi_mr_attr( mr_query_p->access_ctrl = map_vapi_acl(mr_info_p->acl); mr_query_p->lkey = mr_info_p->lkey; - mr_query_p->rkey = mr_info_p->rkey; + mr_query_p->rkey = cl_hton32(mr_info_p->rkey); } ////////////////////////////////////////////////////////////// diff --git a/trunk/tests/alts/registermemregion.c b/trunk/tests/alts/registermemregion.c index dee9813b..ca3ff6e7 100644 --- a/trunk/tests/alts/registermemregion.c +++ b/trunk/tests/alts/registermemregion.c @@ -147,6 +147,24 @@ al_test_register_mem( break; } + if(alts_mr_attr.lkey != lkey || alts_mr_attr.rkey != rkey) + { + + ALTS_PRINT( ALTS_DBG_ERROR, + ("ib_query_mr failed lkey rkey different from reg\n")); + ALTS_PRINT( ALTS_DBG_ERROR, + ("\t\t reg-lkey = %x query-lkey %x reg-rkey%x query-rkey%x\n" , + alts_mr_attr.lkey , lkey , alts_mr_attr.rkey , rkey)); + alts_close_ca(h_ca); + ib_status = IB_INVALID_LKEY; + break; + + } + + ALTS_PRINT( ALTS_DBG_ERROR, + ("ib_query_mr passed\n" + "\t\t lkey = %x rkey%x query-rkey%x\n" , + lkey, rkey)); /* * Re-register the memeory region */ @@ -158,6 +176,8 @@ al_test_register_mem( { ALTS_PRINT( ALTS_DBG_ERROR, ("ib_rereg_mem failed status = %s\n", ib_get_err_str(ib_status)) ); + alts_close_ca(h_ca); + break; } ALTS_PRINT( ALTS_DBG_ERROR, diff --git a/trunk/tests/alts/user/alts_main.c b/trunk/tests/alts/user/alts_main.c index ef97cc39..d7dd1fdf 100644 --- a/trunk/tests/alts/user/alts_main.c +++ b/trunk/tests/alts/user/alts_main.c @@ -212,9 +212,7 @@ ib_api_status_t ib_status = IB_ERROR; ib_status = al_test_create_mem_window(); break; case RegisterSharedMemRegion: - CL_PRINT( ALTS_DBG_VERBOSE, alts_dbg_lvl, - ("altsapp: RegisterSharedMemRegion not implemented.\n") ); - ib_status = IB_SUCCESS; + ib_status = al_test_register_shared_mem(); break; case MultiSend: ib_status = al_test_multi_send_recv(); -- 2.41.0