[lkml]   [2015]   [Aug]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] dax, pmem: add support for msync
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?

 \ /
  Last update: 2015-08-31 21:41    [W:0.158 / U:2.788 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site