Skip to content

Commit

Permalink
[PATCH] constify inode accessors
Browse files Browse the repository at this point in the history
Change the signature of i_size_read(), IMINOR() and IMAJOR() because they,
or the functions they call, will never modify the argument.

Signed-off-by: Jan Engelhardt <jengelh@gmx.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
  • Loading branch information
Jan Engelhardt authored and Linus Torvalds committed Dec 7, 2006
1 parent 89fc9a1 commit 48ed214
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions include/linux/fs.h
Original file line number Diff line number Diff line change
Expand Up @@ -636,7 +636,7 @@ extern void inode_double_unlock(struct inode *inode1, struct inode *inode2);
* cmpxchg8b without the need of the lock prefix). For SMP compiles
* and 64bit archs it makes no difference if preempt is enabled or not.
*/
static inline loff_t i_size_read(struct inode *inode)
static inline loff_t i_size_read(const struct inode *inode)
{
#if BITS_PER_LONG==32 && defined(CONFIG_SMP)
loff_t i_size;
Expand Down Expand Up @@ -679,12 +679,12 @@ static inline void i_size_write(struct inode *inode, loff_t i_size)
#endif
}

static inline unsigned iminor(struct inode *inode)
static inline unsigned iminor(const struct inode *inode)
{
return MINOR(inode->i_rdev);
}

static inline unsigned imajor(struct inode *inode)
static inline unsigned imajor(const struct inode *inode)
{
return MAJOR(inode->i_rdev);
}
Expand Down

0 comments on commit 48ed214

Please sign in to comment.