Re: [PATCH] perf record: Fix memory leak in vDSO

2021-03-24 Thread Arnaldo Carvalho de Melo
Em Tue, Mar 16, 2021 at 02:50:48PM +0100, Jiri Olsa escreveu: > On Tue, Mar 16, 2021 at 09:56:26AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Tue, Mar 16, 2021 at 11:28:12AM +0900, Namhyung Kim escreveu: > > > On Mon, Mar 15, 2021 at 10:28 PM Jiri Olsa wrote: > > > > > > > > On Mon, Mar 15,

Re: [PATCH] perf record: Fix memory leak in vDSO

2021-03-16 Thread Jiri Olsa
On Tue, Mar 16, 2021 at 09:56:26AM -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Mar 16, 2021 at 11:28:12AM +0900, Namhyung Kim escreveu: > > On Mon, Mar 15, 2021 at 10:28 PM Jiri Olsa wrote: > > > > > > On Mon, Mar 15, 2021 at 01:56:41PM +0900, Namhyung Kim wrote: > > > > I got several memory

Re: [PATCH] perf record: Fix memory leak in vDSO

2021-03-16 Thread Arnaldo Carvalho de Melo
Em Tue, Mar 16, 2021 at 11:28:12AM +0900, Namhyung Kim escreveu: > On Mon, Mar 15, 2021 at 10:28 PM Jiri Olsa wrote: > > > > On Mon, Mar 15, 2021 at 01:56:41PM +0900, Namhyung Kim wrote: > > > I got several memory leak reports from Asan with a simple command. It > > > was because VDSO is not

Re: [PATCH] perf record: Fix memory leak in vDSO

2021-03-15 Thread Namhyung Kim
On Mon, Mar 15, 2021 at 10:28 PM Jiri Olsa wrote: > > On Mon, Mar 15, 2021 at 01:56:41PM +0900, Namhyung Kim wrote: > > I got several memory leak reports from Asan with a simple command. It > > was because VDSO is not released due to the refcount. Like in > > __dsos_addnew_id(), it should put

Re: [PATCH] perf record: Fix memory leak in vDSO

2021-03-15 Thread Jiri Olsa
On Mon, Mar 15, 2021 at 01:56:41PM +0900, Namhyung Kim wrote: > I got several memory leak reports from Asan with a simple command. It > was because VDSO is not released due to the refcount. Like in > __dsos_addnew_id(), it should put the refcount after adding to the list. > > $ perf record

[PATCH] perf record: Fix memory leak in vDSO

2021-03-14 Thread Namhyung Kim
I got several memory leak reports from Asan with a simple command. It was because VDSO is not released due to the refcount. Like in __dsos_addnew_id(), it should put the refcount after adding to the list. $ perf record true [ perf record: Woken up 1 times to write data ] [ perf record: