Attachment 13074 Details for Bug 12628 – git-am fix for 4.5.next.

[patch] git-am fix for 4.5.next.

bug-12628-v4,5 (text/plain), 8.36 KB, created by Jeremy Allison on 2017-03-16 00:03:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2017-03-16 00:03:52 UTC

Size: 8.36 KB

patch

obsolete

>From da99b1366cd7979846728ad467f294037bf593d4 Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Tue, 14 Mar 2017 13:23:13 -0700
>Subject: [PATCH 1/2] s3: locking: Move two leases functions into a new file.
>
>map_oplock_to_lease_type(), fsp_lease_type().
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=12628
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>Reviewed-by: Ralph Boehme <slow@samba.org>
>(back ported from commit 125c78ad0b8f9caaef1ba2f1aeb5ec593375fccd)
>---
> source3/locking/leases_util.c | 55 +++++++++++++++++++++++++++++++++++++++++++
> source3/locking/proto.h       |  4 ++++
> source3/smbd/files.c          |  8 -------
> source3/smbd/oplock.c         | 22 -----------------
> source3/smbd/proto.h          |  2 --
> source3/wscript_build         |  6 +++++
> 6 files changed, 65 insertions(+), 32 deletions(-)
> create mode 100644 source3/locking/leases_util.c
>
>diff --git a/source3/locking/leases_util.c b/source3/locking/leases_util.c
>new file mode 100644
>index 00000000000..cb307c88d36
>--- /dev/null
>+++ b/source3/locking/leases_util.c
>@@ -0,0 +1,55 @@
>+/*
>+   Unix SMB/CIFS implementation.
>+   Lease utility functions
>+
>+   Copyright (C) Jeremy Allison 2017.
>+   Copyright (C) Stefan (metze) Metzmacher 2017.
>+
>+   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 3 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, see <http://www.gnu.org/licenses/>.
>+*/
>+
>+#define DBGC_CLASS DBGC_LOCKING
>+#include "includes.h"
>+#include "../librpc/gen_ndr/open_files.h"
>+#include "locking/proto.h"
>+
>+uint32_t map_oplock_to_lease_type(uint16_t op_type)
>+{
>+	uint32_t ret;
>+
>+	switch(op_type) {
>+	case BATCH_OPLOCK:
>+	case BATCH_OPLOCK|EXCLUSIVE_OPLOCK:
>+		ret = SMB2_LEASE_READ|SMB2_LEASE_WRITE|SMB2_LEASE_HANDLE;
>+		break;
>+	case EXCLUSIVE_OPLOCK:
>+		ret = SMB2_LEASE_READ|SMB2_LEASE_WRITE;
>+		break;
>+	case LEVEL_II_OPLOCK:
>+		ret = SMB2_LEASE_READ;
>+		break;
>+	default:
>+		ret = SMB2_LEASE_NONE;
>+		break;
>+	}
>+	return ret;
>+}
>+
>+uint32_t fsp_lease_type(struct files_struct *fsp)
>+{
>+	if (fsp->oplock_type == LEASE_OPLOCK) {
>+		return fsp->lease->lease.lease_state;
>+	}
>+	return map_oplock_to_lease_type(fsp->oplock_type);
>+}
>diff --git a/source3/locking/proto.h b/source3/locking/proto.h
>index 13499cfbf41..17cb1cdf86b 100644
>--- a/source3/locking/proto.h
>+++ b/source3/locking/proto.h
>@@ -248,4 +248,8 @@ bool release_posix_lock_posix_flavour(files_struct *fsp,
> 				const struct lock_struct *plocks,
> 				int num_locks);
> 
>+/* The following definitions come from locking/leases_util.c */
>+uint32_t map_oplock_to_lease_type(uint16_t op_type);
>+uint32_t fsp_lease_type(struct files_struct *fsp);
>+
> #endif /* _LOCKING_PROTO_H_ */
>diff --git a/source3/smbd/files.c b/source3/smbd/files.c
>index 1ef1bc9ff9b..6d0f05b9db1 100644
>--- a/source3/smbd/files.c
>+++ b/source3/smbd/files.c
>@@ -785,14 +785,6 @@ const struct GUID *fsp_client_guid(const files_struct *fsp)
> 	return &fsp->conn->sconn->client->connections->smb2.client.guid;
> }
> 
>-uint32_t fsp_lease_type(struct files_struct *fsp)
>-{
>-	if (fsp->oplock_type == LEASE_OPLOCK) {
>-		return fsp->lease->lease.lease_state;
>-	}
>-	return map_oplock_to_lease_type(fsp->oplock_type);
>-}
>-
> size_t fsp_fullbasepath(struct files_struct *fsp, char *buf, size_t buflen)
> {
> 	int len;
>diff --git a/source3/smbd/oplock.c b/source3/smbd/oplock.c
>index 6d7d17f6fe4..57b53f82c17 100644
>--- a/source3/smbd/oplock.c
>+++ b/source3/smbd/oplock.c
>@@ -149,28 +149,6 @@ static void downgrade_file_oplock(files_struct *fsp)
> 	TALLOC_FREE(fsp->oplock_timeout);
> }
> 
>-uint32_t map_oplock_to_lease_type(uint16_t op_type)
>-{
>-	uint32_t ret;
>-
>-	switch(op_type) {
>-	case BATCH_OPLOCK:
>-	case BATCH_OPLOCK|EXCLUSIVE_OPLOCK:
>-		ret = SMB2_LEASE_READ|SMB2_LEASE_WRITE|SMB2_LEASE_HANDLE;
>-		break;
>-	case EXCLUSIVE_OPLOCK:
>-		ret = SMB2_LEASE_READ|SMB2_LEASE_WRITE;
>-		break;
>-	case LEVEL_II_OPLOCK:
>-		ret = SMB2_LEASE_READ;
>-		break;
>-	default:
>-		ret = SMB2_LEASE_NONE;
>-		break;
>-	}
>-	return ret;
>-}
>-
> uint32_t get_lease_type(const struct share_mode_data *d,
> 			const struct share_mode_entry *e)
> {
>diff --git a/source3/smbd/proto.h b/source3/smbd/proto.h
>index 9301c787d68..277dc6edd85 100644
>--- a/source3/smbd/proto.h
>+++ b/source3/smbd/proto.h
>@@ -409,7 +409,6 @@ NTSTATUS file_name_hash(connection_struct *conn,
> NTSTATUS fsp_set_smb_fname(struct files_struct *fsp,
> 			   const struct smb_filename *smb_fname_in);
> const struct GUID *fsp_client_guid(const files_struct *fsp);
>-uint32_t fsp_lease_type(struct files_struct *fsp);
> size_t fsp_fullbasepath(struct files_struct *fsp, char *buf, size_t buflen);
> 
> /* The following definitions come from smbd/ipc.c  */
>@@ -703,7 +702,6 @@ NTSTATUS get_relative_fid_filename(connection_struct *conn,
> 
> /* The following definitions come from smbd/oplock.c  */
> 
>-uint32_t map_oplock_to_lease_type(uint16_t op_type);
> uint32_t get_lease_type(const struct share_mode_data *d,
> 			const struct share_mode_entry *e);
> bool update_num_read_oplocks(files_struct *fsp, struct share_mode_lock *lck);
>diff --git a/source3/wscript_build b/source3/wscript_build
>index 2918975b8d5..9a196a25b91 100755
>--- a/source3/wscript_build
>+++ b/source3/wscript_build
>@@ -629,6 +629,7 @@ bld.SAMBA3_LIBRARY('smbd_base',
>                    RPC_SERVICE
>                    NDR_SMBXSRV
>                    LEASES_DB
>+		   LEASES_UTIL
>                    LIBASYS
>                    sysquotas
>                    NDR_SMB_ACL
>@@ -650,6 +651,7 @@ bld.SAMBA3_SUBSYSTEM('LOCKING',
>                     tdb
>                     talloc
>                     LEASES_DB
>+		    LEASES_UTIL
>                     NDR_OPEN_FILES
>                     FNAME_UTIL''')
> 
>@@ -657,6 +659,10 @@ bld.SAMBA3_SUBSYSTEM('LEASES_DB',
>                     source='locking/leases_db.c',
>                     deps='NDR_LEASES_DB')
> 
>+bld.SAMBA3_SUBSYSTEM('LEASES_UTIL',
>+                    source='locking/leases_util.c',
>+                    deps='NDR_OPEN_FILES')
>+
> if bld.CONFIG_GET("WITH_PROFILE"):
>     bld.SAMBA3_SUBSYSTEM('PROFILE',
>                          source='profile/profile.c',
>-- 
>2.12.0.367.g23dc2f6d3c-goog
>
>
>From 310184f1b6035b63ba8c58d5d545669e45161859 Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Tue, 14 Mar 2017 13:34:07 -0700
>Subject: [PATCH 2/2] s3: locking: Update oplock optimization for the leases
> era !
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=12628
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>Reviewed-by: Ralph Boehme <slow@samba.org>
>
>Autobuild-User(master): Ralph BÃ¶hme <slow@samba.org>
>Autobuild-Date(master): Wed Mar 15 20:04:32 CET 2017 on sn-devel-144
>
>(cherry picked from commit 1c4b15aa5f6707e7bcfc21435e26929fb7f45c0f)
>---
> source3/locking/locking.c | 22 +++++++++++++---------
> 1 file changed, 13 insertions(+), 9 deletions(-)
>
>diff --git a/source3/locking/locking.c b/source3/locking/locking.c
>index 5a9746094ba..e6d3918db79 100644
>--- a/source3/locking/locking.c
>+++ b/source3/locking/locking.c
>@@ -118,17 +118,21 @@ bool strict_lock_default(files_struct *fsp, struct lock_struct *plock)
> 	}
> 
> 	if (strict_locking == Auto) {
>-		if  (EXCLUSIVE_OPLOCK_TYPE(fsp->oplock_type) &&
>-		     (plock->lock_type == READ_LOCK ||
>-		      plock->lock_type == WRITE_LOCK)) {
>-			DEBUG(10, ("is_locked: optimisation - exclusive oplock "
>-				   "on file %s\n", fsp_str_dbg(fsp)));
>+		uint32_t lease_type = fsp_lease_type(fsp);
>+
>+		if ((lease_type & SMB2_LEASE_READ) &&
>+		     (plock->lock_type == READ_LOCK))
>+		{
>+			DBG_DEBUG("optimisation - read lease on file %s\n",
>+				  fsp_str_dbg(fsp));
> 			return true;
> 		}
>-		if ((fsp->oplock_type == LEVEL_II_OPLOCK) &&
>-		    (plock->lock_type == READ_LOCK)) {
>-			DEBUG(10, ("is_locked: optimisation - level II oplock "
>-				   "on file %s\n", fsp_str_dbg(fsp)));
>+
>+		if ((lease_type & SMB2_LEASE_WRITE) &&
>+		     (plock->lock_type == WRITE_LOCK))
>+		{
>+			DBG_DEBUG("optimisation - write lease on file %s\n",
>+				  fsp_str_dbg(fsp));
> 			return true;
> 		}
> 	}
>-- 
>2.12.0.367.g23dc2f6d3c-goog
>

Flags:

slow: review+

Actions: View

Attachments on bug 12628: 13059 | 13073 | 13074 | 13075