Re: [Intel-gfx] [PATCH v2 4/4] drm/i915/fbc: Register per-crtc debugfs files

Jani Nikula
2021-12-13T14:02:34Z

On Mon, 13 Dec 2021, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Expose FBC debugfs files for each crtc. These may or may not point
> to the same FBC instance depending on the platform.
>
> We leave the old global debugfs files in place until
> igt catches up to the new per-crtc approach.
>
> Cc: Jani Nikula <jani.nikula@intel.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_fbc.c | 31 +++++++++++++++---------
> 1 file changed, 19 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c b/drivers/gpu/drm/i915/display/intel_fbc.c
> index 53c93387710c..eafd84e7f058 100644
> --- a/drivers/gpu/drm/i915/display/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/display/intel_fbc.c
> @@ -1798,25 +1798,32 @@ DEFINE_SIMPLE_ATTRIBUTE(intel_fbc_debugfs_false_color_fops,
> intel_fbc_debugfs_false_color_set,
> "%llu\n");
>
> -static void intel_fbc_debugfs_add(struct intel_fbc *fbc)
> +static void intel_fbc_debugfs_add(struct intel_fbc *fbc,
> + struct dentry *parent)
> {
> - struct drm_i915_private *i915 = fbc->i915;
> - struct drm_minor *minor = i915->drm.primary;
> -
> - debugfs_create_file("i915_fbc_status", 0444,
> - minor->debugfs_root, fbc,
> - &intel_fbc_debugfs_status_fops);
> + debugfs_create_file("i915_fbc_status", 0444, parent,
> + fbc, &intel_fbc_debugfs_status_fops);
>
> if (fbc->funcs->set_false_color)
> - debugfs_create_file("i915_fbc_false_color", 0644,
> - minor->debugfs_root, fbc,
> - &intel_fbc_debugfs_false_color_fops);
> + debugfs_create_file("i915_fbc_false_color", 0644, parent,
> + fbc, &intel_fbc_debugfs_false_color_fops);
> }
>
> void intel_fbc_debugfs_register(struct drm_i915_private *i915)
> {
> - struct intel_fbc *fbc = i915->fbc[INTEL_FBC_A];
> + struct drm_minor *minor = i915->drm.primary;
> + struct intel_crtc *crtc;
> + struct intel_fbc *fbc;
>
> + for_each_intel_crtc(&i915->drm, crtc) {
> + struct intel_plane *plane = to_intel_plane(crtc->base.primary);
> +
> + if (plane->fbc)
> + intel_fbc_debugfs_add(plane->fbc, crtc->base.debugfs_entry);
> + }

I think the per-crtc debugfs files should be registered via
intel_crtc_debugfs_add(). Maybe leave the current
intel_fbc_debugfs_register() function as-is and remove once igt is
fixed.

BR,
Jani.

> +
> + /* FIXME: remove this once igt is on board with per-crtc stuff */
> + fbc = i915->fbc[INTEL_FBC_A];
> if (fbc)
> - intel_fbc_debugfs_add(fbc);
> + intel_fbc_debugfs_add(fbc, minor->debugfs_root);
> }

--
Jani Nikula, Intel Open Source Graphics Center