| 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
| ||
|
Message-ID: <Yukn3vNJ8iGuXgiz@kroah.com>
Date: Tue, 2 Aug 2022 15:34:22 +0200
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Miguel Ojeda <ojeda@...nel.org>
Cc: Linus Torvalds <torvalds@...ux-foundation.org>,
rust-for-linux@...r.kernel.org, linux-kernel@...r.kernel.org,
Jarkko Sakkinen <jarkko@...nel.org>,
Wedson Almeida Filho <wedsonaf@...gle.com>,
Alex Gaynor <alex.gaynor@...il.com>,
Geoffrey Thomas <geofft@...reload.com>,
Finn Behrens <me@...enk.de>,
Adam Bratschi-Kaye <ark.email@...il.com>,
Michael Ellerman <mpe@...erman.id.au>,
Sumera Priyadarsini <sylphrenadin@...il.com>,
Sven Van Asbroeck <thesven73@...il.com>,
Gary Guo <gary@...yguo.net>,
Boris-Chengbiao Zhou <bobo1239@....de>,
Boqun Feng <boqun.feng@...il.com>,
Fox Chen <foxhlchen@...il.com>,
Dan Robertson <daniel.robertson@...rlab.io>,
Viktor Garske <viktor@...ar.de>,
Dariusz Sosnowski <dsosnowski@...snowski.pl>,
Léo Lanteri Thauvin
<leseulartichaut@...il.com>, Niklas Mohrin <dev@...lasmohrin.de>,
Gioh Kim <gurugio@...il.com>, Daniel Xu <dxu@...uu.xyz>,
Milan Landaverde <milan@...verde.com>,
Morgan Bartlett <mjmouse9999@...il.com>,
Maciej Falkowski <m.falkowski@...sung.com>,
Jiapeng Chong <jiapeng.chong@...ux.alibaba.com>,
Nándor István Krácser <bonifaido@...il.com>,
David Gow <davidgow@...gle.com>,
John Baublitz <john.m.baublitz@...il.com>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>
Subject: Re: [PATCH v8 17/31] rust: add `kernel` crate
On Tue, Aug 02, 2022 at 03:50:04AM +0200, Miguel Ojeda wrote:
> From: Wedson Almeida Filho <wedsonaf@...gle.com>
>
> The `kernel` crate currently includes all the abstractions that wrap
> kernel features written in C.
>
> These abstractions call the C side of the kernel via the generated
> bindings with the `bindgen` tool. Modules developed in Rust should
> never call the bindings themselves.
>
> In the future, as the abstractions grow in number, we may need
> to split this crate into several, possibly following a similar
> subdivision in subsystems as the kernel itself and/or moving
> the code to the actual subsystems.
>
> Co-developed-by: Alex Gaynor <alex.gaynor@...il.com>
> Signed-off-by: Alex Gaynor <alex.gaynor@...il.com>
> Co-developed-by: Geoffrey Thomas <geofft@...reload.com>
> Signed-off-by: Geoffrey Thomas <geofft@...reload.com>
> Co-developed-by: Finn Behrens <me@...enk.de>
> Signed-off-by: Finn Behrens <me@...enk.de>
> Co-developed-by: Adam Bratschi-Kaye <ark.email@...il.com>
> Signed-off-by: Adam Bratschi-Kaye <ark.email@...il.com>
> Co-developed-by: Michael Ellerman <mpe@...erman.id.au>
> Signed-off-by: Michael Ellerman <mpe@...erman.id.au>
> Co-developed-by: Sumera Priyadarsini <sylphrenadin@...il.com>
> Signed-off-by: Sumera Priyadarsini <sylphrenadin@...il.com>
> Co-developed-by: Sven Van Asbroeck <thesven73@...il.com>
> Signed-off-by: Sven Van Asbroeck <thesven73@...il.com>
> Co-developed-by: Gary Guo <gary@...yguo.net>
> Signed-off-by: Gary Guo <gary@...yguo.net>
> Co-developed-by: Boris-Chengbiao Zhou <bobo1239@....de>
> Signed-off-by: Boris-Chengbiao Zhou <bobo1239@....de>
> Co-developed-by: Boqun Feng <boqun.feng@...il.com>
> Signed-off-by: Boqun Feng <boqun.feng@...il.com>
> Co-developed-by: Fox Chen <foxhlchen@...il.com>
> Signed-off-by: Fox Chen <foxhlchen@...il.com>
> Co-developed-by: Dan Robertson <daniel.robertson@...rlab.io>
> Signed-off-by: Dan Robertson <daniel.robertson@...rlab.io>
> Co-developed-by: Viktor Garske <viktor@...ar.de>
> Signed-off-by: Viktor Garske <viktor@...ar.de>
> Co-developed-by: Dariusz Sosnowski <dsosnowski@...snowski.pl>
> Signed-off-by: Dariusz Sosnowski <dsosnowski@...snowski.pl>
> Co-developed-by: Léo Lanteri Thauvin <leseulartichaut@...il.com>
> Signed-off-by: Léo Lanteri Thauvin <leseulartichaut@...il.com>
> Co-developed-by: Niklas Mohrin <dev@...lasmohrin.de>
> Signed-off-by: Niklas Mohrin <dev@...lasmohrin.de>
> Co-developed-by: Gioh Kim <gurugio@...il.com>
> Signed-off-by: Gioh Kim <gurugio@...il.com>
> Co-developed-by: Daniel Xu <dxu@...uu.xyz>
> Signed-off-by: Daniel Xu <dxu@...uu.xyz>
> Co-developed-by: Milan Landaverde <milan@...verde.com>
> Signed-off-by: Milan Landaverde <milan@...verde.com>
> Co-developed-by: Morgan Bartlett <mjmouse9999@...il.com>
> Signed-off-by: Morgan Bartlett <mjmouse9999@...il.com>
> Co-developed-by: Maciej Falkowski <m.falkowski@...sung.com>
> Signed-off-by: Maciej Falkowski <m.falkowski@...sung.com>
> Co-developed-by: Jiapeng Chong <jiapeng.chong@...ux.alibaba.com>
> Signed-off-by: Jiapeng Chong <jiapeng.chong@...ux.alibaba.com>
> Co-developed-by: Nándor István Krácser <bonifaido@...il.com>
> Signed-off-by: Nándor István Krácser <bonifaido@...il.com>
> Co-developed-by: David Gow <davidgow@...gle.com>
> Signed-off-by: David Gow <davidgow@...gle.com>
> Co-developed-by: John Baublitz <john.m.baublitz@...il.com>
> Signed-off-by: John Baublitz <john.m.baublitz@...il.com>
> Co-developed-by: Björn Roy Baron <bjorn3_gh@...tonmail.com>
> Signed-off-by: Björn Roy Baron <bjorn3_gh@...tonmail.com>
> Signed-off-by: Wedson Almeida Filho <wedsonaf@...gle.com>
> Co-developed-by: Miguel Ojeda <ojeda@...nel.org>
> Signed-off-by: Miguel Ojeda <ojeda@...nel.org>
> ---
> rust/kernel/allocator.rs | 64 ++
> rust/kernel/amba.rs | 261 +++++++
> rust/kernel/build_assert.rs | 83 +++
> rust/kernel/chrdev.rs | 206 ++++++
> rust/kernel/clk.rs | 79 ++
> rust/kernel/cred.rs | 46 ++
> rust/kernel/delay.rs | 104 +++
> rust/kernel/device.rs | 527 ++++++++++++++
> rust/kernel/driver.rs | 442 +++++++++++
> rust/kernel/error.rs | 564 ++++++++++++++
> rust/kernel/file.rs | 887 +++++++++++++++++++++++
> rust/kernel/fs.rs | 846 +++++++++++++++++++++
> rust/kernel/fs/param.rs | 553 ++++++++++++++
> rust/kernel/gpio.rs | 505 +++++++++++++
> rust/kernel/hwrng.rs | 210 ++++++
> rust/kernel/io_buffer.rs | 153 ++++
> rust/kernel/io_mem.rs | 278 +++++++
> rust/kernel/iov_iter.rs | 81 +++
> rust/kernel/irq.rs | 681 +++++++++++++++++
> rust/kernel/kasync.rs | 50 ++
> rust/kernel/kasync/executor.rs | 154 ++++
> rust/kernel/kasync/executor/workqueue.rs | 291 ++++++++
> rust/kernel/kasync/net.rs | 322 ++++++++
> rust/kernel/kunit.rs | 91 +++
> rust/kernel/lib.rs | 267 +++++++
> rust/kernel/linked_list.rs | 247 +++++++
> rust/kernel/miscdev.rs | 290 ++++++++
> rust/kernel/mm.rs | 149 ++++
> rust/kernel/module_param.rs | 499 +++++++++++++
> rust/kernel/net.rs | 392 ++++++++++
> rust/kernel/net/filter.rs | 447 ++++++++++++
> rust/kernel/of.rs | 63 ++
> rust/kernel/pages.rs | 144 ++++
> rust/kernel/platform.rs | 223 ++++++
> rust/kernel/power.rs | 118 +++
> rust/kernel/prelude.rs | 36 +
> rust/kernel/print.rs | 406 +++++++++++
> rust/kernel/random.rs | 42 ++
> rust/kernel/raw_list.rs | 361 +++++++++
> rust/kernel/rbtree.rs | 563 ++++++++++++++
> rust/kernel/revocable.rs | 425 +++++++++++
> rust/kernel/security.rs | 38 +
> rust/kernel/static_assert.rs | 34 +
> rust/kernel/std_vendor.rs | 161 ++++
> rust/kernel/str.rs | 597 +++++++++++++++
> rust/kernel/sync.rs | 48 +-
> rust/kernel/sysctl.rs | 199 +++++
> rust/kernel/task.rs | 239 ++++++
> rust/kernel/types.rs | 705 ++++++++++++++++++
> rust/kernel/unsafe_list.rs | 680 +++++++++++++++++
> rust/kernel/user_ptr.rs | 175 +++++
> rust/kernel/workqueue.rs | 512 +++++++++++++
> 52 files changed, 15518 insertions(+), 20 deletions(-)
This is huge for a single commit, and pretty much impossible to review
as-is.
Any chance you can turn this into a series of commits, that starts with
the basics and builds on top of that? Right now you are mixing many
many different subsystems all at once into one commit, guaranteeing that
no one can review the whole thing properly :(
thanks,
greg k-h
Powered by blists - more mailing lists