Re: [perf-tools-next:tmp.perf-tools-next] [perf dlfilter] f178a76b05: perf-sanity-tests.dlfilter_C_API.fail

[Adrian Hunter]

On 24/08/23 04:41, Oliver Sang wrote:
> hi, Adrian Hunter,
> 
> On Wed, Aug 23, 2023 at 06:37:49PM +0300, Adrian Hunter wrote:
>> On 23/08/23 17:57, kernel test robot wrote:
>>>
>>>
>>> Hello,
>>>
>>> kernel test robot noticed "perf-sanity-tests.dlfilter_C_API.fail" on:
>>>
>>> commit: f178a76b054fd046d212c3c67745146ff191a443 ("perf dlfilter: Add a test for resolve_address()")
>>> https://git.kernel.org/cgit/linux/kernel/git/perf/perf-tools-next.git tmp.perf-tools-next
>>>
>>> [test failed on linux-next/master 28c736b0e92e11bfe2b9997688213dc43cb22182]
>>
>> We sometimes make a test for an issue then fix it, so the new test fails,
>> but is fixed in a subsequent patch.  If you read the commit it says as much:
>>
>> commit f178a76b054fd046d212c3c67745146ff191a443
>> Author: Adrian Hunter <adrian.hunter@xxxxxxxxx>
>> Date:   Mon Jul 31 12:18:55 2023 +0300
>>
>>     perf dlfilter: Add a test for resolve_address()
>>     
>>     Extend the "dlfilter C API" test to test
>>     perf_dlfilter_fns.resolve_address(). The test currently fails, but passes
>>     after a subsequent patch.
>>     
>>     Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>
>>     Signed-off-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>
>>     Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
>>     Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
>>     Link: https://lore.kernel.org/r/20230731091857.10681-1-adrian.hunter@xxxxxxxxx
>>     Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>>
>> The fix is:
>>
>> 42c6dd9d23019ff339d0aca80a444eb71087050e perf dlfilter: Initialize addr_location before passing it to thread__find_symbol_fb()
> 
> thanks a lot for information!
> 
> as above mentioned, when this auto-bisect done, it confirmed the issue still
> exists linux-next/master:
> [test failed on linux-next/master 28c736b0e92e11bfe2b9997688213dc43cb22182]
> which includes 42c6dd9d23019ff339d0aca80a444eb71087050e
> 
> we will test again on latest linux-next/master:
> e3f80d3eae76c (tag: next-20230823, linux-next/master, broonie-ci/next-master) Add linux-next specific files for 20230823
> and 42c6dd9d23019ff339d0aca80a444eb71087050e
> 
> Thanks

Could you include the output from:

	perf test -v dlfilter

> 
>>>
>>> in testcase: perf-sanity-tests
>>> version: perf-x86_64-00c7b5f4ddc5-1_20230402
>>> with following parameters:
>>>
>>> 	perf_compiler: gcc
>>>
>>>
>>>
>>> compiler: gcc-12
>>> test machine: 224 threads 2 sockets Intel(R) Xeon(R) Platinum 8480+ (Sapphire Rapids) with 256G memory
>>>
>>> (please refer to attached dmesg/kmsg for entire log/backtrace)
>>>
>>>
>>>
>>>
>>> If you fix the issue in a separate patch/commit (i.e. not just a new version of
>>> the same patch/commit), kindly add following tags
>>> | Reported-by: kernel test robot <oliver.sang@xxxxxxxxx>
>>> | Closes: https://lore.kernel.org/oe-lkp/202308232146.94d82cb4-oliver.sang@xxxxxxxxx
>>>
>>>
>>>
>>>  66: Convert perf time to TSC                                        :
>>>  66.1: TSC support                                                   : Ok
>>>  66.2: Perf time to TSC                                              : Ok
>>>  67: dlfilter C API                                                  : FAILED!  <---
>>>  68: Sigtrap                                                         : Ok
>>>
>>>
>>>
>>> The kernel config and materials to reproduce are available at:
>>> https://download.01.org/0day-ci/archive/20230823/202308232146.94d82cb4-oliver.sang@xxxxxxxxx
>>>
>>>
>>>
>>