From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: [PATCH 1/1] vfs: umount_begin BKL pushdown v2 Date: Fri, 24 Apr 2009 08:18:53 +0100 Message-ID: <20090424071853.GG8633@ZenIV.linux.org.uk> References: <20090423191934.GW1926@parisc-linux.org> <1240556813-8739-1-git-send-email-abogani@texware.it> <1240556813-8739-2-git-send-email-abogani@texware.it> <20090424071312.GE8633@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jonathan Corbet , Fr??d??ric Weisbecker , Peter Zijlstra , LKML , LFSDEV , Ingo Molnar , Matthew Wilcox To: Alessio Igor Bogani Return-path: Received: from zeniv.linux.org.uk ([195.92.253.2]:46835 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750799AbZDXHS4 (ORCPT ); Fri, 24 Apr 2009 03:18:56 -0400 Content-Disposition: inline In-Reply-To: <20090424071312.GE8633@ZenIV.linux.org.uk> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Fri, Apr 24, 2009 at 08:13:12AM +0100, Al Viro wrote: > AFAICS, both CIFS_SB(sb) and ->tcon are assign-once, so lock_kernel() should > really go here (if it can't be removed completely, of course, but that's up > to CIFS folks). Applied with such modification. commit 208f6be8f9244f4a3e8de7b4c6ca97069698303a in git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6.git/ if you want to see the version after that change (or wait for linux-next to pick it).