From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl1-x644.google.com ([2607:f8b0:4864:20::644]) by bombadil.infradead.org with esmtps (Exim 4.92.2 #3 (Red Hat Linux)) id 1iHOG9-00061Y-3P for linux-um@lists.infradead.org; Mon, 07 Oct 2019 08:20:50 +0000 Received: by mail-pl1-x644.google.com with SMTP id j11so6514239plk.3 for ; Mon, 07 Oct 2019 01:20:46 -0700 (PDT) MIME-Version: 1.0 References: <56e2e1a7-f8fe-765b-8452-1710b41895bf@kernel.org> <20191004222714.GA107737@google.com> <20191004232955.GC12012@mit.edu> <63e59b0b-b51e-01f4-6359-a134a1f903fd@kernel.org> <544bdfcb-fb35-5008-ec94-8d404a08fd14@kernel.org> <20191006165436.GA29585@mit.edu> In-Reply-To: <20191006165436.GA29585@mit.edu> From: Brendan Higgins Date: Mon, 7 Oct 2019 01:20:34 -0700 Message-ID: Subject: Re: [PATCH v18 00/19] kunit: introduce KUnit, the Linux kernel unit testing framework List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-um" Errors-To: linux-um-bounces+geert=linux-m68k.org@lists.infradead.org To: "Theodore Y. Ts'o" Cc: Petr Mladek , "open list:DOCUMENTATION" , Peter Zijlstra , Amir Goldstein , dri-devel , Sasha Levin , Masahiro Yamada , Michael Ellerman , "open list:KERNEL SELFTEST FRAMEWORK" , Frank Rowand , Rob Herring , linux-nvdimm , shuah , Knut Omang , Kieran Bingham , wfg@linux.intel.com, Joel Stanley , David Rientjes , Jeff Dike , Dan Carpenter , devicetree , Linux Kbuild mailing list , "Bird, Timothy , linux-um@lists.infradead.org, Steven Rostedt" , Julia Lawall , Josh Poimboeuf , kunit-dev@googlegroups.com, Logan Gunthorpe , Richard Weinberger , Stephen Boyd , Greg Kroah-Hartman , Randy Dunlap , Linux Kernel Mailing List , Luis Chamberlain , Daniel Vetter , Kees Cook , linux-fsdevel , Linus Torvalds , Kevin Hilman On Sun, Oct 6, 2019 at 9:55 AM Theodore Y. Ts'o wrote: > > On Fri, Oct 04, 2019 at 06:18:04PM -0700, Brendan Higgins wrote: > > > Let's talk about current state. Right now kunit is in linux-next and > > > we want to add a few more tests. We will have to coordinate the effort. > > > Once kunit get into mainline, then the need for this coordination goes > > > down. > > > > Sure, I was just thinking that getting other people to write the tests > > would be better. Since not only is it then useful to someone else, it > > provides the best possible exercise of KUnit. > > Well, one thing we *can* do is if (a) if we can create a kselftest > branch which we know is stable and won't change, and (b) we can get > assurances that Linus *will* accept that branch during the next merge > window, those subsystems which want to use kself test can simply pull > it into their tree. Yeah, I can't think of any reason that you haven't outlined already why that might not work, but that seems kind of like circumventing Linus. > We've done this before in the file system world, when there has been > some common set of changes needed to improve, say, Direct I/O, where > the changes are put into a standalone branch, say, in the xfs tree, > and those file systems which need it as a building block can pull it > into their tree, and then add the changes needed to use those changes > into their file system git tree. These changes are generally not > terribly controversial, and we've not had to worry about people want > to bikeshed the changes. > > There is a risk with doing this of course, which is that if the branch > *is* controversial, or gets bike-shedded for some reason, then Linus > gets upset and any branches which depended on said branch will get > rejected at the next merge window. Which is the requirement for (a) > and (b) above. Presumably, the fact that people were unwilling to let > Kunit land during this merge window might will *because* we think more > changes might be pending? My understanding, based on what I have been told, is that we were simply unlucky with the timing when Linus pulled the branch in the first week of the 5.4 merge window (Friday), such that once I fixed the directory naming issue, the updated changes didn't spend enough time in linux-next. And now with this issue fixed and KUnit back in linux-next, if nothing interesting happens between now and 5.5, it will be accepted in the 5.5 merge window. I do not think that anyone is expecting anymore changes before merging. Shuah, Linus, is my understanding correct? > The other thing I suppose I can do is to let the ext4 kunit tests land > in ext4 tree, but with the necessary #ifdef's around things like > "#include " so that the build won't blow up w/o kunit > changes being in the tree that I'm building. It means I won't be able > to run the tests without creating a test integration branch and > merging in kunit by hand, which will super-annoying, of course. And > if some of the bike-shedding is in Kunit's interfaces, then that > becomes problematic as well, since any tests that are in ext4.git tree > might change if people want to rename Kunit's publically exported > functions (for example). Yeah, that seems even worse. I'm sorry to have caused this frustration. > > Hey Ted, do you know if that ext4 timestamp test can go in through > > linux-kselftest? It seemed fairly self-contained. Or is that what you > > were saying wouldn't work for you? > > Well, I was hoping that we might start creating more tests beyond just > the ext4 timestamp tests.... Okay, that's what I thought (and what I hoped) you were saying :-) I hope we can figure out something that will work for you. Or otherwise that you won't mind waiting until 5.5. Sorry _______________________________________________ linux-um mailing list linux-um@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-um