[ewg] [PATCH] DAPL v2.0: common: srq create asserts with !dapl_llist_is_empty(head) failed

Davis, Arlin R arlin.r.davis at intel.com
Wed Oct 26 14:06:21 PDT 2011


return DAT_NOT_IMPLEMENTED before allocating any resources
until there is a provider that supports SRQ's.

Signed-off-by: Arlin Davis <arlin.r.davis at intel.com>
---
 dapl/common/dapl_srq_create.c |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/dapl/common/dapl_srq_create.c b/dapl/common/dapl_srq_create.c
index 7631a5e..77aeacd 100644
--- a/dapl/common/dapl_srq_create.c
+++ b/dapl/common/dapl_srq_create.c
@@ -114,6 +114,10 @@ dapl_srq_create(IN DAT_IA_HANDLE ia_handle,
 		goto bail;
 	}
 
+	/* SRQ provider not implemented */
+	dat_status = DAT_ERROR(DAT_NOT_IMPLEMENTED, DAT_NO_SUBTYPE);
+	goto bail;
+
 	/* Allocate SRQ */
 	srq_ptr = dapl_srq_alloc(ia_ptr, srq_attr);
 	if (srq_ptr == NULL) {
@@ -129,9 +133,6 @@ dapl_srq_create(IN DAT_IA_HANDLE ia_handle,
 	/*
 	 * XXX Allocate provider resource here!!!
 	 */
-	/* XXX */ dat_status = DAT_ERROR(DAT_NOT_IMPLEMENTED, DAT_NO_SUBTYPE);
-	/* XXX */ dapl_srq_dealloc(srq_ptr);
-	/* XXX */ goto bail;
 
 	/* Link it onto the IA */
 	dapl_ia_link_srq(ia_ptr, srq_ptr);
-- 
1.7.3





More information about the ewg mailing list