diff options
author | Jeff Mahoney <jeffm@suse.com> | 2009-03-30 14:02:17 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-03-30 12:16:35 -0700 |
commit | 600ed41675d8c384519d8f0b3c76afed39ef2f4b (patch) | |
tree | 106f17dcaaee07671efdef651ff7f78b1afffb2f /include/linux/reiserfs_fs_i.h | |
parent | 702d21c6f6c790b12c4820cd2f29bc8472aed633 (diff) | |
download | op-kernel-dev-600ed41675d8c384519d8f0b3c76afed39ef2f4b.zip op-kernel-dev-600ed41675d8c384519d8f0b3c76afed39ef2f4b.tar.gz |
reiserfs: audit transaction ids to always be unsigned ints
This patch fixes up the reiserfs code such that transaction ids are
always unsigned ints. In places they can currently be signed ints or
unsigned longs.
The former just causes an annoying clm-2200 warning and may join a
transaction when it should wait.
The latter is just for correctness since the disk format uses a 32-bit
transaction id. There aren't any runtime problems that result from it
not wrapping at the correct location since the value is truncated
correctly even on big endian systems. The 0 value might make it to
disk, but the mount-time checks will bump it to 10 itself.
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux/reiserfs_fs_i.h')
-rw-r--r-- | include/linux/reiserfs_fs_i.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/reiserfs_fs_i.h b/include/linux/reiserfs_fs_i.h index ce3663f..201dd91 100644 --- a/include/linux/reiserfs_fs_i.h +++ b/include/linux/reiserfs_fs_i.h @@ -51,7 +51,7 @@ struct reiserfs_inode_info { /* we use these for fsync or O_SYNC to decide which transaction ** needs to be committed in order for this inode to be properly ** flushed */ - unsigned long i_trans_id; + unsigned int i_trans_id; struct reiserfs_journal_list *i_jl; struct mutex i_mmap; #ifdef CONFIG_REISERFS_FS_POSIX_ACL |