All of lore.kernel.org
 help / color / mirror / Atom feed
* Pull patches from tip/perf/core to bpf-next
@ 2019-02-05 22:47 Song Liu
  2019-02-06  3:36 ` Alexei Starovoitov
  0 siblings, 1 reply; 5+ messages in thread
From: Song Liu @ 2019-02-05 22:47 UTC (permalink / raw)
  To: Daniel Borkmann, Alexei Starovoitov; +Cc: Kernel Team, Netdev

Hi Alexei and Daniel, 

The following patches are required for BPF introspection in perf tools. 
Please pull them to bpf-next, so that we get all the dependencies in one
tree. 

Thanks,
Song

(from 1/10 to 10/10)
76193a94522f perf, bpf: Introduce PERF_RECORD_KSYMBOL
d764ac646491 tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
6ee52e2a3fe4 perf, bpf: Introduce PERF_RECORD_BPF_EVENT
df063c83aa2c tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
9aa0bfa370b2 perf tools: Handle PERF_RECORD_KSYMBOL
45178a928a4b perf tools: Handle PERF_RECORD_BPF_EVENT
7b612e291a5a perf tools: Synthesize PERF_RECORD_* for loaded BPF programs
a40b95bcd30c perf top: Synthesize BPF events for pre-existing loaded BPF programs
6934058d9fb6 bpf: Add module name [bpf] to ksymbols for bpf programs
811184fb6977 perf bpf: Fix synthesized PERF_RECORD_KSYMBOL/BPF_EVENT

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Pull patches from tip/perf/core to bpf-next
  2019-02-05 22:47 Pull patches from tip/perf/core to bpf-next Song Liu
@ 2019-02-06  3:36 ` Alexei Starovoitov
  2019-02-06  3:44   ` Song Liu
  0 siblings, 1 reply; 5+ messages in thread
From: Alexei Starovoitov @ 2019-02-06  3:36 UTC (permalink / raw)
  To: Song Liu; +Cc: Daniel Borkmann, Alexei Starovoitov, Kernel Team, Netdev

On Tue, Feb 05, 2019 at 10:47:06PM +0000, Song Liu wrote:
> Hi Alexei and Daniel, 
> 
> The following patches are required for BPF introspection in perf tools. 
> Please pull them to bpf-next, so that we get all the dependencies in one
> tree. 
> 
> Thanks,
> Song
> 
> (from 1/10 to 10/10)
> 76193a94522f perf, bpf: Introduce PERF_RECORD_KSYMBOL
> d764ac646491 tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
> 6ee52e2a3fe4 perf, bpf: Introduce PERF_RECORD_BPF_EVENT
> df063c83aa2c tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
> 9aa0bfa370b2 perf tools: Handle PERF_RECORD_KSYMBOL
> 45178a928a4b perf tools: Handle PERF_RECORD_BPF_EVENT
> 7b612e291a5a perf tools: Synthesize PERF_RECORD_* for loaded BPF programs
> a40b95bcd30c perf top: Synthesize BPF events for pre-existing loaded BPF programs
> 6934058d9fb6 bpf: Add module name [bpf] to ksymbols for bpf programs
> 811184fb6977 perf bpf: Fix synthesized PERF_RECORD_KSYMBOL/BPF_EVENT

yes. we can certainly do that.
Do you have bpf specific patches that depend on that ?
Since it's rc5 already. Are you planning to send them within next week?


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Pull patches from tip/perf/core to bpf-next
  2019-02-06  3:36 ` Alexei Starovoitov
@ 2019-02-06  3:44   ` Song Liu
  2019-02-06  3:47     ` Alexei Starovoitov
  0 siblings, 1 reply; 5+ messages in thread
From: Song Liu @ 2019-02-06  3:44 UTC (permalink / raw)
  To: Alexei Starovoitov
  Cc: Daniel Borkmann, Alexei Starovoitov, Kernel Team, Netdev



> On Feb 5, 2019, at 7:36 PM, Alexei Starovoitov <alexei.starovoitov@gmail.com> wrote:
> 
> On Tue, Feb 05, 2019 at 10:47:06PM +0000, Song Liu wrote:
>> Hi Alexei and Daniel, 
>> 
>> The following patches are required for BPF introspection in perf tools. 
>> Please pull them to bpf-next, so that we get all the dependencies in one
>> tree. 
>> 
>> Thanks,
>> Song
>> 
>> (from 1/10 to 10/10)
>> 76193a94522f perf, bpf: Introduce PERF_RECORD_KSYMBOL
>> d764ac646491 tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
>> 6ee52e2a3fe4 perf, bpf: Introduce PERF_RECORD_BPF_EVENT
>> df063c83aa2c tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
>> 9aa0bfa370b2 perf tools: Handle PERF_RECORD_KSYMBOL
>> 45178a928a4b perf tools: Handle PERF_RECORD_BPF_EVENT
>> 7b612e291a5a perf tools: Synthesize PERF_RECORD_* for loaded BPF programs
>> a40b95bcd30c perf top: Synthesize BPF events for pre-existing loaded BPF programs
>> 6934058d9fb6 bpf: Add module name [bpf] to ksymbols for bpf programs
>> 811184fb6977 perf bpf: Fix synthesized PERF_RECORD_KSYMBOL/BPF_EVENT
> 
> yes. we can certainly do that.
> Do you have bpf specific patches that depend on that ?
> Since it's rc5 already. Are you planning to send them within next week?

BPF introspection work depends on these patches. I have been hopping 
between perf tree and bpf-next tree. I think basing the series up on 
bpf-next plus these patches leads least conflicts. 

I do plan to send the series within next week. 

On a second thought, maybe I should send based on perf tree, and worry
about the conflicts later? It is really heavier on perf side. 

Thanks,
Song


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Pull patches from tip/perf/core to bpf-next
  2019-02-06  3:44   ` Song Liu
@ 2019-02-06  3:47     ` Alexei Starovoitov
  2019-02-06  4:07       ` Song Liu
  0 siblings, 1 reply; 5+ messages in thread
From: Alexei Starovoitov @ 2019-02-06  3:47 UTC (permalink / raw)
  To: Song Liu; +Cc: Daniel Borkmann, Alexei Starovoitov, Kernel Team, Netdev

On Wed, Feb 06, 2019 at 03:44:29AM +0000, Song Liu wrote:
> 
> 
> > On Feb 5, 2019, at 7:36 PM, Alexei Starovoitov <alexei.starovoitov@gmail.com> wrote:
> > 
> > On Tue, Feb 05, 2019 at 10:47:06PM +0000, Song Liu wrote:
> >> Hi Alexei and Daniel, 
> >> 
> >> The following patches are required for BPF introspection in perf tools. 
> >> Please pull them to bpf-next, so that we get all the dependencies in one
> >> tree. 
> >> 
> >> Thanks,
> >> Song
> >> 
> >> (from 1/10 to 10/10)
> >> 76193a94522f perf, bpf: Introduce PERF_RECORD_KSYMBOL
> >> d764ac646491 tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
> >> 6ee52e2a3fe4 perf, bpf: Introduce PERF_RECORD_BPF_EVENT
> >> df063c83aa2c tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
> >> 9aa0bfa370b2 perf tools: Handle PERF_RECORD_KSYMBOL
> >> 45178a928a4b perf tools: Handle PERF_RECORD_BPF_EVENT
> >> 7b612e291a5a perf tools: Synthesize PERF_RECORD_* for loaded BPF programs
> >> a40b95bcd30c perf top: Synthesize BPF events for pre-existing loaded BPF programs
> >> 6934058d9fb6 bpf: Add module name [bpf] to ksymbols for bpf programs
> >> 811184fb6977 perf bpf: Fix synthesized PERF_RECORD_KSYMBOL/BPF_EVENT
> > 
> > yes. we can certainly do that.
> > Do you have bpf specific patches that depend on that ?
> > Since it's rc5 already. Are you planning to send them within next week?
> 
> BPF introspection work depends on these patches. I have been hopping 
> between perf tree and bpf-next tree. I think basing the series up on 
> bpf-next plus these patches leads least conflicts. 
> 
> I do plan to send the series within next week. 
> 
> On a second thought, maybe I should send based on perf tree, and worry
> about the conflicts later? It is really heavier on perf side. 

whichever way is easier.
if bpf-next is the best use that as a base with above patches.
Once your set gets Acks from perf folks we can push above patches
from tip first and then apply your set.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Pull patches from tip/perf/core to bpf-next
  2019-02-06  3:47     ` Alexei Starovoitov
@ 2019-02-06  4:07       ` Song Liu
  0 siblings, 0 replies; 5+ messages in thread
From: Song Liu @ 2019-02-06  4:07 UTC (permalink / raw)
  To: Alexei Starovoitov
  Cc: Daniel Borkmann, Alexei Starovoitov, Kernel Team, Netdev



> On Feb 5, 2019, at 7:47 PM, Alexei Starovoitov <alexei.starovoitov@gmail.com> wrote:
> 
> On Wed, Feb 06, 2019 at 03:44:29AM +0000, Song Liu wrote:
>> 
>> 
>>> On Feb 5, 2019, at 7:36 PM, Alexei Starovoitov <alexei.starovoitov@gmail.com> wrote:
>>> 
>>> On Tue, Feb 05, 2019 at 10:47:06PM +0000, Song Liu wrote:
>>>> Hi Alexei and Daniel, 
>>>> 
>>>> The following patches are required for BPF introspection in perf tools. 
>>>> Please pull them to bpf-next, so that we get all the dependencies in one
>>>> tree. 
>>>> 
>>>> Thanks,
>>>> Song
>>>> 
>>>> (from 1/10 to 10/10)
>>>> 76193a94522f perf, bpf: Introduce PERF_RECORD_KSYMBOL
>>>> d764ac646491 tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
>>>> 6ee52e2a3fe4 perf, bpf: Introduce PERF_RECORD_BPF_EVENT
>>>> df063c83aa2c tools headers uapi: Sync tools/include/uapi/linux/perf_event.h
>>>> 9aa0bfa370b2 perf tools: Handle PERF_RECORD_KSYMBOL
>>>> 45178a928a4b perf tools: Handle PERF_RECORD_BPF_EVENT
>>>> 7b612e291a5a perf tools: Synthesize PERF_RECORD_* for loaded BPF programs
>>>> a40b95bcd30c perf top: Synthesize BPF events for pre-existing loaded BPF programs
>>>> 6934058d9fb6 bpf: Add module name [bpf] to ksymbols for bpf programs
>>>> 811184fb6977 perf bpf: Fix synthesized PERF_RECORD_KSYMBOL/BPF_EVENT
>>> 
>>> yes. we can certainly do that.
>>> Do you have bpf specific patches that depend on that ?
>>> Since it's rc5 already. Are you planning to send them within next week?
>> 
>> BPF introspection work depends on these patches. I have been hopping 
>> between perf tree and bpf-next tree. I think basing the series up on 
>> bpf-next plus these patches leads least conflicts. 
>> 
>> I do plan to send the series within next week. 
>> 
>> On a second thought, maybe I should send based on perf tree, and worry
>> about the conflicts later? It is really heavier on perf side. 
> 
> whichever way is easier.
> if bpf-next is the best use that as a base with above patches.
> Once your set gets Acks from perf folks we can push above patches
> from tip first and then apply your set.

I see. Let's wait until the patchset is acked. 

Thanks,
Song

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2019-02-06  4:08 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-05 22:47 Pull patches from tip/perf/core to bpf-next Song Liu
2019-02-06  3:36 ` Alexei Starovoitov
2019-02-06  3:44   ` Song Liu
2019-02-06  3:47     ` Alexei Starovoitov
2019-02-06  4:07       ` Song Liu

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.