Re: [PATCH 1/3] perf core: Fix a mmap and munmap mismatched bug

2013-10-14 Thread Arnaldo Carvalho de Melo
Please use "perf tools: " as the prefix, I was assuming that "perf core: " was related to the kernel code and was skipping those patches. I'll fix up the prefix to use the standard used for the various files in tools/perf, "perf symbols: " for this patch, for instance. Applied this one and I'm ch

[PATCH 1/3] perf core: Fix a mmap and munmap mismatched bug

2013-10-10 Thread Chenggang Qin
From: root In function filename__read_debuglink(), while the ELF file is opend and mmapped in elf_begin(), but if this file is considered to not be usable during the following code, we will goto the close(fd) directly. The elf_end() is skipped. So, the mmaped ELF file cannot be munmapped. The mem

Re: [PATCH 1/3] perf core: Fix a mmap and munmap mismatched bug

2013-09-03 Thread Namhyung Kim
Hi Chenggang, On Sun, 1 Sep 2013 23:29:43 +0800, Chenggang Qin wrote: > From: root > > In function filename__read_debuglink(), while the ELF file is opend and > mmapped > in elf_begin(), but if this file is considered to not be usable during the > following code, we will goto the close(fd) dire

[PATCH 1/3] perf core: Fix a mmap and munmap mismatched bug

2013-09-01 Thread Chenggang Qin
From: root In function filename__read_debuglink(), while the ELF file is opend and mmapped in elf_begin(), but if this file is considered to not be usable during the following code, we will goto the close(fd) directly. The elf_end() is skipped. So, the mmaped ELF file cannot be munmapped. The mem