[PATCH] drm/i915: reinstate call to trace_i915_vma_bind

[PATCH] drm/i915: reinstate call to trace_i915_vma_bind

[daniele.ceraolospurio]

From: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>

The call went away in:

commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Thu Aug 4 16:32:25 2016 +0100

    drm/i915: Split insertion/binding of an object into the VM

It is useful to have this trace as it pairs nicely with the vma_unbind
one to track vma activity.
Added inside the i915_vma_bind function (was outside before) to keep a
similar placement as trace_i915_vma_unbind.

Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
---
 drivers/gpu/drm/i915/i915_vma.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
index b74eeb7..b593748 100644
--- a/drivers/gpu/drm/i915/i915_vma.c
+++ b/drivers/gpu/drm/i915/i915_vma.c
@@ -207,6 +207,7 @@ int i915_vma_bind(struct i915_vma *vma, enum i915_cache_level cache_level,
 			return ret;
 	}
 
+	trace_i915_vma_bind(vma, flags);
 	ret = vma->vm->bind_vma(vma, cache_level, bind_flags);
 	if (ret)
 		return ret;
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✓ Fi.CI.BAT: success for drm/i915: reinstate call to trace_i915_vma_bind

[Patchwork]

== Series Details ==

Series: drm/i915: reinstate call to trace_i915_vma_bind
URL   : https://patchwork.freedesktop.org/series/17929/
State : success

== Summary ==

Series 17929v1 drm/i915: reinstate call to trace_i915_vma_bind
https://patchwork.freedesktop.org/api/1.0/series/17929/revisions/1/mbox/


fi-bdw-5557u     total:246  pass:232  dwarn:0   dfail:0   fail:0   skip:14 
fi-bsw-n3050     total:246  pass:207  dwarn:0   dfail:0   fail:0   skip:39 
fi-bxt-j4205     total:246  pass:224  dwarn:0   dfail:0   fail:0   skip:22 
fi-bxt-t5700     total:82   pass:69   dwarn:0   dfail:0   fail:0   skip:12 
fi-byt-j1900     total:246  pass:219  dwarn:0   dfail:0   fail:0   skip:27 
fi-byt-n2820     total:246  pass:215  dwarn:0   dfail:0   fail:0   skip:31 
fi-hsw-4770      total:246  pass:227  dwarn:0   dfail:0   fail:0   skip:19 
fi-hsw-4770r     total:246  pass:227  dwarn:0   dfail:0   fail:0   skip:19 
fi-ivb-3520m     total:246  pass:225  dwarn:0   dfail:0   fail:0   skip:21 
fi-ivb-3770      total:246  pass:225  dwarn:0   dfail:0   fail:0   skip:21 
fi-kbl-7500u     total:246  pass:225  dwarn:0   dfail:0   fail:0   skip:21 
fi-skl-6260u     total:246  pass:233  dwarn:0   dfail:0   fail:0   skip:13 
fi-skl-6700hq    total:246  pass:226  dwarn:0   dfail:0   fail:0   skip:20 
fi-skl-6700k     total:246  pass:222  dwarn:3   dfail:0   fail:0   skip:21 
fi-snb-2520m     total:246  pass:215  dwarn:0   dfail:0   fail:0   skip:31 
fi-snb-2600      total:246  pass:214  dwarn:0   dfail:0   fail:0   skip:32 

b72db2710635164c7b0495192fbf44e3a17362f6 drm-tip: 2017y-01m-12d-20h-39m-13s UTC integration manifest
6c31af4 drm/i915: reinstate call to trace_i915_vma_bind

== Logs ==

For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_3508/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915: reinstate call to trace_i915_vma_bind

[Daniele Ceraolo Spurio]

Ping. can anyone review/comment on this?

Thanks,
Daniele

On 12/01/17 14:21, daniele.ceraolospurio@intel.com wrote:
> From: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>
> The call went away in:
>
> commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Thu Aug 4 16:32:25 2016 +0100
>
>     drm/i915: Split insertion/binding of an object into the VM
>
> It is useful to have this trace as it pairs nicely with the vma_unbind
> one to track vma activity.
> Added inside the i915_vma_bind function (was outside before) to keep a
> similar placement as trace_i915_vma_unbind.
>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_vma.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
> index b74eeb7..b593748 100644
> --- a/drivers/gpu/drm/i915/i915_vma.c
> +++ b/drivers/gpu/drm/i915/i915_vma.c
> @@ -207,6 +207,7 @@ int i915_vma_bind(struct i915_vma *vma, enum i915_cache_level cache_level,
>  			return ret;
>  	}
>
> +	trace_i915_vma_bind(vma, flags);
>  	ret = vma->vm->bind_vma(vma, cache_level, bind_flags);
>  	if (ret)
>  		return ret;
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915: reinstate call to trace_i915_vma_bind

[Chris Wilson]

On Fri, Jan 20, 2017 at 01:16:13PM -0800, Daniele Ceraolo Spurio wrote:
> Ping. can anyone review/comment on this?
> 
> Thanks,
> Daniele
> 
> On 12/01/17 14:21, daniele.ceraolospurio@intel.com wrote:
> >From: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> >
> >The call went away in:
> >
> >commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
> >Author: Chris Wilson <chris@chris-wilson.co.uk>
> >Date:   Thu Aug 4 16:32:25 2016 +0100
> >
> >    drm/i915: Split insertion/binding of an object into the VM
> >
> >It is useful to have this trace as it pairs nicely with the vma_unbind
> >one to track vma activity.
> >Added inside the i915_vma_bind function (was outside before) to keep a
> >similar placement as trace_i915_vma_unbind.
> >
> >Cc: Chris Wilson <chris@chris-wilson.co.uk>
> >Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> >---
> > drivers/gpu/drm/i915/i915_vma.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> >diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
> >index b74eeb7..b593748 100644
> >--- a/drivers/gpu/drm/i915/i915_vma.c
> >+++ b/drivers/gpu/drm/i915/i915_vma.c
> >@@ -207,6 +207,7 @@ int i915_vma_bind(struct i915_vma *vma, enum i915_cache_level cache_level,
> > 			return ret;
> > 	}
> >
> >+	trace_i915_vma_bind(vma, flags);

We need it but that's not the information being used in the bind.
(I know that's what it used to be, but it's a long time since it was
correct.)

> > 	ret = vma->vm->bind_vma(vma, cache_level, bind_flags);
> > 	if (ret)
> > 		return ret;
> >
> 

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915: reinstate call to trace_i915_vma_bind

[Daniele Ceraolo Spurio]

On 20/01/17 13:28, Chris Wilson wrote:
> On Fri, Jan 20, 2017 at 01:16:13PM -0800, Daniele Ceraolo Spurio wrote:
>> Ping. can anyone review/comment on this?
>>
>> Thanks,
>> Daniele
>>
>> On 12/01/17 14:21, daniele.ceraolospurio@intel.com wrote:
>>> From: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>>>
>>> The call went away in:
>>>
>>> commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
>>> Author: Chris Wilson <chris@chris-wilson.co.uk>
>>> Date:   Thu Aug 4 16:32:25 2016 +0100
>>>
>>>    drm/i915: Split insertion/binding of an object into the VM
>>>
>>> It is useful to have this trace as it pairs nicely with the vma_unbind
>>> one to track vma activity.
>>> Added inside the i915_vma_bind function (was outside before) to keep a
>>> similar placement as trace_i915_vma_unbind.
>>>
>>> Cc: Chris Wilson <chris@chris-wilson.co.uk>
>>> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>>> ---
>>> drivers/gpu/drm/i915/i915_vma.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
>>> index b74eeb7..b593748 100644
>>> --- a/drivers/gpu/drm/i915/i915_vma.c
>>> +++ b/drivers/gpu/drm/i915/i915_vma.c
>>> @@ -207,6 +207,7 @@ int i915_vma_bind(struct i915_vma *vma, enum i915_cache_level cache_level,
>>> 			return ret;
>>> 	}
>>>
>>> +	trace_i915_vma_bind(vma, flags);
>
> We need it but that's not the information being used in the bind.
> (I know that's what it used to be, but it's a long time since it was
> correct.)
>

would s/flags/bind_flags/ be enough or do you have any other info in 
mind that you want to pass (e.g. vma->flags)?

Thanks,
Daniele

>>> 	ret = vma->vm->bind_vma(vma, cache_level, bind_flags);
>>> 	if (ret)
>>> 		return ret;
>>>
>>
>

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH] drm/i915: reinstate call to trace_i915_vma_bind

[Chris Wilson]

On Fri, Jan 20, 2017 at 01:35:21PM -0800, Daniele Ceraolo Spurio wrote:
> 
> 
> On 20/01/17 13:28, Chris Wilson wrote:
> >On Fri, Jan 20, 2017 at 01:16:13PM -0800, Daniele Ceraolo Spurio wrote:
> >>Ping. can anyone review/comment on this?
> >>
> >>Thanks,
> >>Daniele
> >>
> >>On 12/01/17 14:21, daniele.ceraolospurio@intel.com wrote:
> >>>From: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> >>>
> >>>The call went away in:
> >>>
> >>>commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
> >>>Author: Chris Wilson <chris@chris-wilson.co.uk>
> >>>Date:   Thu Aug 4 16:32:25 2016 +0100
> >>>
> >>>   drm/i915: Split insertion/binding of an object into the VM
> >>>
> >>>It is useful to have this trace as it pairs nicely with the vma_unbind
> >>>one to track vma activity.
> >>>Added inside the i915_vma_bind function (was outside before) to keep a
> >>>similar placement as trace_i915_vma_unbind.
> >>>
> >>>Cc: Chris Wilson <chris@chris-wilson.co.uk>
> >>>Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> >>>---
> >>>drivers/gpu/drm/i915/i915_vma.c | 1 +
> >>>1 file changed, 1 insertion(+)
> >>>
> >>>diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
> >>>index b74eeb7..b593748 100644
> >>>--- a/drivers/gpu/drm/i915/i915_vma.c
> >>>+++ b/drivers/gpu/drm/i915/i915_vma.c
> >>>@@ -207,6 +207,7 @@ int i915_vma_bind(struct i915_vma *vma, enum i915_cache_level cache_level,
> >>>			return ret;
> >>>	}
> >>>
> >>>+	trace_i915_vma_bind(vma, flags);
> >
> >We need it but that's not the information being used in the bind.
> >(I know that's what it used to be, but it's a long time since it was
> >correct.)
> >
> 
> would s/flags/bind_flags/ be enough or do you have any other info in
> mind that you want to pass (e.g. vma->flags)?

Bind flags is more suitable, and will do for the moment. It's a little
bit more complicated since the actual bind may be more than the flags we
set. Bleh.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[PATCH v2] drm/i915: reinstate call to trace_i915_vma_bind

[Daniele Ceraolo Spurio]

The call went away in:

commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Thu Aug 4 16:32:25 2016 +0100

    drm/i915: Split insertion/binding of an object into the VM

It is useful to have this trace as it pairs nicely with the vma_unbind
one to track vma activity.
Added inside the i915_vma_bind function (was outside before) to keep a
similar placement as trace_i915_vma_unbind.

v2: print bind_flags instead of flags (Chris)

Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
---
 drivers/gpu/drm/i915/i915_vma.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
index 635f263..9d32696 100644
--- a/drivers/gpu/drm/i915/i915_vma.c
+++ b/drivers/gpu/drm/i915/i915_vma.c
@@ -253,6 +253,7 @@ int i915_vma_bind(struct i915_vma *vma, enum i915_cache_level cache_level,
 			return ret;
 	}
 
+	trace_i915_vma_bind(vma, bind_flags);
 	ret = vma->vm->bind_vma(vma, cache_level, bind_flags);
 	if (ret)
 		return ret;
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✓ Fi.CI.BAT: success for drm/i915: reinstate call to trace_i915_vma_bind (rev2)

[Patchwork]

== Series Details ==

Series: drm/i915: reinstate call to trace_i915_vma_bind (rev2)
URL   : https://patchwork.freedesktop.org/series/17929/
State : success

== Summary ==

Series 17929v2 drm/i915: reinstate call to trace_i915_vma_bind
https://patchwork.freedesktop.org/api/1.0/series/17929/revisions/2/mbox/


fi-bdw-5557u     total:246  pass:232  dwarn:0   dfail:0   fail:0   skip:14 
fi-bsw-n3050     total:246  pass:207  dwarn:0   dfail:0   fail:0   skip:39 
fi-bxt-j4205     total:246  pass:224  dwarn:0   dfail:0   fail:0   skip:22 
fi-bxt-t5700     total:79   pass:66   dwarn:0   dfail:0   fail:0   skip:12 
fi-byt-j1900     total:246  pass:219  dwarn:0   dfail:0   fail:0   skip:27 
fi-byt-n2820     total:246  pass:215  dwarn:0   dfail:0   fail:0   skip:31 
fi-hsw-4770      total:246  pass:227  dwarn:0   dfail:0   fail:0   skip:19 
fi-hsw-4770r     total:246  pass:227  dwarn:0   dfail:0   fail:0   skip:19 
fi-ivb-3520m     total:246  pass:225  dwarn:0   dfail:0   fail:0   skip:21 
fi-ivb-3770      total:246  pass:225  dwarn:0   dfail:0   fail:0   skip:21 
fi-skl-6260u     total:246  pass:233  dwarn:0   dfail:0   fail:0   skip:13 
fi-skl-6700hq    total:246  pass:226  dwarn:0   dfail:0   fail:0   skip:20 
fi-skl-6700k     total:246  pass:222  dwarn:3   dfail:0   fail:0   skip:21 
fi-skl-6770hq    total:246  pass:233  dwarn:0   dfail:0   fail:0   skip:13 
fi-snb-2520m     total:246  pass:215  dwarn:0   dfail:0   fail:0   skip:31 
fi-snb-2600      total:246  pass:214  dwarn:0   dfail:0   fail:0   skip:32 

b32b21b117a110873013e5e6d4fa880b6a81a404 drm-tip: 2017y-01m-20d-20h-00m-57s UTC integration manifest
aca8d0f drm/i915: reinstate call to trace_i915_vma_bind

== Logs ==

For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_3567/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH v2] drm/i915: reinstate call to trace_i915_vma_bind

[Chris Wilson]

On Fri, Jan 20, 2017 at 01:51:23PM -0800, Daniele Ceraolo Spurio wrote:
> The call went away in:
> 
> commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
> Author: Chris Wilson <chris@chris-wilson.co.uk>
> Date:   Thu Aug 4 16:32:25 2016 +0100
> 
>     drm/i915: Split insertion/binding of an object into the VM
> 
> It is useful to have this trace as it pairs nicely with the vma_unbind
> one to track vma activity.
> Added inside the i915_vma_bind function (was outside before) to keep a
> similar placement as trace_i915_vma_unbind.
> 
> v2: print bind_flags instead of flags (Chris)
> 
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>

Pushed, thanks for the patch.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx