[Intel-gfx] [PATCH 2/2] Simplification of guc logger design

Krzysztof E. Olinski krzysztof.e.olinski at intel.com
Thu Apr 27 08:59:20 UTC 2017


There are some compile problems for Android platform. The aim of this patch is to simplify the current design and make it compilable both on Linux and Android.

Signed-off-by: Krzysztof E. Olinski <krzysztof.e.olinski at intel.com>
---
 lib/Makefile.sources     |   4 +
 lib/igt_guclaw.c         | 272 +++++++++++++++++++++++++++
 lib/igt_guclaw.h         |  81 +++++++++
 tools/intel_guc_logger.c | 465 +++++++++--------------------------------------
 4 files changed, 443 insertions(+), 379 deletions(-)
 mode change 100644 => 100755 lib/Makefile.sources
 create mode 100755 lib/igt_guclaw.c
 create mode 100755 lib/igt_guclaw.h
 mode change 100644 => 100755 tools/intel_guc_logger.c

diff --git a/lib/Makefile.sources b/lib/Makefile.sources
old mode 100644
new mode 100755
index 6348487..89a0fee
--- a/lib/Makefile.sources
+++ b/lib/Makefile.sources
@@ -83,6 +83,10 @@ lib_source_list =	 	\
 	uwildmat/uwildmat.c	\
 	igt_kmod.c		\
 	igt_kmod.h		\
+	buc.c			\
+	buc.h			\
+	igt_guclaw.c		\
+	igt_guclaw.h 		\
 	$(NULL)
 
 if HAVE_CHAMELIUM
diff --git a/lib/igt_guclaw.c b/lib/igt_guclaw.c
new file mode 100755
index 0000000..6880f4d
--- /dev/null
+++ b/lib/igt_guclaw.c
@@ -0,0 +1,272 @@
+#define _GNU_SOURCE
+#include <inttypes.h>
+#include <stdio.h>
+#include <fcntl.h>
+#include <sched.h>
+#include <sys/ioctl.h>
+#include <string.h>
+#include <errno.h>
+#include <signal.h>
+#include <stdlib.h>
+#include <unistd.h>
+#include <sys/poll.h>
+#include <sys/wait.h>
+#include <sys/types.h>
+#include <sys/syscall.h>
+#include <sys/mman.h>
+#include <assert.h>
+#include <pthread.h>
+
+#include "igt.h"
+#include "buc.h"
+#include "igt_guclaw.h"
+
+#define MB(x) ((uint64_t)(x) * 1024 * 1024)
+#ifndef PAGE_SIZE
+  #define PAGE_SIZE 4096
+#endif
+/* Currently the size of GuC log buffer is 19 pages & so is the size of relay
+ * subbuffer. If the size changes in future, then this define also needs to be
+ * updated accordingly.
+ */
+#define SUBBUF_SIZE (19*PAGE_SIZE)
+/* Need large buffering from logger side to hide the DISK IO latency, Driver
+ * can only store 8 snapshots of GuC log buffer in relay.
+ */
+
+#define NUM_SUBBUFS 100
+
+#define RELAY_FILE_NAME  "guc_log"
+#define DEFAULT_OUTPUT_FILE_NAME  "guc_log_dump.dat"
+#define CONTROL_FILE_NAME "i915_guc_log_control"
+
+buc_t* my_buc = NULL;
+
+static guclaw_options_t guclaw_ops;
+
+uint64_t total_bytes_written;
+int relay_fd, drm_fd, outfile_fd = -1;
+bool stop_logging = false, suspend_logging = true;
+pthread_t guclaw_thread;
+
+
+static void guc_log_control(bool enable_logging)
+{
+    int control_fd;
+    char data[19];
+    uint64_t val;
+    int ret;
+
+    control_fd = igt_debugfs_open(drm_fd, CONTROL_FILE_NAME, O_WRONLY);
+    igt_assert_f(control_fd >= 0, "couldn't open the guc log control file\n");
+
+    val = enable_logging ? ((guclaw_ops.verbosity_level << 4) | 0x1) : 0;
+
+    ret = snprintf(data, sizeof(data), "0x%" PRIx64, val);
+    igt_assert(ret > 2 && ret < sizeof(data));
+
+    ret = write(control_fd, data, ret);
+    igt_assert_f(ret > 0, "couldn't write to the log control file\n");
+
+    close(control_fd);
+}
+
+static void pull_leftover_data(void)
+{
+    unsigned int bytes_read = 0;
+    int ret;
+    char rbuf[SUBBUF_SIZE];
+
+    do {
+            /* Read the logs from relay buffer */
+            ret = read(relay_fd, rbuf, SUBBUF_SIZE);
+            if (!ret)
+                break;
+
+            igt_assert_f(ret > 0, "failed to read from the guc log file\n");
+
+            bytes_read += ret;
+
+            if(guclaw_ops.discard_oldlogs)
+            {
+                total_bytes_written += ret;
+                if (outfile_fd >= 0)
+                   buc__append(my_buc, rbuf, ret);
+            }
+
+    } while(1);
+
+    igt_info("%u bytes flushed\n", bytes_read);
+}
+
+static void pull_data(void)
+{
+    int ret;
+    char rbuf[SUBBUF_SIZE];
+
+    do
+    {
+        /* Read the logs from relay buffer */
+        ret = read(relay_fd, rbuf, SUBBUF_SIZE);
+        if (!ret)
+            break;
+
+        igt_assert_f(ret >= 0, "failed to read from the guc log file\n");
+        total_bytes_written += ret;
+
+        if(!suspend_logging)
+            buc__append(my_buc, rbuf, ret);
+
+    } while(1);
+}
+
+static void open_relay_file(void)
+{
+    relay_fd = igt_debugfs_open(drm_fd, RELAY_FILE_NAME, O_RDONLY);
+    igt_assert_f(relay_fd >= 0, "couldn't open the guc log file\n");
+
+    /* Purge the old/boot-time logs from the relay buffer.
+     * This is more for Val team's requirement, where they have to first
+     * purge the existing logs before starting the tests for which the logs
+     * are actually needed. After this logger will enter into a loop and
+     * wait for the new data, at that point benchmark can be launched from
+     * a different shell.
+     */
+
+     pull_leftover_data();
+}
+
+static void open_output_file(void)
+{
+    outfile_fd = open(guclaw_ops.out_filename ? : DEFAULT_OUTPUT_FILE_NAME,
+                      O_CREAT | O_WRONLY | O_TRUNC, S_IWUSR|S_IROTH);
+
+    igt_assert_f(outfile_fd >= 0, "couldn't open the output file\n");
+    free(guclaw_ops.out_filename);
+}
+
+void* guclaw_routine(void *p)
+{
+    struct pollfd relay_poll_fd;
+    int nfds;
+    int ret;
+
+
+    open_relay_file();
+    relay_poll_fd.fd = relay_fd;
+    relay_poll_fd.events = POLLIN;
+    relay_poll_fd.revents = 0;
+    nfds = 1; /* only one fd to poll */
+
+    guc_log_control(true);   
+   
+    do {
+            /* Wait/poll for the new data to be available, relay doesn't
+            * provide a blocking read.
+            * On older kernels need to do polling with a timeout instead of
+            * indefinite wait to avoid relying on relay for the wakeup, as
+            * relay used to do the wakeup in a deferred manner on jiffies
+            * granularity by scheduling a timer and moreover that timer was
+            * re-scheduled on every newly produced buffer and so was pushed
+            * out if there were multiple flush interrupts in a very quick
+            * succession (less than a jiffy gap between 2 flush interrupts)
+            * causing relay to run out of sub buffers to store new logs.
+            */
+            ret = poll(&relay_poll_fd, nfds, guclaw_ops.poll_timeout);
+            if (ret < 0) {
+                if (errno == EINTR)
+                    break;
+                igt_assert_f(0, "poll call failed\n");
+            }
+
+            /* No data available yet, poll again, hopefully new data is round the corner */
+            if (!relay_poll_fd.revents)
+                continue;
+
+            pull_data();
+
+            if (guclaw_ops.max_filesize && (total_bytes_written > MB(guclaw_ops.max_filesize))) {
+                igt_debug("reached the target of %" PRIu64 " bytes\n", MB(guclaw_ops.max_filesize));
+                stop_logging = true;
+            }
+
+     } while (!stop_logging);
+
+     return NULL;
+}
+
+
+int guclaw_init(guclaw_options_t *op)
+{
+    int r;
+
+    guclaw_ops.verbosity_level= op->verbosity_level;
+    guclaw_ops.out_filename = (op->out_filename)?strdup(op->out_filename):NULL;
+    guclaw_ops.num_buffers = op->num_buffers;
+    guclaw_ops.poll_timeout = op->poll_timeout;
+    guclaw_ops.max_filesize = op->max_filesize;
+    guclaw_ops.discard_oldlogs = op->discard_oldlogs;
+
+    open_output_file();
+    my_buc = buc__create(outfile_fd, SUBBUF_SIZE*guclaw_ops.num_buffers);
+
+    /* Just to make sure we open the right debugfs files */
+    drm_fd = drm_open_driver(DRIVER_INTEL);
+    /* Enable the logging, it may not have been enabled from boot and so
+     * the relay file also wouldn't have been created.
+     */
+   
+    /* Start guclaw thread */
+    r = pthread_create(&guclaw_thread,
+                       NULL,
+                       guclaw_routine,
+                       NULL);
+    igt_assert(r==0);
+ 
+    return 0;
+}
+
+void guclaw_suspend(void)
+{
+    suspend_logging = true;
+}
+
+void guclaw_resume(void)
+{
+    suspend_logging = false;
+}
+
+void guclaw_stop(void)
+{
+        stop_logging = true;
+}
+
+void guclaw_wait(void)
+{
+    pthread_join(guclaw_thread, NULL);
+}
+
+void guclaw_finalize(void)
+{
+    int overflows;
+
+    /* Pause logging on the GuC side */
+    guc_log_control(false);
+
+    if(!stop_logging)
+    {
+        stop_logging = true;
+        pthread_join(guclaw_thread, NULL);
+    }
+
+    pull_leftover_data();
+    igt_info("total bytes written %" PRIu64 "\n", total_bytes_written);
+
+    overflows = buc__get_overflow(my_buc);
+    igt_info("overflows counter: %u\n", overflows);
+
+    buc__finalize(my_buc);
+    close(relay_fd);
+    close(outfile_fd);
+    close(drm_fd);
+}
diff --git a/lib/igt_guclaw.h b/lib/igt_guclaw.h
new file mode 100755
index 0000000..d01df15
--- /dev/null
+++ b/lib/igt_guclaw.h
@@ -0,0 +1,81 @@
+#pragma once
+/**
+ * GuC Logger Auxiliary Wrapper (GuCLAW)
+ *
+ * This library contains helper functions for GuC logs dumping.
+ *
+ * Usage pattern:
+ *
+ * 1. Create and fill GuCLAW options structure (guclaw_options_t).
+ *
+ * 2. Call 'guclaw_init' to initialize the logger
+ *    (Logger starts as suspended).
+ *
+ * 3. Call 'guclaw_resume'/'guclaw_suspend' to gate logging process.
+ *    If you want to create a standalone tool place 'guclaw_wait'
+ *    in a main loop to wait for terminate conditions
+ *    to be satisfied.
+ *    Optionally, you can call 'guclaw_stop' to terminate guclaw thread.
+ *
+ * 4. Call 'guclaw_finalize' to finish all logger jobs.
+ */
+
+typedef struct _guclaw_options
+{
+    // GuC verbosity level
+    int verbosity_level;
+
+    // path to output file
+    char* out_filename;
+
+    // number of buffers for storing logs
+    int num_buffers;
+
+    // polling timeout in ms
+    int poll_timeout;
+
+    // max allowable size of output file in MBs
+    uint32_t max_filesize;
+
+    // discard the old/boot-time logs flag
+    int discard_oldlogs;
+} guclaw_options_t;
+
+
+/**
+ * Initializes GuCLAW utility guclaw_options_t structure.
+ * GuCLAW is suspended after initialization.
+ *
+ * @return zero value on success, negative otherwise.
+ */
+int  guclaw_init(guclaw_options_t *op);
+
+/**
+ * Finalizes GuCLAW utility.
+ */
+void guclaw_finalize(void);
+
+/**
+ * Suspends log dumping activity
+ * (GuC logs are still pulled but not
+ * streamed to the destination file).
+ */
+void guclaw_suspend(void);
+
+/**
+ * Resumes log dumping activity.
+ */
+void guclaw_resume(void);
+
+/**
+ * Stops the GuCLAW thread.
+ */
+void guclaw_stop(void);
+
+/**
+ * Waits for logging terminal conditions
+ * to be satisfied.
+ */
+void guclaw_wait(void);
+
+void* guclaw_routine(void *p);
\ No newline at end of file
diff --git a/tools/intel_guc_logger.c b/tools/intel_guc_logger.c
old mode 100644
new mode 100755
index 3079878..9e29445
--- a/tools/intel_guc_logger.c
+++ b/tools/intel_guc_logger.c
@@ -1,24 +1,16 @@
-
-#define _GNU_SOURCE  /* For using O_DIRECT */
+#define _GNU_SOURCE
 #include <inttypes.h>
 #include <stdio.h>
-#include <fcntl.h>
-#include <sched.h>
-#include <sys/ioctl.h>
 #include <string.h>
 #include <errno.h>
 #include <signal.h>
 #include <stdlib.h>
 #include <unistd.h>
-#include <sys/poll.h>
-#include <sys/wait.h>
-#include <sys/types.h>
-#include <sys/syscall.h>
-#include <sys/mman.h>
-#include <assert.h>
-#include <pthread.h>
+
+#include <sys/resource.h>
 
 #include "igt.h"
+#include "igt_guclaw.h"
 
 #define MB(x) ((uint64_t)(x) * 1024 * 1024)
 #ifndef PAGE_SIZE
@@ -32,411 +24,126 @@
 /* Need large buffering from logger side to hide the DISK IO latency, Driver
  * can only store 8 snapshots of GuC log buffer in relay.
  */
+
 #define NUM_SUBBUFS 100
 
 #define RELAY_FILE_NAME  "guc_log"
 #define DEFAULT_OUTPUT_FILE_NAME  "guc_log_dump.dat"
 #define CONTROL_FILE_NAME "i915_guc_log_control"
 
-char *read_buffer;
 char *out_filename;
 int poll_timeout = 2; /* by default 2ms timeout */
-pthread_mutex_t mutex;
-pthread_t flush_thread;
 int verbosity_level = 3; /* by default capture logs at max verbosity */
-uint32_t produced, consumed;
-uint64_t total_bytes_written;
 int num_buffers = NUM_SUBBUFS;
-int relay_fd, drm_fd, outfile_fd = -1;
 uint32_t test_duration, max_filesize;
-pthread_cond_t underflow_cond, overflow_cond;
-bool stop_logging, discard_oldlogs, capturing_stopped;
-
-static void guc_log_control(bool enable_logging)
-{
-	int control_fd;
-	char data[19];
-	uint64_t val;
-	int ret;
-
-	control_fd = igt_debugfs_open(drm_fd, CONTROL_FILE_NAME, O_WRONLY);
-	igt_assert_f(control_fd >= 0, "couldn't open the guc log control file\n");
-
-	val = enable_logging ? ((verbosity_level << 4) | 0x1) : 0;
-
-	ret = snprintf(data, sizeof(data), "0x%" PRIx64, val);
-	igt_assert(ret > 2 && ret < sizeof(data));
-
-	ret = write(control_fd, data, ret);
-	igt_assert_f(ret > 0, "couldn't write to the log control file\n");
-
-	close(control_fd);
-}
+bool stop_logging, discard_oldlogs = false;
 
 static void int_sig_handler(int sig)
 {
-	igt_info("received signal %d\n", sig);
-
-	stop_logging = true;
-}
-
-static void pull_leftover_data(void)
-{
-	unsigned int bytes_read = 0;
-	int ret;
-
-	do {
-		/* Read the logs from relay buffer */
-		ret = read(relay_fd, read_buffer, SUBBUF_SIZE);
-		if (!ret)
-			break;
-
-		igt_assert_f(ret > 0, "failed to read from the guc log file\n");
-		igt_assert_f(ret == SUBBUF_SIZE, "invalid read from relay file\n");
-
-		bytes_read += ret;
-
-		if (outfile_fd >= 0) {
-			ret = write(outfile_fd, read_buffer, SUBBUF_SIZE);
-			igt_assert_f(ret == SUBBUF_SIZE, "couldn't dump the logs in a file\n");
-			total_bytes_written += ret;
-		}
-	} while(1);
-
-	igt_debug("%u bytes flushed\n", bytes_read);
-}
-
-static int num_filled_bufs(void)
-{
-	return (produced - consumed);
-}
-
-static void pull_data(void)
-{
-	char *ptr;
-	int ret;
-
-	pthread_mutex_lock(&mutex);
-	while (num_filled_bufs() >= num_buffers) {
-		igt_debug("overflow, will wait, produced %u, consumed %u\n", produced, consumed);
-		/* Stall the main thread in case of overflow, as there are no
-		 * buffers available to store the new logs, otherwise there
-		 * could be corruption if both threads work on the same buffer.
-		 */
-		pthread_cond_wait(&overflow_cond, &mutex);
-	};
-	pthread_mutex_unlock(&mutex);
-
-	ptr = read_buffer + (produced % num_buffers) * SUBBUF_SIZE;
-
-	/* Read the logs from relay buffer */
-	ret = read(relay_fd, ptr, SUBBUF_SIZE);
-	igt_assert_f(ret >= 0, "failed to read from the guc log file\n");
-	igt_assert_f(!ret || ret == SUBBUF_SIZE, "invalid read from relay file\n");
-
-	if (ret) {
-		pthread_mutex_lock(&mutex);
-		produced++;
-		pthread_cond_signal(&underflow_cond);
-		pthread_mutex_unlock(&mutex);
-	} else {
-		/* Occasionally (very rare) read from the relay file returns no
-		 * data, albeit the polling done prior to read call indicated
-		 * availability of data.
-		 */
-		igt_debug("no data read from the relay file\n");
-	}
-}
-
-static void *flusher(void *arg)
-{
-	char *ptr;
-	int ret;
-
-	igt_debug("execution started of flusher thread\n");
-
-	do {
-		pthread_mutex_lock(&mutex);
-		while (!num_filled_bufs()) {
-			/* Exit only after completing the flush of all the filled
-			 * buffers as User would expect that all logs captured up
-			 * till the point of interruption/exit are written out to
-			 * the disk file.
-			 */
-			if (capturing_stopped) {
-				igt_debug("flusher to exit now\n");
-				pthread_mutex_unlock(&mutex);
-				return NULL;
-			}
-			pthread_cond_wait(&underflow_cond, &mutex);
-		};
-		pthread_mutex_unlock(&mutex);
-
-		ptr = read_buffer + (consumed % num_buffers) * SUBBUF_SIZE;
-
-		ret = write(outfile_fd, ptr, SUBBUF_SIZE);
-		igt_assert_f(ret == SUBBUF_SIZE, "couldn't dump the logs in a file\n");
-
-		total_bytes_written += ret;
-		if (max_filesize && (total_bytes_written > MB(max_filesize))) {
-			igt_debug("reached the target of %" PRIu64 " bytes\n", MB(max_filesize));
-			stop_logging = true;
-		}
-
-		pthread_mutex_lock(&mutex);
-		consumed++;
-		pthread_cond_signal(&overflow_cond);
-		pthread_mutex_unlock(&mutex);
-	} while(1);
-
-	return NULL;
-}
-
-static void init_flusher_thread(void)
-{
-	struct sched_param	thread_sched;
-	pthread_attr_t		p_attr;
-	int ret;
-
-	pthread_cond_init(&underflow_cond, NULL);
-	pthread_cond_init(&overflow_cond, NULL);
-	pthread_mutex_init(&mutex, NULL);
-
-	ret = pthread_attr_init(&p_attr);
-	igt_assert_f(ret == 0, "error obtaining default thread attributes\n");
-
-	ret = pthread_attr_setinheritsched(&p_attr, PTHREAD_EXPLICIT_SCHED);
-	igt_assert_f(ret == 0, "couldn't set inheritsched\n");
-
-	ret = pthread_attr_setschedpolicy(&p_attr, SCHED_RR);
-	igt_assert_f(ret == 0, "couldn't set thread scheduling policy\n");
-
-	/* Keep the flusher task also at rt priority, so that it doesn't get
-	 * too late in flushing the collected logs in local buffers to the disk,
-	 * and so main thread always have spare buffers to collect the logs.
-	 */
-	thread_sched.sched_priority = 5;
-	ret = pthread_attr_setschedparam(&p_attr, &thread_sched);
-	igt_assert_f(ret == 0, "couldn't set thread priority\n");
-
-	ret = pthread_create(&flush_thread, &p_attr, flusher, NULL);
-	igt_assert_f(ret == 0, "thread creation failed\n");
-
-	ret = pthread_attr_destroy(&p_attr);
-	igt_assert_f(ret == 0, "error destroying thread attributes\n");
-}
-
-static void open_relay_file(void)
-{
-	relay_fd = igt_debugfs_open(drm_fd, RELAY_FILE_NAME, O_RDONLY);
-	igt_assert_f(relay_fd >= 0, "couldn't open the guc log file\n");
-
-	/* Purge the old/boot-time logs from the relay buffer.
-	 * This is more for Val team's requirement, where they have to first
-	 * purge the existing logs before starting the tests for which the logs
-	 * are actually needed. After this logger will enter into a loop and
-	 * wait for the new data, at that point benchmark can be launched from
-	 * a different shell.
-	 */
-	if (discard_oldlogs)
-		pull_leftover_data();
-}
-
-static void open_output_file(void)
-{
-	/* Use Direct IO mode for the output file, as the data written is not
-	 * supposed to be accessed again, this saves a copy of data from App's
-	 * buffer to kernel buffer (Page cache). Due to no buffering on kernel
-	 * side, data is flushed out to disk faster and more buffering can be
-	 * done on the logger side to hide the disk IO latency.
-	 */
-	outfile_fd = open(out_filename ? : DEFAULT_OUTPUT_FILE_NAME,
-			  O_CREAT | O_WRONLY | O_TRUNC | O_DIRECT,
-			  0440);
-	igt_assert_f(outfile_fd >= 0, "couldn't open the output file\n");
-
-	free(out_filename);
-}
-
-static void init_main_thread(void)
-{
-	struct sched_param	thread_sched;
-	int ret;
-
-	/* Run the main thread at highest priority to ensure that it always
-	 * gets woken-up at earliest on arrival of new data and so is always
-	 * ready to pull the logs, otherwise there could be loss logs if
-	 * GuC firmware is generating logs at a very high rate.
-	 */
-	thread_sched.sched_priority = 1;
-	ret = sched_setscheduler(getpid(), SCHED_FIFO, &thread_sched);
-	igt_assert_f(ret == 0, "couldn't set the priority\n");
-
-	if (signal(SIGINT, int_sig_handler) == SIG_ERR)
-		igt_assert_f(0, "SIGINT handler registration failed\n");
-
-	if (signal(SIGALRM, int_sig_handler) == SIG_ERR)
-		igt_assert_f(0, "SIGALRM handler registration failed\n");
-
-	/* Need an aligned pointer for direct IO */
-	ret = posix_memalign((void **)&read_buffer, PAGE_SIZE, num_buffers * SUBBUF_SIZE);
-	igt_assert_f(ret == 0, "couldn't allocate the read buffer\n");
-
-	/* Keep the pages locked in RAM, avoid page fault overhead */
-	ret = mlock(read_buffer, num_buffers * SUBBUF_SIZE);
-	igt_assert_f(ret == 0, "failed to lock memory\n");
-
-	/* Enable the logging, it may not have been enabled from boot and so
-	 * the relay file also wouldn't have been created.
-	 */
-	guc_log_control(true);
-
-	open_relay_file();
-	open_output_file();
+    igt_info("received signal %d\n", sig);
+    guclaw_stop();
 }
 
 static int parse_options(int opt, int opt_index, void *data)
 {
-	igt_debug("opt %c optarg %s\n", opt, optarg);
+    igt_debug("opt %c optarg %s\n", opt, optarg);
 
-	switch(opt) {
-	case 'v':
-		verbosity_level = atoi(optarg);
-		igt_assert_f(verbosity_level >= 0 && verbosity_level <= 3, "invalid input for -v option\n");
-		igt_debug("verbosity level to be used is %d\n", verbosity_level);
-		break;
-	case 'o':
-		out_filename = strdup(optarg);
-		igt_assert_f(out_filename, "Couldn't allocate the o/p filename\n");
-		igt_debug("logs to be stored in file %s\n", out_filename);
-		break;
-	case 'b':
-		num_buffers = atoi(optarg);
-		igt_assert_f(num_buffers > 0, "invalid input for -b option\n");
-		igt_debug("number of buffers to be used is %d\n", num_buffers);
-		break;
-	case 't':
-		test_duration = atoi(optarg);
-		igt_assert_f(test_duration > 0, "invalid input for -t option\n");
-		igt_debug("logger to run for %d second\n", test_duration);
-		break;
-	case 'p':
-		poll_timeout = atoi(optarg);
-		igt_assert_f(poll_timeout != 0, "invalid input for -p option\n");
-		if (poll_timeout > 0)
-			igt_debug("polling to be done with %d millisecond timeout\n", poll_timeout);
-		break;
-	case 's':
-		max_filesize = atoi(optarg);
-		igt_assert_f(max_filesize > 0, "invalid input for -s option\n");
-		igt_debug("max allowed size of the output file is %d MB\n", max_filesize);
-		break;
-	case 'd':
-		discard_oldlogs = true;
-		igt_debug("old/boot-time logs will be discarded\n");
-		break;
-	}
+    switch(opt) {
+    case 'v':
+            verbosity_level = atoi(optarg);
+            igt_assert_f(verbosity_level >= 0 && verbosity_level <= 3, "invalid input for -v option\n");
+            igt_debug("verbosity level to be used is %d\n", verbosity_level);
+            break;
+    case 'o':
+            out_filename = strdup(optarg);
+            igt_assert_f(out_filename, "Couldn't allocate the o/p filename\n");
+            igt_debug("logs to be stored in file %s\n", out_filename);
+            break;
+    case 'b':
+            num_buffers = atoi(optarg);
+            igt_assert_f(num_buffers > 0, "invalid input for -b option\n");
+            igt_debug("number of buffers to be used is %d\n", num_buffers);
+            break;
+    case 't':
+            test_duration = atoi(optarg);
+            igt_assert_f(test_duration > 0, "invalid input for -t option\n");
+            igt_debug("logger to run for %d second\n", test_duration);
+            break;
+    case 'p':
+            poll_timeout = atoi(optarg);
+            igt_assert_f(poll_timeout != 0, "invalid input for -p option\n");
+            if (poll_timeout > 0)
+                    igt_debug("polling to be done with %d millisecond timeout\n", poll_timeout);
+            break;
+    case 's':
+            max_filesize = atoi(optarg);
+            igt_assert_f(max_filesize > 0, "invalid input for -s option\n");
+            igt_debug("max allowed size of the output file is %d MB\n", max_filesize);
+            break;
+    case 'd':
+            discard_oldlogs = true;
+            igt_debug("old/boot-time logs will be discarded\n");
+            break;
+    }
 
-	return 0;
+    return 0;
 }
 
 static void process_command_line(int argc, char **argv)
 {
-	static struct option long_options[] = {
-		{"verbosity", required_argument, 0, 'v'},
-		{"outputfile", required_argument, 0, 'o'},
-		{"buffers", required_argument, 0, 'b'},
-		{"testduration", required_argument, 0, 't'},
-		{"polltimeout", required_argument, 0, 'p'},
-		{"size", required_argument, 0, 's'},
-		{"discard", no_argument, 0, 'd'},
-		{ 0, 0, 0, 0 }
-	};
+    static struct option long_options[] = {
+            {"verbosity", required_argument, 0, 'v'},
+            {"outputfile", required_argument, 0, 'o'},
+            {"buffers", required_argument, 0, 'b'},
+            {"testduration", required_argument, 0, 't'},
+            {"polltimeout", required_argument, 0, 'p'},
+            {"size", required_argument, 0, 's'},
+            {"discard", no_argument, 0, 'd'},
+            { 0, 0, 0, 0 }
+    };
 
-	const char *help =
-		"  -v --verbosity=level   verbosity level of GuC logging (0-3)\n"
-		"  -o --outputfile=name   name of the output file, including the location, where logs will be stored\n"
-		"  -b --buffers=num       number of buffers to be maintained on logger side for storing logs\n"
-		"  -t --testduration=sec  max duration in seconds for which the logger should run\n"
-		"  -p --polltimeout=ms    polling timeout in ms, -1 == indefinite wait for the new data\n"
-		"  -s --size=MB           max size of output file in MBs after which logging will be stopped\n"
-		"  -d --discard           discard the old/boot-time logs before entering into the capture loop\n";
+    const char *help =
+            "  -v --verbosity=level   verbosity level of GuC logging (0-3)\n"
+            "  -o --outputfile=name   name of the output file, including the location, where logs will be stored\n"
+            "  -b --buffers=num       number of buffers to be maintained on logger side for storing logs\n"
+            "  -t --testduration=sec  max duration in seconds for which the logger should run\n"
+            "  -p --polltimeout=ms    polling timeout in ms, -1 == indefinite wait for the new data\n"
+            "  -s --size=MB           max size of output file in MBs after which logging will be stopped\n"
+            "  -d --discard           discard the old/boot-time logs before entering into the capture loop\n";
 
-	igt_simple_init_parse_opts(&argc, argv, "v:o:b:t:p:s:d", long_options,
-				   help, parse_options, NULL);
+    igt_simple_init_parse_opts(&argc, argv, "v:o:b:t:p:s:d", long_options,
+                               help, parse_options, NULL);
 }
 
 int main(int argc, char **argv)
 {
-	struct pollfd relay_poll_fd;
-	int nfds;
-	int ret;
+    guclaw_options_t gops;
 
-	process_command_line(argc, argv);
+    process_command_line(argc, argv);
 
-	/* Just to make sure we open the right debugfs files */
-	drm_fd = drm_open_driver_master(DRIVER_INTEL);
+    gops.verbosity_level = verbosity_level;
+    gops.out_filename = out_filename;
+    gops.num_buffers = num_buffers;
+    gops.poll_timeout = poll_timeout;
+    gops.max_filesize = max_filesize;
+    gops.discard_oldlogs = discard_oldlogs;
 
-	init_main_thread();
+    if (signal(SIGINT, int_sig_handler) == SIG_ERR)
+        igt_assert_f(0, "SIGINT handler registration failed\n");
 
-	/* Use a separate thread for flushing the logs to a file on disk.
-	 * Main thread will buffer the data from relay file in its pool of
-	 * buffers and other thread will flush the data to disk in background.
-	 * This is needed, albeit by default data is written out to disk in
-	 * async mode, as when there are too many dirty pages in the RAM,
-	 * (/proc/sys/vm/dirty_ratio), kernel starts blocking the processes
-	 * doing the file writes.
-	 */
-	init_flusher_thread();
+    if (signal(SIGALRM, int_sig_handler) == SIG_ERR)
+        igt_assert_f(0, "SIGALRM handler registration failed\n");
 
-	relay_poll_fd.fd = relay_fd;
-	relay_poll_fd.events = POLLIN;
-	relay_poll_fd.revents = 0;
 
-	nfds = 1; /* only one fd to poll */
+    alarm(test_duration);
 
-	alarm(test_duration); /* Start the alarm */
+    guclaw_init(&gops);
+    free(gops.out_filename);
 
-	do {
-		/* Wait/poll for the new data to be available, relay doesn't
-		 * provide a blocking read.
-		 * On older kernels need to do polling with a timeout instead of
-		 * indefinite wait to avoid relying on relay for the wakeup, as
-		 * relay used to do the wakeup in a deferred manner on jiffies
-		 * granularity by scheduling a timer and moreover that timer was
-		 * re-scheduled on every newly produced buffer and so was pushed
-		 * out if there were multiple flush interrupts in a very quick
-		 * succession (less than a jiffy gap between 2 flush interrupts)
-		 * causing relay to run out of sub buffers to store new logs.
-		 */
-		ret = poll(&relay_poll_fd, nfds, poll_timeout);
-		if (ret < 0) {
-			if (errno == EINTR)
-				break;
-			igt_assert_f(0, "poll call failed\n");
-		}
+    guclaw_resume();
+    guclaw_wait();
 
-		/* No data available yet, poll again, hopefully new data is round the corner */
-		if (!relay_poll_fd.revents)
-			continue;
+    guclaw_finalize();
 
-		pull_data();
-	} while (!stop_logging);
-
-	/* Pause logging on the GuC side */
-	guc_log_control(false);
-
-	/* Signal flusher thread to make an exit */
-	capturing_stopped = 1;
-	pthread_cond_signal(&underflow_cond);
-	pthread_join(flush_thread, NULL);
-
-	pull_leftover_data();
-	igt_info("total bytes written %" PRIu64 "\n", total_bytes_written);
-
-	free(read_buffer);
-	close(relay_fd);
-	close(outfile_fd);
-	close(drm_fd);
-	igt_exit();
+    igt_exit();
 }
-- 
2.9.3



More information about the Intel-gfx mailing list