On 25/03/15 08:00AM, Greg Kroah-Hartman wrote: > On Fri, Mar 14, 2025 at 05:57:46PM -0400, Ethan Carter Edwards wrote: > > Hello everyone, > > > > This is a follow up patchset to the driver I sent an email about a few > > weeks ago [0]. I understand this patchset will probably get rejected, > > but I wanted to report on what I have done thus far. I have got the > > upstream module imported and building, and it passes some basic tests > > so far (I have not tried getting XFS/FStests running yet). > > > > Like mentioned earlier, some of the files have been moved to folios, but > > a large majority of them still use bufferheads. I would like to have > > them completely removed before moved from staging/ into fs/. > > > > I have split everything up into separate commits as best as I could. > > Most of the C files rely in functions from other C files, so I included > > them all in one patch/commit. > > > > I am curious to hear everyone's thoughts on this and to start getting > > the ball rolling for the code-review process. Please feel free to > > include/CC anyone who may be interested in this driver/the review > > process. I have included a few people, but have certainly missed others. > > > > [0]: https://lore.kernel.org/lkml/20250307165054.GA9774@eaf/ > > > > Signed-off-by: Ethan Carter Edwards <ethan@xxxxxxxxxxxxxxxxx> > > I don't mind adding this to staging from this series, thanks for > breaking it up! > > But I'll wait for an ACK from the filesystem developers before doing it > as having filesystem code in drivers/staging/ feels odd, and they kind > of need to know what's going on here for when they change api stuff. No problem. That makes sense. I used the process that erofs used as a reference for how the fs development lifecycle should look. They started in staging/ and ended in fs/ after. Thanks, Ethan > > thanks, > > greg k-h