* [PATCH] drm/amdgpu: change trace event parameter name from 'driect' to 'immediate'
@ 2020-12-07 6:56 Kevin Wang
2020-12-07 8:23 ` Christian König
0 siblings, 1 reply; 3+ messages in thread
From: Kevin Wang @ 2020-12-07 6:56 UTC (permalink / raw)
To: amd-gfx; +Cc: Kevin Wang, christian.koenig
s/direct/immediate/g
amdgpu vm has renamed parameter name from 'direct' to 'immedate'.
however, the trace event is not updated yet.
Signed-off-by: Kevin Wang <kevin1.wang@amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 24 +++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
index ee9480d14cbc..324d5e3f3579 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
@@ -366,15 +366,15 @@ TRACE_EVENT(amdgpu_vm_update_ptes,
TRACE_EVENT(amdgpu_vm_set_ptes,
TP_PROTO(uint64_t pe, uint64_t addr, unsigned count,
- uint32_t incr, uint64_t flags, bool direct),
- TP_ARGS(pe, addr, count, incr, flags, direct),
+ uint32_t incr, uint64_t flags, bool immediate),
+ TP_ARGS(pe, addr, count, incr, flags, immediate),
TP_STRUCT__entry(
__field(u64, pe)
__field(u64, addr)
__field(u32, count)
__field(u32, incr)
__field(u64, flags)
- __field(bool, direct)
+ __field(bool, immediate)
),
TP_fast_assign(
@@ -383,32 +383,32 @@ TRACE_EVENT(amdgpu_vm_set_ptes,
__entry->count = count;
__entry->incr = incr;
__entry->flags = flags;
- __entry->direct = direct;
+ __entry->immediate = immediate;
),
TP_printk("pe=%010Lx, addr=%010Lx, incr=%u, flags=%llx, count=%u, "
- "direct=%d", __entry->pe, __entry->addr, __entry->incr,
- __entry->flags, __entry->count, __entry->direct)
+ "immediate=%d", __entry->pe, __entry->addr, __entry->incr,
+ __entry->flags, __entry->count, __entry->immediate)
);
TRACE_EVENT(amdgpu_vm_copy_ptes,
- TP_PROTO(uint64_t pe, uint64_t src, unsigned count, bool direct),
- TP_ARGS(pe, src, count, direct),
+ TP_PROTO(uint64_t pe, uint64_t src, unsigned count, bool immediate),
+ TP_ARGS(pe, src, count, immediate),
TP_STRUCT__entry(
__field(u64, pe)
__field(u64, src)
__field(u32, count)
- __field(bool, direct)
+ __field(bool, immediate)
),
TP_fast_assign(
__entry->pe = pe;
__entry->src = src;
__entry->count = count;
- __entry->direct = direct;
+ __entry->immediate = immediate;
),
- TP_printk("pe=%010Lx, src=%010Lx, count=%u, direct=%d",
+ TP_printk("pe=%010Lx, src=%010Lx, count=%u, immediate=%d",
__entry->pe, __entry->src, __entry->count,
- __entry->direct)
+ __entry->immediate)
);
TRACE_EVENT(amdgpu_vm_flush,
--
2.17.1
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/amdgpu: change trace event parameter name from 'driect' to 'immediate'
2020-12-07 6:56 [PATCH] drm/amdgpu: change trace event parameter name from 'driect' to 'immediate' Kevin Wang
@ 2020-12-07 8:23 ` Christian König
2020-12-07 8:33 ` Wang, Kevin(Yang)
0 siblings, 1 reply; 3+ messages in thread
From: Christian König @ 2020-12-07 8:23 UTC (permalink / raw)
To: Kevin Wang, amd-gfx
Am 07.12.20 um 07:56 schrieb Kevin Wang:
> s/direct/immediate/g
>
> amdgpu vm has renamed parameter name from 'direct' to 'immedate'.
> however, the trace event is not updated yet.
>
> Signed-off-by: Kevin Wang <kevin1.wang@amd.com>
There is a small typo in the subject line "driect", apart from that the
patch is Reviewed-by: Christian König <christian.koenig@amd.com>.
Thanks for removing one item from my TODO list :)
Regards,
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 24 +++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> index ee9480d14cbc..324d5e3f3579 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> @@ -366,15 +366,15 @@ TRACE_EVENT(amdgpu_vm_update_ptes,
>
> TRACE_EVENT(amdgpu_vm_set_ptes,
> TP_PROTO(uint64_t pe, uint64_t addr, unsigned count,
> - uint32_t incr, uint64_t flags, bool direct),
> - TP_ARGS(pe, addr, count, incr, flags, direct),
> + uint32_t incr, uint64_t flags, bool immediate),
> + TP_ARGS(pe, addr, count, incr, flags, immediate),
> TP_STRUCT__entry(
> __field(u64, pe)
> __field(u64, addr)
> __field(u32, count)
> __field(u32, incr)
> __field(u64, flags)
> - __field(bool, direct)
> + __field(bool, immediate)
> ),
>
> TP_fast_assign(
> @@ -383,32 +383,32 @@ TRACE_EVENT(amdgpu_vm_set_ptes,
> __entry->count = count;
> __entry->incr = incr;
> __entry->flags = flags;
> - __entry->direct = direct;
> + __entry->immediate = immediate;
> ),
> TP_printk("pe=%010Lx, addr=%010Lx, incr=%u, flags=%llx, count=%u, "
> - "direct=%d", __entry->pe, __entry->addr, __entry->incr,
> - __entry->flags, __entry->count, __entry->direct)
> + "immediate=%d", __entry->pe, __entry->addr, __entry->incr,
> + __entry->flags, __entry->count, __entry->immediate)
> );
>
> TRACE_EVENT(amdgpu_vm_copy_ptes,
> - TP_PROTO(uint64_t pe, uint64_t src, unsigned count, bool direct),
> - TP_ARGS(pe, src, count, direct),
> + TP_PROTO(uint64_t pe, uint64_t src, unsigned count, bool immediate),
> + TP_ARGS(pe, src, count, immediate),
> TP_STRUCT__entry(
> __field(u64, pe)
> __field(u64, src)
> __field(u32, count)
> - __field(bool, direct)
> + __field(bool, immediate)
> ),
>
> TP_fast_assign(
> __entry->pe = pe;
> __entry->src = src;
> __entry->count = count;
> - __entry->direct = direct;
> + __entry->immediate = immediate;
> ),
> - TP_printk("pe=%010Lx, src=%010Lx, count=%u, direct=%d",
> + TP_printk("pe=%010Lx, src=%010Lx, count=%u, immediate=%d",
> __entry->pe, __entry->src, __entry->count,
> - __entry->direct)
> + __entry->immediate)
> );
>
> TRACE_EVENT(amdgpu_vm_flush,
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/amdgpu: change trace event parameter name from 'driect' to 'immediate'
2020-12-07 8:23 ` Christian König
@ 2020-12-07 8:33 ` Wang, Kevin(Yang)
0 siblings, 0 replies; 3+ messages in thread
From: Wang, Kevin(Yang) @ 2020-12-07 8:33 UTC (permalink / raw)
To: Koenig, Christian, amd-gfx
[-- Attachment #1.1: Type: text/plain, Size: 4208 bytes --]
[AMD Official Use Only - Internal Distribution Only]
________________________________
From: Koenig, Christian <Christian.Koenig@amd.com>
Sent: Monday, December 7, 2020 4:23 PM
To: Wang, Kevin(Yang) <Kevin1.Wang@amd.com>; amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>
Subject: Re: [PATCH] drm/amdgpu: change trace event parameter name from 'driect' to 'immediate'
Am 07.12.20 um 07:56 schrieb Kevin Wang:
> s/direct/immediate/g
>
> amdgpu vm has renamed parameter name from 'direct' to 'immedate'.
> however, the trace event is not updated yet.
>
> Signed-off-by: Kevin Wang <kevin1.wang@amd.com>
There is a small typo in the subject line "driect", apart from that the
patch is Reviewed-by: Christian König <christian.koenig@amd.com>.
Thanks for removing one item from my TODO list :)
Regards,
Christian.
[kevin]:
ok, thank you for reminding me.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 24 +++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> index ee9480d14cbc..324d5e3f3579 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
> @@ -366,15 +366,15 @@ TRACE_EVENT(amdgpu_vm_update_ptes,
>
> TRACE_EVENT(amdgpu_vm_set_ptes,
> TP_PROTO(uint64_t pe, uint64_t addr, unsigned count,
> - uint32_t incr, uint64_t flags, bool direct),
> - TP_ARGS(pe, addr, count, incr, flags, direct),
> + uint32_t incr, uint64_t flags, bool immediate),
> + TP_ARGS(pe, addr, count, incr, flags, immediate),
> TP_STRUCT__entry(
> __field(u64, pe)
> __field(u64, addr)
> __field(u32, count)
> __field(u32, incr)
> __field(u64, flags)
> - __field(bool, direct)
> + __field(bool, immediate)
> ),
>
> TP_fast_assign(
> @@ -383,32 +383,32 @@ TRACE_EVENT(amdgpu_vm_set_ptes,
> __entry->count = count;
> __entry->incr = incr;
> __entry->flags = flags;
> - __entry->direct = direct;
> + __entry->immediate = immediate;
> ),
> TP_printk("pe=%010Lx, addr=%010Lx, incr=%u, flags=%llx, count=%u, "
> - "direct=%d", __entry->pe, __entry->addr, __entry->incr,
> - __entry->flags, __entry->count, __entry->direct)
> + "immediate=%d", __entry->pe, __entry->addr, __entry->incr,
> + __entry->flags, __entry->count, __entry->immediate)
> );
>
> TRACE_EVENT(amdgpu_vm_copy_ptes,
> - TP_PROTO(uint64_t pe, uint64_t src, unsigned count, bool direct),
> - TP_ARGS(pe, src, count, direct),
> + TP_PROTO(uint64_t pe, uint64_t src, unsigned count, bool immediate),
> + TP_ARGS(pe, src, count, immediate),
> TP_STRUCT__entry(
> __field(u64, pe)
> __field(u64, src)
> __field(u32, count)
> - __field(bool, direct)
> + __field(bool, immediate)
> ),
>
> TP_fast_assign(
> __entry->pe = pe;
> __entry->src = src;
> __entry->count = count;
> - __entry->direct = direct;
> + __entry->immediate = immediate;
> ),
> - TP_printk("pe=%010Lx, src=%010Lx, count=%u, direct=%d",
> + TP_printk("pe=%010Lx, src=%010Lx, count=%u, immediate=%d",
> __entry->pe, __entry->src, __entry->count,
> - __entry->direct)
> + __entry->immediate)
> );
>
> TRACE_EVENT(amdgpu_vm_flush,
[-- Attachment #1.2: Type: text/html, Size: 10870 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-12-07 8:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-07 6:56 [PATCH] drm/amdgpu: change trace event parameter name from 'driect' to 'immediate' Kevin Wang
2020-12-07 8:23 ` Christian König
2020-12-07 8:33 ` Wang, Kevin(Yang)
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.