Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Message-ID: <b99b5960-b1ec-b968-1d9c-d125a23c59fe@redhat.com>
Date:   Fri, 15 Oct 2021 17:47:03 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.1.0
Subject: Re: selftests/vm madv_populate.c test
Content-Language: en-US
To:     Shuah Khan <skhan@linuxfoundation.org>
Cc:     linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org
References: <b703a326-66f7-bf35-58ee-f60e504ea5ef@linuxfoundation.org>
 <0a20f6b6-5985-8b3e-a577-7495dcf7d2b8@redhat.com>
 <3a06d58e-7301-6fbc-a305-d9f7c7220843@linuxfoundation.org>
From:   David Hildenbrand <david@redhat.com>
Organization: Red Hat
In-Reply-To: <3a06d58e-7301-6fbc-a305-d9f7c7220843@linuxfoundation.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 15.10.21 17:45, Shuah Khan wrote:
> On 9/18/21 1:41 AM, David Hildenbrand wrote:
>> On 18.09.21 00:45, Shuah Khan wrote:
>>> Hi David,
>>>
>>> I am running into the following warning when try to build this test:
>>>
>>> madv_populate.c:334:2: warning: #warning "missing MADV_POPULATE_READ or MADV_POPULATE_WRITE definition" [-Wcpp]
>>>     334 | #warning "missing MADV_POPULATE_READ or MADV_POPULATE_WRITE definition"
>>>         |  ^~~~~~~
>>>
>>>
>>> I see that the following handling is in place. However there is no
>>> other information to explain why the check is necessary.
>>>
>>> #if defined(MADV_POPULATE_READ) && defined(MADV_POPULATE_WRITE)
>>>
>>> #else /* defined(MADV_POPULATE_READ) && defined(MADV_POPULATE_WRITE) */
>>>
>>> #warning "missing MADV_POPULATE_READ or MADV_POPULATE_WRITE definition"
>>>
>>> I do see these defined in:
>>>
>>> include/uapi/asm-generic/mman-common.h:#define MADV_POPULATE_READ       22
>>> include/uapi/asm-generic/mman-common.h:#define MADV_POPULATE_WRITE      23
>>>
>>> Is this the case of missing include from madv_populate.c?
>>
>> Hi Shuan,
>>
>> note that we're including "#include <sys/mman.h>", which in my
>> understanding maps to the version installed on your system instead
>> of the one in our build environment.ing.
>>
>> So as soon as you have a proper kernel + the proper headers installed
>> and try to build, it would pick up MADV_POPULATE_READ and
>> MADV_POPULATE_WRITE from the updated headers. That makes sense: you
>> annot run any MADV_POPULATE_READ/MADV_POPULATE_WRITE tests on a kernel
>> that doesn't support it.
>>
>> See vm/userfaultfd.c where we do something similar.
>>
> 
> Kselftest is for testing the kernel with kernel headers. That is the
> reason why there is the dependency on header install.
> 
>>
>> As soon as we have a proper environment, it seems to work just fine:
>>
>> Linux vm-0 5.15.0-0.rc1.20210915git3ca706c189db.13.fc36.x86_64 #1 SMP Thu Sep 16 11:32:54 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
>> [root@vm-0 linux]# cat /etc/redhat-release
>> Fedora release 36 (Rawhide)
> 
> This is a distro release. We don't want to have dependency on headers
> from the distro to run selftests. Hope this makes sense.
> 
> I still see this on my test system running Linux 5.15-rc5.

Did you also install Linux headers? I assume no, correct?

-- 
Thanks,

David / dhildenb