On Wed, Sep 05, 2018 at 10:39:25AM +0300, Alexey Budankov wrote: SNIP
> + } else if (errno != EAGAIN) { > + cblock->aio_fildes = -1; > + pr_err("failed to queue perf data, error: %m\n"); > + break; > + } > + } while (1); > + > + return rc; > +} > + > +static int record__aio_sync(struct perf_mmap *md) > +{ this is almost identical to record__aio_sync function, it looks like we should be able to do the sync with single function.. for both the in-between syncs and the final one jirka