[Mesa-dev] [PATCH 1/5] intel: aubinator: add missing return characters

[Gandikota, Sirisha]

>-----Original Message-----
>From: mesa-dev [mailto:mesa-dev-bounces at lists.freedesktop.org] On Behalf Of
>Lionel Landwerlin
>Sent: Wednesday, October 05, 2016 8:55 AM
>To: mesa-dev at lists.freedesktop.org
>Cc: Landwerlin, Lionel G <lionel.g.landwerlin at intel.com>
>Subject: [Mesa-dev] [PATCH 1/5] intel: aubinator: add missing return characters
>
>Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
>---
> src/intel/tools/aubinator.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
>diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c index
>6fc0208..b74885a 100644
>--- a/src/intel/tools/aubinator.c
>+++ b/src/intel/tools/aubinator.c
>@@ -807,7 +807,7 @@ handle_trace_block(struct gen_spec *spec, uint32_t *p)
>       if (address_space != AUB_TRACE_MEMTYPE_GTT)
>          break;
>       if (gtt_size < offset + size) {
>-         fprintf(stderr, "overflow gtt space: %s", strerror(errno));
>+         fprintf(stderr, "overflow gtt space: %s\n", strerror(errno));
>          exit(EXIT_FAILURE);
>       }
>       memcpy((char *) gtt + offset, data, size); @@ -849,19 +849,19 @@
>aub_file_open(const char *filename)
>    file->filename = strdup(filename);
>    file->fd = open(file->filename, O_RDONLY);
>    if (file->fd == -1) {
>-      fprintf(stderr, "open %s failed: %s", file->filename, strerror(errno));
>+      fprintf(stderr, "open %s failed: %s\n", file->filename,
>+ strerror(errno));
>       exit(EXIT_FAILURE);
>    }
>
>    if (fstat(file->fd, &file->sb) == -1) {
>-      fprintf(stderr, "stat failed: %s", strerror(errno));
>+      fprintf(stderr, "stat failed: %s\n", strerror(errno));
>       exit(EXIT_FAILURE);
>    }
>
>    file->map = mmap(NULL, file->sb.st_size,
>                     PROT_READ, MAP_SHARED, file->fd, 0);
>    if (file->map == MAP_FAILED) {
>-      fprintf(stderr, "mmap failed: %s", strerror(errno));
>+      fprintf(stderr, "mmap failed: %s\n", strerror(errno));
>       exit(EXIT_FAILURE);
>    }
>
>@@ -873,7 +873,7 @@ aub_file_open(const char *filename)
>    gtt = mmap(NULL, gtt_size, PROT_READ | PROT_WRITE,
>               MAP_PRIVATE | MAP_ANONYMOUS |  MAP_NORESERVE, -1, 0);
>    if (gtt == MAP_FAILED) {
>-      fprintf(stderr, "failed to alloc gtt space: %s", strerror(errno));
>+      fprintf(stderr, "failed to alloc gtt space: %s\n",
>+ strerror(errno));
>       exit(1);
>    }
>
>--
>2.9.3

[SG] Works for me
Reviewed-by: Sirisha Gandikota<Sirisha.gandikota at intel.com>
>_______________________________________________
>mesa-dev mailing list
>mesa-dev at lists.freedesktop.org
>https://lists.freedesktop.org/mailman/listinfo/mesa-dev