|
@@ -33,11 +33,10 @@ static void guc_log_capture_logs(struct intel_guc *guc);
|
|
|
/**
|
|
|
* DOC: GuC firmware log
|
|
|
*
|
|
|
- * Firmware log is enabled by setting i915.guc_log_level to non-negative level.
|
|
|
+ * Firmware log is enabled by setting i915.guc_log_level to the positive level.
|
|
|
* Log data is printed out via reading debugfs i915_guc_log_dump. Reading from
|
|
|
* i915_guc_load_status will print out firmware loading status and scratch
|
|
|
* registers value.
|
|
|
- *
|
|
|
*/
|
|
|
|
|
|
static int guc_log_flush_complete(struct intel_guc *guc)
|
|
@@ -59,11 +58,15 @@ static int guc_log_flush(struct intel_guc *guc)
|
|
|
return intel_guc_send(guc, action, ARRAY_SIZE(action));
|
|
|
}
|
|
|
|
|
|
-static int guc_log_control(struct intel_guc *guc, u32 control_val)
|
|
|
+static int guc_log_control(struct intel_guc *guc, bool enable, u32 verbosity)
|
|
|
{
|
|
|
+ union guc_log_control control_val = {
|
|
|
+ .logging_enabled = enable,
|
|
|
+ .verbosity = verbosity,
|
|
|
+ };
|
|
|
u32 action[] = {
|
|
|
INTEL_GUC_ACTION_UK_LOG_ENABLE_LOGGING,
|
|
|
- control_val
|
|
|
+ control_val.value
|
|
|
};
|
|
|
|
|
|
return intel_guc_send(guc, action, ARRAY_SIZE(action));
|
|
@@ -78,7 +81,8 @@ static int subbuf_start_callback(struct rchan_buf *buf,
|
|
|
void *prev_subbuf,
|
|
|
size_t prev_padding)
|
|
|
{
|
|
|
- /* Use no-overwrite mode by default, where relay will stop accepting
|
|
|
+ /*
|
|
|
+ * Use no-overwrite mode by default, where relay will stop accepting
|
|
|
* new data if there are no empty sub buffers left.
|
|
|
* There is no strict synchronization enforced by relay between Consumer
|
|
|
* and Producer. In overwrite mode, there is a possibility of getting
|
|
@@ -104,7 +108,8 @@ static struct dentry *create_buf_file_callback(const char *filename,
|
|
|
{
|
|
|
struct dentry *buf_file;
|
|
|
|
|
|
- /* This to enable the use of a single buffer for the relay channel and
|
|
|
+ /*
|
|
|
+ * This to enable the use of a single buffer for the relay channel and
|
|
|
* correspondingly have a single file exposed to User, through which
|
|
|
* it can collect the logs in order without any post-processing.
|
|
|
* Need to set 'is_global' even if parent is NULL for early logging.
|
|
@@ -114,7 +119,8 @@ static struct dentry *create_buf_file_callback(const char *filename,
|
|
|
if (!parent)
|
|
|
return NULL;
|
|
|
|
|
|
- /* Not using the channel filename passed as an argument, since for each
|
|
|
+ /*
|
|
|
+ * Not using the channel filename passed as an argument, since for each
|
|
|
* channel relay appends the corresponding CPU number to the filename
|
|
|
* passed in relay_open(). This should be fine as relay just needs a
|
|
|
* dentry of the file associated with the channel buffer and that file's
|
|
@@ -147,13 +153,16 @@ static int guc_log_relay_file_create(struct intel_guc *guc)
|
|
|
struct dentry *log_dir;
|
|
|
int ret;
|
|
|
|
|
|
- if (i915_modparams.guc_log_level < 0)
|
|
|
+ if (!i915_modparams.guc_log_level)
|
|
|
return 0;
|
|
|
|
|
|
+ mutex_lock(&guc->log.runtime.relay_lock);
|
|
|
+
|
|
|
/* For now create the log file in /sys/kernel/debug/dri/0 dir */
|
|
|
log_dir = dev_priv->drm.primary->debugfs_root;
|
|
|
|
|
|
- /* If /sys/kernel/debug/dri/0 location do not exist, then debugfs is
|
|
|
+ /*
|
|
|
+ * If /sys/kernel/debug/dri/0 location do not exist, then debugfs is
|
|
|
* not mounted and so can't create the relay file.
|
|
|
* The relay API seems to fit well with debugfs only, for availing relay
|
|
|
* there are 3 requirements which can be met for debugfs file only in a
|
|
@@ -166,25 +175,41 @@ static int guc_log_relay_file_create(struct intel_guc *guc)
|
|
|
*/
|
|
|
if (!log_dir) {
|
|
|
DRM_ERROR("Debugfs dir not available yet for GuC log file\n");
|
|
|
- return -ENODEV;
|
|
|
+ ret = -ENODEV;
|
|
|
+ goto out_unlock;
|
|
|
}
|
|
|
|
|
|
ret = relay_late_setup_files(guc->log.runtime.relay_chan, "guc_log", log_dir);
|
|
|
if (ret < 0 && ret != -EEXIST) {
|
|
|
DRM_ERROR("Couldn't associate relay chan with file %d\n", ret);
|
|
|
- return ret;
|
|
|
+ goto out_unlock;
|
|
|
}
|
|
|
|
|
|
- return 0;
|
|
|
+ ret = 0;
|
|
|
+
|
|
|
+out_unlock:
|
|
|
+ mutex_unlock(&guc->log.runtime.relay_lock);
|
|
|
+ return ret;
|
|
|
+}
|
|
|
+
|
|
|
+static bool guc_log_has_relay(struct intel_guc *guc)
|
|
|
+{
|
|
|
+ lockdep_assert_held(&guc->log.runtime.relay_lock);
|
|
|
+
|
|
|
+ return guc->log.runtime.relay_chan != NULL;
|
|
|
}
|
|
|
|
|
|
static void guc_move_to_next_buf(struct intel_guc *guc)
|
|
|
{
|
|
|
- /* Make sure the updates made in the sub buffer are visible when
|
|
|
+ /*
|
|
|
+ * Make sure the updates made in the sub buffer are visible when
|
|
|
* Consumer sees the following update to offset inside the sub buffer.
|
|
|
*/
|
|
|
smp_wmb();
|
|
|
|
|
|
+ if (!guc_log_has_relay(guc))
|
|
|
+ return;
|
|
|
+
|
|
|
/* All data has been written, so now move the offset of sub buffer. */
|
|
|
relay_reserve(guc->log.runtime.relay_chan, guc->log.vma->obj->base.size);
|
|
|
|
|
@@ -194,10 +219,11 @@ static void guc_move_to_next_buf(struct intel_guc *guc)
|
|
|
|
|
|
static void *guc_get_write_buffer(struct intel_guc *guc)
|
|
|
{
|
|
|
- if (!guc->log.runtime.relay_chan)
|
|
|
+ if (!guc_log_has_relay(guc))
|
|
|
return NULL;
|
|
|
|
|
|
- /* Just get the base address of a new sub buffer and copy data into it
|
|
|
+ /*
|
|
|
+ * Just get the base address of a new sub buffer and copy data into it
|
|
|
* ourselves. NULL will be returned in no-overwrite mode, if all sub
|
|
|
* buffers are full. Could have used the relay_write() to indirectly
|
|
|
* copy the data, but that would have been bit convoluted, as we need to
|
|
@@ -262,15 +288,30 @@ static void guc_read_update_log_buffer(struct intel_guc *guc)
|
|
|
/* Get the pointer to shared GuC log buffer */
|
|
|
log_buf_state = src_data = guc->log.runtime.buf_addr;
|
|
|
|
|
|
+ mutex_lock(&guc->log.runtime.relay_lock);
|
|
|
+
|
|
|
/* Get the pointer to local buffer to store the logs */
|
|
|
log_buf_snapshot_state = dst_data = guc_get_write_buffer(guc);
|
|
|
|
|
|
+ if (unlikely(!log_buf_snapshot_state)) {
|
|
|
+ /*
|
|
|
+ * Used rate limited to avoid deluge of messages, logs might be
|
|
|
+ * getting consumed by User at a slow rate.
|
|
|
+ */
|
|
|
+ DRM_ERROR_RATELIMITED("no sub-buffer to capture logs\n");
|
|
|
+ guc->log.capture_miss_count++;
|
|
|
+ mutex_unlock(&guc->log.runtime.relay_lock);
|
|
|
+
|
|
|
+ return;
|
|
|
+ }
|
|
|
+
|
|
|
/* Actual logs are present from the 2nd page */
|
|
|
src_data += PAGE_SIZE;
|
|
|
dst_data += PAGE_SIZE;
|
|
|
|
|
|
for (type = GUC_ISR_LOG_BUFFER; type < GUC_MAX_LOG_BUFFER; type++) {
|
|
|
- /* Make a copy of the state structure, inside GuC log buffer
|
|
|
+ /*
|
|
|
+ * Make a copy of the state structure, inside GuC log buffer
|
|
|
* (which is uncached mapped), on the stack to avoid reading
|
|
|
* from it multiple times.
|
|
|
*/
|
|
@@ -290,14 +331,12 @@ static void guc_read_update_log_buffer(struct intel_guc *guc)
|
|
|
log_buf_state->flush_to_file = 0;
|
|
|
log_buf_state++;
|
|
|
|
|
|
- if (unlikely(!log_buf_snapshot_state))
|
|
|
- continue;
|
|
|
-
|
|
|
/* First copy the state structure in snapshot buffer */
|
|
|
memcpy(log_buf_snapshot_state, &log_buf_state_local,
|
|
|
sizeof(struct guc_log_buffer_state));
|
|
|
|
|
|
- /* The write pointer could have been updated by GuC firmware,
|
|
|
+ /*
|
|
|
+ * The write pointer could have been updated by GuC firmware,
|
|
|
* after sending the flush interrupt to Host, for consistency
|
|
|
* set write pointer value to same value of sampled_write_ptr
|
|
|
* in the snapshot buffer.
|
|
@@ -332,15 +371,9 @@ static void guc_read_update_log_buffer(struct intel_guc *guc)
|
|
|
dst_data += buffer_size;
|
|
|
}
|
|
|
|
|
|
- if (log_buf_snapshot_state)
|
|
|
- guc_move_to_next_buf(guc);
|
|
|
- else {
|
|
|
- /* Used rate limited to avoid deluge of messages, logs might be
|
|
|
- * getting consumed by User at a slow rate.
|
|
|
- */
|
|
|
- DRM_ERROR_RATELIMITED("no sub-buffer to capture logs\n");
|
|
|
- guc->log.capture_miss_count++;
|
|
|
- }
|
|
|
+ guc_move_to_next_buf(guc);
|
|
|
+
|
|
|
+ mutex_unlock(&guc->log.runtime.relay_lock);
|
|
|
}
|
|
|
|
|
|
static void capture_logs_work(struct work_struct *work)
|
|
@@ -360,19 +393,21 @@ static int guc_log_runtime_create(struct intel_guc *guc)
|
|
|
{
|
|
|
struct drm_i915_private *dev_priv = guc_to_i915(guc);
|
|
|
void *vaddr;
|
|
|
- struct rchan *guc_log_relay_chan;
|
|
|
- size_t n_subbufs, subbuf_size;
|
|
|
int ret;
|
|
|
|
|
|
lockdep_assert_held(&dev_priv->drm.struct_mutex);
|
|
|
|
|
|
+ if (!guc->log.vma)
|
|
|
+ return -ENODEV;
|
|
|
+
|
|
|
GEM_BUG_ON(guc_log_has_runtime(guc));
|
|
|
|
|
|
ret = i915_gem_object_set_to_wc_domain(guc->log.vma->obj, true);
|
|
|
if (ret)
|
|
|
return ret;
|
|
|
|
|
|
- /* Create a WC (Uncached for read) vmalloc mapping of log
|
|
|
+ /*
|
|
|
+ * Create a WC (Uncached for read) vmalloc mapping of log
|
|
|
* buffer pages, so that we can directly get the data
|
|
|
* (up-to-date) from memory.
|
|
|
*/
|
|
@@ -384,17 +419,55 @@ static int guc_log_runtime_create(struct intel_guc *guc)
|
|
|
|
|
|
guc->log.runtime.buf_addr = vaddr;
|
|
|
|
|
|
+ return 0;
|
|
|
+}
|
|
|
+
|
|
|
+static void guc_log_runtime_destroy(struct intel_guc *guc)
|
|
|
+{
|
|
|
+ /*
|
|
|
+ * It's possible that the runtime stuff was never allocated because
|
|
|
+ * GuC log was disabled at the boot time.
|
|
|
+ */
|
|
|
+ if (!guc_log_has_runtime(guc))
|
|
|
+ return;
|
|
|
+
|
|
|
+ i915_gem_object_unpin_map(guc->log.vma->obj);
|
|
|
+ guc->log.runtime.buf_addr = NULL;
|
|
|
+}
|
|
|
+
|
|
|
+void intel_guc_log_init_early(struct intel_guc *guc)
|
|
|
+{
|
|
|
+ mutex_init(&guc->log.runtime.relay_lock);
|
|
|
+ INIT_WORK(&guc->log.runtime.flush_work, capture_logs_work);
|
|
|
+}
|
|
|
+
|
|
|
+int intel_guc_log_relay_create(struct intel_guc *guc)
|
|
|
+{
|
|
|
+ struct drm_i915_private *dev_priv = guc_to_i915(guc);
|
|
|
+ struct rchan *guc_log_relay_chan;
|
|
|
+ size_t n_subbufs, subbuf_size;
|
|
|
+ int ret;
|
|
|
+
|
|
|
+ if (!i915_modparams.guc_log_level)
|
|
|
+ return 0;
|
|
|
+
|
|
|
+ mutex_lock(&guc->log.runtime.relay_lock);
|
|
|
+
|
|
|
+ GEM_BUG_ON(guc_log_has_relay(guc));
|
|
|
+
|
|
|
/* Keep the size of sub buffers same as shared log buffer */
|
|
|
- subbuf_size = guc->log.vma->obj->base.size;
|
|
|
+ subbuf_size = GUC_LOG_SIZE;
|
|
|
|
|
|
- /* Store up to 8 snapshots, which is large enough to buffer sufficient
|
|
|
+ /*
|
|
|
+ * Store up to 8 snapshots, which is large enough to buffer sufficient
|
|
|
* boot time logs and provides enough leeway to User, in terms of
|
|
|
* latency, for consuming the logs from relay. Also doesn't take
|
|
|
* up too much memory.
|
|
|
*/
|
|
|
n_subbufs = 8;
|
|
|
|
|
|
- /* Create a relay channel, so that we have buffers for storing
|
|
|
+ /*
|
|
|
+ * Create a relay channel, so that we have buffers for storing
|
|
|
* the GuC firmware logs, the channel will be linked with a file
|
|
|
* later on when debugfs is registered.
|
|
|
*/
|
|
@@ -404,33 +477,39 @@ static int guc_log_runtime_create(struct intel_guc *guc)
|
|
|
DRM_ERROR("Couldn't create relay chan for GuC logging\n");
|
|
|
|
|
|
ret = -ENOMEM;
|
|
|
- goto err_vaddr;
|
|
|
+ goto err;
|
|
|
}
|
|
|
|
|
|
GEM_BUG_ON(guc_log_relay_chan->subbuf_size < subbuf_size);
|
|
|
guc->log.runtime.relay_chan = guc_log_relay_chan;
|
|
|
|
|
|
- INIT_WORK(&guc->log.runtime.flush_work, capture_logs_work);
|
|
|
+ mutex_unlock(&guc->log.runtime.relay_lock);
|
|
|
+
|
|
|
return 0;
|
|
|
|
|
|
-err_vaddr:
|
|
|
- i915_gem_object_unpin_map(guc->log.vma->obj);
|
|
|
- guc->log.runtime.buf_addr = NULL;
|
|
|
+err:
|
|
|
+ mutex_unlock(&guc->log.runtime.relay_lock);
|
|
|
+ /* logging will be off */
|
|
|
+ i915_modparams.guc_log_level = 0;
|
|
|
return ret;
|
|
|
}
|
|
|
|
|
|
-static void guc_log_runtime_destroy(struct intel_guc *guc)
|
|
|
+void intel_guc_log_relay_destroy(struct intel_guc *guc)
|
|
|
{
|
|
|
+ mutex_lock(&guc->log.runtime.relay_lock);
|
|
|
+
|
|
|
/*
|
|
|
- * It's possible that the runtime stuff was never allocated because
|
|
|
- * guc_log_level was < 0 at the time
|
|
|
- **/
|
|
|
- if (!guc_log_has_runtime(guc))
|
|
|
- return;
|
|
|
+ * It's possible that the relay was never allocated because
|
|
|
+ * GuC log was disabled at the boot time.
|
|
|
+ */
|
|
|
+ if (!guc_log_has_relay(guc))
|
|
|
+ goto out_unlock;
|
|
|
|
|
|
relay_close(guc->log.runtime.relay_chan);
|
|
|
- i915_gem_object_unpin_map(guc->log.vma->obj);
|
|
|
- guc->log.runtime.buf_addr = NULL;
|
|
|
+ guc->log.runtime.relay_chan = NULL;
|
|
|
+
|
|
|
+out_unlock:
|
|
|
+ mutex_unlock(&guc->log.runtime.relay_lock);
|
|
|
}
|
|
|
|
|
|
static int guc_log_late_setup(struct intel_guc *guc)
|
|
@@ -438,16 +517,24 @@ static int guc_log_late_setup(struct intel_guc *guc)
|
|
|
struct drm_i915_private *dev_priv = guc_to_i915(guc);
|
|
|
int ret;
|
|
|
|
|
|
- lockdep_assert_held(&dev_priv->drm.struct_mutex);
|
|
|
-
|
|
|
if (!guc_log_has_runtime(guc)) {
|
|
|
- /* If log_level was set as -1 at boot time, then setup needed to
|
|
|
- * handle log buffer flush interrupts would not have been done yet,
|
|
|
- * so do that now.
|
|
|
+ /*
|
|
|
+ * If log was disabled at boot time, then setup needed to handle
|
|
|
+ * log buffer flush interrupts would not have been done yet, so
|
|
|
+ * do that now.
|
|
|
*/
|
|
|
- ret = guc_log_runtime_create(guc);
|
|
|
+ ret = intel_guc_log_relay_create(guc);
|
|
|
if (ret)
|
|
|
goto err;
|
|
|
+
|
|
|
+ mutex_lock(&dev_priv->drm.struct_mutex);
|
|
|
+ intel_runtime_pm_get(dev_priv);
|
|
|
+ ret = guc_log_runtime_create(guc);
|
|
|
+ intel_runtime_pm_put(dev_priv);
|
|
|
+ mutex_unlock(&dev_priv->drm.struct_mutex);
|
|
|
+
|
|
|
+ if (ret)
|
|
|
+ goto err_relay;
|
|
|
}
|
|
|
|
|
|
ret = guc_log_relay_file_create(guc);
|
|
@@ -457,10 +544,14 @@ static int guc_log_late_setup(struct intel_guc *guc)
|
|
|
return 0;
|
|
|
|
|
|
err_runtime:
|
|
|
+ mutex_lock(&dev_priv->drm.struct_mutex);
|
|
|
guc_log_runtime_destroy(guc);
|
|
|
+ mutex_unlock(&dev_priv->drm.struct_mutex);
|
|
|
+err_relay:
|
|
|
+ intel_guc_log_relay_destroy(guc);
|
|
|
err:
|
|
|
/* logging will remain off */
|
|
|
- i915_modparams.guc_log_level = -1;
|
|
|
+ i915_modparams.guc_log_level = 0;
|
|
|
return ret;
|
|
|
}
|
|
|
|
|
@@ -470,7 +561,8 @@ static void guc_log_capture_logs(struct intel_guc *guc)
|
|
|
|
|
|
guc_read_update_log_buffer(guc);
|
|
|
|
|
|
- /* Generally device is expected to be active only at this
|
|
|
+ /*
|
|
|
+ * Generally device is expected to be active only at this
|
|
|
* time, so get/put should be really quick.
|
|
|
*/
|
|
|
intel_runtime_pm_get(dev_priv);
|
|
@@ -482,20 +574,26 @@ static void guc_flush_logs(struct intel_guc *guc)
|
|
|
{
|
|
|
struct drm_i915_private *dev_priv = guc_to_i915(guc);
|
|
|
|
|
|
- if (!USES_GUC_SUBMISSION(dev_priv) ||
|
|
|
- (i915_modparams.guc_log_level < 0))
|
|
|
+ if (!USES_GUC_SUBMISSION(dev_priv) || !i915_modparams.guc_log_level)
|
|
|
return;
|
|
|
|
|
|
/* First disable the interrupts, will be renabled afterwards */
|
|
|
+ mutex_lock(&dev_priv->drm.struct_mutex);
|
|
|
+ intel_runtime_pm_get(dev_priv);
|
|
|
gen9_disable_guc_interrupts(dev_priv);
|
|
|
+ intel_runtime_pm_put(dev_priv);
|
|
|
+ mutex_unlock(&dev_priv->drm.struct_mutex);
|
|
|
|
|
|
- /* Before initiating the forceful flush, wait for any pending/ongoing
|
|
|
+ /*
|
|
|
+ * Before initiating the forceful flush, wait for any pending/ongoing
|
|
|
* flush to complete otherwise forceful flush may not actually happen.
|
|
|
*/
|
|
|
flush_work(&guc->log.runtime.flush_work);
|
|
|
|
|
|
/* Ask GuC to update the log buffer state */
|
|
|
+ intel_runtime_pm_get(dev_priv);
|
|
|
guc_log_flush(guc);
|
|
|
+ intel_runtime_pm_put(dev_priv);
|
|
|
|
|
|
/* GuC would have updated log buffer by now, so capture it */
|
|
|
guc_log_capture_logs(guc);
|
|
@@ -506,21 +604,12 @@ int intel_guc_log_create(struct intel_guc *guc)
|
|
|
struct i915_vma *vma;
|
|
|
unsigned long offset;
|
|
|
u32 flags;
|
|
|
- u32 size;
|
|
|
int ret;
|
|
|
|
|
|
GEM_BUG_ON(guc->log.vma);
|
|
|
|
|
|
- if (i915_modparams.guc_log_level > GUC_LOG_VERBOSITY_MAX)
|
|
|
- i915_modparams.guc_log_level = GUC_LOG_VERBOSITY_MAX;
|
|
|
-
|
|
|
- /* The first page is to save log buffer state. Allocate one
|
|
|
- * extra page for others in case for overlap */
|
|
|
- size = (1 + GUC_LOG_DPC_PAGES + 1 +
|
|
|
- GUC_LOG_ISR_PAGES + 1 +
|
|
|
- GUC_LOG_CRASH_PAGES + 1) << PAGE_SHIFT;
|
|
|
-
|
|
|
- /* We require SSE 4.1 for fast reads from the GuC log buffer and
|
|
|
+ /*
|
|
|
+ * We require SSE 4.1 for fast reads from the GuC log buffer and
|
|
|
* it should be present on the chipsets supporting GuC based
|
|
|
* submisssions.
|
|
|
*/
|
|
@@ -529,7 +618,7 @@ int intel_guc_log_create(struct intel_guc *guc)
|
|
|
goto err;
|
|
|
}
|
|
|
|
|
|
- vma = intel_guc_allocate_vma(guc, size);
|
|
|
+ vma = intel_guc_allocate_vma(guc, GUC_LOG_SIZE);
|
|
|
if (IS_ERR(vma)) {
|
|
|
ret = PTR_ERR(vma);
|
|
|
goto err;
|
|
@@ -537,7 +626,7 @@ int intel_guc_log_create(struct intel_guc *guc)
|
|
|
|
|
|
guc->log.vma = vma;
|
|
|
|
|
|
- if (i915_modparams.guc_log_level >= 0) {
|
|
|
+ if (i915_modparams.guc_log_level) {
|
|
|
ret = guc_log_runtime_create(guc);
|
|
|
if (ret < 0)
|
|
|
goto err_vma;
|
|
@@ -558,7 +647,7 @@ err_vma:
|
|
|
i915_vma_unpin_and_release(&guc->log.vma);
|
|
|
err:
|
|
|
/* logging will be off */
|
|
|
- i915_modparams.guc_log_level = -1;
|
|
|
+ i915_modparams.guc_log_level = 0;
|
|
|
return ret;
|
|
|
}
|
|
|
|
|
@@ -568,35 +657,46 @@ void intel_guc_log_destroy(struct intel_guc *guc)
|
|
|
i915_vma_unpin_and_release(&guc->log.vma);
|
|
|
}
|
|
|
|
|
|
-int i915_guc_log_control(struct drm_i915_private *dev_priv, u64 control_val)
|
|
|
+int intel_guc_log_control(struct intel_guc *guc, u64 control_val)
|
|
|
{
|
|
|
- struct intel_guc *guc = &dev_priv->guc;
|
|
|
-
|
|
|
- union guc_log_control log_param;
|
|
|
+ struct drm_i915_private *dev_priv = guc_to_i915(guc);
|
|
|
+ bool enable_logging = control_val > 0;
|
|
|
+ u32 verbosity;
|
|
|
int ret;
|
|
|
|
|
|
- log_param.value = control_val;
|
|
|
+ if (!guc->log.vma)
|
|
|
+ return -ENODEV;
|
|
|
|
|
|
- if (log_param.verbosity < GUC_LOG_VERBOSITY_MIN ||
|
|
|
- log_param.verbosity > GUC_LOG_VERBOSITY_MAX)
|
|
|
+ BUILD_BUG_ON(GUC_LOG_VERBOSITY_MIN);
|
|
|
+ if (control_val > 1 + GUC_LOG_VERBOSITY_MAX)
|
|
|
return -EINVAL;
|
|
|
|
|
|
/* This combination doesn't make sense & won't have any effect */
|
|
|
- if (!log_param.logging_enabled && (i915_modparams.guc_log_level < 0))
|
|
|
+ if (!enable_logging && !i915_modparams.guc_log_level)
|
|
|
return 0;
|
|
|
|
|
|
- ret = guc_log_control(guc, log_param.value);
|
|
|
+ verbosity = enable_logging ? control_val - 1 : 0;
|
|
|
+
|
|
|
+ ret = mutex_lock_interruptible(&dev_priv->drm.struct_mutex);
|
|
|
+ if (ret)
|
|
|
+ return ret;
|
|
|
+ intel_runtime_pm_get(dev_priv);
|
|
|
+ ret = guc_log_control(guc, enable_logging, verbosity);
|
|
|
+ intel_runtime_pm_put(dev_priv);
|
|
|
+ mutex_unlock(&dev_priv->drm.struct_mutex);
|
|
|
+
|
|
|
if (ret < 0) {
|
|
|
DRM_DEBUG_DRIVER("guc_logging_control action failed %d\n", ret);
|
|
|
return ret;
|
|
|
}
|
|
|
|
|
|
- if (log_param.logging_enabled) {
|
|
|
- i915_modparams.guc_log_level = log_param.verbosity;
|
|
|
+ if (enable_logging) {
|
|
|
+ i915_modparams.guc_log_level = 1 + verbosity;
|
|
|
|
|
|
- /* If log_level was set as -1 at boot time, then the relay channel file
|
|
|
- * wouldn't have been created by now and interrupts also would not have
|
|
|
- * been enabled. Try again now, just in case.
|
|
|
+ /*
|
|
|
+ * If log was disabled at boot time, then the relay channel file
|
|
|
+ * wouldn't have been created by now and interrupts also would
|
|
|
+ * not have been enabled. Try again now, just in case.
|
|
|
*/
|
|
|
ret = guc_log_late_setup(guc);
|
|
|
if (ret < 0) {
|
|
@@ -605,9 +705,14 @@ int i915_guc_log_control(struct drm_i915_private *dev_priv, u64 control_val)
|
|
|
}
|
|
|
|
|
|
/* GuC logging is currently the only user of Guc2Host interrupts */
|
|
|
+ mutex_lock(&dev_priv->drm.struct_mutex);
|
|
|
+ intel_runtime_pm_get(dev_priv);
|
|
|
gen9_enable_guc_interrupts(dev_priv);
|
|
|
+ intel_runtime_pm_put(dev_priv);
|
|
|
+ mutex_unlock(&dev_priv->drm.struct_mutex);
|
|
|
} else {
|
|
|
- /* Once logging is disabled, GuC won't generate logs & send an
|
|
|
+ /*
|
|
|
+ * Once logging is disabled, GuC won't generate logs & send an
|
|
|
* interrupt. But there could be some data in the log buffer
|
|
|
* which is yet to be captured. So request GuC to update the log
|
|
|
* buffer state and then collect the left over logs.
|
|
@@ -615,7 +720,7 @@ int i915_guc_log_control(struct drm_i915_private *dev_priv, u64 control_val)
|
|
|
guc_flush_logs(guc);
|
|
|
|
|
|
/* As logging is disabled, update log level to reflect that */
|
|
|
- i915_modparams.guc_log_level = -1;
|
|
|
+ i915_modparams.guc_log_level = 0;
|
|
|
}
|
|
|
|
|
|
return ret;
|
|
@@ -623,23 +728,27 @@ int i915_guc_log_control(struct drm_i915_private *dev_priv, u64 control_val)
|
|
|
|
|
|
void i915_guc_log_register(struct drm_i915_private *dev_priv)
|
|
|
{
|
|
|
- if (!USES_GUC_SUBMISSION(dev_priv) ||
|
|
|
- (i915_modparams.guc_log_level < 0))
|
|
|
+ if (!USES_GUC_SUBMISSION(dev_priv) || !i915_modparams.guc_log_level)
|
|
|
return;
|
|
|
|
|
|
- mutex_lock(&dev_priv->drm.struct_mutex);
|
|
|
guc_log_late_setup(&dev_priv->guc);
|
|
|
- mutex_unlock(&dev_priv->drm.struct_mutex);
|
|
|
}
|
|
|
|
|
|
void i915_guc_log_unregister(struct drm_i915_private *dev_priv)
|
|
|
{
|
|
|
+ struct intel_guc *guc = &dev_priv->guc;
|
|
|
+
|
|
|
if (!USES_GUC_SUBMISSION(dev_priv))
|
|
|
return;
|
|
|
|
|
|
mutex_lock(&dev_priv->drm.struct_mutex);
|
|
|
/* GuC logging is currently the only user of Guc2Host interrupts */
|
|
|
+ intel_runtime_pm_get(dev_priv);
|
|
|
gen9_disable_guc_interrupts(dev_priv);
|
|
|
- guc_log_runtime_destroy(&dev_priv->guc);
|
|
|
+ intel_runtime_pm_put(dev_priv);
|
|
|
+
|
|
|
+ guc_log_runtime_destroy(guc);
|
|
|
mutex_unlock(&dev_priv->drm.struct_mutex);
|
|
|
+
|
|
|
+ intel_guc_log_relay_destroy(guc);
|
|
|
}
|