From mboxrd@z Thu Jan 1 00:00:00 1970 From: namhyung@kernel.org (Namhyung Kim) Date: Thu, 29 Jan 2015 15:54:50 +0900 Subject: [PATCH V3 2/2] perf symbols: debuglink should take symfs option into account In-Reply-To: References: <1422340442-4673-1-git-send-email-victor.kamensky@linaro.org> <1422340442-4673-3-git-send-email-victor.kamensky@linaro.org> <20150129013809.GB24182@sejong> Message-ID: <20150129065450.GD24182@sejong> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Jan 28, 2015 at 08:08:30PM -0800, Victor Kamensky wrote: > Hi Namhyung, > > On 28 January 2015 at 17:38, Namhyung Kim wrote: > > Hi Victor, > > > > On Mon, Jan 26, 2015 at 10:34:02PM -0800, Victor Kamensky wrote: > >> Currently code that tries to read corresponding debug symbol > >> file from .gnu_debuglink section (DSO_BINARY_TYPE__DEBUGLINK) > >> does not take in account symfs option, so filename__read_debuglink > >> function cannot open ELF file, if symfs option is used. > >> > >> Fix is to add proper handling of symfs as it is done in other > >> places: use __symbol__join_symfs function to get real file name > >> of target ELF file. > >> > >> Signed-off-by: Victor Kamensky > >> Cc: Peter Zijlstra > >> Cc: Paul Mackerras > >> Cc: Ingo Molnar > >> Cc: Arnaldo Carvalho de Melo > >> Cc: Jiri Olsa > >> Cc: Adrian Hunter > >> Cc: Waiman Long > >> Cc: David Ahern > >> Acked-and-tested-by: David Ahern > > > > Acked-by: Namhyung Kim > > Thank you for the Acks! Is there any action items on my side? > Should I repost patches with your Acks? Or you or Arnaldo > will add them once they got merged in proper tree for > upstreaming? I think Arnaldo will merge them with Acks. ;-) Thanks, Namhyung