[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240703170804.3621083-2-yutingtseng@google.com>
Date: Wed, 3 Jul 2024 10:08:04 -0700
From: Yu-Ting Tseng <yutingtseng@...gle.com>
To: cmllamas@...gle.com, tkjos@...gle.com, gregkh@...uxfoundation.org
Cc: arve@...roid.com, maco@...roid.com, joel@...lfernandes.org,
brauner@...nel.org, surenb@...gle.com, aliceryhl@...gle.com,
kernel-team@...roid.com, linux-kernel@...r.kernel.org,
Yu-Ting Tseng <yutingtseng@...gle.com>
Subject: [PATCH v5] binder: frozen notification
> can you merge the branches into a single printk?
Done
> I believe you meant to use the 'is_frozen' here
Fixed
> I think you want to add a kfree(ref->freeze) to binder_free_ref().
Done
> Typo s/An/A/
Done
> The memset should have a blank line after the declarations. This should trigger a checkpatch issue I think.
Fixed. Reran checkpatch and there is no other issues flagged now.
(except for line-too-long which doesn't really have a fix)
Also merged the binder_features change into this patch as requested.
Yu-Ting Tseng (1):
binder: frozen notification
drivers/android/binder.c | 284 +++++++++++++++++-
drivers/android/binder_internal.h | 21 +-
drivers/android/binderfs.c | 8 +
include/uapi/linux/android/binder.h | 36 +++
.../filesystems/binderfs/binderfs_test.c | 1 +
5 files changed, 346 insertions(+), 4 deletions(-)
base-commit: 14d7c92f8df9c0964ae6f8b813c1b3ac38120825
--
2.45.2.803.g4e1b14247a-goog
Powered by blists - more mailing lists