diff mbox series

kunit: Add missing newline in summary message

Message ID 20200416114256.226329-1-elver@google.com
State Mainlined
Commit 6cb1818798812fe7e2c8fe3f489ad1c86adfd6c4
Headers show
Series kunit: Add missing newline in summary message | expand

Commit Message

Marco Elver April 16, 2020, 11:42 a.m. UTC
Add missing newline, as otherwise flushing of the final summary message
to the console log can be delayed.

Fixes: e2219db280e3 ("kunit: add debugfs /sys/kernel/debug/kunit/<suite>/results display")
Signed-off-by: Marco Elver <elver@google.com>
---
 lib/kunit/test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

David Gow April 17, 2020, 2:55 a.m. UTC | #1
On Thu, Apr 16, 2020 at 7:43 PM 'Marco Elver' via KUnit Development
<kunit-dev@googlegroups.com> wrote:
>
> Add missing newline, as otherwise flushing of the final summary message
> to the console log can be delayed.
>
> Fixes: e2219db280e3 ("kunit: add debugfs /sys/kernel/debug/kunit/<suite>/results display")
> Signed-off-by: Marco Elver <elver@google.com>

Tested-by: David Gow <davidgow@google.com>

Thanks a lot: this issue was starting to get annoying, and I can
confirm this patch fixes it for me!

-- David
Alan Maguire April 17, 2020, 10:03 a.m. UTC | #2
On Thu, 16 Apr 2020, Marco Elver wrote:

> Add missing newline, as otherwise flushing of the final summary message
> to the console log can be delayed.
> 
> Fixes: e2219db280e3 ("kunit: add debugfs /sys/kernel/debug/kunit/<suite>/results display")
> Signed-off-by: Marco Elver <elver@google.com>

Reviewed-by: Alan Maguire <alan.maguire@oracle.com>

Thanks for finding/fixing this!

Alan

> ---
>  lib/kunit/test.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/kunit/test.c b/lib/kunit/test.c
> index 7a6430a7fca0..ccb2ffad8dcf 100644
> --- a/lib/kunit/test.c
> +++ b/lib/kunit/test.c
> @@ -93,7 +93,7 @@ static void kunit_print_ok_not_ok(void *test_or_suite,
>  	 * representation.
>  	 */
>  	if (suite)
> -		pr_info("%s %zd - %s",
> +		pr_info("%s %zd - %s\n",
>  			kunit_status_to_string(is_ok),
>  			test_number, description);
>  	else
> -- 
> 2.26.1.301.g55bc3eb7cb9-goog
> 
>
Marco Elver April 22, 2020, 11:05 a.m. UTC | #3
On Fri, 17 Apr 2020 at 12:03, Alan Maguire <alan.maguire@oracle.com> wrote:
>
> On Thu, 16 Apr 2020, Marco Elver wrote:
>
> > Add missing newline, as otherwise flushing of the final summary message
> > to the console log can be delayed.
> >
> > Fixes: e2219db280e3 ("kunit: add debugfs /sys/kernel/debug/kunit/<suite>/results display")
> > Signed-off-by: Marco Elver <elver@google.com>
>
> Reviewed-by: Alan Maguire <alan.maguire@oracle.com>
>
> Thanks for finding/fixing this!
>
> Alan
>
> > ---
> >  lib/kunit/test.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)

Which tree will pick this up? I cannot yet see this in -next.

Thanks,
-- Marco


> > diff --git a/lib/kunit/test.c b/lib/kunit/test.c
> > index 7a6430a7fca0..ccb2ffad8dcf 100644
> > --- a/lib/kunit/test.c
> > +++ b/lib/kunit/test.c
> > @@ -93,7 +93,7 @@ static void kunit_print_ok_not_ok(void *test_or_suite,
> >        * representation.
> >        */
> >       if (suite)
> > -             pr_info("%s %zd - %s",
> > +             pr_info("%s %zd - %s\n",
> >                       kunit_status_to_string(is_ok),
> >                       test_number, description);
> >       else
> > --
> > 2.26.1.301.g55bc3eb7cb9-goog
> >
> >
Brendan Higgins April 22, 2020, 7:35 p.m. UTC | #4
On Thu, Apr 16, 2020 at 4:43 AM Marco Elver <elver@google.com> wrote:
>
> Add missing newline, as otherwise flushing of the final summary message
> to the console log can be delayed.
>
> Fixes: e2219db280e3 ("kunit: add debugfs /sys/kernel/debug/kunit/<suite>/results display")
> Signed-off-by: Marco Elver <elver@google.com>

Acked-by: Brendan Higgins <brendanhiggins@google.com>

Thanks!
Brendan Higgins April 22, 2020, 7:37 p.m. UTC | #5
On Wed, Apr 22, 2020 at 4:05 AM Marco Elver <elver@google.com> wrote:
>
> On Fri, 17 Apr 2020 at 12:03, Alan Maguire <alan.maguire@oracle.com> wrote:
> >
> > On Thu, 16 Apr 2020, Marco Elver wrote:
> >
> > > Add missing newline, as otherwise flushing of the final summary message
> > > to the console log can be delayed.
> > >
> > > Fixes: e2219db280e3 ("kunit: add debugfs /sys/kernel/debug/kunit/<suite>/results display")
> > > Signed-off-by: Marco Elver <elver@google.com>
> >
> > Reviewed-by: Alan Maguire <alan.maguire@oracle.com>
> >
> > Thanks for finding/fixing this!
> >
> > Alan
> >
> > > ---
> > >  lib/kunit/test.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Which tree will pick this up? I cannot yet see this in -next.

This will be picked up by kselftest/kunit:

https://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git/log/?h=kunit
Shuah Khan April 23, 2020, 10:07 p.m. UTC | #6
On 4/22/20 1:35 PM, Brendan Higgins wrote:
> On Thu, Apr 16, 2020 at 4:43 AM Marco Elver <elver@google.com> wrote:
>>
>> Add missing newline, as otherwise flushing of the final summary message
>> to the console log can be delayed.
>>
>> Fixes: e2219db280e3 ("kunit: add debugfs /sys/kernel/debug/kunit/<suite>/results display")
>> Signed-off-by: Marco Elver <elver@google.com>
> 
> Acked-by: Brendan Higgins <brendanhiggins@google.com>
> 
> Thanks!
> 

Applied. Now in

git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest kunit
branch.

thanks,
-- Shuah
diff mbox series

Patch

diff --git a/lib/kunit/test.c b/lib/kunit/test.c
index 7a6430a7fca0..ccb2ffad8dcf 100644
--- a/lib/kunit/test.c
+++ b/lib/kunit/test.c
@@ -93,7 +93,7 @@  static void kunit_print_ok_not_ok(void *test_or_suite,
 	 * representation.
 	 */
 	if (suite)
-		pr_info("%s %zd - %s",
+		pr_info("%s %zd - %s\n",
 			kunit_status_to_string(is_ok),
 			test_number, description);
 	else