On Wed, Mar 24, 2021 at 09:40:50AM +0200, Amir Goldstein wrote: > On Mon, Mar 22, 2021 at 3:46 PM Christian Brauner <brauner@xxxxxxxxxx> wrote: > > > > From: Christian Brauner <christian.brauner@xxxxxxxxxx> > > > > Regression test to verify that creating a series of detached mounts, > > attaching them to the filesystem, and unmounting them does not trigger an > > integer overflow in ns->mounts causing the kernel to block any new mounts in > > count_mounts() and returning ENOSPC because it falsely assumes that the > > maximum number of mounts in the mount namespace has been reached, i.e. it > > thinks it can't fit the new mounts into the mount namespace anymore. > > > > The test is written in a way that it will leave the host's mount > > namespace intact so we are sure to never make the host's mount namespace > > unuseable! > > > > Link: https://git.kernel.org/torvalds/c/ee2e3f50629f17b0752b55b2566c15ce8dafb557 > > Cc: Eryu Guan <guan@xxxxxxx> > > Cc: David Howells <dhowells@xxxxxxxxxx> > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> > > Signed-off-by: Christian Brauner <christian.brauner@xxxxxxxxxx> > > --- > > /* v1 - v8 */ > > patch not present > > > > /* v9 */ > > - Christian Brauner <christian.brauner@xxxxxxxxxx>: > > - Rebased on current master. > > > > /* v10 */ > > - Eryu Guan <guan@xxxxxxx>: > > - Add missing checks whether test is supported. > > - Move status=$? assignment up. > > --- > > Technical nit: why did you add this extra --- line? > It causes all the patch changelog to appear in the commit message. > I don't think that was your intention? and I don't think it adds valuable > into to git log. I've not done that in any of the other patches or on any other patches I ever wrote so I think this was just a copy-paste error when I updated the changelog. Thanks for pointing this out. Christian
