On Mar 11, 2010, at 19:59 , Roland Dreier wrote:
> I think we can worry about that if/when an HCA comes along that supports
> global atomics for ordinary atomics but not enhanced atomics.

With the proposed patches in place, how do you know if masked atomics are 
implemented or not? Guess apps need to know this information already on todays 
HCAs.

> Although perhaps it would be cleaner to change the atomic_cap enum to:
> 
>       /*
>        * IB_ATOMIC_NONE:      no atomic capability
>        * IB_ATOMIC_HCA:       all ops are atomic within HCA

But IB_ATOMIC_HCA  does not tell you if the masked ones are supported or not.

>        * IB_ATOMIC_GLOB:      standard ops atomic with respect to all
>                               memory ops; masked ops atomic within HCA

What if an HCA supports standards ops with respect to all memory ops, but the 
HCA does not support masked atomics?

Hence, I think it would be cleaner if a new capability, masked_atomic_cap, were 
introduced, using the original definitions (NONE, HCA, GLOB).


Thanks, Håkon


--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to