1
linux/fs/ocfs2
Christoph Lameter eebd2aa355 Pagecache zeroing: zero_user_segment, zero_user_segments and zero_user
Simplify page cache zeroing of segments of pages through 3 functions

zero_user_segments(page, start1, end1, start2, end2)

        Zeros two segments of the page. It takes the position where to
        start and end the zeroing which avoids length calculations and
	makes code clearer.

zero_user_segment(page, start, end)

        Same for a single segment.

zero_user(page, start, length)

        Length variant for the case where we know the length.

We remove the zero_user_page macro. Issues:

1. Its a macro. Inline functions are preferable.

2. The KM_USER0 macro is only defined for HIGHMEM.

   Having to treat this special case everywhere makes the
   code needlessly complex. The parameter for zeroing is always
   KM_USER0 except in one single case that we open code.

Avoiding KM_USER0 makes a lot of code not having to be dealing
with the special casing for HIGHMEM anymore. Dealing with
kmap is only necessary for HIGHMEM configurations. In those
configurations we use KM_USER0 like we do for a series of other
functions defined in highmem.h.

Since KM_USER0 is depends on HIGHMEM the existing zero_user_page
function could not be a macro. zero_user_* functions introduced
here can be be inline because that constant is not used when these
functions are called.

Also extract the flushing of the caches to be outside of the kmap.

[akpm@linux-foundation.org: fix nfs and ntfs build]
[akpm@linux-foundation.org: fix ntfs build some more]
Signed-off-by: Christoph Lameter <clameter@sgi.com>
Cc: Steven French <sfrench@us.ibm.com>
Cc: Michael Halcrow <mhalcrow@us.ibm.com>
Cc: <linux-ext4@vger.kernel.org>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
Cc: "J. Bruce Fields" <bfields@fieldses.org>
Cc: Anton Altaparmakov <aia21@cantab.net>
Cc: Mark Fasheh <mark.fasheh@oracle.com>
Cc: David Chinner <dgc@sgi.com>
Cc: Michael Halcrow <mhalcrow@us.ibm.com>
Cc: Steven French <sfrench@us.ibm.com>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2008-02-05 09:44:13 -08:00
..
cluster ocfs2: Fix userspace ABI breakage in sysfs 2008-01-28 19:10:23 -08:00
dlm ocfs2: bump version number 2008-01-25 15:05:46 -08:00
alloc.c Pagecache zeroing: zero_user_segment, zero_user_segments and zero_user 2008-02-05 09:44:13 -08:00
alloc.h
aops.c Pagecache zeroing: zero_user_segment, zero_user_segments and zero_user 2008-02-05 09:44:13 -08:00
aops.h
buffer_head_io.c ocfs2: clean up bh null checks 2008-01-25 15:05:48 -08:00
buffer_head_io.h [PATCH 1/2] ocfs2: Add group extend for online resize 2008-01-25 14:53:35 -08:00
dcache.c ocfs2: Remove mount/unmount votes 2008-01-25 14:45:34 -08:00
dcache.h
dir.c fs/: Spelling fixes 2008-02-03 17:33:42 +02:00
dir.h
dlmglue.c [PATCH 1/2] ocfs2: add flock lock type 2008-01-25 15:05:43 -08:00
dlmglue.h [PATCH 1/2] ocfs2: add flock lock type 2008-01-25 15:05:43 -08:00
endian.h ocfs2: convert byte order of constant instead of variable 2008-01-25 15:05:46 -08:00
export.c ocfs2: Silence false lockdep warnings 2008-01-25 15:05:44 -08:00
export.h
extent_map.c
extent_map.h
file.c ocfs2: printf fixes 2008-01-25 15:05:45 -08:00
file.h [PATCH 1/2] ocfs2: add flock lock type 2008-01-25 15:05:43 -08:00
heartbeat.c ocfs2: Remove mount/unmount votes 2008-01-25 14:45:34 -08:00
heartbeat.h ocfs2: Remove fs dependency on ocfs2_heartbeat module 2008-01-25 14:36:40 -08:00
inode.c ocfs2: convert byte order of constant instead of variable 2008-01-25 15:05:46 -08:00
inode.h ocfs2: Silence false lockdep warnings 2008-01-25 15:05:44 -08:00
ioctl.c ocfs2: Add missing permission checks 2008-01-25 15:05:19 -08:00
ioctl.h
journal.c ocfs2: Silence false lockdep warnings 2008-01-25 15:05:44 -08:00
journal.h [PATCH 2/2] ocfs2: Implement group add for online resize 2008-01-25 15:04:24 -08:00
localalloc.c ocfs2: Local alloc window size changeable via mount option 2008-01-25 15:05:43 -08:00
localalloc.h
locks.c [PATCH 2/2] ocfs2: cluster aware flock() 2008-01-25 15:05:43 -08:00
locks.h [PATCH 2/2] ocfs2: cluster aware flock() 2008-01-25 15:05:43 -08:00
Makefile [PATCH 2/2] ocfs2: cluster aware flock() 2008-01-25 15:05:43 -08:00
mmap.c ocfs2: Rename ocfs2_meta_[un]lock 2008-01-25 14:46:01 -08:00
mmap.h
namei.c ocfs2: Silence false lockdep warnings 2008-01-25 15:05:44 -08:00
namei.h
ocfs1_fs_compat.h fs/: Spelling fixes 2008-02-03 17:33:42 +02:00
ocfs2_fs.h ocfs2: Local alloc window size changeable via mount option 2008-01-25 15:05:43 -08:00
ocfs2_lockid.h [PATCH 1/2] ocfs2: add flock lock type 2008-01-25 15:05:43 -08:00
ocfs2.h ocfs2: document access rules for blocked_lock_list 2008-01-25 15:05:48 -08:00
resize.c ocfs2: clean up bh null checks 2008-01-25 15:05:48 -08:00
resize.h [PATCH 2/2] ocfs2: Implement group add for online resize 2008-01-25 15:04:24 -08:00
slot_map.c ocfs2: Remove mount/unmount votes 2008-01-25 14:45:34 -08:00
slot_map.h ocfs2: Remove mount/unmount votes 2008-01-25 14:45:34 -08:00
suballoc.c fs/: Spelling fixes 2008-02-03 17:33:42 +02:00
suballoc.h [PATCH 1/2] ocfs2: Add group extend for online resize 2008-01-25 14:53:35 -08:00
super.c ocfs2: Silence false lockdep warnings 2008-01-25 15:05:44 -08:00
super.h
symlink.c
symlink.h
sysfile.c ocfs2: Silence false lockdep warnings 2008-01-25 15:05:44 -08:00
sysfile.h
uptodate.c
uptodate.h
ver.c ocfs2: bump version number 2008-01-25 15:05:46 -08:00
ver.h