[ofa-general] [PATCH] DAT 2.0 change to include extended event data with all builds.

Arlin Davis ardavis at ichips.intel.com
Tue Oct 16 15:35:53 PDT 2007


James,

Can you comment on the following patch to DAT v2.0. Do you see any 
issues? If ok, I would like to get this accepted and rolled back into 
the specification.

Thanks,

-arlin


--
Modify dat.h dat_event to include event_extension_data[8].
Extend struct dat_event outside of extension build
switch to enable non-extended applications to work
with extended libraries. Otherwise, there is a potential
for the event callee to write back too much event data
and exceed callers non-extended event buffer.
--

Signed-off by: Arlin Davis <ardavis at ichips.intel.com>

--- a/dat/include/dat/dat.h
+++ b/dat/include/dat/dat.h
@@ -944,9 +944,7 @@ typedef struct dat_event
      DAT_EVENT_NUMBER            event_number;
      DAT_EVD_HANDLE              evd_handle;
      DAT_EVENT_DATA              event_data;
-#ifdef DAT_EXTENSIONS
      DAT_UINT64                  event_extension_data[8];
-#endif /* DAT_EXTENSIONS */
  } DAT_EVENT;

  /* Provider/registration info */



More information about the general mailing list