From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f48.google.com ([209.85.220.48]:33838 "EHLO mail-pa0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751732AbaJJHNg (ORCPT ); Fri, 10 Oct 2014 03:13:36 -0400 Received: by mail-pa0-f48.google.com with SMTP id eu11so1203975pac.7 for ; Fri, 10 Oct 2014 00:13:36 -0700 (PDT) Received: from localhost ([203.114.244.88]) by mx.google.com with ESMTPSA id ip3sm2451978pbc.58.2014.10.10.00.13.34 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 10 Oct 2014 00:13:35 -0700 (PDT) Date: Fri, 10 Oct 2014 15:13:31 +0800 From: Eryu Guan To: linux-btrfs@vger.kernel.org Subject: Re: [PATCH] Btrfs: return failure if btrfs_dev_replace_finishing() failed Message-ID: <20141010071331.GG13950@dhcp-13-216.nay.redhat.com> References: <1411640894-5160-1-git-send-email-guaneryu@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1411640894-5160-1-git-send-email-guaneryu@gmail.com> Sender: linux-btrfs-owner@vger.kernel.org List-ID: On Thu, Sep 25, 2014 at 06:28:14PM +0800, Eryu Guan wrote: > device replace could fail due to another running scrub process, but this > failure doesn't get returned to userspace. > > The following steps could reproduce this issue > > mkfs -t btrfs -f /dev/sdb1 /dev/sdb2 > mount /dev/sdb1 /mnt/btrfs > while true; do > btrfs scrub start -B /mnt/btrfs >/dev/null 2>&1 > done & > btrfs replace start -Bf /dev/sdb2 /dev/sdb3 /mnt/btrfs > # if this replace succeeded, do the following and repeat until > # you see this log in dmesg > # BTRFS: btrfs_scrub_dev(/dev/sdb2, 2, /dev/sdb3) failed -115 > #btrfs replace start -Bf /dev/sdb3 /dev/sdb2 /mnt/btrfs > > # once you see the error log in dmesg, check return value of > # replace > echo $? > > Also only WARN_ON if the return code is not -EINPROGRESS. > > Signed-off-by: Eryu Guan Ping, any comments on this patch? Thanks, Eryu > --- > fs/btrfs/dev-replace.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c > index eea26e1..44d32ab 100644 > --- a/fs/btrfs/dev-replace.c > +++ b/fs/btrfs/dev-replace.c > @@ -418,9 +418,11 @@ int btrfs_dev_replace_start(struct btrfs_root *root, > &dev_replace->scrub_progress, 0, 1); > > ret = btrfs_dev_replace_finishing(root->fs_info, ret); > - WARN_ON(ret); > + /* don't warn if EINPROGRESS, someone else might be running scrub */ > + if (ret != -EINPROGRESS) > + WARN_ON(ret); > > - return 0; > + return ret; > > leave: > dev_replace->srcdev = NULL; > @@ -538,7 +540,7 @@ static int btrfs_dev_replace_finishing(struct btrfs_fs_info *fs_info, > btrfs_destroy_dev_replace_tgtdev(fs_info, tgt_device); > mutex_unlock(&dev_replace->lock_finishing_cancel_unmount); > > - return 0; > + return scrub_ret; > } > > printk_in_rcu(KERN_INFO > -- > 1.8.3.1 >