From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BAC5A143748 for ; Wed, 9 Apr 2025 19:34:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744227275; cv=none; b=UwPyKsUBYbOuIjI8c9iIRSUFA6go9JqFAUm8ADP9JABLDGOqat0uNsju5aMFnlCpEGDxiibZ5h9VUC6G7bqUzQL+7eNJHrhrl/+OqHCzxQm9wwuTUq6DpN3PYwrG/1TYCE6SVLj6RQondlmrJfatvhK7BLBQowph0sk2gz5UkY8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744227275; c=relaxed/simple; bh=zeqZPAlZf4HKoho3LumOJpu4t5tcceiRT2fxSvqBXCI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=U9ZgeuqarZb+93+q71e5xvsszM99/JjEYvm1Lg2CXRTEY6Ie3gWuiXaJQ79ytyBJpXNjgYMBUdqUfpLVHyyZHrIAxG7+ebUOXdB+znFuSDf1iPsw8Cm8bishD613L6jVoLbyRcGumYsVUWVtWlBZB0Q6dFsTCMrEL4Aaib6DRnw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=JdY4gIzc; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="JdY4gIzc" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2FBBEC4CEE2; Wed, 9 Apr 2025 19:34:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1744227275; bh=zeqZPAlZf4HKoho3LumOJpu4t5tcceiRT2fxSvqBXCI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=JdY4gIzcXgN/Yh6OrElLzQuWnzOG2S1wvQOydjZSBwuCuRGDpVlZ0b3IV5thtEPqt VHOBizJWO34Sp0Lz8F6fgemO1puJZYmkfZhuv7n+vONIr1F/5VnVs8C/vvswF5A38w 3orXyNeiRRmC399GfWOmbprpNTb4BHq6bAxxBjnvBKHcwwryug0dgcOIDARF4RTHj5 y0S2NkFhY2ZYiTCszL70ohNvcsPbnwM8LsOxue01kU91U2v9+Uj9rS8X1HyGPUgrC3 NgEYTsCfs39d8yBBWIHnA+81twjeRdw/3b7puwoWlKR/n6CsDAK/69GiH3k4svJaDu JQ8HcM66mURrA== Date: Wed, 9 Apr 2025 12:34:34 -0700 From: "Darrick J. Wong" To: Christoph Hellwig Cc: Andrey Albershteyn , Hans Holmberg , linux-xfs@vger.kernel.org Subject: Re: [PATCH 43/45] xfs_scrub: handle internal RT devices Message-ID: <20250409193434.GO6283@frogsfrogsfrogs> References: <20250409075557.3535745-1-hch@lst.de> <20250409075557.3535745-44-hch@lst.de> Precedence: bulk X-Mailing-List: linux-xfs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250409075557.3535745-44-hch@lst.de> On Wed, Apr 09, 2025 at 09:55:46AM +0200, Christoph Hellwig wrote: > Handle the synthetic fmr_device values. > > Signed-off-by: Christoph Hellwig Looks fine, Reviewed-by: "Darrick J. Wong" --D > --- > scrub/phase6.c | 65 ++++++++++++++++++++++++++++-------------------- > scrub/phase7.c | 28 +++++++++++++++------ > scrub/spacemap.c | 17 +++++++++---- > 3 files changed, 70 insertions(+), 40 deletions(-) > > diff --git a/scrub/phase6.c b/scrub/phase6.c > index 2a52b2c92419..abf6f9713f1a 100644 > --- a/scrub/phase6.c > +++ b/scrub/phase6.c > @@ -56,12 +56,21 @@ dev_to_pool( > struct media_verify_state *vs, > dev_t dev) > { > - if (dev == ctx->fsinfo.fs_datadev) > - return vs->rvp_data; > - else if (dev == ctx->fsinfo.fs_logdev) > - return vs->rvp_log; > - else if (dev == ctx->fsinfo.fs_rtdev) > - return vs->rvp_realtime; > + if (ctx->mnt.fsgeom.rtstart) { > + if (dev == XFS_DEV_DATA) > + return vs->rvp_data; > + if (dev == XFS_DEV_LOG) > + return vs->rvp_log; > + if (dev == XFS_DEV_RT) > + return vs->rvp_realtime; > + } else { > + if (dev == ctx->fsinfo.fs_datadev) > + return vs->rvp_data; > + if (dev == ctx->fsinfo.fs_logdev) > + return vs->rvp_log; > + if (dev == ctx->fsinfo.fs_rtdev) > + return vs->rvp_realtime; > + } > abort(); > } > > @@ -71,12 +80,21 @@ disk_to_dev( > struct scrub_ctx *ctx, > struct disk *disk) > { > - if (disk == ctx->datadev) > - return ctx->fsinfo.fs_datadev; > - else if (disk == ctx->logdev) > - return ctx->fsinfo.fs_logdev; > - else if (disk == ctx->rtdev) > - return ctx->fsinfo.fs_rtdev; > + if (ctx->mnt.fsgeom.rtstart) { > + if (disk == ctx->datadev) > + return XFS_DEV_DATA; > + if (disk == ctx->logdev) > + return XFS_DEV_LOG; > + if (disk == ctx->rtdev) > + return XFS_DEV_RT; > + } else { > + if (disk == ctx->datadev) > + return ctx->fsinfo.fs_datadev; > + if (disk == ctx->logdev) > + return ctx->fsinfo.fs_logdev; > + if (disk == ctx->rtdev) > + return ctx->fsinfo.fs_rtdev; > + } > abort(); > } > > @@ -87,11 +105,9 @@ bitmap_for_disk( > struct disk *disk, > struct media_verify_state *vs) > { > - dev_t dev = disk_to_dev(ctx, disk); > - > - if (dev == ctx->fsinfo.fs_datadev) > + if (disk == ctx->datadev) > return vs->d_bad; > - else if (dev == ctx->fsinfo.fs_rtdev) > + if (disk == ctx->rtdev) > return vs->r_bad; > return NULL; > } > @@ -501,14 +517,11 @@ report_ioerr( > .length = length, > }; > struct disk_ioerr_report *dioerr = arg; > - dev_t dev; > - > - dev = disk_to_dev(dioerr->ctx, dioerr->disk); > > /* Go figure out which blocks are bad from the fsmap. */ > - keys[0].fmr_device = dev; > + keys[0].fmr_device = disk_to_dev(dioerr->ctx, dioerr->disk); > keys[0].fmr_physical = start; > - keys[1].fmr_device = dev; > + keys[1].fmr_device = keys[0].fmr_device; > keys[1].fmr_physical = start + length - 1; > keys[1].fmr_owner = ULLONG_MAX; > keys[1].fmr_offset = ULLONG_MAX; > @@ -675,14 +688,12 @@ remember_ioerr( > int ret; > > if (!length) { > - dev_t dev = disk_to_dev(ctx, disk); > - > - if (dev == ctx->fsinfo.fs_datadev) > + if (disk == ctx->datadev) > vs->d_trunc = true; > - else if (dev == ctx->fsinfo.fs_rtdev) > - vs->r_trunc = true; > - else if (dev == ctx->fsinfo.fs_logdev) > + else if (disk == ctx->logdev) > vs->l_trunc = true; > + else if (disk == ctx->rtdev) > + vs->r_trunc = true; > return; > } > > diff --git a/scrub/phase7.c b/scrub/phase7.c > index 01097b678798..e25502668b1c 100644 > --- a/scrub/phase7.c > +++ b/scrub/phase7.c > @@ -68,25 +68,37 @@ count_block_summary( > void *arg) > { > struct summary_counts *counts; > + bool is_rt = false; > unsigned long long len; > int ret; > > + if (ctx->mnt.fsgeom.rtstart) { > + if (fsmap->fmr_device == XFS_DEV_LOG) > + return 0; > + if (fsmap->fmr_device == XFS_DEV_RT) > + is_rt = true; > + } else { > + if (fsmap->fmr_device == ctx->fsinfo.fs_logdev) > + return 0; > + if (fsmap->fmr_device == ctx->fsinfo.fs_rtdev) > + is_rt = true; > + } > + > counts = ptvar_get((struct ptvar *)arg, &ret); > if (ret) { > str_liberror(ctx, -ret, _("retrieving summary counts")); > return -ret; > } > - if (fsmap->fmr_device == ctx->fsinfo.fs_logdev) > - return 0; > + > if ((fsmap->fmr_flags & FMR_OF_SPECIAL_OWNER) && > fsmap->fmr_owner == XFS_FMR_OWN_FREE) { > uint64_t blocks; > > blocks = cvt_b_to_off_fsbt(&ctx->mnt, fsmap->fmr_length); > - if (fsmap->fmr_device == ctx->fsinfo.fs_datadev) > - hist_add(&counts->datadev_hist, blocks); > - else if (fsmap->fmr_device == ctx->fsinfo.fs_rtdev) > + if (is_rt) > hist_add(&counts->rtdev_hist, blocks); > + else > + hist_add(&counts->datadev_hist, blocks); > return 0; > } > > @@ -94,10 +106,10 @@ count_block_summary( > > /* freesp btrees live in free space, need to adjust counters later. */ > if ((fsmap->fmr_flags & FMR_OF_SPECIAL_OWNER) && > - fsmap->fmr_owner == XFS_FMR_OWN_AG) { > + fsmap->fmr_owner == XFS_FMR_OWN_AG) > counts->agbytes += fsmap->fmr_length; > - } > - if (fsmap->fmr_device == ctx->fsinfo.fs_rtdev) { > + > + if (is_rt) { > /* Count realtime extents. */ > counts->rbytes += len; > } else { > diff --git a/scrub/spacemap.c b/scrub/spacemap.c > index c293ab44a528..1ee4d1946d3d 100644 > --- a/scrub/spacemap.c > +++ b/scrub/spacemap.c > @@ -103,9 +103,12 @@ scan_ag_rmaps( > bperag = (off_t)ctx->mnt.fsgeom.agblocks * > (off_t)ctx->mnt.fsgeom.blocksize; > > - keys[0].fmr_device = ctx->fsinfo.fs_datadev; > + if (ctx->mnt.fsgeom.rtstart) > + keys[0].fmr_device = XFS_DEV_DATA; > + else > + keys[0].fmr_device = ctx->fsinfo.fs_datadev; > keys[0].fmr_physical = agno * bperag; > - keys[1].fmr_device = ctx->fsinfo.fs_datadev; > + keys[1].fmr_device = keys[0].fmr_device; > keys[1].fmr_physical = ((agno + 1) * bperag) - 1; > keys[1].fmr_owner = ULLONG_MAX; > keys[1].fmr_offset = ULLONG_MAX; > @@ -140,9 +143,12 @@ scan_rtg_rmaps( > off_t bperrg = bytes_per_rtgroup(&ctx->mnt.fsgeom); > int ret; > > - keys[0].fmr_device = ctx->fsinfo.fs_rtdev; > + if (ctx->mnt.fsgeom.rtstart) > + keys[0].fmr_device = XFS_DEV_RT; > + else > + keys[0].fmr_device = ctx->fsinfo.fs_rtdev; > keys[0].fmr_physical = (xfs_rtblock_t)rgno * bperrg; > - keys[1].fmr_device = ctx->fsinfo.fs_rtdev; > + keys[1].fmr_device = keys[0].fmr_device; > keys[1].fmr_physical = ((rgno + 1) * bperrg) - 1; > keys[1].fmr_owner = ULLONG_MAX; > keys[1].fmr_offset = ULLONG_MAX; > @@ -216,7 +222,8 @@ scan_log_rmaps( > { > struct scrub_ctx *ctx = (struct scrub_ctx *)wq->wq_ctx; > > - scan_dev_rmaps(ctx, ctx->fsinfo.fs_logdev, arg); > + scan_dev_rmaps(ctx, ctx->mnt.fsgeom.rtstart ? 2 : ctx->fsinfo.fs_logdev, > + arg); > } > > /* > -- > 2.47.2 > >