> Now, as a general rule, I would call timestamps general.  They should be
> added in a fashion that anyone can implement.  They should also be well
> defined.  Sean's questions raise a very valid point.  Exactly what is
> being timestamped, and do we care about different timestamp options?  Is
> it completion of message, start of message, transfer from HCA to main
> system memory completion, etc.  The 00/10 header to this patch series
> was probably answering Sean's question, but just based on the name of
> the TIMESTAMP flag to the CQ creation attr struct it isn't clear that
> this is the case.

I didn't see the information that I was looking for in the patch header to this 
series.  As Jason pointed out, the use case is lacking.

IMO, it could make just as much sense to associate/enable time stamping with 
the QP as with the CQ, or even make it configurable per operation or operation 
type.

If Christoph has a clear use case and wants to go to the 'bare metal', then a 
vendor specific option seems ideal.  At least until there are other 
implementations or the driving use case is clearer.

- Sean
N�����r��y����b�X��ǧv�^�)޺{.n�+����{��ٚ�{ay�ʇڙ�,j��f���h���z��w���
���j:+v���w�j�m��������zZ+�����ݢj"��!�i

Reply via email to