On Fri, Apr 24, 2015 at 10:29:46PM +0300, Adrian Hunter wrote:
SNIP
> +
> +int auxtrace_index__process(int fd, u64 size, struct perf_session *session,
> + bool needs_swap)
> +{
> + struct list_head *head = &session->auxtrace_index;
> + u64 nr;
> +
> + if (readn(fd, &nr, sizeof(u64)) != sizeof(u64))
> + return -1;
> +
> + if (needs_swap)
> + nr = bswap_64(nr);
> +
> + if (sizeof(u64) + nr * sizeof(struct auxtrace_index_entry) != size)
> + return -1;
so the total nr is only used to double check the data size is correct,
it could be omited and you could use just the size of the section..
also the auxtrace_index__write would be shorter of one loop
but no big deal.. just asking ;-)
jirka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
