Hi Christoph, On Mon, Sep 02, 2019 at 05:10:21AM -0700, Christoph Hellwig wrote: > > { > > - struct erofs_vnode *vi = ptr; > > - > > - inode_init_once(&vi->vfs_inode); > > + inode_init_once(&((struct erofs_inode *)ptr)->vfs_inode); > > Why doesn't this use EROFS_I? This looks a little odd. Thanks for your reply and suggestion... EROFS_I seems the revert direction ---> inode to erofs_inode here we need "erofs_inode" to inode... Am I missing something?.... Hope not.... Thanks, Gao Xiang