[PATCH v2] File Server Remote VSS Protocol server

David Disseldorp ddiss at suse.de
Tue Mar 24 07:19:59 MDT 2015


Thanks for the feedback Volker.

On Tue, 24 Mar 2015 13:37:11 +0100, Volker Lendecke wrote:

> On Mon, Mar 23, 2015 at 08:37:37PM +0100, David Disseldorp wrote:
> > This patch-set adds a new File Server Remote VSS (Volume Shadow Copy
> > Service) Protocol server, test infrastructure, documentation, and
> > corresponding VFS layer changes to handle remote share snapshot
> > requests.
> 
> As discussed privately already: This patchset adds a persistent tdb
> for fss_state using tdb_pack/unpack.  tdb_pack/unpack is really a bad
> interface that I'd like to get rid of eventually. In new code we do
> marshalling of structures with idl/ndr usually. Can you change the code
> to idl before it goes in? This way necessary upgrade code can be avoided?

I agree that tdb_pack/unpack is a pretty ugly interface, and I'm fine
with using idl for [un]marshalling. As mentioned though, I don't think
this should be a show-stopper for merging this feature - conversion to
IDL should be possible without a change to the on-disk format.
If you disagree, then I'll go ahead and make these changes for the next
round, but it'll take a little while to fully test this.

> Also, looking a bit closer at fss_state_store: There's a dbwrap_wipe
> without a transaction. Isn't that racy? Do we have an overview somewhere
> about the data model stored in fss_state?

The database is only manipulated by the single FSRVP server process, so
this shouldn't be an issue. I haven't written an overview of the
fss_state data model, but I'd suggest looking at the local.fss_state
smbtorture test suite for interface details. Otherwise there's the
MS-FSRVP specification.

Cheers, David


More information about the samba-technical mailing list