[PATCH] rust_binderfs: add transaction_report feature entry

Carlos Llamas posted 1 patch 1 week, 1 day ago
drivers/android/binder/rust_binderfs.c | 8 ++++++++
1 file changed, 8 insertions(+)
[PATCH] rust_binderfs: add transaction_report feature entry
Posted by Carlos Llamas 1 week, 1 day ago
rust_binderfs is missing the equivalent of commit f37b55ded8ed ("binder:
add transaction_report feature entry"), which adds "transaction_report"
to the binderfs feature list. This helps userspace determine if the
BINDER_CMD_REPORT from the generic netlink API is supported.

Cc: stable@vger.kernel.org
Fixes: f14e0c8183bc ("rust_binder: report netlink transactions")
Signed-off-by: Carlos Llamas <cmllamas@google.com>
---
 drivers/android/binder/rust_binderfs.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/drivers/android/binder/rust_binderfs.c b/drivers/android/binder/rust_binderfs.c
index 300cc65562d1..46a37f1858dd 100644
--- a/drivers/android/binder/rust_binderfs.c
+++ b/drivers/android/binder/rust_binderfs.c
@@ -72,6 +72,7 @@ struct binder_features {
 	bool oneway_spam_detection;
 	bool extended_error;
 	bool freeze_notification;
+	bool transaction_report;
 };
 
 static const struct constant_table binderfs_param_stats[] = {
@@ -89,6 +90,7 @@ static struct binder_features binder_features = {
 	.oneway_spam_detection = true,
 	.extended_error = true,
 	.freeze_notification = true,
+	.transaction_report = true,
 };
 
 static inline struct binderfs_info *BINDERFS_SB(const struct super_block *sb)
@@ -566,6 +568,12 @@ static int init_binder_features(struct super_block *sb)
 	if (IS_ERR(dentry))
 		return PTR_ERR(dentry);
 
+	dentry = rust_binderfs_create_file(dir, "transaction_report",
+				      &binder_features_fops,
+				      &binder_features.transaction_report);
+	if (IS_ERR(dentry))
+		return PTR_ERR(dentry);
+
 	return 0;
 }
 
-- 
2.55.0.1032.g73a4cd73de-goog
Re: [PATCH] rust_binderfs: add transaction_report feature entry
Posted by Alice Ryhl 1 week, 1 day ago
On Wed, Sep 16, 2026 at 1:08 PM Carlos Llamas <cmllamas@google.com> wrote:
>
> rust_binderfs is missing the equivalent of commit f37b55ded8ed ("binder:
> add transaction_report feature entry"), which adds "transaction_report"
> to the binderfs feature list. This helps userspace determine if the
> BINDER_CMD_REPORT from the generic netlink API is supported.
>
> Cc: stable@vger.kernel.org
> Fixes: f14e0c8183bc ("rust_binder: report netlink transactions")
> Signed-off-by: Carlos Llamas <cmllamas@google.com>

Reviewed-by: Alice Ryhl <aliceryhl@google.com>