On 1/12/2017 9:17 AM, Jerin Jacob wrote: <...> > > @@ -1560,11 +1556,9 @@ bnx2x_reg_read32(struct bnx2x_softc *sc, size_t offset) > #define DPM_TRIGGER_TYPE 0x40 > > /* Doorbell macro */ > -#define BNX2X_DB_WRITE(db_bar, val) \ > - *((volatile uint32_t *)(db_bar)) = (val) > +#define BNX2X_DB_WRITE(db_bar, val) rte_write32_relaxed((val), (db_bar))
What is the rule to use relaxed version or not? I don't know about memory barrier requirements, if it is easy, would you mind explaining? Because I have same question for many different parts of this patchset. Thanks, ferruh