On Wed, 2023-01-11 at 02:12 +0000, Al Viro wrote: > On Thu, Jan 05, 2023 at 04:19:29PM -0500, Jeff Layton wrote: > > The file locking definitions have lived in fs.h since the dawn of time, > > but they are only used by a small subset of the source files that > > include it. > > > > Move the file locking definitions to a new header file, and add the > > appropriate #include directives to the source files that need them. By > > doing this we trim down fs.h a bit and limit the amount of rebuilding > > that has to be done when we make changes to the file locking APIs. > > > > Reviewed-by: Xiubo Li <xiubli@xxxxxxxxxx> > > Reviewed-by: Christian Brauner (Microsoft) <brauner@xxxxxxxxxx> > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> > > Reviewed-by: David Howells <dhowells@xxxxxxxxxx> > > Acked-by: Chuck Lever <chuck.lever@xxxxxxxxxx> > > Acked-by: Joseph Qi <joseph.qi@xxxxxxxxxxxxxxxxx> > > Acked-by: Steve French <stfrench@xxxxxxxxxxxxx> > > Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx> > > Same question re git tree preferences (and my Acked-by in any case) I'll take this one via the file locking tree as well. FWIW, I have some other filelock API changes I'd like to propose, so I think that'll be easier. Thanks, -- Jeff Layton <jlayton@xxxxxxxxxx>