[ofw] [PATCH 2/12] ib_types/events: remove trap data from event record

Sean Hefty sean.hefty at intel.com
Mon Jul 21 09:11:05 PDT 2008


The trap data in the ib_event_rec is both incomplete (wrt the types of
traps that it can report) and unused.  Remove it to save stack space.

This will also make it cleaner to have event handlers per QP, SRQ, CQ, etc.

Signed-off-by: Sean Hefty <sean.hefty at intel.com>
---
diff -up -r -X trunk\docs\dontdiff.txt -I '\$Id:' trunk/hw/mlx4/kernel/hca/data.c branches\winverbs/hw/mlx4/kernel/hca/data.c
--- trunk/hw/mlx4/kernel/hca/data.c	2008-06-03 20:28:30.744750000 -0700
+++ branches\winverbs/hw/mlx4/kernel/hca/data.c	2008-06-03 20:29:37.557250000 -0700
@@ -317,7 +317,6 @@ void ca_event_handler(struct ib_event *e
 
 	// prepare parameters
 	event_rec.context = (void *)p_hca->ca_context;
-	event_rec.trap.info.port_num = ev->element.port_num;
 	event_rec.type = ev->event;
 	if (event_rec.type > IB_AE_UNKNOWN) {
 		// CL_ASSERT(0); // This shouldn't happen
diff -up -r -X trunk\docs\dontdiff.txt -I '\$Id:' trunk/hw/mthca/kernel/hca_data.c branches\winverbs/hw/mthca/kernel/hca_data.c
--- trunk/hw/mthca/kernel/hca_data.c	2008-06-03 20:28:30.760375000 -0700
+++ branches\winverbs/hw/mthca/kernel/hca_data.c	2008-06-03 20:29:37.588500000 -0700
@@ -354,7 +354,6 @@ void ca_event_handler(struct ib_event *e
 
 	// prepare parameters
 	event_rec.context = (void *)hob_p->ca_context;
-	event_rec.trap.info.port_num = ev->element.port_num;
 	event_rec.type = ev->event;
 	if (event_rec.type > IB_AE_UNKNOWN) {
 		// CL_ASSERT(0); // This shouldn't happen
diff -up -r -X trunk\docs\dontdiff.txt -I '\$Id:' trunk/inc/iba/ib_types.h branches\winverbs/inc/iba/ib_types.h
--- trunk/inc/iba/ib_types.h	2008-06-03 20:28:30.713500000 -0700
+++ branches\winverbs/inc/iba/ib_types.h	2008-06-03 20:29:37.526000000 -0700
@@ -8973,43 +8973,6 @@ typedef struct _ib_event_rec
 	//HCA vendor specific event information. 
 	uint64_t				vendor_specific;
 
-	/* The following structures are valid only for trap types. */
-	union _trap
-	{
-		struct
-		{
-			uint16_t			lid;
-			ib_net64_t			port_guid;
-			uint8_t				port_num;
-
-			/*
-			 * The following structure is valid only for
-			 * P_KEY, Q_KEY, and M_KEY violation traps.
-			 */
-			struct
-			{
-				uint8_t			sl;
-				uint16_t		src_lid;
-				uint16_t		dest_lid;
-				union _key
-				{
-					uint16_t	pkey;
-					uint32_t	qkey;
-					uint64_t	mkey;
-				} key;
-				uint32_t		src_qp;
-				uint32_t		dest_qp;
-				ib_gid_t		src_gid;
-				ib_gid_t		dest_gid;
-
-			}	violation;
-
-		} info;
-
-		ib_net64_t	sysimg_guid;
-
-	}	trap;
-
 }	ib_event_rec_t;
 /*******/
 





More information about the ofw mailing list