On 2018年01月26日 16:35, Su Yue wrote:
> Original check forgets to clear bit I_ERR_FILE_EXTENT_DISCOUNT
> in rec->errors after repair.
> 
> So just do it.
> 
> Signed-off-by: Su Yue <suy.f...@cn.fujitsu.com>

Reviewed-by: Qu Wenruo <w...@suse.com>

And a test case for this would help us to avoid further regression.

Thanks,
Qu

> ---
>  cmds-check.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/cmds-check.c b/cmds-check.c
> index ae0a9e146399..d8d9a3227c06 100644
> --- a/cmds-check.c
> +++ b/cmds-check.c
> @@ -3978,6 +3978,7 @@ static int repair_inode_discount_extent(struct 
> btrfs_trans_handle *trans,
>               if (ret < 0)
>                       goto out;
>       }
> +     rec->errors &= ~I_ERR_FILE_EXTENT_DISCOUNT;
>       printf("Fixed discount file extents for inode: %llu in root: %llu\n",
>              rec->ino, root->objectid);
>  out:
> 

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to