From: "Jason A. Donenfeld" <Jason@zx2c4.com>
To: Kent Overstreet <kent.overstreet@linux.dev>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Theodore Ts'o <tytso@mit.edu>,
linux-bcachefs@vger.kernel.org, linux-fsdevel@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [GIT PULL] bcachefs fixes for 6.12-rc2
Date: Mon, 7 Oct 2024 17:01:55 +0200 [thread overview]
Message-ID: <ZwP341bZ9eQ0Qyej@zx2c4.com> (raw)
In-Reply-To: <dcfwznpfogbtbsiwbtj56fa3dxnba4aptkcq5a5buwnkma76nc@rjon67szaahh>
On Sun, Oct 06, 2024 at 03:29:51PM -0400, Kent Overstreet wrote:
> But - a big gap right now is endian /portability/, and that one is a
> pain to cover with automated tests because you either need access to
> both big and little endian hardware (at a minumm for creating test
> images), or you need to run qemu in full-emulation mode, which is pretty
> unbearably slow.
It's really not that bad, at least for my use cases:
https://www.wireguard.com/build-status/
This thing sends pings to my cellphone too. You can poke around in
tools/testing/selftests/wireguard/qemu/ if you're curious. It's kinda
gnarly but has proven very very flexible to hack up for whatever
additional testing I need. For example, I've been using it for some of
my recent non-wireguard work here: https://git.zx2c4.com/linux-rng/commit/?h=jd/vdso-test-harness
Taking this straight-up probably won't fit for your filesystem work, but
maybe it can act as a bit of motivation that automated qemu'ing can
generally work. It has definitely caught a lot of silly bugs during
development time.
If for your cases, this winds up taking 3 days to run instead of the
minutes mine needs, so be it, that's a small workflow adjustment thing.
You might not get the same dopamine feedback loop of seeing your changes
in action and deployed to users _now_, but maybe delaying the
gratification a bit will be good anyway.
Jason
next prev parent reply other threads:[~2024-10-07 15:02 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-05 18:35 [GIT PULL] bcachefs fixes for 6.12-rc2 Kent Overstreet
2024-10-05 22:34 ` Linus Torvalds
2024-10-05 22:54 ` Kent Overstreet
2024-10-05 23:15 ` Linus Torvalds
2024-10-05 23:40 ` Kent Overstreet
2024-10-05 23:47 ` Kent Overstreet
2024-10-06 0:14 ` Linus Torvalds
2024-10-06 0:54 ` Kent Overstreet
2024-10-06 4:30 ` Theodore Ts'o
2024-10-06 4:33 ` Kent Overstreet
2024-10-06 19:04 ` Linus Torvalds
2024-10-06 19:29 ` Kent Overstreet
2024-10-06 21:31 ` Alan Huang
2024-10-07 15:01 ` Jason A. Donenfeld [this message]
2024-10-07 19:59 ` Kent Overstreet
2024-10-07 21:21 ` Jason A. Donenfeld
2024-10-07 23:33 ` Jann Horn
2024-10-09 3:51 ` Theodore Ts'o
2024-10-09 4:17 ` Kent Overstreet
2024-10-09 17:54 ` Theodore Ts'o
2024-10-10 8:51 ` Daniel Gomez
2024-10-06 11:49 ` Martin Steigerwald
2024-10-06 17:18 ` Kent Overstreet
2024-10-07 15:13 ` Martin Steigerwald
2024-10-06 1:20 ` Carl E. Thompson
2024-10-06 1:56 ` Kent Overstreet
2024-10-06 3:06 ` Carl E. Thompson
2024-10-06 3:42 ` Kent Overstreet
2024-10-07 14:58 ` Josef Bacik
2024-10-07 20:21 ` Kent Overstreet
2024-10-05 22:36 ` pr-tracker-bot
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=ZwP341bZ9eQ0Qyej@zx2c4.com \
--to=jason@zx2c4.com \
--cc=kent.overstreet@linux.dev \
--cc=linux-bcachefs@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@linux-foundation.org \
--cc=tytso@mit.edu \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox