linux-kernel - Re: [RFC PATCH 19/19] drivers: android: Binder IPC in Rust

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Ya40GOSIrWVC2ZSq@kroah.com>
Date:   Mon, 6 Dec 2021 17:02:32 +0100
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Wedson Almeida Filho <wedsonaf@...gle.com>
Cc:     Miguel Ojeda <ojeda@...nel.org>,
        Linus Torvalds <torvalds@...ux-foundation.org>,
        rust-for-linux@...r.kernel.org, linux-kbuild@...r.kernel.org,
        linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
        Alex Gaynor <alex.gaynor@...il.com>,
        Finn Behrens <me@...enk.de>,
        Sven Van Asbroeck <thesven73@...il.com>,
        Gary Guo <gary@...yguo.net>,
        Wayne Campbell <wcampbell1995@...il.com>
Subject: Re: [RFC PATCH 19/19] drivers: android: Binder IPC in Rust

On Mon, Dec 06, 2021 at 03:59:46PM +0000, Wedson Almeida Filho wrote:
> On Mon, Dec 06, 2021 at 04:01:15PM +0100, Greg Kroah-Hartman wrote:
> > On Mon, Dec 06, 2021 at 03:03:13PM +0100, Miguel Ojeda wrote:
> > > From: Wedson Almeida Filho <wedsonaf@...gle.com>
> > > 
> > > A port to Rust of the Android Binder IPC mechanism.
> > > 
> > > This module is a work in progress and will be sent for review later
> > > on, as well as separately from the Rust support.
> > > 
> > > However, it is included to show how an actual working module
> > > written in Rust may look like.
> > 
> > Have you all tested this against the userspace binder tests?  And is it
> > up to date with the features of the in-kernel binder driver?
> 
> Very little has changed here since the last submission, namely: using
> credentials for security callbacks, and replacing `Arc` with `Ref` (i.e., using
> refcount_t to manage ref-counted allocations).
> 
> As the message tries to indicate, this is submitted as an example and WIP, it
> doesn't have feature parity with the C version yet.

You might want to say that in the changelog text so I don't keep asking
every time it gets posted :)

thanks,

greg k-h