Home » Mailing list archives » linux-fsdevel » 2007 » April » 11

[PATCH 8/17] jfs: use locking read_mapping_page

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Nate Diller <nate.diller@...>
To: Andrew Morton <akpm@...>, Alexander Viro <viro@...>, Christoph Hellwig <hch@...>, Roman Zippel <zippel@...>, Mikulas Patocka <mikulas@...>, David Woodhouse <dwmw2@...>, Dave Kleikamp <shaggy@...>, Anton Altaparmakov <aia21@...>, Evgeniy Dushistov <dushistov@...>
Cc: <linux-kernel@...>, <linux-fsdevel@...>, <reiserfs-dev@...>
Subject: [PATCH 8/17] jfs: use locking read_mapping_page
Date: Wednesday, April 11, 2007 - 10:49 pm

Use the new locking variant of read_mapping_page to avoid doing extra work.

Signed-off-by: Nate Diller <nate.diller@gmail.com>

---

diff -urpN -X dontdiff linux-2.6.21-rc6-mm1/fs/jfs/jfs_metapage.c linux-2.6.21-rc6-mm1-test/fs/jfs/jfs_metapage.c
--- linux-2.6.21-rc6-mm1/fs/jfs/jfs_metapage.c	2007-04-09 17:23:48.000000000 -0700
+++ linux-2.6.21-rc6-mm1-test/fs/jfs/jfs_metapage.c	2007-04-09 21:37:09.000000000 -0700
@@ -632,12 +632,11 @@ struct metapage *__get_metapage(struct i
 		}
 		SetPageUptodate(page);
 	} else {
-		page = read_mapping_page(mapping, page_index, NULL);
-		if (IS_ERR(page) || !PageUptodate(page)) {
+		page = __read_mapping_page(mapping, page_index, NULL);
+		if (IS_ERR(page)) {
 			jfs_err("read_mapping_page failed!");
 			return NULL;
 		}
-		lock_page(page);
 	}
 
 	mp = page_to_mp(page, page_offset);
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 0/17] fs: cleanup single page synchronous read interf..., Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 6/17] hfs: remove redundant read_mapping_page error c..., Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 14/17] reiserfs: convert reiserfs_get_page to read_km..., Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 11/17] ntfs: convert ntfs_map_page to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 16/17] ufs: convert ufs_get_page to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 17/17] vxfs: convert vxfs_get_page to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 12/17] partition: remove redundant read_mapping_page ..., Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 13/17] reiser4: remove redundant read_mapping_page er..., Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 15/17] sysv: convert dir_get_page to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 10/17] mtd: convert page_read to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 9/17] minix: convert dir_get_page to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 8/17] jfs: use locking read_mapping_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 7/17] jffs2: convert jffs2_gc_fetch_page to read_cach..., Nate Diller, (Wed Apr 11, 10:49 pm)
Re: [PATCH 7/17] jffs2: convert jffs2_gc_fetch_page to read_..., David Woodhouse, (Sun Apr 15, 6:52 am)
Re: [PATCH 7/17] jffs2: convert jffs2_gc_fetch_page to read_..., Phillip Lougher, (Thu Apr 12, 7:40 am)
Re: [PATCH 7/17] jffs2: convert jffs2_gc_fetch_page to read_..., Nate Diller, (Thu Apr 12, 2:29 pm)
Re: [PATCH 7/17] jffs2: convert jffs2_gc_fetch_page to read_..., Phillip Lougher, (Thu Apr 12, 2:53 pm)
[PATCH 5/17] hfsplus: remove redundant read_mapping_page err..., Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
Re: [PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_..., David Howells, (Thu Apr 12, 6:58 am)
Re: [PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_..., Nate Diller, (Thu Apr 12, 2:23 pm)
Re: [PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_..., David Howells, (Thu Apr 12, 2:57 pm)
Re: [PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_..., Nate Diller, (Thu Apr 12, 3:27 pm)
Re: [PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_..., David Howells, (Thu Apr 12, 3:43 pm)
Re: [PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_..., Andrew Morton, (Thu Apr 12, 3:21 pm)
Re: [PATCH 3/17] afs: convert afs_dir_get_page to read_kmap_..., David Howells, (Thu Apr 12, 3:29 pm)
[PATCH 4/17] ext2: convert ext2_get_page to read_kmap_page, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 2/17] fs: introduce new read_cache_page interface, Nate Diller, (Wed Apr 11, 10:49 pm)
[PATCH 1/17] cramfs: use read_mapping_page, Nate Diller, (Wed Apr 11, 10:49 pm)
Re: [PATCH 1/17] cramfs: use read_mapping_page, Christoph Hellwig, (Thu Apr 12, 5:54 am)
Re: [PATCH 1/17] cramfs: use read_mapping_page, Roman Zippel, (Thu Apr 12, 7:26 am)
Re: [PATCH 1/17] cramfs: use read_mapping_page, Nate Diller, (Thu Apr 12, 2:36 pm)