diff mbox

[2/2] drm/i915: Add runtime PM's usage_count in i915_runtime_pm_status

Message ID 1433438638-17434-2-git-send-email-damien.lespiau@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lespiau, Damien June 4, 2015, 5:23 p.m. UTC
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 drivers/gpu/drm/i915/i915_debugfs.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Paulo Zanoni June 5, 2015, 12:24 p.m. UTC | #1
2015-06-04 14:23 GMT-03:00 Damien Lespiau <damien.lespiau@intel.com>:
> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

For both patches: Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>

> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index d8bd4e1..92cf273 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -2493,6 +2493,8 @@ static int i915_runtime_pm_status(struct seq_file *m, void *unused)
>         seq_printf(m, "GPU idle: %s\n", yesno(!dev_priv->mm.busy));
>         seq_printf(m, "IRQs disabled: %s\n",
>                    yesno(!intel_irqs_enabled(dev_priv)));
> +       seq_printf(m, "Usage count: %d\n",
> +                  atomic_read(&dev->dev->power.usage_count));
>
>         return 0;
>  }
> --
> 2.1.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Shuang He June 5, 2015, 6:38 p.m. UTC | #2
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang.he@intel.com)
Task id: 6540
-------------------------------------Summary-------------------------------------
Platform          Delta          drm-intel-nightly          Series Applied
PNV                 -1              270/270              269/270
ILK                                  303/303              303/303
SNB                                  312/312              312/312
IVB                                  343/343              343/343
BYT                                  287/287              287/287
BDW                                  318/318              318/318
-------------------------------------Detailed-------------------------------------
Platform  Test                                drm-intel-nightly          Series Applied
*PNV  igt@gem_tiled_pread_pwrite      PASS(1)      FAIL(1)
Note: You need to pay more attention to line start with '*'
Jani Nikula June 10, 2015, 1:56 p.m. UTC | #3
On Fri, 05 Jun 2015, Paulo Zanoni <przanoni@gmail.com> wrote:
> 2015-06-04 14:23 GMT-03:00 Damien Lespiau <damien.lespiau@intel.com>:
>> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
>
> For both patches: Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>

Pushed to drm-intel-next-queued, thanks for the patches and review.

BR,
Jani.

>
>> ---
>>  drivers/gpu/drm/i915/i915_debugfs.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
>> index d8bd4e1..92cf273 100644
>> --- a/drivers/gpu/drm/i915/i915_debugfs.c
>> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
>> @@ -2493,6 +2493,8 @@ static int i915_runtime_pm_status(struct seq_file *m, void *unused)
>>         seq_printf(m, "GPU idle: %s\n", yesno(!dev_priv->mm.busy));
>>         seq_printf(m, "IRQs disabled: %s\n",
>>                    yesno(!intel_irqs_enabled(dev_priv)));
>> +       seq_printf(m, "Usage count: %d\n",
>> +                  atomic_read(&dev->dev->power.usage_count));
>>
>>         return 0;
>>  }
>> --
>> 2.1.0
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
>
> -- 
> Paulo Zanoni
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Daniel Vetter June 15, 2015, 12:44 p.m. UTC | #4
On Thu, Jun 04, 2015 at 06:23:58PM +0100, Damien Lespiau wrote:
> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index d8bd4e1..92cf273 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -2493,6 +2493,8 @@ static int i915_runtime_pm_status(struct seq_file *m, void *unused)
>  	seq_printf(m, "GPU idle: %s\n", yesno(!dev_priv->mm.busy));
>  	seq_printf(m, "IRQs disabled: %s\n",
>  		   yesno(!intel_irqs_enabled(dev_priv)));
> +	seq_printf(m, "Usage count: %d\n",
> +		   atomic_read(&dev->dev->power.usage_count));

Needs a check for CONFIG_PM. Since I pulled in what Jani merged as a merge
commit, can you please do a follow-up patch?

Thanks, Daniel
Lespiau, Damien June 15, 2015, 12:56 p.m. UTC | #5
On Mon, Jun 15, 2015 at 02:44:05PM +0200, Daniel Vetter wrote:
> On Thu, Jun 04, 2015 at 06:23:58PM +0100, Damien Lespiau wrote:
> > Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
> > ---
> >  drivers/gpu/drm/i915/i915_debugfs.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> > index d8bd4e1..92cf273 100644
> > --- a/drivers/gpu/drm/i915/i915_debugfs.c
> > +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> > @@ -2493,6 +2493,8 @@ static int i915_runtime_pm_status(struct seq_file *m, void *unused)
> >  	seq_printf(m, "GPU idle: %s\n", yesno(!dev_priv->mm.busy));
> >  	seq_printf(m, "IRQs disabled: %s\n",
> >  		   yesno(!intel_irqs_enabled(dev_priv)));
> > +	seq_printf(m, "Usage count: %d\n",
> > +		   atomic_read(&dev->dev->power.usage_count));
> 
> Needs a check for CONFIG_PM. Since I pulled in what Jani merged as a merge
> commit, can you please do a follow-up patch?

Ooops indeed, Chris just sent one.
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index d8bd4e1..92cf273 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -2493,6 +2493,8 @@  static int i915_runtime_pm_status(struct seq_file *m, void *unused)
 	seq_printf(m, "GPU idle: %s\n", yesno(!dev_priv->mm.busy));
 	seq_printf(m, "IRQs disabled: %s\n",
 		   yesno(!intel_irqs_enabled(dev_priv)));
+	seq_printf(m, "Usage count: %d\n",
+		   atomic_read(&dev->dev->power.usage_count));
 
 	return 0;
 }