Commit 48ed214d authored by Jan Engelhardt's avatar Jan Engelhardt Committed by Linus Torvalds

[PATCH] constify inode accessors

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: default avatarJan Engelhardt <jengelh@gmx.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 89fc9a1a
...@@ -636,7 +636,7 @@ extern void inode_double_unlock(struct inode *inode1, struct inode *inode2); ...@@ -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 * cmpxchg8b without the need of the lock prefix). For SMP compiles
* and 64bit archs it makes no difference if preempt is enabled or not. * 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) #if BITS_PER_LONG==32 && defined(CONFIG_SMP)
loff_t i_size; loff_t i_size;
...@@ -679,12 +679,12 @@ static inline void i_size_write(struct inode *inode, loff_t i_size) ...@@ -679,12 +679,12 @@ static inline void i_size_write(struct inode *inode, loff_t i_size)
#endif #endif
} }
static inline unsigned iminor(struct inode *inode) static inline unsigned iminor(const struct inode *inode)
{ {
return MINOR(inode->i_rdev); 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); return MAJOR(inode->i_rdev);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment