| Date | Wed, 16 Jul 2014 16:20:31 +0200 | From | Jiri Olsa <> | Subject | Re: [PATCH 18/41] perf tools: Record whether a dso has data |
| |
On Mon, Jul 14, 2014 at 01:02:42PM +0300, Adrian Hunter wrote: > Add 'data_status' to record whether a dso has data > (i.e. an object file) as follows: > > dso->data_status meaning > ---------------------------------- > 0 don't know > -1 no data > 1 has data
please add enum for this
> > Signed-off-by: Adrian Hunter <adrian.hunter@intel.com> > --- > tools/perf/util/dso.c | 5 ++++- > tools/perf/util/dso.h | 1 + > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/tools/perf/util/dso.c b/tools/perf/util/dso.c > index fc006fe..91a4eb4 100644 > --- a/tools/perf/util/dso.c > +++ b/tools/perf/util/dso.c > @@ -556,8 +556,11 @@ static ssize_t data_read_offset(struct dso *dso, u64 offset, > ssize_t dso__data_read_offset(struct dso *dso, struct machine *machine, > u64 offset, u8 *data, ssize_t size) > { > - if (dso__data_fd(dso, machine) < 0) > + if (dso__data_fd(dso, machine) < 0) { > + dso->data.data_status = -1; > return -1; > + } > + dso->data.data_status = 1;
we have more callers to dso__data_fd, so I think we want to setup dso->data.data_status within dso__data_fd
jirka
|