Messages in this thread | | | Date | Tue, 2 Aug 2022 16:36:25 +0200 | From | Greg Kroah-Hartman <> | Subject | Re: [PATCH v8 17/31] rust: add `kernel` crate |
| |
On Tue, Aug 02, 2022 at 04:33:17PM +0200, Miguel Ojeda wrote: > Hi Greg, > > On Tue, Aug 2, 2022 at 3:34 PM Greg Kroah-Hartman > <gregkh@linuxfoundation.org> wrote: > > > > 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 :( > > Yeah, of course, I am happy to do that -- I did it for the `sync` > module, but stopped splitting further since I understood you wanted it > this way from what you told me on February.
I did? Maybe I didn't realize we would end up with 15k lines of code from different parts of the kernel like amba bus and filesystem apis all in a single commit. If so, my fault, but as other parts of this discussion show, it is going to be required to get the subsystem maintainers to be able to review the interfaces to their code in order to get any further.
thanks,
greg k-h
| |