On 02/10/20 18:43, Matthew Rosato wrote:
>> diff --git
>> a/include/standard-headers/drivers/infiniband/hw/vmw_pvrdma/pvrdma_ring.h
>> b/include/standard-headers/drivers/infiniband/hw/vmw_pvrdma/pvrdma_ring.h
>> index acd4c8346d..7b4062a1a1 100644
>> ---
>> a/include/standard-headers/drivers/infiniband/hw/vmw_pvrdma/pvrdma_ring.h
>> +++
>> b/include/standard-headers/drivers/infiniband/hw/vmw_pvrdma/pvrdma_ring.h
>> @@ -68,7 +68,7 @@ static inline int pvrdma_idx_valid(uint32_t idx,
>> uint32_t max_elems)
>> static inline int32_t pvrdma_idx(int *var, uint32_t max_elems)
>> {
>> - const unsigned int idx = atomic_read(var);
>> + const unsigned int idx = qatomic_read(var);
>> if (pvrdma_idx_valid(idx, max_elems))
>> return idx & (max_elems - 1);
>> @@ -77,17 +77,17 @@ static inline int32_t pvrdma_idx(int *var,
>> uint32_t max_elems)
>> static inline void pvrdma_idx_ring_inc(int *var, uint32_t max_elems)
>> {
>> - uint32_t idx = atomic_read(var) + 1; /* Increment. */
>> + uint32_t idx = qatomic_read(var) + 1; /* Increment. */
>> idx &= (max_elems << 1) - 1; /* Modulo size, flip gen. */
>> - atomic_set(var, idx);
>> + qatomic_set(var, idx);
>> }
>> static inline int32_t pvrdma_idx_ring_has_space(const struct
>> pvrdma_ring *r,
>> uint32_t max_elems, uint32_t *out_tail)
>> {
>> - const uint32_t tail = atomic_read(&r->prod_tail);
>> - const uint32_t head = atomic_read(&r->cons_head);
>> + const uint32_t tail = qatomic_read(&r->prod_tail);
>> + const uint32_t head = qatomic_read(&r->cons_head);
>> if (pvrdma_idx_valid(tail, max_elems) &&
>> pvrdma_idx_valid(head, max_elems)) {
>> @@ -100,8 +100,8 @@ static inline int32_t
>> pvrdma_idx_ring_has_space(const struct pvrdma_ring *r,
>> static inline int32_t pvrdma_idx_ring_has_data(const struct
>> pvrdma_ring *r,
>> uint32_t max_elems, uint32_t *out_head)
>> {
>> - const uint32_t tail = atomic_read(&r->prod_tail);
>> - const uint32_t head = atomic_read(&r->cons_head);
>> + const uint32_t tail = qatomic_read(&r->prod_tail);
>> + const uint32_t head = qatomic_read(&r->cons_head);
>> if (pvrdma_idx_valid(tail, max_elems) &&
>> pvrdma_idx_valid(head, max_elems)) {
>
>
> It looks like the changes in this file are going to get reverted the
> next time someone does a linux header sync.
Source code should not be at all imported from Linux. The hacks that
accomodate pvrdma in update-linux-headers.sh (like s/atomic_t/u32/)
really have no place there; the files in
include/standard-headers/drivers/infiniband/hw/vmw_pvrdma should all be
moved in hw/.
Paolo
