Re: [PATCH] dax: Fix Xarray conversion of dax_unlock_mapping_entry()
From: Matthew Wilcox
Date: Fri Nov 30 2018 - 11:24:39 EST
On Fri, Nov 30, 2018 at 07:54:49AM -0800, Dan Williams wrote:
> Looks good to me, although can we make that cookie an actual type? I
> think it's mostly ok to pass around (void *) for 'entry' inside of
> fs/dax.c, but once an entry leaves that file I'd like it to have an
> explicit type to catch people that might accidentally pass a (struct
> page *) to the unlock routine.
That's a really good idea. Something like this?
typedef struct {
void *v;
} dax_entry_t;
I could see us making good use of that within dax.c.