Re: [PATCH 7/7] perf tools: add feature header record to pipe-mode

From: Jiri Olsa
Date: Thu May 18 2017 - 12:13:19 EST


On Wed, May 17, 2017 at 09:16:02PM -0700, David Carrillo-Cisneros wrote:

SNIP

> struct header_print_data {
> @@ -2218,33 +2258,33 @@ int perf_header__fprintf_info(struct perf_session *session, FILE *fp, bool full)
> return 0;
> }
>
> -static int do_write_feat(struct feat_fd *fd, struct perf_header *h, int type,
> +static int do_write_feat(struct feat_fd *fdd, int type,
> struct perf_file_section **p,
> struct perf_evlist *evlist)
> {
> int err;
> int ret = 0;
>
> - if (perf_header__has_feat(h, type)) {
> + if (perf_header__has_feat(fdd->ph, type)) {

please do these changes in the earlier patch that introduced
the feat_fd as parameter in do_write_feat function

thanks,
jirka

> if (!feat_ops[type].write)
> return -1;
>
> - if (fd->buf) {
> - pr_err("do_write_feat to memory buffer\n");
> + if (fdd->buf) {
> + pr_debug("Called do_write_feat for memory buffer\n");
> return -1;
> }
> - (*p)->offset = lseek(fd->fd, 0, SEEK_CUR);
> + (*p)->offset = lseek(fdd->fd, 0, SEEK_CUR);
>
> - err = feat_ops[type].write(fd, evlist);
> + err = feat_ops[type].write(fdd, evlist);
> if (err < 0) {
> pr_debug("failed to write feature %s\n", feat_ops[type].name);
>
> /* undo anything written */
> - lseek(fd->fd, (*p)->offset, SEEK_SET);
> + lseek(fdd->fd, (*p)->offset, SEEK_SET);
>
> return -1;
> }
> - (*p)->size = lseek(fd->fd, 0, SEEK_CUR) - (*p)->offset;
> + (*p)->size = lseek(fdd->fd, 0, SEEK_CUR) - (*p)->offset;
> (*p)++;
> }
> return ret;
> @@ -2261,10 +2301,6 @@ static int perf_header__adds_write(struct perf_header *header,
> int feat;
> int err;
>
> - /*
> - * may write more than needed due to dropped feature, but
> - * this is okay, reader will skip the mising entries
> - */
> fdd = (struct feat_fd){
> .fd = fd,
> .ph = header,
> @@ -2284,7 +2320,7 @@ static int perf_header__adds_write(struct perf_header *header,
> lseek(fd, sec_start + sec_size, SEEK_SET);
>
> for_each_set_bit(feat, header->adds_features, HEADER_FEAT_BITS) {
> - if (do_write_feat(&fdd, header, feat, &p, evlist))
> + if (do_write_feat(&fdd, feat, &p, evlist))
> perf_header__clear_feat(header, feat);

SNIP