In order to provide alignment between IGT intel_guc_logger tool and
i915 kernel's guc log relay channels without requiring updating both
repositories everytime a sizing change is made, provide that info
via debugfs files.
Signed-off-by: Alan Previn <alan.previn.teres.alexis@xxxxxxxxx>
---
drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 10 ++++--
drivers/gpu/drm/i915/gt/uc/intel_guc_log.h | 2 ++
.../drm/i915/gt/uc/intel_guc_log_debugfs.c | 34 +++++++++++++++++++
3 files changed, 44 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
index 2fa952916120..6e880d9f42d4 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
@@ -141,7 +141,7 @@ u32 intel_guc_log_section_size_capture(struct intel_guc_log *log)
return log->sizes[GUC_LOG_SECTIONS_CAPTURE].bytes;
}
-static u32 intel_guc_log_size(struct intel_guc_log *log)
+u32 intel_guc_log_size(struct intel_guc_log *log)
{
/*
* GuC Log buffer Layout:
@@ -170,6 +170,12 @@ static u32 intel_guc_log_size(struct intel_guc_log *log)
intel_guc_log_section_size_capture(log);
}
+#define GUC_LOG_RELAY_SUBBUF_COUNT 8
+u32 intel_guc_log_relay_subbuf_count(struct intel_guc_log *log)
+{
+ return GUC_LOG_RELAY_SUBBUF_COUNT;
+}
+
/**
* DOC: GuC firmware log
*
@@ -543,7 +549,7 @@ static int guc_log_relay_create(struct intel_guc_log *log)
* latency, for consuming the logs from relay. Also doesn't take
* up too much memory.
*/
- n_subbufs = 8;
+ n_subbufs = intel_guc_log_relay_subbuf_count(log);
guc_log_relay_chan = relay_open("guc_log",
dev_priv->drm.primary->debugfs_root,
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h
index 02127703be80..c981eb8f4990 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h
@@ -81,6 +81,8 @@ unsigned int intel_guc_get_log_buffer_size(struct intel_guc_log *log,
size_t intel_guc_get_log_buffer_offset(struct intel_guc_log *log, enum guc_log_buffer_type type);
int intel_guc_log_create(struct intel_guc_log *log);
void intel_guc_log_destroy(struct intel_guc_log *log);
+u32 intel_guc_log_size(struct intel_guc_log *log);
+u32 intel_guc_log_relay_subbuf_count(struct intel_guc_log *log);
int intel_guc_log_set_level(struct intel_guc_log *log, u32 level);
bool intel_guc_log_relay_created(const struct intel_guc_log *log);
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c
index ddfbe334689f..27756640338e 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c
@@ -105,6 +105,38 @@ DEFINE_SIMPLE_ATTRIBUTE(guc_log_level_fops,
guc_log_level_get, guc_log_level_set,
"%lld\n");
+static int guc_log_relay_subbuf_size_get(void *data, u64 *val)
+{
+ struct intel_guc_log *log = data;
+
+ if (!log->vma)
+ return -ENODEV;
+
+ *val = (u64)intel_guc_log_size(log);
+
+ return 0;
+}
+
+DEFINE_SIMPLE_ATTRIBUTE(guc_log_relay_subbuf_size_fops,
+ guc_log_relay_subbuf_size_get, NULL,
+ "%lld\n");
+
+static int guc_log_relay_subbuf_count_get(void *data, u64 *val)
+{
+ struct intel_guc_log *log = data;
+
+ if (!log->vma)
+ return -ENODEV;
+
+ *val = intel_guc_log_relay_subbuf_count(log);
+
+ return 0;
+}
+
+DEFINE_SIMPLE_ATTRIBUTE(guc_log_relay_subbuf_count_fops,
+ guc_log_relay_subbuf_count_get, NULL,
+ "%lld\n");
+
static int guc_log_relay_open(struct inode *inode, struct file *file)
{
struct intel_guc_log *log = inode->i_private;
@@ -166,6 +198,8 @@ void intel_guc_log_debugfs_register(struct intel_guc_log *log,
{ "guc_load_err_log_dump", &guc_load_err_log_dump_fops, NULL },
{ "guc_log_level", &guc_log_level_fops, NULL },
{ "guc_log_relay", &guc_log_relay_fops, NULL },
+ { "guc_log_relay_subbuf_size", &guc_log_relay_subbuf_size_fops, NULL },
+ { "guc_log_relay_subbuf_count", &guc_log_relay_subbuf_count_fops, NULL },
};
if (!intel_guc_is_supported(log_to_guc(log)))
--
2.34.1
