> Can you share what the libfabric change would look like to add a
> timestamp field to the completion? That might be informative,
> libfabric is also consuming the kernel UAPI.

Libfabric uses a CQ format option. so this would just require a new format.

There is no type checking on CQ poll (read).  The provider interface looks like 
this:

struct fi_ops_cq {
        size_t  size;
        ssize_t (*read)(struct fid_cq *cq, void *buf, size_t count);
        ssize_t (*readfrom)(struct fid_cq *cq, void *buf, size_t count,
                        fi_addr_t *src_addr);

(We could construct static inline calls with type checking if someone wanted 
that.)

If I were adding timestamps, I would probably define a new completion structure 
with 2 u64 time stamp fields (start and end times), and figure out when start 
occurred, end occurred, and the timing metric later.  :)

I would assume that these are non-wrapping values.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to