linux-ext4 - Re: [PATCH RFC 05/30] ext4: snapshot hooks

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.LFD.2.00.1106071306510.4754@dhcp-27-109.brq.redhat.com>
Date:	Tue, 7 Jun 2011 13:24:39 +0200 (CEST)
From:	Lukas Czerner <lczerner@...hat.com>
To:	amir73il@...rs.sourceforge.net
cc:	linux-ext4@...r.kernel.org, tytso@....edu,
	Amir Goldstein <amir73il@...rs.sf.net>,
	Yongqiang Yang <xiaoqiangnk@...il.com>
Subject: Re: [PATCH RFC 05/30] ext4: snapshot hooks - delete blocks

On Mon, 9 May 2011, amir73il@...rs.sourceforge.net wrote:

> From: Amir Goldstein <amir73il@...rs.sf.net>
> 
> Before deleting file blocks in ext4_free_blocks(),
> we call the snapshot API ext4_snapshot_get_delete_access(),
> to optionally move the block to the snapshot file instead of
> freeing them.
> 
> Signed-off-by: Amir Goldstein <amir73il@...rs.sf.net>
> Signed-off-by: Yongqiang Yang <xiaoqiangnk@...il.com>
> ---
>  fs/ext4/ext4.h    |   10 +++++++---
>  fs/ext4/mballoc.c |   30 +++++++++++++++++++++++++++---
>  2 files changed, 34 insertions(+), 6 deletions(-)
> 
> diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
> index ca25e67..4e9e46a 100644
> --- a/fs/ext4/ext4.h
> +++ b/fs/ext4/ext4.h
> @@ -1729,9 +1729,13 @@ extern int ext4_mb_reserve_blocks(struct super_block *, int);
>  extern void ext4_discard_preallocations(struct inode *);
>  extern int __init ext4_init_mballoc(void);
>  extern void ext4_exit_mballoc(void);
> -extern void ext4_free_blocks(handle_t *handle, struct inode *inode,
> -			     struct buffer_head *bh, ext4_fsblk_t block,
> -			     unsigned long count, int flags);
> +extern void __ext4_free_blocks(const char *where, unsigned int line,
> +			       handle_t *handle,  struct inode *inode,
> +			       struct buffer_head *bh, ext4_fsblk_t block,
> +			       unsigned long count, int flags);
> +#define ext4_free_blocks(handle, inode, bh, block, count, flags) \
> +	__ext4_free_blocks(__func__, __LINE__ , (handle), (inode), (bh), \
> +			   (block), (count), (flags))
>  extern int ext4_mb_add_groupinfo(struct super_block *sb,
>  		ext4_group_t i, struct ext4_group_desc *desc);
>  extern int ext4_trim_fs(struct super_block *, struct fstrim_range *);
> diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c
> index e8bfd8d..3b1c6d1 100644
> --- a/fs/ext4/mballoc.c
> +++ b/fs/ext4/mballoc.c
> @@ -4445,9 +4445,9 @@ ext4_mb_free_metadata(handle_t *handle, struct ext4_buddy *e4b,
>   * @count:		number of blocks to count
>   * @metadata: 		Are these metadata blocks
>   */
> -void ext4_free_blocks(handle_t *handle, struct inode *inode,
> -		      struct buffer_head *bh, ext4_fsblk_t block,
> -		      unsigned long count, int flags)
> +void __ext4_free_blocks(const char *where, unsigned int line, handle_t *handle,
> +			struct inode *inode, struct buffer_head *bh,
> +			ext4_fsblk_t block, unsigned long count, int flags)
>  {
>  	struct buffer_head *bitmap_bh = NULL;
>  	struct super_block *sb = inode->i_sb;
> @@ -4461,6 +4461,7 @@ void ext4_free_blocks(handle_t *handle, struct inode *inode,
>  	struct ext4_buddy e4b;
>  	int err = 0;
>  	int ret;
> +	int maxblocks;
>  
>  	if (bh) {
>  		if (block)
> @@ -4543,6 +4544,29 @@ do_more:
>  		goto error_return;
>  	}
>  
> +	maxblocks = count;
> +	ret = ext4_snapshot_get_delete_access(handle, inode,
> +					      block, &maxblocks);

It would be nice to have this defined in the same commit, so I know what
is it doing.

> +	if (ret < 0) {
> +		ext4_journal_abort_handle(where, line, __func__,
> +					  NULL, handle, ret);
> +		err = ret;
> +		goto error_return;
> +	}
> +	if (ret > 0) {
> +		/* 'ret' blocks were moved to snapshot - skip them */
> +		block += maxblocks;
> +		count -= maxblocks;
> +		count += overflow;
> +		cond_resched();
> +		if (count > 0)
> +			goto do_more;
> +		/* no more blocks to free/move to snapshot */
> +		ext4_mark_super_dirty(sb);
> +		goto error_return;
> +	}
> +	overflow += count - maxblocks;
> +	count = maxblocks;
>  	BUFFER_TRACE(bitmap_bh, "getting write access");
>  	err = ext4_handle_get_bitmap_access(handle, sb, block_group, bitmap_bh);
>  	if (err)
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html