Re: [PATCH v6 5/5] selftests/mm: add UFFDIO_MOVE ioctl test

From: Mark Brown
Date: Mon Dec 11 2023 - 15:22:19 EST


On Mon, Dec 11, 2023 at 10:46:23AM -0800, John Hubbard wrote:

> Or (4) Hack in little ifdef snippets, into the selftests, like we used
> to do. Peter Zijlstra seems to be asking for this, if I understand his
> (much) earlier comments about this.

I can't help but think that if we're having to manually copy bits of
the uapi headers (which are already separated out in the source) into
another part of the same source tree in order to use them then there's
room for improvement somewhere. TBH it also doesn't seem great to add
additional variables that depend on the user's build environment, we
already have enough build issues. It ought to be mostly tedious rather
than hard but it's still a pain, especially given the issues we have
getting kselftest fixes merged promptly.

Attachment: signature.asc
Description: PGP signature