[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8a2ce635-58f4-44e1-a646-6527936c5836@redhat.com>
Date: Mon, 11 Dec 2023 19:00:32 +0100
From: David Hildenbrand <david@...hat.com>
To: Mark Brown <broonie@...nel.org>
Cc: Suren Baghdasaryan <surenb@...gle.com>, akpm@...ux-foundation.org,
viro@...iv.linux.org.uk, brauner@...nel.org, shuah@...nel.org,
aarcange@...hat.com, lokeshgidra@...gle.com, peterx@...hat.com,
ryan.roberts@....com, hughd@...gle.com, mhocko@...e.com,
axelrasmussen@...gle.com, rppt@...nel.org, willy@...radead.org,
Liam.Howlett@...cle.com, jannh@...gle.com, zhangpeng362@...wei.com,
bgeffon@...gle.com, kaleshsingh@...gle.com, ngeoffray@...gle.com,
jdduke@...gle.com, linux-mm@...ck.org,
linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-kselftest@...r.kernel.org, kernel-team@...roid.com,
John Hubbard <jhubbard@...dia.com>,
Peter Zijlstra <peterz@...radead.org>
Subject: Re: [PATCH v6 5/5] selftests/mm: add UFFDIO_MOVE ioctl test
On 11.12.23 18:32, Mark Brown wrote:
> On Mon, Dec 11, 2023 at 05:53:59PM +0100, David Hildenbrand wrote:
>
>>>> (3) avoids dirtying the tree as a "make headers_install" would, but it also
>>>> means that each test that makes use of new uapi has to update the relevant
>>>> headers (what people working on QEMU are used to).
>
>>> Note that you can do an out of tree build to avoid dirtying things.
>
>> Yes, but apparently the simple "make headers_install" will dirty the kernel.
>
>> See (and ideally comment on)
>
>> https://lkml.kernel.org/r/20231209020144.244759-1-jhubbard@nvidia.com
>
> I mean, I guess people who don't want to install the headers are just
> not going to be able to build a bunch of tests? There definitely are a
> bunch of tests where it's not needed so I can see why people would not
> like being forced to do the headers step if they're only interested in
> those tests.
Yes. And before that, people mostly had no clue that headers had to be
installed in order to compile successfully.
So maybe a warning to give at least some hint might be reasonable.
--
Cheers,
David / dhildenb
Powered by blists - more mailing lists