[PATCH 2/2] drm/amdgpu: Allow to reset to EERPOM table.

[Zhou1, Tao]

> -----Original Message-----
> From: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
> Sent: 2019年9月10日 4:04
> To: amd-gfx at lists.freedesktop.org
> Cc: Chen, Guchun <Guchun.Chen at amd.com>; Zhou1, Tao
> <Tao.Zhou1 at amd.com>; Deucher, Alexander
> <Alexander.Deucher at amd.com>; Grodzovsky, Andrey
> <Andrey.Grodzovsky at amd.com>
> Subject: [PATCH 2/2] drm/amdgpu: Allow to reset to EERPOM table.
> 
> The table grows quickly during debug/development effort when multiple RAS
> errors are injected. Allow to avoid this by setting table header back to empty
> if needed.
> 
> v2: Switch to debugfs entry instead of load time parameter.
> 
> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
> index 119bedc..52c5c61 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
> @@ -303,6 +303,17 @@ static ssize_t amdgpu_ras_debugfs_ctrl_write(struct
> file *f, const char __user *
>  	return size;
>  }
> 
> +static ssize_t amdgpu_ras_debugfs_eeprom_write(struct file *f, const char
> __user *buf,
> +		size_t size, loff_t *pos)
> +{
> +	struct amdgpu_device *adev = (struct amdgpu_device *)file_inode(f)-
> >i_private;
> +	int ret;
> +
> +	ret =
> +amdgpu_ras_eeprom_reset_table(&adev->psp.ras.ras->eeprom_control);

[Tao] It's better to add table key in front of the function, with this fixed, the series is:

Reviewed-by: Tao Zhou <tao.zhou1 at amd.com>

> +
> +	return ret == 1 ? size : -EIO;
> +}
> +
>  static const struct file_operations amdgpu_ras_debugfs_ctrl_ops = {
>  	.owner = THIS_MODULE,
>  	.read = NULL,
> @@ -310,6 +321,13 @@ static const struct file_operations
> amdgpu_ras_debugfs_ctrl_ops = {
>  	.llseek = default_llseek
>  };
> 
> +static const struct file_operations amdgpu_ras_debugfs_eeprom_ops = {
> +	.owner = THIS_MODULE,
> +	.read = NULL,
> +	.write = amdgpu_ras_debugfs_eeprom_write,
> +	.llseek = default_llseek
> +};
> +
>  static ssize_t amdgpu_ras_sysfs_read(struct device *dev,
>  		struct device_attribute *attr, char *buf)  { @@ -951,6 +969,8
> @@ static void amdgpu_ras_debugfs_create_ctrl_node(struct
> amdgpu_device *adev)
>  	con->dir = debugfs_create_dir("ras", minor->debugfs_root);
>  	con->ent = debugfs_create_file("ras_ctrl", S_IWUGO | S_IRUGO, con-
> >dir,
>  				       adev, &amdgpu_ras_debugfs_ctrl_ops);
> +	con->ent = debugfs_create_file("ras_eeprom_reset", S_IWUGO |
> S_IRUGO, con->dir,
> +					       adev,
> &amdgpu_ras_debugfs_eeprom_ops);
>  }
> 
>  void amdgpu_ras_debugfs_create(struct amdgpu_device *adev,
> --
> 2.7.4