All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@infradead.org>
To: "Darrick J. Wong" <djwong@kernel.org>
Cc: Christoph Hellwig <hch@infradead.org>,
	cem@kernel.org, stable@vger.kernel.org,
	linux-xfs@vger.kernel.org
Subject: Re: [PATCH 6/6] xfs: adjust datadev sector count to reflect internal rt volumes
Date: Sun, 23 Aug 2026 21:47:16 -0700	[thread overview]
Message-ID: <aovM1MZumYu3AsVB@infradead.org> (raw)
In-Reply-To: <20260819230839.GB6072@frogsfrogsfrogs>

On Wed, Aug 19, 2026 at 04:08:39PM -0700, Darrick J. Wong wrote:
> IOWS: 512M data volume, 3.1G internal rt section
> Now let's try some media verification:
> 
> # xfs_io -c 'verifymedia -d' -c 'verifymedia -r' /mnt
> verified 536870912/536870912 bytes at offset 0
> 512 MiB, 1 ops; 0.0496 sec (10.067 GiB/sec and 20.1345 ops/sec)
> verified 536870912/536870912 bytes at offset 0
> 512 MiB, 1 ops; 0.0409 sec (12.222 GiB/sec and 24.4439 ops/sec)
> 
> Notice how xfs_io says we only verified 512M of the rt volume?
> If you run btrace in the background you'll see that we read the first
> 512M of the volume (aka the data section) twice and never read anything
> from the rt section.
> 
> Ofc my solution does too /much/ work:

We should probably wire this up to xfstests..


      reply	other threads:[~2026-08-24  4:47 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-13  5:23 [PATCHSET v2 1/2] xfs: LLM-inspired bug fixes, part 5 Darrick J. Wong
2026-08-13  5:24 ` [PATCH 1/6] xfs: fix unit conversions in per_binval computation Darrick J. Wong
2026-08-13  5:24 ` [PATCH 2/6] xfs: fix short ifork reaping computation in xreap_bmapi_binval Darrick J. Wong
2026-08-13  5:24 ` [PATCH 3/6] xfs: fix name string recording in slowpath pptr tracepoints Darrick J. Wong
2026-08-13  5:24 ` [PATCH 4/6] xfs: don't leak dqacct if rhashtable insertion fails Darrick J. Wong
2026-08-13  5:25 ` [PATCH 5/6] xfs: fix the rtrmap and rtrefcount _maxlevels_ondisk functions Darrick J. Wong
2026-08-13  5:25 ` [PATCH 6/6] xfs: adjust datadev sector count to reflect internal rt volumes Darrick J. Wong
2026-08-19  6:33   ` Christoph Hellwig
2026-08-19  6:36     ` Christoph Hellwig
2026-08-19 23:08     ` Darrick J. Wong
2026-08-24  4:47       ` Christoph Hellwig [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aovM1MZumYu3AsVB@infradead.org \
    --to=hch@infradead.org \
    --cc=cem@kernel.org \
    --cc=djwong@kernel.org \
    --cc=linux-xfs@vger.kernel.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.