Loading ...
Sorry, an error occurred while loading the content.

Re: [PATCH] DAT 2.0 change to include extended event data with all builds.

Expand Messages
  • James Lentini
    This makes to me.
    Message 1 of 2 , Oct 17, 2007
    View Source
    • 0 Attachment
      This makes to me.

      On Tue, 16 Oct 2007, Arlin Davis wrote:

      > 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@...>
      >
      > --- 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 */
      >
    • James Lentini
      ... ^ sense :)
      Message 2 of 2 , Oct 17, 2007
      View Source
      • 0 Attachment
        On Wed, 17 Oct 2007, James Lentini wrote:

        >
        > This makes to me.
        ^
        sense :)

        > On Tue, 16 Oct 2007, Arlin Davis wrote:
        >
        > > 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@...>
        > >
        > > --- 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 */
        > >
        > _______________________________________________
        > general mailing list
        > general@...
        > http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
        >
        > To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
        >
      Your message has been successfully submitted and would be delivered to recipients shortly.