![]() | ||||||||||
Messages in this thread
|
Hi Josef,
The names of some of these macros are somewhat unreadable.
On 10/7/06, Josef Jeff Sipek <jsipek@cs.sunysb.edu> wrote:
> +/* Inode to private data */
> +static inline struct unionfs_inode_info *itopd(const struct inode *inode)
> +{
> + return
> + &(container_of(inode, struct unionfs_inode_container, vfs_inode)->
> + info);
> +}
Rename to unionfs_inode?
> +#define itohi_ptr(ino) (itopd(ino)->uii_inode)
unionfs_hidden_inodes ?
> +#define ibstart(ino) (itopd(ino)->b_start)
unionfs_inode_branch_start ?
> +#define ibend(ino) (itopd(ino)->b_end)
unionfs_inode_branch_end ?
> +#define stopd(super) ((struct unionfs_sb_info *)(super)->s_fs_info)
unionfs_sb_info ?
> +#define stopd_lhs(super) ((super)->s_fs_info)
This you should drop.
> +#define sbstart(sb) 0
unionfs_sb_branch_start?
> +#define sbend(sb) stopd(sb)->b_end
unionfs_sb_branch_end?
> +#define sbmax(sb) (stopd(sb)->b_end + 1)
unionfs_sb_branch_max?
> +#define ftopd(file) ((struct unionfs_file_info *)((file)->private_data))
unionfs_file_info ?
> +#define ftopd_lhs(file) ((file)->private_data)
This you should drop.
> +#define ftohf_ptr(file) (ftopd(file)->ufi_file)
unionfs_hidden_files ?
> +#define fbstart(file) (ftopd(file)->b_start)
unionfs_file_branch_start ?
> +#define fbend(file) (ftopd(file)->b_end)
unionfs_file_branch_end ?
> +
> +/* File to hidden file. */
> +static inline struct file *ftohf(struct file *f)
> +{
> + return ftopd(f)->ufi_file[fbstart(f)];
> +}
unionfs_hidden_file ?
> +
> +static inline struct file *ftohf_index(const struct file *f, int index)
> +{
> + return ftopd(f)->ufi_file[index];
> +}
__unionfs_hidden_file (and make the above use this too)?
> +
> +static inline void set_ftohf_index(struct file *f, int index, struct file *val)
> +{
> + ftopd(f)->ufi_file[index] = val;
> +}
__unionfs_set_hidden_file ?
> +
> +static inline void set_ftohf(struct file *f, struct file *val)
> +{
> + ftopd(f)->ufi_file[fbstart(f)] = val;
> +}
unionfs_set_hidden_file ?
[Same comments apply for the rest.]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
| |||||||||
| Last update: 2006-10-13 08:27 [from the cache] ©2003-2008 | ||||||||||