r20891: enable multiple brlock backends. The tdb backend is the default. The

prototype ctdb backend is in cluster/ctdb/brlock_ctdb.c
(This used to be commit 84d0e5316299931dc26f2a7b86962d2fffcc4b71)

diff --git a/source4/ntvfs/common/brlock.h b/source4/ntvfs/common/brlock.h
new file mode 100644 (file)
index 0000000..a462254
--- /dev/null
+++ b/source4/ntvfs/common/brlock.h
@@ -0,0 +1,53 @@
+/* 
+   Unix SMB/CIFS implementation.
+
+   generic byte range locking code - common include
+
+   Copyright (C) Andrew Tridgell 2006
+   
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+   
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+   
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+*/
+
+struct brlock_ops {
+       struct brl_context *(*brl_init)(TALLOC_CTX *, struct server_id , 
+                                       struct messaging_context *);
+       struct brl_handle *(*brl_create_handle)(TALLOC_CTX *, struct ntvfs_handle *, DATA_BLOB *);
+       NTSTATUS (*brl_lock)(struct brl_context *,
+                            struct brl_handle *,
+                            uint16_t ,
+                            uint64_t , uint64_t , 
+                            enum brl_type ,
+                            void *);
+       NTSTATUS (*brl_unlock)(struct brl_context *,
+                              struct brl_handle *, 
+                              uint16_t ,
+                              uint64_t , uint64_t );
+       NTSTATUS (*brl_remove_pending)(struct brl_context *,
+                                      struct brl_handle *, 
+                                      void *);
+       NTSTATUS (*brl_locktest)(struct brl_context *,
+                                struct brl_handle *,
+                                uint16_t , 
+                                uint64_t , uint64_t , 
+                                enum brl_type );
+       NTSTATUS (*brl_close)(struct brl_context *,
+                             struct brl_handle *);
+};
+
+
+void brl_set_ops(const struct brlock_ops *new_ops);
+void brl_tdb_init_ops(void);
+void brl_ctdb_init_ops(void);
+

diff --git a/source4/ntvfs/common/brlock.c b/source4/ntvfs/common/brlock_tdb.c
similarity index 84%
rename from source4/ntvfs/common/brlock.c
rename to source4/ntvfs/common/brlock_tdb.c
index c42c2f3a118204bcac75a9aeddfe53a41dfaa020..34c0b7e979f9047ab404c180234c68ae46c0d5d2 100644 (file)
--- a/source4/ntvfs/common/brlock.c
+++ b/source4/ntvfs/common/brlock_tdb.c
@@ -1,9 +1,9 @@
 /* 
    Unix SMB/CIFS implementation.
 
-   generic byte range locking code
+   generic byte range locking code - tdb backend
 
-   Copyright (C) Andrew Tridgell 1992-2004
+   Copyright (C) Andrew Tridgell 1992-2006
    Copyright (C) Jeremy Allison 1992-2000
    
    This program is free software; you can redistribute it and/or modify
@@ -33,6 +33,7 @@
 #include "lib/messaging/irpc.h"
 #include "libcli/libcli.h"
 #include "cluster/cluster.h"
+#include "ntvfs/common/brlock.h"
 
 /*
   in this module a "DATA_BLOB *file_key" is a blob that uniquely identifies
@@ -42,7 +43,13 @@
   as it is applied consistently.
 */
 
-struct brl_context;
+/* this struct is typicaly attached to tcon */
+struct brl_context {
+       struct tdb_wrap *w;
+       struct server_id server;
+       struct messaging_context *messaging_ctx;
+};
+
 /*
   the lock context contains the elements that define whether one
   lock is the same as another lock
@@ -72,20 +79,13 @@ struct brl_handle {
        struct lock_struct last_lock;
 };
 
-/* this struct is typicaly attached to tcon */
-struct brl_context {
-       struct tdb_wrap *w;
-       struct server_id server;
-       struct messaging_context *messaging_ctx;
-};
-
 /*
   Open up the brlock.tdb database. Close it down using
   talloc_free(). We need the messaging_ctx to allow for
   pending lock notifications.
 */
-struct brl_context *brl_init(TALLOC_CTX *mem_ctx, struct server_id server, 
-                            struct messaging_context *messaging_ctx)
+static struct brl_context *brl_tdb_init(TALLOC_CTX *mem_ctx, struct server_id server, 
+                                   struct messaging_context *messaging_ctx)
 {
        char *path;
        struct brl_context *brl;
@@ -110,7 +110,8 @@ struct brl_context *brl_init(TALLOC_CTX *mem_ctx, struct server_id server,
        return brl;
 }
 
-struct brl_handle *brl_create_handle(TALLOC_CTX *mem_ctx, struct ntvfs_handle *ntvfs, DATA_BLOB *file_key)
+static struct brl_handle *brl_tdb_create_handle(TALLOC_CTX *mem_ctx, struct ntvfs_handle *ntvfs, 
+                                                   DATA_BLOB *file_key)
 {
        struct brl_handle *brlh;
 
@@ -129,7 +130,7 @@ struct brl_handle *brl_create_handle(TALLOC_CTX *mem_ctx, struct ntvfs_handle *n
 /*
   see if two locking contexts are equal
 */
-static BOOL brl_same_context(struct lock_context *ctx1, struct lock_context *ctx2)
+static BOOL brl_tdb_same_context(struct lock_context *ctx1, struct lock_context *ctx2)
 {
        return (cluster_id_equal(&ctx1->server, &ctx2->server) &&
                ctx1->smbpid == ctx2->smbpid &&
@@ -139,7 +140,7 @@ static BOOL brl_same_context(struct lock_context *ctx1, struct lock_context *ctx
 /*
   see if lck1 and lck2 overlap
 */
-static BOOL brl_overlap(struct lock_struct *lck1, 
+static BOOL brl_tdb_overlap(struct lock_struct *lck1, 
                        struct lock_struct *lck2)
 {
        /* this extra check is not redundent - it copes with locks
@@ -160,7 +161,7 @@ static BOOL brl_overlap(struct lock_struct *lck1,
 /*
  See if lock2 can be added when lock1 is in place.
 */
-static BOOL brl_conflict(struct lock_struct *lck1, 
+static BOOL brl_tdb_conflict(struct lock_struct *lck1, 
                         struct lock_struct *lck2)
 {
        /* pending locks don't conflict with anything */
@@ -173,12 +174,12 @@ static BOOL brl_conflict(struct lock_struct *lck1,
                return False;
        }
 
-       if (brl_same_context(&lck1->context, &lck2->context) &&
+       if (brl_tdb_same_context(&lck1->context, &lck2->context) &&
            lck2->lock_type == READ_LOCK && lck1->ntvfs == lck2->ntvfs) {
                return False;
        }
 
-       return brl_overlap(lck1, lck2);
+       return brl_tdb_overlap(lck1, lck2);
 } 
 
 
@@ -186,7 +187,7 @@ static BOOL brl_conflict(struct lock_struct *lck1,
  Check to see if this lock conflicts, but ignore our own locks on the
  same fnum only.
 */
-static BOOL brl_conflict_other(struct lock_struct *lck1, struct lock_struct *lck2)
+static BOOL brl_tdb_conflict_other(struct lock_struct *lck1, struct lock_struct *lck2)
 {
        /* pending locks don't conflict with anything */
        if (lck1->lock_type >= PENDING_READ_LOCK ||
@@ -202,13 +203,13 @@ static BOOL brl_conflict_other(struct lock_struct *lck1, struct lock_struct *lck
         * locks even if the context is the same. JRA. See LOCKTEST7
         * in smbtorture.
         */
-       if (brl_same_context(&lck1->context, &lck2->context) &&
+       if (brl_tdb_same_context(&lck1->context, &lck2->context) &&
            lck1->ntvfs == lck2->ntvfs &&
            (lck2->lock_type == READ_LOCK || lck1->lock_type == WRITE_LOCK)) {
                return False;
        }
 
-       return brl_overlap(lck1, lck2);
+       return brl_tdb_overlap(lck1, lck2);
 } 
 
 
@@ -217,7 +218,7 @@ static BOOL brl_conflict_other(struct lock_struct *lck1, struct lock_struct *lck
   is the same as this one and changes its error code. I wonder if any
   app depends on this?
 */
-static NTSTATUS brl_lock_failed(struct brl_handle *brlh, struct lock_struct *lock)
+static NTSTATUS brl_tdb_lock_failed(struct brl_handle *brlh, struct lock_struct *lock)
 {
        /*
         * this function is only called for non pending lock!
@@ -268,12 +269,12 @@ static NTSTATUS brl_lock_failed(struct brl_handle *brlh, struct lock_struct *loc
   someone else closing an overlapping lock range) a messaging
   notification is sent, identified by the notify_ptr
 */
-NTSTATUS brl_lock(struct brl_context *brl,
-                 struct brl_handle *brlh,
-                 uint16_t smbpid,
-                 uint64_t start, uint64_t size, 
-                 enum brl_type lock_type,
-                 void *notify_ptr)
+static NTSTATUS brl_tdb_lock(struct brl_context *brl,
+                        struct brl_handle *brlh,
+                        uint16_t smbpid,
+                        uint64_t start, uint64_t size, 
+                        enum brl_type lock_type,
+                        void *notify_ptr)
 {
        TDB_DATA kbuf, dbuf;
        int count=0, i;
@@ -297,7 +298,7 @@ NTSTATUS brl_lock(struct brl_context *brl,
 
                /* here we need to force that the last_lock isn't overwritten */
                lock = brlh->last_lock;
-               status = brl_lock(brl, brlh, smbpid, start, size, rw, NULL);
+               status = brl_tdb_lock(brl, brlh, smbpid, start, size, rw, NULL);
                brlh->last_lock = lock;
 
                if (NT_STATUS_IS_OK(status)) {
@@ -323,8 +324,8 @@ NTSTATUS brl_lock(struct brl_context *brl,
                locks = (struct lock_struct *)dbuf.dptr;
                count = dbuf.dsize / sizeof(*locks);
                for (i=0; i<count; i++) {
-                       if (brl_conflict(&locks[i], &lock)) {
-                               status = brl_lock_failed(brlh, &lock);
+                       if (brl_tdb_conflict(&locks[i], &lock)) {
+                               status = brl_tdb_lock_failed(brlh, &lock);
                                goto fail;
                        }
                }
@@ -371,7 +372,7 @@ NTSTATUS brl_lock(struct brl_context *brl,
   locks that cover this range and if we find any then notify the server that it should
   retry the lock
 */
-static void brl_notify_unlock(struct brl_context *brl,
+static void brl_tdb_notify_unlock(struct brl_context *brl,
                              struct lock_struct *locks, int count, 
                              struct lock_struct *removed_lock)
 {
@@ -385,8 +386,8 @@ static void brl_notify_unlock(struct brl_context *brl,
 
        for (i=0;i<count;i++) {
                if (locks[i].lock_type >= PENDING_READ_LOCK &&
-                   brl_overlap(&locks[i], removed_lock)) {
-                       if (last_notice != -1 && brl_overlap(&locks[i], &locks[last_notice])) {
+                   brl_tdb_overlap(&locks[i], removed_lock)) {
+                       if (last_notice != -1 && brl_tdb_overlap(&locks[i], &locks[last_notice])) {
                                continue;
                        }
                        if (locks[i].lock_type == PENDING_WRITE_LOCK) {
@@ -403,13 +404,13 @@ static void brl_notify_unlock(struct brl_context *brl,
   send notifications for all pending locks - the file is being closed by this
   user
 */
-static void brl_notify_all(struct brl_context *brl,
+static void brl_tdb_notify_all(struct brl_context *brl,
                           struct lock_struct *locks, int count)
 {
        int i;
        for (i=0;i<count;i++) {
                if (locks->lock_type >= PENDING_READ_LOCK) {
-                       brl_notify_unlock(brl, locks, count, &locks[i]);
+                       brl_tdb_notify_unlock(brl, locks, count, &locks[i]);
                }
        }
 }
@@ -419,10 +420,10 @@ static void brl_notify_all(struct brl_context *brl,
 /*
  Unlock a range of bytes.
 */
-NTSTATUS brl_unlock(struct brl_context *brl,
-                   struct brl_handle *brlh, 
-                   uint16_t smbpid,
-                   uint64_t start, uint64_t size)
+static NTSTATUS brl_tdb_unlock(struct brl_context *brl,
+                          struct brl_handle *brlh, 
+                          uint16_t smbpid,
+                          uint64_t start, uint64_t size)
 {
        TDB_DATA kbuf, dbuf;
        int count, i;
@@ -454,7 +455,7 @@ NTSTATUS brl_unlock(struct brl_context *brl,
        for (i=0; i<count; i++) {
                struct lock_struct *lock = &locks[i];
                
-               if (brl_same_context(&lock->context, &context) &&
+               if (brl_tdb_same_context(&lock->context, &context) &&
                    lock->ntvfs == brlh->ntvfs &&
                    lock->start == start &&
                    lock->size == size &&
@@ -474,7 +475,7 @@ NTSTATUS brl_unlock(struct brl_context *brl,
                                count--;
 
                                /* send notifications for any relevant pending locks */
-                               brl_notify_unlock(brl, locks, count, &removed_lock);
+                               brl_tdb_notify_unlock(brl, locks, count, &removed_lock);
 
                                dbuf.dsize = count * sizeof(*locks);
 
@@ -505,9 +506,9 @@ NTSTATUS brl_unlock(struct brl_context *brl,
   given up trying to establish a lock or when they have succeeded in
   getting it. In either case they no longer need to be notified.
 */
-NTSTATUS brl_remove_pending(struct brl_context *brl,
-                           struct brl_handle *brlh, 
-                           void *notify_ptr)
+static NTSTATUS brl_tdb_remove_pending(struct brl_context *brl,
+                                  struct brl_handle *brlh, 
+                                  void *notify_ptr)
 {
        TDB_DATA kbuf, dbuf;
        int count, i;
@@ -575,11 +576,11 @@ NTSTATUS brl_remove_pending(struct brl_context *brl,
 /*
   Test if we are allowed to perform IO on a region of an open file
 */
-NTSTATUS brl_locktest(struct brl_context *brl,
-                     struct brl_handle *brlh,
-                     uint16_t smbpid, 
-                     uint64_t start, uint64_t size, 
-                     enum brl_type lock_type)
+static NTSTATUS brl_tdb_locktest(struct brl_context *brl,
+                            struct brl_handle *brlh,
+                            uint16_t smbpid, 
+                            uint64_t start, uint64_t size, 
+                            enum brl_type lock_type)
 {
        TDB_DATA kbuf, dbuf;
        int count, i;
@@ -606,7 +607,7 @@ NTSTATUS brl_locktest(struct brl_context *brl,
        count = dbuf.dsize / sizeof(*locks);
 
        for (i=0; i<count; i++) {
-               if (brl_conflict_other(&locks[i], &lock)) {
+               if (brl_tdb_conflict_other(&locks[i], &lock)) {
                        free(dbuf.dptr);
                        return NT_STATUS_FILE_LOCK_CONFLICT;
                }
@@ -620,8 +621,8 @@ NTSTATUS brl_locktest(struct brl_context *brl,
 /*
  Remove any locks associated with a open file.
 */
-NTSTATUS brl_close(struct brl_context *brl,
-                  struct brl_handle *brlh)
+static NTSTATUS brl_tdb_close(struct brl_context *brl,
+                         struct brl_handle *brlh)
 {
        TDB_DATA kbuf, dbuf;
        int count, i, dcount=0;
@@ -672,7 +673,7 @@ NTSTATUS brl_close(struct brl_context *brl,
                /* tell all pending lock holders for this file that
                   they have a chance now. This is a bit indiscriminant,
                   but works OK */
-               brl_notify_all(brl, locks, count);
+               brl_tdb_notify_all(brl, locks, count);
 
                dbuf.dsize = count * sizeof(*locks);
 
@@ -687,3 +688,19 @@ NTSTATUS brl_close(struct brl_context *brl,
        return status;
 }
 
+
+static const struct brlock_ops brlock_tdb_ops = {
+       .brl_init           = brl_tdb_init,
+       .brl_create_handle  = brl_tdb_create_handle,
+       .brl_lock           = brl_tdb_lock,
+       .brl_unlock         = brl_tdb_unlock,
+       .brl_remove_pending = brl_tdb_remove_pending,
+       .brl_locktest       = brl_tdb_locktest,
+       .brl_close          = brl_tdb_close
+};
+
+
+void brl_tdb_init_ops(void)
+{
+       brl_set_ops(&brlock_tdb_ops);
+}

diff --git a/source4/ntvfs/common/config.mk b/source4/ntvfs/common/config.mk
index c16cc09dfec30756c9667d64ad28881ff9211925..fd1b55a4f7aff2ca10361306d82561d34e733420 100644 (file)
--- a/source4/ntvfs/common/config.mk
+++ b/source4/ntvfs/common/config.mk
@@ -5,6 +5,7 @@ PRIVATE_PROTO_HEADER = proto.h
 OBJ_FILES = \
                init.o \
                brlock.o \
+               brlock_tdb.o \
                opendb.o \
                notify.o
 PUBLIC_DEPENDENCIES = NDR_OPENDB NDR_NOTIFY sys_notify share