On Mon, Oct 17, 2022 at 09:54:38PM -0400, Theodore Ts'o wrote: > On Sun, Oct 16, 2022 at 11:07:15AM -0400, Jeff Layton wrote: > > > > I think the chance of regression here is very low, but changing the > > > > default setting for this in a stable release seems a bit cavalier. > > > > > > Why did you have the cc: stable marking then? > > > > > > > Oh...I didn't add that. Maybe Ted did? Ted, did you really want this > > backported to stable releases? > > The reason why I added the cc:stable was because there was a later > bug-fix patches for i_version handling that was dependent on this one. > I guess I probably could have re-ordered the patches, but I decided, > eh, it should be fine. > > However, based on your question, I decided to do a bit more research, > and it occurs to me that patch to fix the i_version overhead landed in > Linus's tree in 2018 --- in 4.16. So it would probably be a good idea > *not* to backport this to the 4.14 and 4.9 LTS kernels. OTOH, because > of the the fact I don't think we backported the changes to support the > new mount API's, I don't think this patch would apply that far back > anyway. I didn't backport it to older kernels, right?