[Ocfs2-devel] [PATCH] ocfs2: fix undefined struct variable in inode.h

Eric Ren zren at suse.com
Wed Sep 21 18:13:22 PDT 2016


Hi,

On 09/21/2016 09:38 AM, Joseph Qi wrote:
> The extern struct variable ocfs2_inode_cache is not defined. It meant to
> use ocfs2_inode_cachep defined in super.c, I think. Fortunately it is
> not used anywhere now, so no impact actually. Clean it up to fix this
> mistake.
>
> Signed-off-by: Joseph Qi <joseph.qi at huawei.com>

LGTM
Reviewed-by: Eric Ren<zren at suse.com>

> ---
>   fs/ocfs2/inode.h | 2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/fs/ocfs2/inode.h b/fs/ocfs2/inode.h
> index 50cc550..5af68fc 100644
> --- a/fs/ocfs2/inode.h
> +++ b/fs/ocfs2/inode.h
> @@ -123,8 +123,6 @@ static inline struct ocfs2_inode_info *OCFS2_I(struct inode *inode)
>   #define INODE_JOURNAL(i) (OCFS2_I(i)->ip_flags & OCFS2_INODE_JOURNAL)
>   #define SET_INODE_JOURNAL(i) (OCFS2_I(i)->ip_flags |= OCFS2_INODE_JOURNAL)
>
> -extern struct kmem_cache *ocfs2_inode_cache;
> -
>   extern const struct address_space_operations ocfs2_aops;
>   extern const struct ocfs2_caching_operations ocfs2_inode_caching_ops;
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://oss.oracle.com/pipermail/ocfs2-devel/attachments/20160922/d0acd4f0/attachment.html 


More information about the Ocfs2-devel mailing list