Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-03-25 Thread Jens Wiklander
On Thu, Mar 25, 2021 at 3:50 AM Jisheng Zhang wrote: > > On Wed, 24 Mar 2021 10:53:13 -0400 > Steven Rostedt wrote: > > > > > > On Wed, 24 Mar 2021 07:48:53 -0700 > > Guenter Roeck wrote: > > > > > On Wed, Mar 24, 2021 at 07:34:07AM -0700, Guenter Roeck wrote: > > > > On Wed, Feb 10, 2021 at

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-03-24 Thread Jisheng Zhang
On Wed, 24 Mar 2021 10:53:13 -0400 Steven Rostedt wrote: > > On Wed, 24 Mar 2021 07:48:53 -0700 > Guenter Roeck wrote: > > > On Wed, Mar 24, 2021 at 07:34:07AM -0700, Guenter Roeck wrote: > > > On Wed, Feb 10, 2021 at 02:44:09PM +0800, Jisheng Zhang wrote: > > > > Add tracepoints to

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-03-24 Thread Steven Rostedt
On Wed, 24 Mar 2021 07:48:53 -0700 Guenter Roeck wrote: > On Wed, Mar 24, 2021 at 07:34:07AM -0700, Guenter Roeck wrote: > > On Wed, Feb 10, 2021 at 02:44:09PM +0800, Jisheng Zhang wrote: > > > Add tracepoints to retrieve information about the invoke_fn. This would > > > help to measure how

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-03-24 Thread Guenter Roeck
On Wed, Mar 24, 2021 at 07:34:07AM -0700, Guenter Roeck wrote: > On Wed, Feb 10, 2021 at 02:44:09PM +0800, Jisheng Zhang wrote: > > Add tracepoints to retrieve information about the invoke_fn. This would > > help to measure how many invoke_fn are triggered and how long it takes > > to complete one

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-03-24 Thread Guenter Roeck
On Wed, Feb 10, 2021 at 02:44:09PM +0800, Jisheng Zhang wrote: > Add tracepoints to retrieve information about the invoke_fn. This would > help to measure how many invoke_fn are triggered and how long it takes > to complete one invoke_fn call. > > Signed-off-by: Jisheng Zhang arm64:defconfig:

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-02-24 Thread Jens Wiklander
On Tue, Feb 23, 2021 at 09:19:36AM -0500, Steven Rostedt wrote: > On Tue, 23 Feb 2021 14:11:24 +0100 > Jens Wiklander wrote: > > > I used the -strict option. > > > > ./scripts/checkpatch.pl -strict > > 0001-tee-optee-add-invoke_fn-tracepoints.patch > > WARNING: added, moved or deleted file(s),

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-02-23 Thread Steven Rostedt
On Tue, 23 Feb 2021 14:11:24 +0100 Jens Wiklander wrote: > I used the -strict option. > > ./scripts/checkpatch.pl -strict 0001-tee-optee-add-invoke_fn-tracepoints.patch > WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? > #44: > new file mode 100644 The above is just

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-02-23 Thread Jens Wiklander
On Tue, Feb 23, 2021 at 06:40:26PM +0800, Jisheng Zhang wrote: > On Tue, 23 Feb 2021 08:59:22 +0100 Jens Wiklander wrote: > > > > > > Hi Jisheng, > > Hi Jens, > > > > > On Wed, Feb 10, 2021 at 7:44 AM Jisheng Zhang > > wrote: > > > > > > Add tracepoints to retrieve information about the

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-02-23 Thread Jisheng Zhang
On Tue, 23 Feb 2021 08:59:22 +0100 Jens Wiklander wrote: > > Hi Jisheng, Hi Jens, > > On Wed, Feb 10, 2021 at 7:44 AM Jisheng Zhang > wrote: > > > > Add tracepoints to retrieve information about the invoke_fn. This would > > help to measure how many invoke_fn are triggered and how long it

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-02-23 Thread Jens Wiklander
Hi Jisheng, On Wed, Feb 10, 2021 at 7:44 AM Jisheng Zhang wrote: > > Add tracepoints to retrieve information about the invoke_fn. This would > help to measure how many invoke_fn are triggered and how long it takes > to complete one invoke_fn call. > > Signed-off-by: Jisheng Zhang > --- > >

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-02-10 Thread Steven Rostedt
On Wed, 10 Feb 2021 14:44:09 +0800 Jisheng Zhang wrote: > Add tracepoints to retrieve information about the invoke_fn. This would > help to measure how many invoke_fn are triggered and how long it takes > to complete one invoke_fn call. > > Signed-off-by: Jisheng Zhang > --- > > Since v1: >

[PATCH] tee: optee: add invoke_fn tracepoints

2021-02-09 Thread Jisheng Zhang
Add tracepoints to retrieve information about the invoke_fn. This would help to measure how many invoke_fn are triggered and how long it takes to complete one invoke_fn call. Signed-off-by: Jisheng Zhang --- Since v1: - add BUILD_BUG_ON() macro usage to make sure that the size of what is being

Re: [PATCH] tee: optee: add invoke_fn tracepoints

2021-02-09 Thread Steven Rostedt
On Tue, 9 Feb 2021 19:02:15 +0800 Jisheng Zhang wrote: > Add tracepoints to retrieve information about the invoke_fn. This would > help to measure how many invoke_fn are triggered and how long it takes > to complete one invoke_fn call. > > Signed-off-by: Jisheng Zhang > --- >

[PATCH] tee: optee: add invoke_fn tracepoints

2021-02-09 Thread Jisheng Zhang
Add tracepoints to retrieve information about the invoke_fn. This would help to measure how many invoke_fn are triggered and how long it takes to complete one invoke_fn call. Signed-off-by: Jisheng Zhang --- drivers/tee/optee/call.c| 4 ++ drivers/tee/optee/optee_trace.h | 65