[lkml]   [2018]   [Mar]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v5 3/5] bnx2x: Eliminate duplicate barriers on weakly-ordered archs
From: Sinan Kaya <>
Date: Fri, 23 Mar 2018 12:31:12 -0400

> Sorry, you got me confused now.
> If you look at the code closer, you'll see this.
> wmb();
> txdata-> += nbd;
> barrier();
> DOORBELL(bp, txdata->cid, txdata->tx_db.raw);
> and you also asked me to rename DOORBELL to DOORBELL_RELAXED() to make
> it obvious that we have a relaxed operator inside the macro.

This still doesn't match the stated pattern.

/* no other memory or I/O or IOMEM operation */

There is a write to a producer index there and then no non-compiler
barrier or any kind before the writel().

So, in fact, it might really need that implicit writel() barrier here!

 \ /
  Last update: 2018-03-23 17:44    [W:0.060 / U:0.784 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site