[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Z8-v92gIPgrxWq1w@google.com>
Date: Tue, 11 Mar 2025 03:37:27 +0000
From: Carlos Llamas <cmllamas@...gle.com>
To: Li Li <dualli@...omium.org>
Cc: dualli@...gle.com, corbet@....net, davem@...emloft.net,
edumazet@...gle.com, kuba@...nel.org, pabeni@...hat.com,
donald.hunter@...il.com, gregkh@...uxfoundation.org,
arve@...roid.com, tkjos@...roid.com, maco@...roid.com,
joel@...lfernandes.org, brauner@...nel.org, surenb@...gle.com,
omosnace@...hat.com, shuah@...nel.org, arnd@...db.de,
masahiroy@...nel.org, bagasdotme@...il.com, horms@...nel.org,
tweek@...gle.com, linux-kernel@...r.kernel.org,
linux-doc@...r.kernel.org, netdev@...r.kernel.org,
selinux@...r.kernel.org, hridya@...gle.com, smoreland@...gle.com,
ynaffit@...gle.com, kernel-team@...roid.com
Subject: Re: [PATCH v16 3/3] binder: generic netlink binder_features flag
On Mon, Mar 03, 2025 at 12:02:12PM -0800, Li Li wrote:
> From: Li Li <dualli@...gle.com>
>
> Add a flag to binder_features to indicate that the generic netlink
> feature is available.
>
> Signed-off-by: Li Li <dualli@...gle.com>
> ---
The implementation looks correct. However, the feature you are adding is
not "generic_netlink", maybe "transaction_report" or something similar?
Future features might use this same interface so lets avoid confusion.
Thanks,
Powered by blists - more mailing lists