2020-11-10 15:17:18

by Greg Kroah-Hartman

Subject: Re: [PATCH v12 2/3] Since the original trace_binder_transaction_received cannot precisely present the real finished time of transaction, adding a trace_binder_txn_latency_free at the point of free transaction may be more close to it.

On Tue, Nov 10, 2020 at 10:19:14PM +0800, Frankie Chang wrote:
> From: "Frankie.Chang" <[email protected]>
>
> Signed-off-by: Frankie.Chang <[email protected]>
> Acked-by: Todd Kjos <[email protected]>

What happened to the subject line here?

The changelog body is in the subject line, and the subject is gone?

Can you fix this up for all of these patches and resend?

thanks,

greg k-h

2020-11-11 03:05:06

by Frankie Chang

Subject: [PATCH v13] binder: add transaction latency tracer

Frankie.Chang (3):
binder: move structs from core file to header file
binder: add trace at free transaction.
binder: add transaction latency tracer

drivers/android/Kconfig | 8 +
drivers/android/Makefile | 1 +
drivers/android/binder.c | 430 ++----------------------
drivers/android/binder_internal.h | 419 +++++++++++++++++++++++
drivers/android/binder_latency_tracer.c | 107 ++++++
drivers/android/binder_trace.h | 49 +++
6 files changed, 608 insertions(+), 406 deletions(-)
create mode 100644 drivers/android/binder_latency_tracer.c

2020-11-11 07:38:21

by Greg Kroah-Hartman

Subject: Re: [PATCH v13] binder: add transaction latency tracer

On Wed, Nov 11, 2020 at 11:02:41AM +0800, Frankie Chang wrote:
>
> Frankie.Chang (3):
> binder: move structs from core file to header file
> binder: add trace at free transaction.
> binder: add transaction latency tracer
>
> drivers/android/Kconfig | 8 +
> drivers/android/Makefile | 1 +
> drivers/android/binder.c | 430 ++----------------------
> drivers/android/binder_internal.h | 419 +++++++++++++++++++++++
> drivers/android/binder_latency_tracer.c | 107 ++++++
> drivers/android/binder_trace.h | 49 +++
> 6 files changed, 608 insertions(+), 406 deletions(-)
> create mode 100644 drivers/android/binder_latency_tracer.c

Your cover letter here lost all of the information that used to be in
other versions of the series :(