From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Layton Subject: Re: [PATCH] fs: fix data races on inode->i_flctx Date: Mon, 19 Oct 2015 13:46:06 -0400 Message-ID: <1445276766.10328.0.camel@poochiereds.net> References: <1442821386-24807-1-git-send-email-dvyukov@google.com> <20150921065009.5767c196@synchrony.poochiereds.net> <20150921065653.39084dde@synchrony.poochiereds.net> <20150921071740.38d21093@synchrony.poochiereds.net> <20150921074409.20bf2576@synchrony.poochiereds.net> <1445273077.11141.10.camel@poochiereds.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Dmitry Vyukov , Bruce Fields , Al Viro , "linux-fsdevel@vger.kernel.org" , LKML , Kostya Serebryany , Andrey Konovalov , Alexander Potapenko , ktsan , Paul McKenney To: William Dauchy Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Mon, 2015-10-19 at 19:24 +0200, William Dauchy wrote: > On Mon, Oct 19, 2015 at 6:53 PM, William Dauchy > wrote: > > On Mon, Oct 19, 2015 at 6:44 PM, Jeff Layton < > > jlayton@poochiereds.net> wrote: > > > This should be fixed by this series of four commits that are > > > already in > > > mainline: > > > bcd7f78d078ff6197715c1ed070c92aca57ec12c..ee296d7c5709440f8abd36b > > > 5b65c6 > > > b3e388538d9 > > > > Am I missing something, I see three of them between > > bcd7f78d078ff6197715c1ed070c92aca57ec12c..ee296d7c5709440f8abd36b5b > > 65c6b3e388538d9 > > (and not four) > > > > ee296d7c5709440f8abd36b5b65c6b3e388538d9 locks: inline > > posix_lock_file_wait and flock_lock_file_wait > > 83bfff23e9ed19f37c4ef0bba84e75bd88e5cf21 nfs4: have do_vfs_lock > > take > > an inode pointer > > 29d01b22eaa18d8b46091d3c98c6001c49f78e4a locks: new helpers - > > flock_lock_inode_wait and posix_lock_inode_wait > > I suppose I found the missing one > bcd7f78 locks: have flock_lock_file take an inode pointer instead of > a filp > Ahh yes, sorry -- that one is also required. Thanks! -- Jeff Layton