<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Hi,<br>
      <br>
      On 09/21/2016 09:38 AM, Joseph Qi wrote:<br>
    </div>
    <blockquote cite="mid:57E1E49D.8050503@huawei.com" type="cite">
      <pre wrap="">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 <a class="moz-txt-link-rfc2396E" href="mailto:joseph.qi@huawei.com">&lt;joseph.qi@huawei.com&gt;</a></pre>
    </blockquote>
    <pre wrap="">LGTM
Reviewed-by: Eric Ren <a class="moz-txt-link-rfc2396E" href="mailto:zren@suse.com">&lt;zren@suse.com&gt;</a></pre>
    <blockquote cite="mid:57E1E49D.8050503@huawei.com" type="cite">
      <pre wrap="">
---
 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)-&gt;ip_flags &amp; OCFS2_INODE_JOURNAL)
 #define SET_INODE_JOURNAL(i) (OCFS2_I(i)-&gt;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;

</pre>
    </blockquote>
    <p><br>
    </p>
  </body>
</html>