Commit 2f137e31 authored by Christoph Hellwig's avatar Christoph Hellwig

freevxfs: implement ->alloc_inode and ->destroy_inode

This driver predates those methods and was trying to be clever allocating
it's own private data.  Switch to the generic scheme used by other file
systems.

Based on an earlier patch from Krzysztof Błaszkowski <kb@sysmikro.com.pl>.
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent f2bf2c70
...@@ -272,13 +272,6 @@ enum { ...@@ -272,13 +272,6 @@ enum {
#define VXFS_ISIMMED(ip) VXFS_IS_ORG((ip), VXFS_ORG_IMMED) #define VXFS_ISIMMED(ip) VXFS_IS_ORG((ip), VXFS_ORG_IMMED)
#define VXFS_ISTYPED(ip) VXFS_IS_ORG((ip), VXFS_ORG_TYPED) #define VXFS_ISTYPED(ip) VXFS_IS_ORG((ip), VXFS_ORG_TYPED)
/*
* Get filesystem private data from VFS inode.
*/
#define VXFS_INO(ip) \
((struct vxfs_inode_info *)(ip)->i_private)
/* /*
* Get filesystem private data from VFS superblock. * Get filesystem private data from VFS superblock.
*/ */
......
...@@ -52,7 +52,6 @@ extern int vxfs_read_fshead(struct super_block *); ...@@ -52,7 +52,6 @@ extern int vxfs_read_fshead(struct super_block *);
/* vxfs_inode.c */ /* vxfs_inode.c */
extern const struct address_space_operations vxfs_immed_aops; extern const struct address_space_operations vxfs_immed_aops;
extern struct kmem_cache *vxfs_inode_cachep;
extern void vxfs_dumpi(struct vxfs_inode_info *, ino_t); extern void vxfs_dumpi(struct vxfs_inode_info *, ino_t);
extern struct inode *vxfs_blkiget(struct super_block *, u_long, ino_t); extern struct inode *vxfs_blkiget(struct super_block *, u_long, ino_t);
extern struct inode *vxfs_stiget(struct super_block *, ino_t); extern struct inode *vxfs_stiget(struct super_block *, ino_t);
......
...@@ -42,11 +42,6 @@ ...@@ -42,11 +42,6 @@
#include "vxfs_extern.h" #include "vxfs_extern.h"
struct kmem_cache *vxfs_inode_cachep;
static struct inode * vxfs_get_fake_inode(struct super_block *,
struct vxfs_inode_info *);
#ifdef DIAGNOSTIC #ifdef DIAGNOSTIC
/* /*
* Dump inode contents (partially). * Dump inode contents (partially).
...@@ -70,9 +65,42 @@ vxfs_dumpi(struct vxfs_inode_info *vip, ino_t ino) ...@@ -70,9 +65,42 @@ vxfs_dumpi(struct vxfs_inode_info *vip, ino_t ino)
} }
#endif #endif
/**
* vxfs_transmod - mode for a VxFS inode
* @vip: VxFS inode
*
* Description:
* vxfs_transmod returns a Linux mode_t for a given
* VxFS inode structure.
*/
static __inline__ umode_t
vxfs_transmod(struct vxfs_inode_info *vip)
{
umode_t ret = vip->vii_mode & ~VXFS_TYPE_MASK;
if (VXFS_ISFIFO(vip))
ret |= S_IFIFO;
if (VXFS_ISCHR(vip))
ret |= S_IFCHR;
if (VXFS_ISDIR(vip))
ret |= S_IFDIR;
if (VXFS_ISBLK(vip))
ret |= S_IFBLK;
if (VXFS_ISLNK(vip))
ret |= S_IFLNK;
if (VXFS_ISREG(vip))
ret |= S_IFREG;
if (VXFS_ISSOC(vip))
ret |= S_IFSOCK;
return (ret);
}
static inline void dip2vip_cpy(struct vxfs_sb_info *sbi, static inline void dip2vip_cpy(struct vxfs_sb_info *sbi,
struct vxfs_inode_info *vip, struct vxfs_dinode *dip) struct vxfs_inode_info *vip, struct vxfs_dinode *dip)
{ {
struct inode *inode = &vip->vfs_inode;
vip->vii_mode = fs32_to_cpu(sbi, dip->vdi_mode); vip->vii_mode = fs32_to_cpu(sbi, dip->vdi_mode);
vip->vii_nlink = fs32_to_cpu(sbi, dip->vdi_nlink); vip->vii_nlink = fs32_to_cpu(sbi, dip->vdi_nlink);
vip->vii_uid = fs32_to_cpu(sbi, dip->vdi_uid); vip->vii_uid = fs32_to_cpu(sbi, dip->vdi_uid);
...@@ -96,6 +124,23 @@ static inline void dip2vip_cpy(struct vxfs_sb_info *sbi, ...@@ -96,6 +124,23 @@ static inline void dip2vip_cpy(struct vxfs_sb_info *sbi,
/* don't endian swap the fields that differ by orgtype */ /* don't endian swap the fields that differ by orgtype */
memcpy(&vip->vii_org, &dip->vdi_org, sizeof(vip->vii_org)); memcpy(&vip->vii_org, &dip->vdi_org, sizeof(vip->vii_org));
inode->i_mode = vxfs_transmod(vip);
i_uid_write(inode, (uid_t)vip->vii_uid);
i_gid_write(inode, (gid_t)vip->vii_gid);
set_nlink(inode, vip->vii_nlink);
inode->i_size = vip->vii_size;
inode->i_atime.tv_sec = vip->vii_atime;
inode->i_ctime.tv_sec = vip->vii_ctime;
inode->i_mtime.tv_sec = vip->vii_mtime;
inode->i_atime.tv_nsec = 0;
inode->i_ctime.tv_nsec = 0;
inode->i_mtime.tv_nsec = 0;
inode->i_blocks = vip->vii_blocks;
inode->i_generation = vip->vii_gen;
} }
/** /**
...@@ -121,48 +166,48 @@ vxfs_blkiget(struct super_block *sbp, u_long extent, ino_t ino) ...@@ -121,48 +166,48 @@ vxfs_blkiget(struct super_block *sbp, u_long extent, ino_t ino)
struct inode *inode; struct inode *inode;
u_long block, offset; u_long block, offset;
inode = new_inode(sbp);
if (!inode)
return NULL;
inode->i_ino = get_next_ino();
block = extent + ((ino * VXFS_ISIZE) / sbp->s_blocksize); block = extent + ((ino * VXFS_ISIZE) / sbp->s_blocksize);
offset = ((ino % (sbp->s_blocksize / VXFS_ISIZE)) * VXFS_ISIZE); offset = ((ino % (sbp->s_blocksize / VXFS_ISIZE)) * VXFS_ISIZE);
bp = sb_bread(sbp, block); bp = sb_bread(sbp, block);
if (bp && buffer_mapped(bp)) { if (bp && buffer_mapped(bp)) {
struct vxfs_inode_info *vip; struct vxfs_inode_info *vip = VXFS_INO(inode);
struct vxfs_dinode *dip; struct vxfs_dinode *dip;
if (!(vip = kmem_cache_alloc(vxfs_inode_cachep, GFP_KERNEL)))
goto fail;
dip = (struct vxfs_dinode *)(bp->b_data + offset); dip = (struct vxfs_dinode *)(bp->b_data + offset);
dip2vip_cpy(VXFS_SBI(sbp), vip, dip); dip2vip_cpy(VXFS_SBI(sbp), vip, dip);
vip->vfs_inode.i_mapping->a_ops = &vxfs_aops;
#ifdef DIAGNOSTIC #ifdef DIAGNOSTIC
vxfs_dumpi(vip, ino); vxfs_dumpi(vip, ino);
#endif #endif
brelse(bp); brelse(bp);
inode = vxfs_get_fake_inode(sbp, vip);
if (!inode)
kmem_cache_free(vxfs_inode_cachep, vip);
return inode; return inode;
} }
fail:
printk(KERN_WARNING "vxfs: unable to read block %ld\n", block); printk(KERN_WARNING "vxfs: unable to read block %ld\n", block);
brelse(bp); brelse(bp);
iput(inode);
return NULL; return NULL;
} }
/** /**
* __vxfs_iget - generic find inode facility * __vxfs_iget - generic find inode facility
* @sbp: VFS superblock
* @ino: inode number
* @ilistp: inode list * @ilistp: inode list
* @vip: VxFS inode to fill in
* @ino: inode number
* *
* Description: * Description:
* Search the for inode number @ino in the filesystem * Search the for inode number @ino in the filesystem
* described by @sbp. Use the specified inode table (@ilistp). * described by @sbp. Use the specified inode table (@ilistp).
* Returns the matching inode on success, else an error code. * Returns the matching inode on success, else an error code.
*/ */
static struct vxfs_inode_info * static int
__vxfs_iget(ino_t ino, struct inode *ilistp) __vxfs_iget(struct inode *ilistp, struct vxfs_inode_info *vip, ino_t ino)
{ {
struct page *pp; struct page *pp;
u_long offset; u_long offset;
...@@ -171,28 +216,22 @@ __vxfs_iget(ino_t ino, struct inode *ilistp) ...@@ -171,28 +216,22 @@ __vxfs_iget(ino_t ino, struct inode *ilistp)
pp = vxfs_get_page(ilistp->i_mapping, ino * VXFS_ISIZE / PAGE_SIZE); pp = vxfs_get_page(ilistp->i_mapping, ino * VXFS_ISIZE / PAGE_SIZE);
if (!IS_ERR(pp)) { if (!IS_ERR(pp)) {
struct vxfs_inode_info *vip;
struct vxfs_dinode *dip; struct vxfs_dinode *dip;
caddr_t kaddr = (char *)page_address(pp); caddr_t kaddr = (char *)page_address(pp);
if (!(vip = kmem_cache_alloc(vxfs_inode_cachep, GFP_KERNEL)))
goto fail;
dip = (struct vxfs_dinode *)(kaddr + offset); dip = (struct vxfs_dinode *)(kaddr + offset);
dip2vip_cpy(VXFS_SBI(ilistp->i_sb), vip, dip); dip2vip_cpy(VXFS_SBI(ilistp->i_sb), vip, dip);
vip->vfs_inode.i_mapping->a_ops = &vxfs_aops;
#ifdef DIAGNOSTIC #ifdef DIAGNOSTIC
vxfs_dumpi(vip, ino); vxfs_dumpi(vip, ino);
#endif #endif
vxfs_put_page(pp); vxfs_put_page(pp);
return (vip); return 0;
} }
printk(KERN_WARNING "vxfs: error on page %p\n", pp); printk(KERN_WARNING "vxfs: error on page 0x%p for inode %ld\n",
return ERR_CAST(pp); pp, (unsigned long)ino);
return PTR_ERR(pp);
fail:
printk(KERN_WARNING "vxfs: unable to read inode %ld\n", (unsigned long)ino);
vxfs_put_page(pp);
return ERR_PTR(-ENOMEM);
} }
/** /**
...@@ -208,104 +247,21 @@ __vxfs_iget(ino_t ino, struct inode *ilistp) ...@@ -208,104 +247,21 @@ __vxfs_iget(ino_t ino, struct inode *ilistp)
struct inode * struct inode *
vxfs_stiget(struct super_block *sbp, ino_t ino) vxfs_stiget(struct super_block *sbp, ino_t ino)
{ {
struct vxfs_inode_info *vip;
struct inode *inode; struct inode *inode;
int error;
vip = __vxfs_iget(ino, VXFS_SBI(sbp)->vsi_stilist); inode = new_inode(sbp);
if (IS_ERR(vip))
return NULL;
inode = vxfs_get_fake_inode(sbp, vip);
if (!inode) if (!inode)
kmem_cache_free(vxfs_inode_cachep, vip); return NULL;
return inode; inode->i_ino = get_next_ino();
}
/**
* vxfs_transmod - mode for a VxFS inode
* @vip: VxFS inode
*
* Description:
* vxfs_transmod returns a Linux mode_t for a given
* VxFS inode structure.
*/
static __inline__ umode_t
vxfs_transmod(struct vxfs_inode_info *vip)
{
umode_t ret = vip->vii_mode & ~VXFS_TYPE_MASK;
if (VXFS_ISFIFO(vip))
ret |= S_IFIFO;
if (VXFS_ISCHR(vip))
ret |= S_IFCHR;
if (VXFS_ISDIR(vip))
ret |= S_IFDIR;
if (VXFS_ISBLK(vip))
ret |= S_IFBLK;
if (VXFS_ISLNK(vip))
ret |= S_IFLNK;
if (VXFS_ISREG(vip))
ret |= S_IFREG;
if (VXFS_ISSOC(vip))
ret |= S_IFSOCK;
return (ret);
}
/**
* vxfs_iinit- helper to fill inode fields
* @ip: VFS inode
* @vip: VxFS inode
*
* Description:
* vxfs_instino is a helper function to fill in all relevant
* fields in @ip from @vip.
*/
static void
vxfs_iinit(struct inode *ip, struct vxfs_inode_info *vip)
{
ip->i_mode = vxfs_transmod(vip);
i_uid_write(ip, (uid_t)vip->vii_uid);
i_gid_write(ip, (gid_t)vip->vii_gid);
set_nlink(ip, vip->vii_nlink);
ip->i_size = vip->vii_size;
ip->i_atime.tv_sec = vip->vii_atime;
ip->i_ctime.tv_sec = vip->vii_ctime;
ip->i_mtime.tv_sec = vip->vii_mtime;
ip->i_atime.tv_nsec = 0;
ip->i_ctime.tv_nsec = 0;
ip->i_mtime.tv_nsec = 0;
ip->i_blocks = vip->vii_blocks;
ip->i_generation = vip->vii_gen;
ip->i_private = vip;
}
/**
* vxfs_get_fake_inode - get fake inode structure
* @sbp: filesystem superblock
* @vip: fspriv inode
*
* Description:
* vxfs_fake_inode gets a fake inode (not in the inode hash) for a
* superblock, vxfs_inode pair.
* Returns the filled VFS inode.
*/
static struct inode *
vxfs_get_fake_inode(struct super_block *sbp, struct vxfs_inode_info *vip)
{
struct inode *ip = NULL;
if ((ip = new_inode(sbp))) { error = __vxfs_iget(VXFS_SBI(sbp)->vsi_stilist, VXFS_INO(inode), ino);
ip->i_ino = get_next_ino(); if (error) {
vxfs_iinit(ip, vip); iput(inode);
ip->i_mapping->a_ops = &vxfs_aops; return NULL;
} }
return (ip);
return inode;
} }
/** /**
...@@ -323,6 +279,7 @@ vxfs_iget(struct super_block *sbp, ino_t ino) ...@@ -323,6 +279,7 @@ vxfs_iget(struct super_block *sbp, ino_t ino)
struct vxfs_inode_info *vip; struct vxfs_inode_info *vip;
const struct address_space_operations *aops; const struct address_space_operations *aops;
struct inode *ip; struct inode *ip;
int error;
ip = iget_locked(sbp, ino); ip = iget_locked(sbp, ino);
if (!ip) if (!ip)
...@@ -330,14 +287,13 @@ vxfs_iget(struct super_block *sbp, ino_t ino) ...@@ -330,14 +287,13 @@ vxfs_iget(struct super_block *sbp, ino_t ino)
if (!(ip->i_state & I_NEW)) if (!(ip->i_state & I_NEW))
return ip; return ip;
vip = __vxfs_iget(ino, VXFS_SBI(sbp)->vsi_ilist); vip = VXFS_INO(ip);
if (IS_ERR(vip)) { error = __vxfs_iget(VXFS_SBI(sbp)->vsi_ilist, vip, ino);
if (error) {
iget_failed(ip); iget_failed(ip);
return ERR_CAST(vip); return ERR_PTR(error);
} }
vxfs_iinit(ip, vip);
if (VXFS_ISIMMED(vip)) if (VXFS_ISIMMED(vip))
aops = &vxfs_immed_aops; aops = &vxfs_immed_aops;
else else
...@@ -368,12 +324,6 @@ vxfs_iget(struct super_block *sbp, ino_t ino) ...@@ -368,12 +324,6 @@ vxfs_iget(struct super_block *sbp, ino_t ino)
return ip; return ip;
} }
static void vxfs_i_callback(struct rcu_head *head)
{
struct inode *inode = container_of(head, struct inode, i_rcu);
kmem_cache_free(vxfs_inode_cachep, inode->i_private);
}
/** /**
* vxfs_evict_inode - remove inode from main memory * vxfs_evict_inode - remove inode from main memory
* @ip: inode to discard. * @ip: inode to discard.
...@@ -387,5 +337,4 @@ vxfs_evict_inode(struct inode *ip) ...@@ -387,5 +337,4 @@ vxfs_evict_inode(struct inode *ip)
{ {
truncate_inode_pages_final(&ip->i_data); truncate_inode_pages_final(&ip->i_data);
clear_inode(ip); clear_inode(ip);
call_rcu(&ip->i_rcu, vxfs_i_callback);
} }
...@@ -151,6 +151,8 @@ struct vxfs_dinode { ...@@ -151,6 +151,8 @@ struct vxfs_dinode {
* The inode as represented in the main memory. * The inode as represented in the main memory.
*/ */
struct vxfs_inode_info { struct vxfs_inode_info {
struct inode vfs_inode;
__u32 vii_mode; __u32 vii_mode;
__u32 vii_nlink; /* Link count */ __u32 vii_nlink; /* Link count */
__u32 vii_uid; /* UID */ __u32 vii_uid; /* UID */
...@@ -183,4 +185,9 @@ struct vxfs_inode_info { ...@@ -183,4 +185,9 @@ struct vxfs_inode_info {
#define vii_ext4 vii_org.ext4 #define vii_ext4 vii_org.ext4
#define vii_typed vii_org.typed #define vii_typed vii_org.typed
static inline struct vxfs_inode_info *VXFS_INO(struct inode *inode)
{
return container_of(inode, struct vxfs_inode_info, vfs_inode);
}
#endif /* _VXFS_INODE_H_ */ #endif /* _VXFS_INODE_H_ */
...@@ -52,6 +52,8 @@ MODULE_AUTHOR("Christoph Hellwig"); ...@@ -52,6 +52,8 @@ MODULE_AUTHOR("Christoph Hellwig");
MODULE_DESCRIPTION("Veritas Filesystem (VxFS) driver"); MODULE_DESCRIPTION("Veritas Filesystem (VxFS) driver");
MODULE_LICENSE("Dual BSD/GPL"); MODULE_LICENSE("Dual BSD/GPL");
static struct kmem_cache *vxfs_inode_cachep;
/** /**
* vxfs_put_super - free superblock resources * vxfs_put_super - free superblock resources
* @sbp: VFS superblock. * @sbp: VFS superblock.
...@@ -117,7 +119,31 @@ static int vxfs_remount(struct super_block *sb, int *flags, char *data) ...@@ -117,7 +119,31 @@ static int vxfs_remount(struct super_block *sb, int *flags, char *data)
return 0; return 0;
} }
static struct inode *vxfs_alloc_inode(struct super_block *sb)
{
struct vxfs_inode_info *vi;
vi = kmem_cache_alloc(vxfs_inode_cachep, GFP_KERNEL);
if (!vi)
return NULL;
return &vi->vfs_inode;
}
static void vxfs_i_callback(struct rcu_head *head)
{
struct inode *inode = container_of(head, struct inode, i_rcu);
kmem_cache_free(vxfs_inode_cachep, VXFS_INO(inode));
}
static void vxfs_destroy_inode(struct inode *inode)
{
call_rcu(&inode->i_rcu, vxfs_i_callback);
}
static const struct super_operations vxfs_super_ops = { static const struct super_operations vxfs_super_ops = {
.alloc_inode = vxfs_alloc_inode,
.destroy_inode = vxfs_destroy_inode,
.evict_inode = vxfs_evict_inode, .evict_inode = vxfs_evict_inode,
.put_super = vxfs_put_super, .put_super = vxfs_put_super,
.statfs = vxfs_statfs, .statfs = vxfs_statfs,
...@@ -206,6 +232,7 @@ static int vxfs_fill_super(struct super_block *sbp, void *dp, int silent) ...@@ -206,6 +232,7 @@ static int vxfs_fill_super(struct super_block *sbp, void *dp, int silent)
goto out; goto out;
} }
sbp->s_op = &vxfs_super_ops;
sbp->s_fs_info = infp; sbp->s_fs_info = infp;
if (!vxfs_try_sb_magic(sbp, silent, 1, if (!vxfs_try_sb_magic(sbp, silent, 1,
...@@ -256,7 +283,6 @@ static int vxfs_fill_super(struct super_block *sbp, void *dp, int silent) ...@@ -256,7 +283,6 @@ static int vxfs_fill_super(struct super_block *sbp, void *dp, int silent)
goto out; goto out;
} }
sbp->s_op = &vxfs_super_ops;
root = vxfs_iget(sbp, VXFS_ROOT_INO); root = vxfs_iget(sbp, VXFS_ROOT_INO);
if (IS_ERR(root)) { if (IS_ERR(root)) {
ret = PTR_ERR(root); ret = PTR_ERR(root);
......
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