From: Kan Liang <kan.li...@linux.intel.com>

Using the 'start' and 'end' which are stored in struct perf_mmap to
replace the temporary 'start' and 'end'.
The temporary variables will be discarded later.

No functional change.

Signed-off-by: Kan Liang <kan.li...@linux.intel.com>
---
 tools/perf/util/mmap.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/tools/perf/util/mmap.c b/tools/perf/util/mmap.c
index 642b479..4cb3614 100644
--- a/tools/perf/util/mmap.c
+++ b/tools/perf/util/mmap.c
@@ -294,12 +294,12 @@ int perf_mmap__push(struct perf_mmap *md, bool overwrite,
        if (rc < 0)
                return (rc == -EAGAIN) ? 0 : -1;
 
-       size = end - start;
+       size = md->end - md->start;
 
-       if ((start & md->mask) + size != (end & md->mask)) {
-               buf = &data[start & md->mask];
-               size = md->mask + 1 - (start & md->mask);
-               start += size;
+       if ((md->start & md->mask) + size != (md->end & md->mask)) {
+               buf = &data[md->start & md->mask];
+               size = md->mask + 1 - (md->start & md->mask);
+               md->start += size;
 
                if (push(to, buf, size) < 0) {
                        rc = -1;
@@ -307,9 +307,9 @@ int perf_mmap__push(struct perf_mmap *md, bool overwrite,
                }
        }
 
-       buf = &data[start & md->mask];
-       size = end - start;
-       start += size;
+       buf = &data[md->start & md->mask];
+       size = md->end - md->start;
+       md->start += size;
 
        if (push(to, buf, size) < 0) {
                rc = -1;
-- 
2.4.11

Reply via email to