mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	drm/amd/display: Add support for hw_state logging via debugfs
[Why] We have logging methods for printing hardware state for newer ASICs but no way to trigger the log output. [How] Add support for triggering the output via writing to a debugfs file entry. Log output currently goes into dmesg for convenience, but accessing via a read should be possible later. Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> Reviewed-by: Jordan Lazare <Jordan.Lazare@amd.com> Acked-by: Leo Li <sunpeng.li@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
		
							parent
							
								
									e5d0170e56
								
							
						
					
					
						commit
						e498eb7136
					
				
					 4 changed files with 77 additions and 4 deletions
				
			
		| 
						 | 
					@ -480,6 +480,11 @@ static int amdgpu_dm_init(struct amdgpu_device *adev)
 | 
				
			||||||
		goto error;
 | 
							goto error;
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					#if defined(CONFIG_DEBUG_FS)
 | 
				
			||||||
 | 
						if (dtn_debugfs_init(adev))
 | 
				
			||||||
 | 
							DRM_ERROR("amdgpu: failed initialize dtn debugfs support.\n");
 | 
				
			||||||
 | 
					#endif
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	DRM_DEBUG_DRIVER("KMS initialized.\n");
 | 
						DRM_DEBUG_DRIVER("KMS initialized.\n");
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return 0;
 | 
						return 0;
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -720,3 +720,56 @@ int connector_debugfs_init(struct amdgpu_dm_connector *connector)
 | 
				
			||||||
	return 0;
 | 
						return 0;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static ssize_t dtn_log_read(
 | 
				
			||||||
 | 
						struct file *f,
 | 
				
			||||||
 | 
						char __user *buf,
 | 
				
			||||||
 | 
						size_t size,
 | 
				
			||||||
 | 
						loff_t *pos)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						/* TODO: Write log output to the user supplied buffer. */
 | 
				
			||||||
 | 
						return 0;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static ssize_t dtn_log_write(
 | 
				
			||||||
 | 
						struct file *f,
 | 
				
			||||||
 | 
						const char __user *buf,
 | 
				
			||||||
 | 
						size_t size,
 | 
				
			||||||
 | 
						loff_t *pos)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						struct amdgpu_device *adev = file_inode(f)->i_private;
 | 
				
			||||||
 | 
						struct dc *dc = adev->dm.dc;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						/* Write triggers log output via dmesg. */
 | 
				
			||||||
 | 
						if (size == 0)
 | 
				
			||||||
 | 
							return 0;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (dc->hwss.log_hw_state)
 | 
				
			||||||
 | 
							dc->hwss.log_hw_state(dc);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						return size;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					int dtn_debugfs_init(struct amdgpu_device *adev)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						static const struct file_operations dtn_log_fops = {
 | 
				
			||||||
 | 
							.owner = THIS_MODULE,
 | 
				
			||||||
 | 
							.read = dtn_log_read,
 | 
				
			||||||
 | 
							.write = dtn_log_write,
 | 
				
			||||||
 | 
							.llseek = default_llseek
 | 
				
			||||||
 | 
						};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						struct drm_minor *minor = adev->ddev->primary;
 | 
				
			||||||
 | 
						struct dentry *root = minor->debugfs_root;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						struct dentry *ent = debugfs_create_file(
 | 
				
			||||||
 | 
							"amdgpu_dm_dtn_log",
 | 
				
			||||||
 | 
							0644,
 | 
				
			||||||
 | 
							root,
 | 
				
			||||||
 | 
							adev,
 | 
				
			||||||
 | 
							&dtn_log_fops);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (IS_ERR(ent))
 | 
				
			||||||
 | 
							return PTR_ERR(ent);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						return 0;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -30,5 +30,6 @@
 | 
				
			||||||
#include "amdgpu_dm.h"
 | 
					#include "amdgpu_dm.h"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
int connector_debugfs_init(struct amdgpu_dm_connector *connector);
 | 
					int connector_debugfs_init(struct amdgpu_dm_connector *connector);
 | 
				
			||||||
 | 
					int dtn_debugfs_init(struct amdgpu_device *adev);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -336,14 +336,28 @@ bool dm_helpers_dp_mst_send_payload_allocation(
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void dm_dtn_log_begin(struct dc_context *ctx)
 | 
					void dm_dtn_log_begin(struct dc_context *ctx)
 | 
				
			||||||
{}
 | 
					{
 | 
				
			||||||
 | 
						pr_info("[dtn begin]\n");
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void dm_dtn_log_append_v(struct dc_context *ctx,
 | 
					void dm_dtn_log_append_v(struct dc_context *ctx,
 | 
				
			||||||
		const char *pMsg, ...)
 | 
							const char *msg, ...)
 | 
				
			||||||
{}
 | 
					{
 | 
				
			||||||
 | 
						struct va_format vaf;
 | 
				
			||||||
 | 
						va_list args;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						va_start(args, msg);
 | 
				
			||||||
 | 
						vaf.fmt = msg;
 | 
				
			||||||
 | 
						vaf.va = &args;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						pr_info("%pV", &vaf);
 | 
				
			||||||
 | 
						va_end(args);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void dm_dtn_log_end(struct dc_context *ctx)
 | 
					void dm_dtn_log_end(struct dc_context *ctx)
 | 
				
			||||||
{}
 | 
					{
 | 
				
			||||||
 | 
						pr_info("[dtn end]\n");
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
bool dm_helpers_dp_mst_start_top_mgr(
 | 
					bool dm_helpers_dp_mst_start_top_mgr(
 | 
				
			||||||
		struct dc_context *ctx,
 | 
							struct dc_context *ctx,
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue