On Mon, Aug 31, 2015 at 09:06:19PM +0200, Christoph Hellwig wrote: > On Mon, Aug 31, 2015 at 12:59:44PM -0600, Ross Zwisler wrote: > > For DAX msync we just need to flush the given range using > > wb_cache_pmem(), which is now a public part of the PMEM API. > > > > The inclusion of <linux/dax.h> in fs/dax.c was done to make checkpatch > > happy. Previously it was complaining about a bunch of undeclared > > functions that could be made static. > > Should this be abstracted by adding a ->msync method? Maybe not > worth to do for now, but it might be worth to keep that in mind. Where would we add the ->msync method? Do you mean to the PMEM API, or somewhere else? -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>