From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brendan Higgins Subject: Re: [RFC v2 00/14] kunit: introduce KUnit, the Linux kernel unit testing framework Date: Tue, 13 Nov 2018 02:10:34 -0800 Message-ID: References: <20181023235750.103146-1-brendanhiggins@google.com> <04f677b1-bc44-f004-cf2a-51b47baf0965@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <04f677b1-bc44-f004-cf2a-51b47baf0965-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-nvdimm-bounces-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org Sender: "Linux-nvdimm" To: frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org Cc: brakmo-b10kYP2dOMg@public.gmane.org, dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, linux-kselftest-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, shuah-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Rob Herring , linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org, richard-/L3Ra7n9ekc@public.gmane.org, kieran.bingham-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org, Joel Stanley , jdike-OPE4K8JWMJJBDgjK7y7TUQ@public.gmane.org, Tim.Bird-7U/KSKJipcs@public.gmane.org, Kees Cook , linux-um-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, rostedt-nx8X9YLhiw1AfugRpC6u6w@public.gmane.org, Julia Lawall , kunit-dev-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Greg KH , Linux Kernel Mailing List , mcgrof-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Daniel Vetter , mpe-Gsx/Oe8HsFggBc27wqDAHg@public.gmane.org, joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org, khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org List-Id: linux-nvdimm@lists.01.org On Wed, Nov 7, 2018 at 9:46 AM Frank Rowand wrote: > > On 11/6/18 5:17 PM, Brendan Higgins wrote: > > On Fri, Nov 2, 2018 at 11:23 AM Shuah Khan wrote: > >> > >> Hi Brendan, > > > >> Framework looks good. I think it would be helpful to include a real test > > > > Great to hear! > > > >> in the patch series to get a feel for how effective it is. > > > > Alright, will do. Rob suggested converting > > https://elixir.bootlin.com/linux/v4.19/source/drivers/of/unittest.c to > > KUnit, so that might offer a good comparison. > > drivers/of/unittest.c might be a bit bigger and more complex test than > you want to start with. I already got it working under KUnit. Admittedly, structurally it is not so different, so it will require some work to make it pretty. I think now it is a question of whether you want me to start here or not. Probably best just to share what I have now along with Shuah's requested changes and we can go from there. Let me know what you think. Cheers