Re: [PATCH 11/12] selftests/mm: fix missing UFFDIO_CONTINUE_MODE_WP and similar build failures
From: John Hubbard
Date: Sat Jun 03 2023 - 19:48:22 EST
On 6/3/23 01:27, David Hildenbrand wrote:
> On 03.06.23 00:20, John Hubbard wrote:
>> On 6/2/23 03:23, David Hildenbrand wrote:
>> ...
>>> Unfortunately, that seems to be the ugly way to handle this because
>>> including the in-tree headers seems to not work and I yet haven't
>>> figured out why (there were some changes back and forth so I lost track).
>>
>> Yes, ugly, and based on Muhammad's response, I plan on dropping this patch
>> entirely, in fact.
>
> Ah, finally I know why it sometimes worked and sometimes didn't ... yes,
> let's document that somehow.
>
> Maybe we can even warn from the Makefile when the in-tree headers are
> not installed yet?
>
Yes. And in fact, after some fooling around getting reacquainted with
selftest Makefiles, it turns out that fussy part was setting up to
detect the missing header files. And that seems to be working reliably
now.
So from there, it's just as easy to automatically build any missing
header files and continue as it is to warn about it.
I'll post a patch.
thanks,
--
John Hubbard
NVIDIA