Re: [PATCH 04/10] perf, tools: Handle header line in mapfile

[Sukadev Bhattiprolu]

Jiri Olsa [jolsa@xxxxxxxxxx] wrote:
| >  		if (line[0] == '#' || line[0] == '\n')
| >  			continue;
| > +		if (!strncmp(line, "Family", 6))
| > +			continue;
| 
| I think we should fix mapfiles to put the 'Family' starting
| line as a comment.. the way powerpc mapfile is done

You mean add something like this to the Intel mapfile:

	# Power8 entries
	004d0100,1,power8.json,core

and drop this patch?

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/