From: Jeremy Allison Date: Thu, 9 Dec 2004 22:10:40 +0000 (+0000) Subject: r4121: Optimisation from Nadav Danieli . If we're oplocked, X-Git-Tag: samba-misc-tags/initial-v3-0-unstable~5512 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f4feed8f90be98eaea59bebcc142fd398ffe2a59;p=thirdparty%2Fsamba.git r4121: Optimisation from Nadav Danieli . If we're oplocked, short circuit some is_locked() tests. Jeremy. --- diff --git a/source/locking/locking.c b/source/locking/locking.c index b6e2ced3360..4c6d89571e3 100644 --- a/source/locking/locking.c +++ b/source/locking/locking.c @@ -77,9 +77,17 @@ BOOL is_locked(files_struct *fsp,connection_struct *conn, if (!lp_locking(snum) || !lp_strict_locking(snum)) return(False); - ret = !brl_locktest(fsp->dev, fsp->inode, fsp->fnum, + if (EXCLUSIVE_OPLOCK_TYPE(fsp->oplock_type) && (lock_type == READ_LOCK || lock_type == WRITE_LOCK)) { + DEBUG(10,("is_locked: optimisation - exclusive oplock on file %s\n", fsp->fsp_name )); + ret = 0; + } else if (LEVEL_II_OPLOCK_TYPE(fsp->oplock_type) && (lock_type == READ_LOCK)) { + DEBUG(10,("is_locked: optimisation - level II oplock on file %s\n", fsp->fsp_name )); + ret = 0; + } else { + ret = !brl_locktest(fsp->dev, fsp->inode, fsp->fnum, global_smbpid, sys_getpid(), conn->cnum, offset, count, lock_type); + } DEBUG(10,("is_locked: brl start=%.0f len=%.0f %s for file %s\n", (double)offset, (double)count, ret ? "locked" : "unlocked",