Patch: Fixed patch for moving RSVD/SHVXD to vfs_default
Jeremy Allison
jra at samba.org
Fri Sep 11 16:12:22 UTC 2015
On Fri, Sep 11, 2015 at 08:12:43AM +0200, Michael Adam wrote:
>
> I fully disagree here!
>
> The point of this patch was, according to the commit message, to
>
> "give VFS module writers a chance to handle RSVD opens
> if they want to"
>
> This is not an urgent bugfix. It does not justify introducing
> new layer violations or aggravating those we already have in
> place. Instead, I think this completely reasonable motivation
> would have been a perfect case for discussing and doing the
> conecptually right changes without haste.
>
> I did not manage to come back to this topic earlier, so it has
> now landed this way. I will try to get the discussion going
> again now... ;-)
OK, so I disagree on this being a layering violation.
VFS modules need to get access to the create context
data, and they're already being passed to VFS_CREATE
(initially for vfs_fruit).
So I still don't get your objection here.
More information about the samba-technical
mailing list