[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