leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Convert simple loops over superblocks to list_for_each_entry_safe

Browse Source 
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
Al Viro 2010-03-22 20:09:33 -04:00
parent 8edd64bd60
commit 6754af6464
4 changed files with 11 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
fs/buffer.c
View File

@ -562,12 +562,11 @@ repeat:
static void do_thaw_all(struct work_struct *work) static void do_thaw_all(struct work_struct *work)
{ {
struct super_block *sb; struct super_block *sb, *n;
char b[BDEVNAME_SIZE]; char b[BDEVNAME_SIZE];
spin_lock(&sb_lock); spin_lock(&sb_lock);
restart: list_for_each_entry_safe(sb, n, &super_blocks, s_list) {
list_for_each_entry(sb, &super_blocks, s_list) {
if (list_empty(&sb->s_instances)) if (list_empty(&sb->s_instances))
continue; continue;
sb->s_count++; sb->s_count++;
@ -578,8 +577,6 @@ restart:
bdevname(sb->s_bdev, b)); bdevname(sb->s_bdev, b));
up_read(&sb->s_umount); up_read(&sb->s_umount);
spin_lock(&sb_lock); spin_lock(&sb_lock);
if (__put_super_and_need_restart(sb))
goto restart;
} }
spin_unlock(&sb_lock); spin_unlock(&sb_lock);
kfree(work); kfree(work);

8
fs/drop_caches.c
View File

@ -35,11 +35,10 @@ static void drop_pagecache_sb(struct super_block *sb)
static void drop_pagecache(void) static void drop_pagecache(void)
{ {
struct super_block *sb; struct super_block *sb, *n;
spin_lock(&sb_lock); spin_lock(&sb_lock);
restart: list_for_each_entry_safe(sb, n, &super_blocks, s_list) {
list_for_each_entry(sb, &super_blocks, s_list) {
if (list_empty(&sb->s_instances)) if (list_empty(&sb->s_instances))
continue; continue;
sb->s_count++; sb->s_count++;
@ -49,8 +48,7 @@ restart:
drop_pagecache_sb(sb); drop_pagecache_sb(sb);
up_read(&sb->s_umount); up_read(&sb->s_umount);
spin_lock(&sb_lock); spin_lock(&sb_lock);
if (__put_super_and_need_restart(sb)) __put_super(sb);
goto restart;
} }
spin_unlock(&sb_lock); spin_unlock(&sb_lock);
} }

8
fs/quota/quota.c
View File

@ -47,7 +47,7 @@ static int check_quotactl_permission(struct super_block *sb, int type, int cmd,
static int quota_sync_all(int type) static int quota_sync_all(int type)
{ {
struct super_block *sb; struct super_block *sb, *n;
int ret; int ret;
if (type >= MAXQUOTAS) if (type >= MAXQUOTAS)
@ -57,8 +57,7 @@ static int quota_sync_all(int type)
return ret; return ret;
spin_lock(&sb_lock); spin_lock(&sb_lock);
restart: list_for_each_entry_safe(sb, n, &super_blocks, s_list) {
list_for_each_entry(sb, &super_blocks, s_list) {
if (list_empty(&sb->s_instances)) if (list_empty(&sb->s_instances))
continue; continue;
if (!sb->s_qcop || !sb->s_qcop->quota_sync) if (!sb->s_qcop || !sb->s_qcop->quota_sync)
@ -71,8 +70,7 @@ restart:
sb->s_qcop->quota_sync(sb, type, 1); sb->s_qcop->quota_sync(sb, type, 1);
up_read(&sb->s_umount); up_read(&sb->s_umount);
spin_lock(&sb_lock); spin_lock(&sb_lock);
if (__put_super_and_need_restart(sb)) __put_super(sb);
goto restart;
} }
spin_unlock(&sb_lock); spin_unlock(&sb_lock);

8
fs/super.c
View File

@ -395,11 +395,10 @@ EXPORT_SYMBOL(drop_super);
*/ */
void sync_supers(void) void sync_supers(void)
{ {
struct super_block *sb; struct super_block *sb, *n;
spin_lock(&sb_lock); spin_lock(&sb_lock);
restart: list_for_each_entry_safe(sb, n, &super_blocks, s_list) {
list_for_each_entry(sb, &super_blocks, s_list) {
if (list_empty(&sb->s_instances)) if (list_empty(&sb->s_instances))
continue; continue;
if (sb->s_op->write_super && sb->s_dirt) { if (sb->s_op->write_super && sb->s_dirt) {
@ -412,8 +411,7 @@ restart:
up_read(&sb->s_umount); up_read(&sb->s_umount);
spin_lock(&sb_lock); spin_lock(&sb_lock);
if (__put_super_and_need_restart(sb)) __put_super(sb);
goto restart;
} }
} }
spin_unlock(&sb_lock); spin_unlock(&sb_lock);