All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: George Spelvin <linux@horizon.com>
Cc: kbuild-all@01.org, andi@firstfloor.org, tytso@mit.edu,
	ahferroin7@gmail.com, jepler@unpythonic.net,
	linux-kernel@vger.kernel.org, linux@horizon.com,
	linux@rasmusvillemoes.dk
Subject: Re: [RFC PATCH 3/4] random: Only do mixback once per read
Date: Fri, 16 Oct 2015 14:12:13 +0800	[thread overview]
Message-ID: <201510161442.qhd50o6a%fengguang.wu@intel.com> (raw)
In-Reply-To: <20151016053356.13111.qmail@ns.horizon.com>

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

Hi George,

[auto build test ERROR on v4.3-rc5 -- if it's inappropriate base, please suggest rules for selecting the more suitable base]

url:    https://github.com/0day-ci/linux/commits/George-Spelvin/Alternate-sclable-urandom-patchset/20151016-133627
config: i386-randconfig-i1-201541 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   In file included from include/asm-generic/percpu.h:6:0,
                    from arch/x86/include/asm/percpu.h:551,
                    from arch/x86/include/asm/preempt.h:5,
                    from include/linux/preempt.h:64,
                    from include/linux/spinlock.h:50,
                    from include/linux/seqlock.h:35,
                    from include/linux/time.h:5,
                    from include/uapi/linux/timex.h:56,
                    from include/linux/timex.h:56,
                    from include/linux/sched.h:19,
                    from include/linux/utsname.h:5,
                    from drivers/char/random.c:238:
   drivers/char/random.c: In function 'extract_buf':
   include/linux/percpu-defs.h:91:33: error: section attribute cannot be specified for local variables
     extern __PCPU_DUMMY_ATTRS char __pcpu_unique_##name;  \
                                    ^
   include/linux/percpu-defs.h:116:2: note: in expansion of macro 'DEFINE_PER_CPU_SECTION'
     DEFINE_PER_CPU_SECTION(type, name, "")
     ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
   include/linux/percpu-defs.h:92:26: error: section attribute cannot be specified for local variables
     __PCPU_DUMMY_ATTRS char __pcpu_unique_##name;   \
                             ^
   include/linux/percpu-defs.h:116:2: note: in expansion of macro 'DEFINE_PER_CPU_SECTION'
     DEFINE_PER_CPU_SECTION(type, name, "")
     ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
   include/linux/percpu-defs.h:92:26: error: declaration of '__pcpu_unique_random_nonce' with no linkage follows extern declaration
     __PCPU_DUMMY_ATTRS char __pcpu_unique_##name;   \
                             ^
   include/linux/percpu-defs.h:116:2: note: in expansion of macro 'DEFINE_PER_CPU_SECTION'
     DEFINE_PER_CPU_SECTION(type, name, "")
     ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
   include/linux/percpu-defs.h:91:33: note: previous declaration of '__pcpu_unique_random_nonce' was here
     extern __PCPU_DUMMY_ATTRS char __pcpu_unique_##name;  \
                                    ^
   include/linux/percpu-defs.h:116:2: note: in expansion of macro 'DEFINE_PER_CPU_SECTION'
     DEFINE_PER_CPU_SECTION(type, name, "")
     ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
>> drivers/char/random.c:1134:31: error: section attribute cannot be specified for local variables
     static DEFINE_PER_CPU(__u32, random_nonce);
                                  ^
   include/linux/percpu-defs.h:93:44: note: in definition of macro 'DEFINE_PER_CPU_SECTION'
     extern __PCPU_ATTRS(sec) __typeof__(type) name;   \
                                               ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
>> drivers/char/random.c:1134:31: error: section attribute cannot be specified for local variables
     static DEFINE_PER_CPU(__u32, random_nonce);
                                  ^
   include/linux/percpu-defs.h:95:19: note: in definition of macro 'DEFINE_PER_CPU_SECTION'
     __typeof__(type) name
                      ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
>> drivers/char/random.c:1134:31: error: weak declaration of 'random_nonce' must be public
     static DEFINE_PER_CPU(__u32, random_nonce);
                                  ^
   include/linux/percpu-defs.h:95:19: note: in definition of macro 'DEFINE_PER_CPU_SECTION'
     __typeof__(type) name
                      ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
>> drivers/char/random.c:1134:31: error: declaration of 'random_nonce' with no linkage follows extern declaration
     static DEFINE_PER_CPU(__u32, random_nonce);
                                  ^
   include/linux/percpu-defs.h:95:19: note: in definition of macro 'DEFINE_PER_CPU_SECTION'
     __typeof__(type) name
                      ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^
   drivers/char/random.c:1134:31: note: previous declaration of 'random_nonce' was here
     static DEFINE_PER_CPU(__u32, random_nonce);
                                  ^
   include/linux/percpu-defs.h:93:44: note: in definition of macro 'DEFINE_PER_CPU_SECTION'
     extern __PCPU_ATTRS(sec) __typeof__(type) name;   \
                                               ^
   drivers/char/random.c:1134:9: note: in expansion of macro 'DEFINE_PER_CPU'
     static DEFINE_PER_CPU(__u32, random_nonce);
            ^

vim +1134 drivers/char/random.c

  1128		union {
  1129			__u32 w[5];
  1130			unsigned long l[LONGS(16)];
  1131		} hash;
  1132		__u32 nonce;
  1133		__u32 workspace[SHA_WORKSPACE_WORDS];
> 1134		static DEFINE_PER_CPU(__u32, random_nonce);
  1135	
  1136		/*
  1137		 * If we have an architectural hardware random number

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 18058 bytes --]

  reply	other threads:[~2015-10-16  6:13 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-10 18:45 Updated scalable urandom patchkit George Spelvin
2015-10-11  2:31 ` Theodore Ts'o
2015-10-11  2:53   ` Theodore Ts'o
2015-10-11  4:35   ` George Spelvin
2015-10-11 22:25     ` Theodore Ts'o
2015-10-12  0:16       ` George Spelvin
2015-10-12  4:05         ` Theodore Ts'o
2015-10-12  7:49           ` George Spelvin
2015-10-12 13:54             ` Theodore Ts'o
2015-10-12 20:30               ` George Spelvin
2015-10-12 20:34                 ` George Spelvin
2015-10-13  2:46                 ` Theodore Ts'o
2015-10-13  3:50                   ` Raymond Jennings
2015-10-13  7:50                     ` George Spelvin
2015-10-13  6:24                   ` George Spelvin
2015-10-13 16:20                   ` Andi Kleen
2015-10-13 21:10                     ` George Spelvin
2015-10-14  2:15                       ` Andi Kleen
2015-10-16  5:28                         ` [RFC PATCH 0/4] Alternate sclable urandom patchset George Spelvin
2015-10-16  5:29                           ` [RFC PATCH 1/4] random: Reduce stack usage in _xfer_secondary_pool George Spelvin
2015-10-16  5:30                           ` [RFC PATCH 2/4] random: Remove two unused arguments from extract_entropy() George Spelvin
2015-10-16  5:33                           ` [RFC PATCH 3/4] random: Only do mixback once per read George Spelvin
2015-10-16  6:12                             ` kbuild test robot [this message]
2015-10-16  8:11                               ` George Spelvin
2015-10-16  6:23                             ` kbuild test robot
2015-10-16  5:34                           ` [RFC PATCH 4/4] random: Make non-blocking mixback non-blocking George Spelvin
2015-10-21  8:27                       ` Updated scalable urandom patchkit George Spelvin
2015-10-21 11:47                         ` Andi Kleen
2015-10-21 18:10                           ` George Spelvin

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=201510161442.qhd50o6a%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=ahferroin7@gmail.com \
    --cc=andi@firstfloor.org \
    --cc=jepler@unpythonic.net \
    --cc=kbuild-all@01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@horizon.com \
    --cc=linux@rasmusvillemoes.dk \
    --cc=tytso@mit.edu \
    /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.