On Mon, Jun 02, 2014 at 11:52:47AM -0500, Steve Wise wrote: > > > You're correct. And this bug appears to be in the current upstream code > > > as well. If > an > > > IB_WR_LOCAL_INV wr is used, it must include IB_SEND_FENCE to fence it > > > until the prior > > read > > > completes. > > > > > > Good catch! I'll post V4 soon. > > > > Any chance that can be handled as a separate patch rather than folded > > in? > > > > (Disclaimer: I've been following the discussion only very > > superficially.) > > > > Sure. I'll post the patch soon.
Thanks, and, again, I'm not terribly happy about the monster patch--anything you can split off it is great, even if that thing's small. As long as all the intermediate stages still build and run. (And any bugs you've identified in upstream code are good candidates for separate patches, hopefully preceding the rewrite. That also allows us to apply those fixes to stable kernels if appropriate.) --b. -- 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
