Re: [PATCH 0/5] Remove possible deadlocks in nfs_release_page() - V3
On Wed, 24 Sep 2014 11:28:32 +1000 NeilBrown wrote: > This set includes acked-by's from Andrew and Peter so it should be > OK for all five patches to go upstream through the NFS tree. > > I split the congestion tracking patch out from the wait-for-PG_private > patch as they are conceptually separate. > > This set continues to perform well in my tests and addresses all > issues that have been raised. > > Thanks a lot, > NeilBrown > > > --- > > NeilBrown (5): > SCHED: add some "wait..on_bit...timeout()" interfaces. > MM: export page_wakeup functions > NFS: avoid deadlocks with loop-back mounted NFS filesystems. > NFS: avoid waiting at all in nfs_release_page when congested. > NFS/SUNRPC: Remove other deadlock-avoidance mechanisms in > nfs_release_page() > > > fs/nfs/file.c | 29 +++-- > fs/nfs/write.c |7 +++ > include/linux/pagemap.h | 12 ++-- > include/linux/wait.h|5 - > kernel/sched/wait.c | 36 > mm/filemap.c| 21 +++-- > net/sunrpc/sched.c |2 -- > net/sunrpc/xprtrdma/transport.c |2 -- > net/sunrpc/xprtsock.c | 10 -- > 9 files changed, 91 insertions(+), 33 deletions(-) > Cool! This looks like it'll address my earlier concern about setting the BDI congested inappropriately. You can add this to the set if you like: Acked-by: Jeff Layton -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/5] Remove possible deadlocks in nfs_release_page() - V3
On Wed, 24 Sep 2014 11:28:32 +1000 NeilBrown ne...@suse.de wrote: This set includes acked-by's from Andrew and Peter so it should be OK for all five patches to go upstream through the NFS tree. I split the congestion tracking patch out from the wait-for-PG_private patch as they are conceptually separate. This set continues to perform well in my tests and addresses all issues that have been raised. Thanks a lot, NeilBrown --- NeilBrown (5): SCHED: add some wait..on_bit...timeout() interfaces. MM: export page_wakeup functions NFS: avoid deadlocks with loop-back mounted NFS filesystems. NFS: avoid waiting at all in nfs_release_page when congested. NFS/SUNRPC: Remove other deadlock-avoidance mechanisms in nfs_release_page() fs/nfs/file.c | 29 +++-- fs/nfs/write.c |7 +++ include/linux/pagemap.h | 12 ++-- include/linux/wait.h|5 - kernel/sched/wait.c | 36 mm/filemap.c| 21 +++-- net/sunrpc/sched.c |2 -- net/sunrpc/xprtrdma/transport.c |2 -- net/sunrpc/xprtsock.c | 10 -- 9 files changed, 91 insertions(+), 33 deletions(-) Cool! This looks like it'll address my earlier concern about setting the BDI congested inappropriately. You can add this to the set if you like: Acked-by: Jeff Layton jlay...@primarydata.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/5] Remove possible deadlocks in nfs_release_page() - V3
On Tue, Sep 23, 2014 at 9:28 PM, NeilBrown wrote: > This set includes acked-by's from Andrew and Peter so it should be > OK for all five patches to go upstream through the NFS tree. > > I split the congestion tracking patch out from the wait-for-PG_private > patch as they are conceptually separate. > > This set continues to perform well in my tests and addresses all > issues that have been raised. > > Thanks a lot, > NeilBrown > Thanks Neil! I'll give them a final review tomorrow, and then queue them up for the 3.18 merge window. -- Trond Myklebust Linux NFS client maintainer, PrimaryData trond.mykleb...@primarydata.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 0/5] Remove possible deadlocks in nfs_release_page() - V3
On Tue, Sep 23, 2014 at 9:28 PM, NeilBrown ne...@suse.de wrote: This set includes acked-by's from Andrew and Peter so it should be OK for all five patches to go upstream through the NFS tree. I split the congestion tracking patch out from the wait-for-PG_private patch as they are conceptually separate. This set continues to perform well in my tests and addresses all issues that have been raised. Thanks a lot, NeilBrown Thanks Neil! I'll give them a final review tomorrow, and then queue them up for the 3.18 merge window. -- Trond Myklebust Linux NFS client maintainer, PrimaryData trond.mykleb...@primarydata.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
