Re: [PATCH 14/33] vfs: Implement a filesystem superblock creation/configuration context [ver #11]
From: Sergey Senozhatsky
Date: Tue Sep 18 2018 - 21:26:33 EST
On (09/19/18 10:12), Sergey Senozhatsky wrote:
> On (09/18/18 07:06), Guenter Roeck wrote:
> > > So the check either better be
> > >
> > > if (fc->ops && fc->ops->reconfigure)
> > >
> >
> > Since there are multiple instances of fs_context where fc->ops isn't set,
> > this check would be needed wherever fc->ops is dereferenced.
>
> Right. If fc is always guaranteed to be properly zeroed-out. This is
> true for kzalloc-ed fc's, but not necessarily so in any other case.
What I mean was something like this
void foo(void)
{
struct fs_context fc;
fc.purpose = ...;
fc.fs_type = ...;
fc.root = ...;
fc.sb_flags = ...;
reconfigure_super(&fc);
}
-ss