All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Daniel Latypov <dlatypov@google.com>, andriy.shevchenko@linux.intel.com
Cc: kbuild-all@lists.01.org, brendanhiggins@google.com,
	davidgow@google.com, linux-kernel@vger.kernel.org,
	kunit-dev@googlegroups.com, linux-kselftest@vger.kernel.org,
	skhan@linuxfoundation.org, Daniel Latypov <dlatypov@google.com>
Subject: Re: [PATCH v6] lib: add basic KUnit test for lib/math
Date: Sat, 17 Apr 2021 13:58:35 +0800	[thread overview]
Message-ID: <202104171338.s91up96i-lkp@intel.com> (raw)
In-Reply-To: <20210416180427.1545645-1-dlatypov@google.com>

[-- Attachment #1: Type: text/plain, Size: 2631 bytes --]

Hi Daniel,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on 7e25f40eab52c57ff6772d27d2aef3640a3237d7]

url:    https://github.com/0day-ci/linux/commits/Daniel-Latypov/lib-add-basic-KUnit-test-for-lib-math/20210417-020619
base:   7e25f40eab52c57ff6772d27d2aef3640a3237d7
config: powerpc-randconfig-c004-20210416 (attached as .config)
compiler: powerpc-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/0f1888ffeaa6baa1bc2a99eac8ba7d1df29c8450
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Daniel-Latypov/lib-add-basic-KUnit-test-for-lib-math/20210417-020619
        git checkout 0f1888ffeaa6baa1bc2a99eac8ba7d1df29c8450
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross W=1 ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   lib/math/math_kunit.c: In function 'abs_test':
>> lib/math/math_kunit.c:41:1: warning: the frame size of 1088 bytes is larger than 1024 bytes [-Wframe-larger-than=]
      41 | }
         | ^


vim +41 lib/math/math_kunit.c

    14	
    15	static void abs_test(struct kunit *test)
    16	{
    17		KUNIT_EXPECT_EQ(test, abs((char)0), (char)0);
    18		KUNIT_EXPECT_EQ(test, abs((char)42), (char)42);
    19		KUNIT_EXPECT_EQ(test, abs((char)-42), (char)42);
    20	
    21		/* The expression in the macro is actually promoted to an int. */
    22		KUNIT_EXPECT_EQ(test, abs((short)0),  0);
    23		KUNIT_EXPECT_EQ(test, abs((short)42),  42);
    24		KUNIT_EXPECT_EQ(test, abs((short)-42),  42);
    25	
    26		KUNIT_EXPECT_EQ(test, abs(0),  0);
    27		KUNIT_EXPECT_EQ(test, abs(42),  42);
    28		KUNIT_EXPECT_EQ(test, abs(-42),  42);
    29	
    30		KUNIT_EXPECT_EQ(test, abs(0L), 0L);
    31		KUNIT_EXPECT_EQ(test, abs(42L), 42L);
    32		KUNIT_EXPECT_EQ(test, abs(-42L), 42L);
    33	
    34		KUNIT_EXPECT_EQ(test, abs(0LL), 0LL);
    35		KUNIT_EXPECT_EQ(test, abs(42LL), 42LL);
    36		KUNIT_EXPECT_EQ(test, abs(-42LL), 42LL);
    37	
    38		/* Unsigned types get casted to signed. */
    39		KUNIT_EXPECT_EQ(test, abs(0ULL), 0LL);
    40		KUNIT_EXPECT_EQ(test, abs(42ULL), 42LL);
  > 41	}
    42	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 32245 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v6] lib: add basic KUnit test for lib/math
Date: Sat, 17 Apr 2021 13:58:35 +0800	[thread overview]
Message-ID: <202104171338.s91up96i-lkp@intel.com> (raw)
In-Reply-To: <20210416180427.1545645-1-dlatypov@google.com>

[-- Attachment #1: Type: text/plain, Size: 2699 bytes --]

Hi Daniel,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on 7e25f40eab52c57ff6772d27d2aef3640a3237d7]

url:    https://github.com/0day-ci/linux/commits/Daniel-Latypov/lib-add-basic-KUnit-test-for-lib-math/20210417-020619
base:   7e25f40eab52c57ff6772d27d2aef3640a3237d7
config: powerpc-randconfig-c004-20210416 (attached as .config)
compiler: powerpc-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/0f1888ffeaa6baa1bc2a99eac8ba7d1df29c8450
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Daniel-Latypov/lib-add-basic-KUnit-test-for-lib-math/20210417-020619
        git checkout 0f1888ffeaa6baa1bc2a99eac8ba7d1df29c8450
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross W=1 ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   lib/math/math_kunit.c: In function 'abs_test':
>> lib/math/math_kunit.c:41:1: warning: the frame size of 1088 bytes is larger than 1024 bytes [-Wframe-larger-than=]
      41 | }
         | ^


vim +41 lib/math/math_kunit.c

    14	
    15	static void abs_test(struct kunit *test)
    16	{
    17		KUNIT_EXPECT_EQ(test, abs((char)0), (char)0);
    18		KUNIT_EXPECT_EQ(test, abs((char)42), (char)42);
    19		KUNIT_EXPECT_EQ(test, abs((char)-42), (char)42);
    20	
    21		/* The expression in the macro is actually promoted to an int. */
    22		KUNIT_EXPECT_EQ(test, abs((short)0),  0);
    23		KUNIT_EXPECT_EQ(test, abs((short)42),  42);
    24		KUNIT_EXPECT_EQ(test, abs((short)-42),  42);
    25	
    26		KUNIT_EXPECT_EQ(test, abs(0),  0);
    27		KUNIT_EXPECT_EQ(test, abs(42),  42);
    28		KUNIT_EXPECT_EQ(test, abs(-42),  42);
    29	
    30		KUNIT_EXPECT_EQ(test, abs(0L), 0L);
    31		KUNIT_EXPECT_EQ(test, abs(42L), 42L);
    32		KUNIT_EXPECT_EQ(test, abs(-42L), 42L);
    33	
    34		KUNIT_EXPECT_EQ(test, abs(0LL), 0LL);
    35		KUNIT_EXPECT_EQ(test, abs(42LL), 42LL);
    36		KUNIT_EXPECT_EQ(test, abs(-42LL), 42LL);
    37	
    38		/* Unsigned types get casted to signed. */
    39		KUNIT_EXPECT_EQ(test, abs(0ULL), 0LL);
    40		KUNIT_EXPECT_EQ(test, abs(42ULL), 42LL);
  > 41	}
    42	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 32245 bytes --]

  parent reply	other threads:[~2021-04-17  5:59 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-16 18:04 [PATCH v6] lib: add basic KUnit test for lib/math Daniel Latypov
2021-04-17  4:16 ` David Gow
2021-04-17  5:58 ` kernel test robot [this message]
2021-04-17  5:58   ` kernel test robot
2024-05-16 10:19 ` Devarsh Thakkar
2024-05-16 10:26   ` Andy Shevchenko
2024-05-16 15:16   ` Daniel Latypov

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=202104171338.s91up96i-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=brendanhiggins@google.com \
    --cc=davidgow@google.com \
    --cc=dlatypov@google.com \
    --cc=kbuild-all@lists.01.org \
    --cc=kunit-dev@googlegroups.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=skhan@linuxfoundation.org \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.