From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Wilcox Subject: [PATCH 5/5] vfs: Allow truncate, chomd and chown to be interrupted by fatal signals Date: Mon, 29 Jun 2015 16:02:32 -0400 Message-ID: <1435608152-6982-6-git-send-email-matthew.r.wilcox@intel.com> References: <1435608152-6982-1-git-send-email-matthew.r.wilcox@intel.com> Cc: Matthew Wilcox , Matthew Wilcox To: linux-fsdevel@vger.kernel.org, Alexander Viro Return-path: Received: from mga02.intel.com ([134.134.136.20]:61840 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753183AbbF2UC5 (ORCPT ); Mon, 29 Jun 2015 16:02:57 -0400 In-Reply-To: <1435608152-6982-1-git-send-email-matthew.r.wilcox@intel.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: If another task dies while holding the i_mutex, tasks trying to truncate, chmod or chown the file will hang. Allowing a fatal interrupt to kill the task is beneficial for the system administrator trying to get the machine to shut down more cleanly. Signed-off-by: Matthew Wilcox --- fs/open.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/fs/open.c b/fs/open.c index e0250bd..4b0061e 100644 --- a/fs/open.c +++ b/fs/open.c @@ -56,7 +56,8 @@ int do_truncate(struct dentry *dentry, loff_t length, unsigned int time_attrs, if (ret) newattrs.ia_valid |= ret | ATTR_FORCE; - mutex_lock(&dentry->d_inode->i_mutex); + if (mutex_lock_killable(&dentry->d_inode->i_mutex)) + return -EINTR; /* Note any delegations or leases have already been broken: */ ret = notify_change(dentry, &newattrs, NULL); mutex_unlock(&dentry->d_inode->i_mutex); @@ -508,7 +509,8 @@ static int chmod_common(struct path *path, umode_t mode) if (error) return error; retry_deleg: - mutex_lock(&inode->i_mutex); + if (mutex_lock_killable(&inode->i_mutex)) + return -EINTR; error = security_path_chmod(path, mode); if (error) goto out_unlock; @@ -591,7 +593,8 @@ retry_deleg: if (!S_ISDIR(inode->i_mode)) newattrs.ia_valid |= ATTR_KILL_SUID | ATTR_KILL_SGID | ATTR_KILL_PRIV; - mutex_lock(&inode->i_mutex); + if (mutex_lock_killable(&inode->i_mutex)) + return -EINTR; error = security_path_chown(path, uid, gid); if (!error) error = notify_change(path->dentry, &newattrs, &delegated_inode); -- 2.1.4