* Re: [selftests] 7cb32086e5: kernel-selftests.x86.check_initial_reg_state_32.fail
[not found] <20200709064946.GQ3874@shao2-debian>
@ 2020-07-09 15:36 ` Shuah Khan
2020-07-10 6:02 ` Yauheni Kaliuta
0 siblings, 1 reply; 4+ messages in thread
From: Shuah Khan @ 2020-07-09 15:36 UTC (permalink / raw)
To: kernel test robot, Yauheni Kaliuta; +Cc: LKML, lkp, Shuah Khan
On 7/9/20 12:49 AM, kernel test robot wrote:
> Greeting,
>
> FYI, we noticed the following commit (built with gcc-9):
>
> commit: 7cb32086e59b514a832a3e11f5370d37e7cfe022 ("selftests: simplify run_tests")
> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git master
>
>
Thanks for the report. I will drop this patch for now from next.
Yauheni,
This patch broke x86 32-bit test run
make run_tests -C x86
Please resubmit the patch with the fix.
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [selftests] 7cb32086e5: kernel-selftests.x86.check_initial_reg_state_32.fail
2020-07-09 15:36 ` [selftests] 7cb32086e5: kernel-selftests.x86.check_initial_reg_state_32.fail Shuah Khan
@ 2020-07-10 6:02 ` Yauheni Kaliuta
2020-07-10 14:18 ` Shuah Khan
0 siblings, 1 reply; 4+ messages in thread
From: Yauheni Kaliuta @ 2020-07-10 6:02 UTC (permalink / raw)
To: Shuah Khan; +Cc: kernel test robot, LKML, lkp
On Thu, Jul 9, 2020 at 6:36 PM Shuah Khan <skhan@linuxfoundation.org> wrote:
>
> On 7/9/20 12:49 AM, kernel test robot wrote:
> > Greeting,
> >
> > FYI, we noticed the following commit (built with gcc-9):
> >
> > commit: 7cb32086e59b514a832a3e11f5370d37e7cfe022 ("selftests: simplify run_tests")
> > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git master
> >
> >
>
> Thanks for the report. I will drop this patch for now from next.
>
> Yauheni,
>
> This patch broke x86 32-bit test run
> make run_tests -C x86
>
> Please resubmit the patch with the fix.
I did not check carefully the report, but isn't it expected that some
tests are moved after the patch since they originally were placed
incorrectly?
--
WBR, Yauheni
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [selftests] 7cb32086e5: kernel-selftests.x86.check_initial_reg_state_32.fail
2020-07-10 6:02 ` Yauheni Kaliuta
@ 2020-07-10 14:18 ` Shuah Khan
2020-07-13 11:27 ` Yauheni Kaliuta
0 siblings, 1 reply; 4+ messages in thread
From: Shuah Khan @ 2020-07-10 14:18 UTC (permalink / raw)
To: Yauheni Kaliuta; +Cc: kernel test robot, LKML, lkp, Shuah Khan
On 7/10/20 12:02 AM, Yauheni Kaliuta wrote:
> On Thu, Jul 9, 2020 at 6:36 PM Shuah Khan <skhan@linuxfoundation.org> wrote:
>>
>> On 7/9/20 12:49 AM, kernel test robot wrote:
>>> Greeting,
>>>
>>> FYI, we noticed the following commit (built with gcc-9):
>>>
>>> commit: 7cb32086e59b514a832a3e11f5370d37e7cfe022 ("selftests: simplify run_tests")
>>> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git master
>>>
>>>
>>
>> Thanks for the report. I will drop this patch for now from next.
>>
>> Yauheni,
>>
>> This patch broke x86 32-bit test run
>> make run_tests -C x86
>>
>> Please resubmit the patch with the fix.
>
> I did not check carefully the report, but isn't it expected that some
> tests are moved after the patch since they originally were placed
> incorrectly?
>
>
The failure doesn't have anything to do with test being moved. You can
reproduce this very easily by running make as shown below in x86 dir
under tools/testing/selftests
make run_tests -C x86
I reproduced the problem with your and patch and verified that the
problem tracks your patch. I dropped the patch from linux-next
Your other two patches in the series are fine.
In any case, this patch isn't really adding any functionality and
is a good cleanup. Let's do the cleanup right or not.
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [selftests] 7cb32086e5: kernel-selftests.x86.check_initial_reg_state_32.fail
2020-07-10 14:18 ` Shuah Khan
@ 2020-07-13 11:27 ` Yauheni Kaliuta
0 siblings, 0 replies; 4+ messages in thread
From: Yauheni Kaliuta @ 2020-07-13 11:27 UTC (permalink / raw)
To: Shuah Khan; +Cc: kernel test robot, LKML, lkp
Hi, Shuah!
>>>>> On Fri, 10 Jul 2020 08:18:49 -0600, Shuah Khan wrote:
> On 7/10/20 12:02 AM, Yauheni Kaliuta wrote:
>> On Thu, Jul 9, 2020 at 6:36 PM Shuah Khan <skhan@linuxfoundation.org> wrote:
>>>
>>> On 7/9/20 12:49 AM, kernel test robot wrote:
>>>> Greeting,
>>>>
>>>> FYI, we noticed the following commit (built with gcc-9):
>>>>
>>>> commit: 7cb32086e59b514a832a3e11f5370d37e7cfe022 ("selftests:
>>>> simplify run_tests")
>>>> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git master
>>>>
>>>>
>>>
>>> Thanks for the report. I will drop this patch for now from next.
>>>
>>> Yauheni,
>>>
>>> This patch broke x86 32-bit test run
>>> make run_tests -C x86
>>>
>>> Please resubmit the patch with the fix.
>>
>> I did not check carefully the report, but isn't it expected that some
>> tests are moved after the patch since they originally were placed
>> incorrectly?
>>
>>
> The failure doesn't have anything to do with test being moved. You can
> reproduce this very easily by running make as shown below in x86 dir
> under tools/testing/selftests
> make run_tests -C x86
> I reproduced the problem with your and patch and verified that the
> problem tracks your patch. I dropped the patch from linux-next
> Your other two patches in the series are fine.
> In any case, this patch isn't really adding any functionality and
> is a good cleanup. Let's do the cleanup right or not.
Checked.
That is because with the patch both lib.mk and x86/Makefile add
the $(OUTPUT) prefix.
So the question is to agree about the convention, should lib.mk
targets expect short test names for TEST_PROGS or full path from
the subtests' Makefiles.
The existing code is hackish (incorrectly -- adding $(OUTPUT)
only to the first list members -- tries to handle it only for
out-of-tree build).
I can make the patch without adding $(OUTPUT). It will require to
fix possible tests which provided only one test and rely on that
behaviour for the OOT build. Do you have an easy way to get a
list of such tests?
--
WBR,
Yauheni Kaliuta
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-07-13 11:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20200709064946.GQ3874@shao2-debian>
2020-07-09 15:36 ` [selftests] 7cb32086e5: kernel-selftests.x86.check_initial_reg_state_32.fail Shuah Khan
2020-07-10 6:02 ` Yauheni Kaliuta
2020-07-10 14:18 ` Shuah Khan
2020-07-13 11:27 ` Yauheni Kaliuta
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.