From mboxrd@z Thu Jan  1 00:00:00 1970
From: Christoph Hellwig <hch@lst.de>
Subject: Re: [PATCH] remove s_async_list
Date: Tue, 28 Apr 2009 18:05:55 +0200
Message-ID: <20090428160555.GA26919@lst.de>
References: <20090428155727.GA26363@lst.de> <20090428160221.GE5716@duck.suse.cz>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Christoph Hellwig <hch@lst.de>, Al Viro <viro@ZenIV.linux.org.uk>,
	linux-fsdevel@vger.kernel.org
To: Jan Kara <jack@suse.cz>
Return-path: <linux-fsdevel-owner@vger.kernel.org>
Received: from verein.lst.de ([213.95.11.210]:52058 "EHLO verein.lst.de"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1754657AbZD1QF5 (ORCPT <rfc822;linux-fsdevel@vger.kernel.org>);
	Tue, 28 Apr 2009 12:05:57 -0400
Content-Disposition: inline
In-Reply-To: <20090428160221.GE5716@duck.suse.cz>
Sender: linux-fsdevel-owner@vger.kernel.org
List-ID: <linux-fsdevel.vger.kernel.org>

On Tue, Apr 28, 2009 at 06:02:21PM +0200, Jan Kara wrote:
> On Tue 28-04-09 17:57:27, Christoph Hellwig wrote:
> > Remove the unused s_async_list in the superblock, a leftover of the
> > broken async inode deletion code that leaked into mainline.  Having this
> > in the middle of the sync/unmount path is not helpful for the following
> > cleanups.
>   I was wondering what it's used for when cleaning up the sync path ;)
> BTW, fs/sync.c also includes <linux/async.h> which can be removed now,
> doesn't it?

Yeah.  Updated version below:


Signed-off-by: Christoph Hellwig <hch@lst.de>

Index: linux-2.6/fs/super.c
===================================================================
--- linux-2.6.orig/fs/super.c	2009-04-28 17:36:09.582631160 +0200
+++ linux-2.6/fs/super.c	2009-04-28 18:04:12.238637421 +0200
@@ -37,7 +37,6 @@
 #include <linux/kobject.h>
 #include <linux/mutex.h>
 #include <linux/file.h>
-#include <linux/async.h>
 #include <asm/uaccess.h>
 #include "internal.h"
 
@@ -71,7 +70,6 @@ static struct super_block *alloc_super(s
 		INIT_HLIST_HEAD(&s->s_anon);
 		INIT_LIST_HEAD(&s->s_inodes);
 		INIT_LIST_HEAD(&s->s_dentry_lru);
-		INIT_LIST_HEAD(&s->s_async_list);
 		init_rwsem(&s->s_umount);
 		mutex_init(&s->s_lock);
 		lockdep_set_class(&s->s_umount, &type->s_umount_key);
@@ -281,11 +279,6 @@ void generic_shutdown_super(struct super
 		lock_super(sb);
 		sb->s_flags &= ~MS_ACTIVE;
 
-		/*
-		 * wait for asynchronous fs operations to finish before going further
-		 */
-		async_synchronize_full_domain(&sb->s_async_list);
-
 		/* bad name - it should be evict_inodes() */
 		invalidate_inodes(sb);
 		lock_kernel();
Index: linux-2.6/include/linux/fs.h
===================================================================
--- linux-2.6.orig/include/linux/fs.h	2009-04-28 17:36:09.586630482 +0200
+++ linux-2.6/include/linux/fs.h	2009-04-28 17:36:14.261638117 +0200
@@ -1372,11 +1372,6 @@ struct super_block {
 	 * generic_show_options()
 	 */
 	char *s_options;
-
-	/*
-	 * storage for asynchronous operations
-	 */
-	struct list_head s_async_list;
 };
 
 extern struct timespec current_fs_time(struct super_block *sb);
Index: linux-2.6/fs/sync.c
===================================================================
--- linux-2.6.orig/fs/sync.c	2009-04-28 17:36:37.593482035 +0200
+++ linux-2.6/fs/sync.c	2009-04-28 18:04:06.045133419 +0200
@@ -12,7 +12,6 @@
 #include <linux/linkage.h>
 #include <linux/pagemap.h>
 #include <linux/quotaops.h>
-#include <linux/async.h>
 
 #define VALID_FLAGS (SYNC_FILE_RANGE_WAIT_BEFORE|SYNC_FILE_RANGE_WRITE| \
 			SYNC_FILE_RANGE_WAIT_AFTER)
@@ -94,7 +93,6 @@ restart:
 		sb->s_count++;
 		spin_unlock(&sb_lock);
 		down_read(&sb->s_umount);
-		async_synchronize_full_domain(&sb->s_async_list);
 		if (sb->s_root)
 			__sync_filesystem(sb, wait);
 		up_read(&sb->s_umount);