r3126: in the brlock code I had used a void* for the brl context as I didn't
authorAndrew Tridgell <tridge@samba.org>
Fri, 22 Oct 2004 01:14:49 +0000 (01:14 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 18:02:24 +0000 (13:02 -0500)
want to expose the brl context structure outside the brlock.c
code. Instead, I now use "struct brl_context *" and rely on C being
happy to pass around pointers to unknown structures as long as they
are not dereferenced. I will be interested to see how the build farm
likes this.
(This used to be commit cb155c8ad837285c5a7f5b104968239df0b65fd2)

source4/ntvfs/common/brlock.c
source4/ntvfs/posix/vfs_posix.h

index 2da32891fbad3c746456b924ea4433ef96dcc391..607947615eb076be6d3b82a2f11278560971618c 100644 (file)
@@ -71,8 +71,8 @@ struct brl_context {
   talloc_free(). We need the messaging_ctx to allow for
   pending lock notifications.
 */
   talloc_free(). We need the messaging_ctx to allow for
   pending lock notifications.
 */
-void *brl_init(TALLOC_CTX *mem_ctx, servid_t server, uint16_t tid, 
-              void *messaging_ctx)
+struct brl_context *brl_init(TALLOC_CTX *mem_ctx, servid_t server, uint16_t tid, 
+                            void *messaging_ctx)
 {
        char *path;
        struct brl_context *brl;
 {
        char *path;
        struct brl_context *brl;
@@ -97,7 +97,7 @@ void *brl_init(TALLOC_CTX *mem_ctx, servid_t server, uint16_t tid,
        brl->messaging_ctx = messaging_ctx;
        ZERO_STRUCT(brl->last_lock_failure);
 
        brl->messaging_ctx = messaging_ctx;
        ZERO_STRUCT(brl->last_lock_failure);
 
-       return (void *)brl;
+       return brl;
 }
 
 
 }
 
 
@@ -219,7 +219,7 @@ static NTSTATUS brl_lock_failed(struct brl_context *brl, struct lock_struct *loc
   someone else closing an overlapping lock range) a messaging
   notification is sent, identified by the notify_ptr
 */
   someone else closing an overlapping lock range) a messaging
   notification is sent, identified by the notify_ptr
 */
-NTSTATUS brl_lock(void *brl_ctx,
+NTSTATUS brl_lock(struct brl_context *brl,
                  DATA_BLOB *file_key, 
                  uint16_t smbpid,
                  uint16_t fnum, 
                  DATA_BLOB *file_key, 
                  uint16_t smbpid,
                  uint16_t fnum, 
@@ -227,7 +227,6 @@ NTSTATUS brl_lock(void *brl_ctx,
                  enum brl_type lock_type,
                  void *notify_ptr)
 {
                  enum brl_type lock_type,
                  void *notify_ptr)
 {
-       struct brl_context *brl = brl_ctx;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct lock, *locks;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct lock, *locks;
@@ -248,7 +247,7 @@ NTSTATUS brl_lock(void *brl_ctx,
           preventing the real lock gets removed */
        if (lock_type >= PENDING_READ_LOCK) {
                enum brl_type rw = (lock_type==PENDING_READ_LOCK? READ_LOCK : WRITE_LOCK);
           preventing the real lock gets removed */
        if (lock_type >= PENDING_READ_LOCK) {
                enum brl_type rw = (lock_type==PENDING_READ_LOCK? READ_LOCK : WRITE_LOCK);
-               status = brl_lock(brl_ctx, file_key, smbpid, fnum, start, size, rw, NULL);
+               status = brl_lock(brl, file_key, smbpid, fnum, start, size, rw, NULL);
                if (NT_STATUS_IS_OK(status)) {
                        tdb_chainunlock(brl->w->tdb, kbuf);
                        return NT_STATUS_OK;
                if (NT_STATUS_IS_OK(status)) {
                        tdb_chainunlock(brl->w->tdb, kbuf);
                        return NT_STATUS_OK;
@@ -368,13 +367,12 @@ static void brl_notify_all(struct brl_context *brl,
 /*
  Unlock a range of bytes.
 */
 /*
  Unlock a range of bytes.
 */
-NTSTATUS brl_unlock(void *brl_ctx,
+NTSTATUS brl_unlock(struct brl_context *brl,
                    DATA_BLOB *file_key, 
                    uint16_t smbpid,
                    uint16_t fnum,
                    uint64_t start, uint64_t size)
 {
                    DATA_BLOB *file_key, 
                    uint16_t smbpid,
                    uint16_t fnum,
                    uint64_t start, uint64_t size)
 {
-       struct brl_context *brl = brl_ctx;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct *locks;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct *locks;
@@ -456,11 +454,10 @@ NTSTATUS brl_unlock(void *brl_ctx,
   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.
 */
   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(void *brl_ctx,
+NTSTATUS brl_remove_pending(struct brl_context *brl,
                            DATA_BLOB *file_key, 
                            void *notify_ptr)
 {
                            DATA_BLOB *file_key, 
                            void *notify_ptr)
 {
-       struct brl_context *brl = brl_ctx;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct *locks;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct *locks;
@@ -526,14 +523,13 @@ NTSTATUS brl_remove_pending(void *brl_ctx,
 /*
   Test if we are allowed to perform IO on a region of an open file
 */
 /*
   Test if we are allowed to perform IO on a region of an open file
 */
-NTSTATUS brl_locktest(void *brl_ctx,
+NTSTATUS brl_locktest(struct brl_context *brl,
                      DATA_BLOB *file_key, 
                      uint16_t fnum,
                      uint16 smbpid, 
                      uint64_t start, uint64_t size, 
                      enum brl_type lock_type)
 {
                      DATA_BLOB *file_key, 
                      uint16_t fnum,
                      uint16 smbpid, 
                      uint64_t start, uint64_t size, 
                      enum brl_type lock_type)
 {
-       struct brl_context *brl = brl_ctx;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct lock, *locks;
        TDB_DATA kbuf, dbuf;
        int count, i;
        struct lock_struct lock, *locks;
@@ -573,10 +569,9 @@ NTSTATUS brl_locktest(void *brl_ctx,
 /*
  Remove any locks associated with a open file.
 */
 /*
  Remove any locks associated with a open file.
 */
-NTSTATUS brl_close(void *brl_ctx,
+NTSTATUS brl_close(struct brl_context *brl,
                   DATA_BLOB *file_key, int fnum)
 {
                   DATA_BLOB *file_key, int fnum)
 {
-       struct brl_context *brl = brl_ctx;
        TDB_DATA kbuf, dbuf;
        int count, i, dcount=0;
        struct lock_struct *locks;
        TDB_DATA kbuf, dbuf;
        int count, i, dcount=0;
        struct lock_struct *locks;
index ae601c60c7bcd55a575715bdabd694a15212a885..a0aacf3786f92584495df7fcd13f848ba82d39df 100644 (file)
@@ -36,7 +36,7 @@ struct pvfs_state {
 
        struct pvfs_mangle_context *mangle_ctx;
 
 
        struct pvfs_mangle_context *mangle_ctx;
 
-       void *brl_context;
+       struct brl_context *brl_context;
 
        /* an id tree mapping open search ID to a pvfs_search_state structure */
        void *idtree_search;
 
        /* an id tree mapping open search ID to a pvfs_search_state structure */
        void *idtree_search;