I added this patch series on top of Linux 5.3-rc6 and ran xfstests on orangefs with no regressions.
Acked-by: Mike Marshall <hub...@omnibond.com> -Mike On Wed, Aug 28, 2019 at 10:40 AM Mark Salyzyn <saly...@android.com> wrote: > > On 8/28/19 7:24 AM, Christoph Hellwig wrote: > > On Tue, Aug 27, 2019 at 08:05:15AM -0700, Mark Salyzyn wrote: > >> Replace arguments for get and set xattr methods, and __vfs_getxattr > >> and __vfs_setaxtr functions with a reference to the following now > >> common argument structure: > > Yikes. That looks like a mess. Why can't we pass a kernel-only > > flag in the existing flags field for ₋>set and add a flags field > > to ->get? Passing methods by structure always tends to be a mess. > > This was a response to GregKH@ criticism, an earlier patch set just > added a flag as you stated to get method, until complaints of an > excessively long argument list and fragility to add or change more > arguments. > > So many ways have been tried to skin this cat ... the risk was taken to > please some, and we now have hundreds of stakeholders, when the first > patch set was less than a dozen. A recipe for failure? > > -- Mark > _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel