From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from imap.thunk.org (imap.thunk.org [IPv6:2600:3c02::f03c:91ff:fe96:be03]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 3A0A321E082A6 for ; Sat, 10 Mar 2018 18:07:58 -0800 (PST) Date: Sat, 10 Mar 2018 21:14:06 -0500 From: "Theodore Y. Ts'o" Subject: Re: [RFC v2 00/83] NOVA: a new file system for persistent memory Message-ID: <20180311021406.GA388@thunk.org> References: <1520705944-6723-1-git-send-email-jix024@eng.ucsd.edu> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1520705944-6723-1-git-send-email-jix024@eng.ucsd.edu> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" To: Andiry Xu Cc: coughlan@redhat.com, linux-nvdimm@lists.01.org, Andiry Xu , miklos@szeredi.hu, david@fromorbit.com, linux-kernel@vger.kernel.org, jack@suse.com, linux-fsdevel@vger.kernel.org, swanson@cs.ucsd.edu, swhiteho@redhat.com, andiry.xu@gmail.com List-ID: FYI, your patch set doesn't even compile for me without these fixups. I'm not sure why you were trying to declare inline functions in a header file without the function body? - Ted diff --git a/fs/nova/balloc.c b/fs/nova/balloc.c index 8e992156f28c..9c7b74aa712e 100644 --- a/fs/nova/balloc.c +++ b/fs/nova/balloc.c @@ -74,12 +74,12 @@ static void nova_init_free_list(struct super_block *sb, free_list->block_end -= sbi->tail_reserved_blocks; } -inline struct nova_range_node *nova_alloc_blocknode(struct super_block *sb) +struct nova_range_node *nova_alloc_blocknode(struct super_block *sb) { return nova_alloc_range_node(sb); } -inline void nova_free_blocknode(struct super_block *sb, +void nova_free_blocknode(struct super_block *sb, struct nova_range_node *node) { nova_free_range_node(node); @@ -206,7 +206,7 @@ int nova_insert_range_node(struct rb_root *tree, return 0; } -inline int nova_insert_blocktree(struct nova_sb_info *sbi, +int nova_insert_blocktree(struct nova_sb_info *sbi, struct rb_root *tree, struct nova_range_node *new_node) { int ret; @@ -659,7 +659,7 @@ static int nova_new_blocks(struct super_block *sb, unsigned long *blocknr, // Allocate data blocks. The offset for the allocated block comes back in // blocknr. Return the number of blocks allocated. -inline int nova_new_data_blocks(struct super_block *sb, +int nova_new_data_blocks(struct super_block *sb, struct nova_inode_info_header *sih, unsigned long *blocknr, unsigned long start_blk, unsigned int num, enum nova_alloc_init zero, int cpu, diff --git a/fs/nova/balloc.h b/fs/nova/balloc.h index 463fbac99eff..aca7e8c18dde 100644 --- a/fs/nova/balloc.h +++ b/fs/nova/balloc.h @@ -62,18 +62,18 @@ enum alloc_type { int nova_alloc_block_free_lists(struct super_block *sb); void nova_delete_free_lists(struct super_block *sb); -inline struct nova_range_node *nova_alloc_blocknode(struct super_block *sb); -inline void nova_free_blocknode(struct super_block *sb, +struct nova_range_node *nova_alloc_blocknode(struct super_block *sb); +void nova_free_blocknode(struct super_block *sb, struct nova_range_node *bnode); extern void nova_init_blockmap(struct super_block *sb, int recovery); extern unsigned long nova_count_free_blocks(struct super_block *sb); -inline int nova_insert_blocktree(struct nova_sb_info *sbi, +int nova_insert_blocktree(struct nova_sb_info *sbi, struct rb_root *tree, struct nova_range_node *new_node); extern int nova_free_data_blocks(struct super_block *sb, struct nova_inode_info_header *sih, unsigned long blocknr, int num); extern int nova_free_log_blocks(struct super_block *sb, struct nova_inode_info_header *sih, unsigned long blocknr, int num); -extern inline int nova_new_data_blocks(struct super_block *sb, +extern int nova_new_data_blocks(struct super_block *sb, struct nova_inode_info_header *sih, unsigned long *blocknr, unsigned long start_blk, unsigned int num, enum nova_alloc_init zero, int cpu, diff --git a/fs/nova/inode.c b/fs/nova/inode.c index 21be31a05d26..31ef258978ba 100644 --- a/fs/nova/inode.c +++ b/fs/nova/inode.c @@ -440,7 +440,7 @@ struct inode *nova_iget(struct super_block *sb, unsigned long ino) return ERR_PTR(err); } -inline int nova_insert_inodetree(struct nova_sb_info *sbi, +int nova_insert_inodetree(struct nova_sb_info *sbi, struct nova_range_node *new_node, int cpu) { struct rb_root *tree; diff --git a/fs/nova/inode.h b/fs/nova/inode.h index 086a7cba8ac3..1097e15ff7af 100644 --- a/fs/nova/inode.h +++ b/fs/nova/inode.h @@ -254,7 +254,7 @@ int nova_init_inode_table(struct super_block *sb); int nova_get_inode_address(struct super_block *sb, u64 ino, u64 *pi_addr, int extendable); struct inode *nova_iget(struct super_block *sb, unsigned long ino); -inline int nova_insert_inodetree(struct nova_sb_info *sbi, +int nova_insert_inodetree(struct nova_sb_info *sbi, struct nova_range_node *new_node, int cpu); u64 nova_new_nova_inode(struct super_block *sb, u64 *pi_addr); struct inode *nova_new_vfs_inode(enum nova_new_inode_type type, diff --git a/fs/nova/super.c b/fs/nova/super.c index 039c003b698b..9f06ec847c95 100644 --- a/fs/nova/super.c +++ b/fs/nova/super.c @@ -795,23 +795,23 @@ static void nova_put_super(struct super_block *sb) sb->s_fs_info = NULL; } -inline void nova_free_range_node(struct nova_range_node *node) +void nova_free_range_node(struct nova_range_node *node) { kmem_cache_free(nova_range_node_cachep, node); } -inline void nova_free_inode_node(struct super_block *sb, +void nova_free_inode_node(struct super_block *sb, struct nova_range_node *node) { nova_free_range_node(node); } -inline void nova_free_file_write_item(struct nova_file_write_item *item) +void nova_free_file_write_item(struct nova_file_write_item *item) { kmem_cache_free(nova_file_write_item_cachep, item); } -inline struct nova_file_write_item * +struct nova_file_write_item * nova_alloc_file_write_item(struct super_block *sb) { struct nova_file_write_item *p; diff --git a/fs/nova/super.h b/fs/nova/super.h index bcbe862ac914..dc98346266e1 100644 --- a/fs/nova/super.h +++ b/fs/nova/super.h @@ -160,11 +160,11 @@ static inline struct nova_super_block *nova_get_super(struct super_block *sb) extern void nova_error_mng(struct super_block *sb, const char *fmt, ...); extern struct nova_range_node *nova_alloc_range_node(struct super_block *sb); -extern inline struct nova_range_node *nova_alloc_inode_node(struct super_block *sb); +extern struct nova_range_node *nova_alloc_inode_node(struct super_block *sb); extern struct nova_file_write_item * nova_alloc_file_write_item(struct super_block *sb); extern void nova_free_range_node(struct nova_range_node *node); -extern inline void nova_free_inode_node(struct super_block *sb, +extern void nova_free_inode_node(struct super_block *sb, struct nova_range_node *node); void nova_free_file_write_item(struct nova_file_write_item *item); _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm