Re: perf test LLVM was: Re: [GIT PULL 00/39] perf tools: filtering events using eBPF programs

["Wangnan (F)" <wangnan0@huawei.com>]

Hi Arnaldo,

I noticed that you have rebase your perf/ebpf branch to 4.2-rc3. I have also
done it in my tree. Please see:

  https://github.com/WangNan0/linux.git ebpf

Also, please recollect patch "perf tests: Add LLVM test for eBPF on-the-fly
compiling", which adds verbose control. I'll also post it again.

Thank you.

On 2015/7/21 19:41, Arnaldo Carvalho de Melo wrote:
> Em Tue, Jul 21, 2015 at 07:09:12PM +0800, Wangnan (F) escreveu:
>> Hi Arnaldo,
>>
>> Have you recovered from your cold?
> Much better now, thanks.
>   
>> Could you please repick patch "perf tools: Introduce llvm config options"
>> from my github:
>>
>> https://github.com/WangNan0/linux/commit/f9fb25c518c7e52a47e98c36ce49f51051bbccf4
>>
>> I made a small change that introduce '-D__KERNEL__' in front of
>> $CLANG_OPTIONS,
>> so samples in 'samples/ebpf' should be compilable with perf now.
>>
>> I'll repost a patch by replying the original one also.
> Ok, saw that 12/39 repost, will pick it
>   
>> The following changes since commit 101ef9b4f5c70dbfffa0186102c1014bd81a4ec7:
>>
>>    bpf tools: Link all bpf objects onto a list (2015-07-13 18:48:17 -0300)
>>
>> are available in the git repository at:
>>
>>    https://github.com/WangNan0/linux.git ebpf
>>
>> for you to fetch changes up to 51f169df1d11ea842a808d751f9c620371c55e74:
>>
>>    bpf: Introduce function for outputing data to perf event (2015-07-21
>> 10:55:48 +0000)
>>
>> ----------------------------------------------------------------
>>
>> Thank you.
>>
>>
>> On 2015/7/17 11:48, Arnaldo Carvalho de Melo wrote:
>>> On Jul 16, 2015 11:34 PM, "Wangnan (F)" <wangnan0@huawei.com
>>> <mailto:wangnan0@huawei.com>> wrote:
>>>> On 2015/7/15 19:20, Arnaldo Carvalho de Melo wrote:
>>>>> Em Wed, Jul 15, 2015 at 06:49:40PM +0800, Wangnan (F) escreveu:
>>>>>> On 2015/7/14 23:36, Arnaldo Carvalho de Melo wrote:
>>>>>>> I have the trees cloned and plan to follow these instructions to have
>>> this test
>>>>>>> passing, to then move on to the next patches, but one request, please
>>> change
>>>>>>> the above to produce:
>>>>>>> 38: Test LLVM searching and compiling               :(no clang) Ok
>>> run 'perf test -v LLVM' for instructions on having a clang environment
>>>>>> The output format you desire is hard to implement, because 'Ok' is
>>> printed
>>>>>> by the parent process. We have to find a way to allow child process to
>>> pass
>>>>>> 'run 'perf test -v LLVM' for ...' to parent, then parent can print it
>>>>>> after 'Ok'.
>>>>>
>>>>>> I change this test as follow:
>>>>>   From a quick look: excellent, thanks for improving this, I downloaded
>>>>> llvm and clang, built it, and the test passes, will apply this new
>>> patch
>>>>> and continue from there,
>>>>>
>>>>> - Arnaldo
>>>>
>>>> Hi Arnaldo,
>>>>
>>>> What's the reviewing going on?
>>> Was sidetracked with some other stuff and was a bit slow due to a cold,
>>> hope to get back to work on ebpf+perf in 8 hours ir so.
>>>
>>> Thanks,
>>>
>>> Arnaldo
>>>
>>>> I collected some more patches on my git tree to avoid losting, and will
>>> collect more on it.
>>>> I'll keep this patchset at the bottom so I think you shoud not be
>>> affected.
>>>> Here is the new branch:
>>>>
>>>> https://github.com/WangNan0/linux.git ebpf
>>>>
>>>> Do you want me to keep the original perf/ebpf-for-acme branch?
>>>>
>>>> Thank you.
>>>>
>>>>