Commit e840238d authored May 21, 2015 by Namhyung Kim Committed by Arnaldo Carvalho de Melo May 27, 2015

perf tools: Get rid of dso__data_fd() from dso__data_size()



It seems that the dso__data_fd() was needed to find a binary type
since open in data_file_size() alone used to fail.

But as it can open the dso fine now, the dso__data_fd() can go away.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Acked-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1432137821-10853-2-git-send-email-namhyung@kernel.org


Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

parent 71ff824a

tools/perf/util/dso.c

+0 −6

Original line number	Diff line number	Diff line
		@@ -745,12 +745,6 @@ static int data_file_size(struct dso dso, struct machine machine)
		*/
		off_t dso__data_size(struct dso dso, struct machine machine)
		{
		int fd;

		fd = dso__data_fd(dso, machine);
		if (fd < 0)
		return fd;

		if (data_file_size(dso, machine))
		return -1;