From: David Gow <davidgow@google.com> To: Andrew Morton <akpm@linux-foundation.org> Cc: Shuah Khan <shuah@kernel.org>, Brendan Higgins <brendanhiggins@google.com>, Kees Cook <keescook@chromium.org>, "open list:KERNEL SELFTEST FRAMEWORK" <linux-kselftest@vger.kernel.org>, kunit-dev@googlegroups.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH linux-kselftest/test v2] lib/list-test: add a test for the 'list' doubly linked list Date: Fri, 11 Oct 2019 14:37:25 -0700 [thread overview] Message-ID: <CABVgOS=UwWxwD97c6y-XzbLWVhznPjBO3qvQEzX=8jTJ-gBi3A@mail.gmail.com> (raw) In-Reply-To: <20191011140727.49160042fafa20d5867f8df7@linux-foundation.org> [-- Attachment #1: Type: text/plain, Size: 1805 bytes --] On Fri, Oct 11, 2019 at 2:05 PM Andrew Morton <akpm@linux-foundation.org> wrote: > > <looks at kunit> > > Given that everything runs at late_initcall time, shouldn't everything > be __init, __initdata etc so all the code and data doesn't hang around > for ever? > That's an interesting point. We haven't done this for KUnit tests to date, and there is certainly a possibility down the line that we may want to be able to run these tests in other circumstances. (There's some work being done to allow KUnit and KUnit tests to be built as modules here: https://lkml.org/lkml/2019/10/8/628 for example.) Maybe it'd be worth having macros which wrap __init/__initdata etc as a way of futureproofing tests against such a change? Either way, I suspect this is something that should probably be considered for KUnit as a whole, rather than on a test-by-test basis. On Fri, Oct 11, 2019 at 2:06 PM Andrew Morton <akpm@linux-foundation.org> wrote: > > On Thu, 10 Oct 2019 11:56:31 -0700 David Gow <davidgow@google.com> wrote: > > > Reply-To: 20191007213633.92565-1-davidgow@google.com > > That's a bit irksome. Deliberate? Whoops, this was supposed to be In-Reply-To. Please ignore it. On Fri, Oct 11, 2019 at 2:07 PM Andrew Morton <akpm@linux-foundation.org> wrote: > > On Thu, 10 Oct 2019 11:56:31 -0700 David Gow <davidgow@google.com> wrote: > > lib/list-test.c | 738 ++++++++++++++++++++++++++++++++++++++++++++++ > > Should this be lib/kunit/list-test.c? > The idea here was to have KUnit tests co-located with the code being tested, but with the linked-list code contained to a header, lib/ seemed the best place to put it, being alongside list_debug.c and similar. lib/kunit just contains the implementation for the KUnit framework itself (and tests which test that framework). Cheers, -- David [-- Attachment #2: S/MIME Cryptographic Signature --] [-- Type: application/pkcs7-signature, Size: 4847 bytes --]
next prev parent reply other threads:[~2019-10-11 21:37 UTC|newest] Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-10 18:56 David Gow 2019-10-11 21:05 ` Andrew Morton 2019-10-11 21:07 ` Andrew Morton 2019-10-11 21:37 ` David Gow [this message] 2019-10-11 21:55 ` Andrew Morton 2019-10-16 20:48 ` Brendan Higgins 2019-10-16 21:48 ` David Gow 2019-10-17 0:32 ` Andrew Morton 2019-10-18 21:24 ` shuah 2019-10-18 21:21 ` shuah
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CABVgOS=UwWxwD97c6y-XzbLWVhznPjBO3qvQEzX=8jTJ-gBi3A@mail.gmail.com' \ --to=davidgow@google.com \ --cc=akpm@linux-foundation.org \ --cc=brendanhiggins@google.com \ --cc=keescook@chromium.org \ --cc=kunit-dev@googlegroups.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-kselftest@vger.kernel.org \ --cc=shuah@kernel.org \ --subject='Re: [PATCH linux-kselftest/test v2] lib/list-test: add a test for the '\''list'\'' doubly linked list' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).