On Fri, Nov 6, 2015 at 10:50 PM, Thomas Gleixner <t...@linutronix.de> wrote: > On Fri, 6 Nov 2015, H. Peter Anvin wrote: >> On 11/06/15 15:17, Dan Williams wrote: >> >> >> >> Is it really required to do that on all cpus? >> > >> > I believe it is, but I'll double check. >> > >> >> It's required on all CPUs on which the DAX memory may have been dirtied. >> This is similar to the way we flush TLBs. > > Right. And that's exactly the problem: "may have been dirtied" > > If DAX is used on 50% of the CPUs and the other 50% are plumming away > happily in user space or run low latency RT tasks w/o ever touching > it, then having an unconditional flush on ALL CPUs is just wrong > because you penalize the uninvolved cores with a completely pointless > SMP function call and drain their caches. >
It's not wrong and pointless, it's all we have available outside of having the kernel remember every virtual address that might have been touched since the last fsync and sit in a loop flushing those virtual address cache line by cache line. There is a crossover point where wbinvd is better than a clwb loop that needs to be determined. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/