From patchwork Thu Mar 30 06:21:09 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Qu Wenruo X-Patchwork-Id: 9652949 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 6318E602BD for ; Thu, 30 Mar 2017 06:22:02 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5323A27B2F for ; Thu, 30 Mar 2017 06:22:02 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 47AD028334; Thu, 30 Mar 2017 06:22:02 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 1B19C27B2F for ; Thu, 30 Mar 2017 06:22:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932716AbdC3GV6 (ORCPT ); Thu, 30 Mar 2017 02:21:58 -0400 Received: from cn.fujitsu.com ([59.151.112.132]:15660 "EHLO heian.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S932437AbdC3GVc (ORCPT ); Thu, 30 Mar 2017 02:21:32 -0400 X-IronPort-AV: E=Sophos;i="5.22,518,1449504000"; d="scan'208";a="17165841" Received: from unknown (HELO cn.fujitsu.com) ([10.167.33.5]) by heian.cn.fujitsu.com with ESMTP; 30 Mar 2017 14:21:29 +0800 Received: from G08CNEXCHPEKD01.g08.fujitsu.local (unknown [10.167.33.80]) by cn.fujitsu.com (Postfix) with ESMTP id 3875147EE21B for ; Thu, 30 Mar 2017 14:21:26 +0800 (CST) Received: from localhost.localdomain (10.167.226.34) by G08CNEXCHPEKD01.g08.fujitsu.local (10.167.33.89) with Microsoft SMTP Server (TLS) id 14.3.319.2; Thu, 30 Mar 2017 14:21:26 +0800 From: Qu Wenruo To: Subject: [PATCH v3 13/19] btrfs-progs: scrub: Introduce function to scrub one data stripe Date: Thu, 30 Mar 2017 14:21:09 +0800 Message-ID: <20170330062116.14379-14-quwenruo@cn.fujitsu.com> X-Mailer: git-send-email 2.12.1 In-Reply-To: <20170330062116.14379-1-quwenruo@cn.fujitsu.com> References: <20170330062116.14379-1-quwenruo@cn.fujitsu.com> MIME-Version: 1.0 X-Originating-IP: [10.167.226.34] X-yoursite-MailScanner-ID: 3875147EE21B.AE5FE X-yoursite-MailScanner: Found to be clean X-yoursite-MailScanner-From: quwenruo@cn.fujitsu.com Sender: linux-btrfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-btrfs@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Introduce new function, scrub_one_data_stripe(), to check all data and tree blocks inside the data stripe. This function will not try to recovery any error, but only check if any data/tree blocks has mismatch csum. If data missing csum, which is completely valid for case like nodatasum, it will just record it, but not report as error. Signed-off-by: Qu Wenruo --- scrub.c | 129 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 129 insertions(+) diff --git a/scrub.c b/scrub.c index 2e4850df..f9d6a5a3 100644 --- a/scrub.c +++ b/scrub.c @@ -367,3 +367,132 @@ invalid_arg: error("invalid parameter for %s", __func__); return -EINVAL; } + +/* + * Scrub one full data stripe of RAID5/6. + * This means it will check any data/metadata extent in the data stripe + * spcified by @stripe and @stripe_len + * + * This function will only *CHECK* if the data stripe has any corruption. + * Won't repair at this function. + * + * Return 0 if the full stripe is OK. + * Return <0 if any error is found. + * Note: Missing csum is not counted as error(NODATACSUM is valid) + */ +static int scrub_one_data_stripe(struct btrfs_fs_info *fs_info, + struct btrfs_scrub_progress *scrub_ctx, + struct scrub_stripe *stripe, u32 stripe_len) +{ + struct btrfs_path *path; + struct btrfs_root *extent_root = fs_info->extent_root; + struct btrfs_key key; + u64 extent_start; + u64 extent_len; + u64 orig_csum_discards; + int ret; + + if (!is_data_stripe(stripe)) + return -EINVAL; + + path = btrfs_alloc_path(); + if (!path) + return -ENOMEM; + + key.objectid = stripe->logical + stripe_len; + key.offset = 0; + key.type = 0; + + ret = btrfs_search_slot(NULL, extent_root, &key, path, 0, 0); + if (ret < 0) + goto out; + while (1) { + struct btrfs_extent_item *ei; + struct extent_buffer *eb; + char *data; + int slot; + int metadata = 0; + u64 check_start; + u64 check_len; + + ret = btrfs_previous_extent_item(extent_root, path, 0); + if (ret > 0) { + ret = 0; + goto out; + } + if (ret < 0) + goto out; + eb = path->nodes[0]; + slot = path->slots[0]; + btrfs_item_key_to_cpu(eb, &key, slot); + extent_start = key.objectid; + ei = btrfs_item_ptr(eb, slot, struct btrfs_extent_item); + + /* tree block scrub */ + if (key.type == BTRFS_METADATA_ITEM_KEY || + btrfs_extent_flags(eb, ei) & BTRFS_EXTENT_FLAG_TREE_BLOCK) { + extent_len = extent_root->nodesize; + metadata = 1; + } else { + extent_len = key.offset; + metadata = 0; + } + + /* Current extent is out of our range, loop comes to end */ + if (extent_start + extent_len <= stripe->logical) + break; + + if (metadata) { + /* + * Check crossing stripe first, which can't be scrubbed + */ + if (check_crossing_stripes(fs_info, extent_start, + extent_root->nodesize)) { + error("tree block at %llu is crossing stripe boundary, unable to scrub", + extent_start); + ret = -EIO; + goto out; + } + data = stripe->data + extent_start - stripe->logical; + ret = scrub_tree_mirror(fs_info, scrub_ctx, + data, extent_start, 0); + /* Any csum/verify error means the stripe is screwed */ + if (ret < 0) { + stripe->csum_mismatch = 1; + ret = -EIO; + goto out; + } + ret = 0; + continue; + } + /* Restrict the extent range to fit stripe range */ + check_start = max(extent_start, stripe->logical); + check_len = min(extent_start + extent_len, stripe->logical + + stripe_len) - check_start; + + /* Record original csum_discards to detect missing csum case */ + orig_csum_discards = scrub_ctx->csum_discards; + + data = stripe->data + check_start - stripe->logical; + ret = scrub_data_mirror(fs_info, scrub_ctx, data, check_start, + check_len, 0); + /* Csum mismatch, no need to continue anyway*/ + if (ret < 0) { + stripe->csum_mismatch = 1; + goto out; + } + /* Check if there is any missing csum for data */ + if (scrub_ctx->csum_discards != orig_csum_discards) + stripe->csum_missing = 1; + /* + * Only increase data_extents_scrubbed if we are scrubbing the + * tailing part of the data extent + */ + if (extent_start + extent_len <= stripe->logical + stripe_len) + scrub_ctx->data_extents_scrubbed++; + ret = 0; + } +out: + btrfs_free_path(path); + return ret; +}