From patchwork Tue Nov 27 06:50:25 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Pavel Shilovsky X-Patchwork-Id: 1808951 Return-Path: X-Original-To: patchwork-cifs-client@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id ACF9E3FC54 for ; Tue, 27 Nov 2012 06:50:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757957Ab2K0Guo (ORCPT ); Tue, 27 Nov 2012 01:50:44 -0500 Received: from mail-lb0-f174.google.com ([209.85.217.174]:40431 "EHLO mail-lb0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757842Ab2K0Gun (ORCPT ); Tue, 27 Nov 2012 01:50:43 -0500 Received: by mail-lb0-f174.google.com with SMTP id gi11so6366892lbb.19 for ; Mon, 26 Nov 2012 22:50:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:subject:date:message-id:x-mailer:in-reply-to :references; bh=G3vVoEGtfoQi0Z4IGoyR0PQwgGdJnCKKoAaNItmzh5o=; b=PvOpR5/ZNeLu+AMNCCnh5e/iCWLtwItARRlxZ5hlOOTGY8l9q48FAHhYZIJbbq+EL/ HYaW4jIkSjSdkiWA7ADIosgzTkyMDpUVLZBWi6rtrDYpV0pXLCoAsrVUBPf33+zI7Zrk hl7Gv1Y+UJ0auD5Zrym+CDihWRcYXARRaRZZis4GHf7OvIfYeLPM1/d7lN8KFvZR9Aw6 zOJloc/KtjamPlnLJDuvUOJFeJOwdTD5AKWPFnMI6WWF0x9+909QFXElAGxpHHHemnsV xigrpMWCP5lQU8hKMJzFYSids2kUixQh323ZOcrIbNGdBb8bG8DBHggnHvDlzntyWyfr ZukQ== Received: by 10.112.23.2 with SMTP id i2mr6301340lbf.24.1353999041748; Mon, 26 Nov 2012 22:50:41 -0800 (PST) Received: from localhost.localdomain ([95.84.0.69]) by mx.google.com with ESMTPS id bf3sm6547215lbb.16.2012.11.26.22.50.40 (version=TLSv1/SSLv3 cipher=OTHER); Mon, 26 Nov 2012 22:50:40 -0800 (PST) From: Pavel Shilovsky To: linux-cifs@vger.kernel.org Subject: [PATCH 2/5] CIFS: Separate pushing mandatory locks and lock_sem handling Date: Tue, 27 Nov 2012 10:50:25 +0400 Message-Id: <1353999029-3975-3-git-send-email-piastry@etersoft.ru> X-Mailer: git-send-email 1.7.10.4 In-Reply-To: <1353999029-3975-1-git-send-email-piastry@etersoft.ru> References: <1353999029-3975-1-git-send-email-piastry@etersoft.ru> Sender: linux-cifs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-cifs@vger.kernel.org Signed-off-by: Pavel Shilovsky --- fs/cifs/file.c | 39 ++++++++++----------------------------- fs/cifs/smb2file.c | 12 ------------ 2 files changed, 10 insertions(+), 41 deletions(-) diff --git a/fs/cifs/file.c b/fs/cifs/file.c index 54541a6..cf67f8f 100644 --- a/fs/cifs/file.c +++ b/fs/cifs/file.c @@ -954,7 +954,6 @@ cifs_push_mandatory_locks(struct cifsFileInfo *cfile) int rc = 0, stored_rc; struct cifsLockInfo *li, *tmp; struct cifs_tcon *tcon; - struct cifsInodeInfo *cinode = CIFS_I(cfile->dentry->d_inode); unsigned int num, max_num, max_buf; LOCKING_ANDX_RANGE *buf, *cur; int types[] = {LOCKING_ANDX_LARGE_FILES, @@ -964,21 +963,12 @@ cifs_push_mandatory_locks(struct cifsFileInfo *cfile) xid = get_xid(); tcon = tlink_tcon(cfile->tlink); - /* we are going to update can_cache_brlcks here - need a write access */ - down_write(&cinode->lock_sem); - if (!cinode->can_cache_brlcks) { - up_write(&cinode->lock_sem); - free_xid(xid); - return rc; - } - /* * Accessing maxBuf is racy with cifs_reconnect - need to store value * and check it for zero before using. */ max_buf = tcon->ses->server->maxBuf; if (!max_buf) { - up_write(&cinode->lock_sem); free_xid(xid); return -EINVAL; } @@ -987,7 +977,6 @@ cifs_push_mandatory_locks(struct cifsFileInfo *cfile) sizeof(LOCKING_ANDX_RANGE); buf = kzalloc(max_num * sizeof(LOCKING_ANDX_RANGE), GFP_KERNEL); if (!buf) { - up_write(&cinode->lock_sem); free_xid(xid); return -ENOMEM; } @@ -1024,9 +1013,6 @@ cifs_push_mandatory_locks(struct cifsFileInfo *cfile) } } - cinode->can_cache_brlcks = false; - up_write(&cinode->lock_sem); - kfree(buf); free_xid(xid); return rc; @@ -1047,7 +1033,7 @@ struct lock_to_push { }; static int -cifs_push_posix_locks_locked(struct cifsFileInfo *cfile) +cifs_push_posix_locks(struct cifsFileInfo *cfile) { struct cifs_tcon *tcon = tlink_tcon(cfile->tlink); struct file_lock *flock, **before; @@ -1135,9 +1121,11 @@ err_out: } static int -cifs_push_posix_locks(struct cifsFileInfo *cfile) +cifs_push_locks(struct cifsFileInfo *cfile) { + struct cifs_sb_info *cifs_sb = CIFS_SB(cfile->dentry->d_sb); struct cifsInodeInfo *cinode = CIFS_I(cfile->dentry->d_inode); + struct cifs_tcon *tcon = tlink_tcon(cfile->tlink); int rc = 0; /* we are going to update can_cache_brlcks here - need a write access */ @@ -1146,24 +1134,17 @@ cifs_push_posix_locks(struct cifsFileInfo *cfile) up_write(&cinode->lock_sem); return rc; } - rc = cifs_push_posix_locks_locked(cfile); - cinode->can_cache_brlcks = false; - up_write(&cinode->lock_sem); - return rc; -} - -static int -cifs_push_locks(struct cifsFileInfo *cfile) -{ - struct cifs_sb_info *cifs_sb = CIFS_SB(cfile->dentry->d_sb); - struct cifs_tcon *tcon = tlink_tcon(cfile->tlink); if (cap_unix(tcon->ses) && (CIFS_UNIX_FCNTL_CAP & le64_to_cpu(tcon->fsUnixInfo.Capability)) && ((cifs_sb->mnt_cifs_flags & CIFS_MOUNT_NOPOSIXBRL) == 0)) - return cifs_push_posix_locks(cfile); + rc = cifs_push_posix_locks(cfile); + else + rc = tcon->ses->server->ops->push_mand_locks(cfile); - return tcon->ses->server->ops->push_mand_locks(cfile); + cinode->can_cache_brlcks = false; + up_write(&cinode->lock_sem); + return rc; } static void diff --git a/fs/cifs/smb2file.c b/fs/cifs/smb2file.c index bfe22e8..7dfb50c 100644 --- a/fs/cifs/smb2file.c +++ b/fs/cifs/smb2file.c @@ -260,13 +260,6 @@ smb2_push_mandatory_locks(struct cifsFileInfo *cfile) struct cifs_fid_locks *fdlocks; xid = get_xid(); - /* we are going to update can_cache_brlcks here - need a write access */ - down_write(&cinode->lock_sem); - if (!cinode->can_cache_brlcks) { - up_write(&cinode->lock_sem); - free_xid(xid); - return rc; - } /* * Accessing maxBuf is racy with cifs_reconnect - need to store value @@ -274,7 +267,6 @@ smb2_push_mandatory_locks(struct cifsFileInfo *cfile) */ max_buf = tlink_tcon(cfile->tlink)->ses->server->maxBuf; if (!max_buf) { - up_write(&cinode->lock_sem); free_xid(xid); return -EINVAL; } @@ -282,7 +274,6 @@ smb2_push_mandatory_locks(struct cifsFileInfo *cfile) max_num = max_buf / sizeof(struct smb2_lock_element); buf = kzalloc(max_num * sizeof(struct smb2_lock_element), GFP_KERNEL); if (!buf) { - up_write(&cinode->lock_sem); free_xid(xid); return -ENOMEM; } @@ -293,10 +284,7 @@ smb2_push_mandatory_locks(struct cifsFileInfo *cfile) rc = stored_rc; } - cinode->can_cache_brlcks = false; kfree(buf); - - up_write(&cinode->lock_sem); free_xid(xid); return rc; }