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

Sean Hefty sean.hefty at intel.com
Tue Apr 29 12:57:35 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>
---
This breaks the ABI, so it would be ideal to include it with other ABI
breaking changes (like WDK support).

Index: hw/mthca/kernel/hca_data.c
===================================================================
--- hw/mthca/kernel/hca_data.c	(revision 1006)
+++ hw/mthca/kernel/hca_data.c	(working copy)
@@ -352,7 +334,6 @@
 
 	// 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
@@ -841,7 +822,7 @@
 			&ah_attr_p->grh.traffic_class, &ah_attr_p->grh.flow_label );
 		err = ib_find_cached_gid((struct ib_device *)ib_dev_p, 
 			(union ib_gid	*)ibal_av_p->grh.src_gid.raw, &port_num, &gid_index);
-		if (err) {
+		if (err) {
 
 			HCA_PRINT(TRACE_LEVEL_ERROR ,HCA_DBG_SHIM ,("ib_find_cached_gid failed %d (%#x). Using default:
sgid_index = 0\n", err, err));
 			gid_index = 0;
Index: inc/iba/ib_types.h
===================================================================
--- inc/iba/ib_types.h	(revision 1006)
+++ inc/iba/ib_types.h	(working copy)
@@ -8943,43 +8943,6 @@
 	/* 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