From: kernel test robot <lkp@intel.com>
To: Roger Pau Monne <roger.pau@citrix.com>, linux-kernel@vger.kernel.org
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com,
Roger Pau Monne <roger.pau@citrix.com>,
Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>,
David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
Boris Ostrovsky <boris.ostrovsky@oracle.com>,
Juergen Gross <jgross@suse.com>,
Stefano Stabellini <sstabellini@kernel.org>,
Dan Carpenter <error27@gmail.com>, Wei Liu <wl@xen.org>
Subject: Re: [PATCH v4 2/2] xen: add helpers to allocate unpopulated memory
Date: Thu, 13 Aug 2020 02:13:05 +0800 [thread overview]
Message-ID: <202008130220.OXfV1fy9%lkp@intel.com> (raw)
In-Reply-To: <20200811094447.31208-3-roger.pau@citrix.com>
[-- Attachment #1: Type: text/plain, Size: 10397 bytes --]
Hi Roger,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on xen-tip/linux-next]
[also build test ERROR on linux/master hnaz-linux-mm/master linus/master v5.8 next-20200812]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Roger-Pau-Monne/xen-balloon-fixes-for-memory-hotplug/20200811-174616
base: https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
config: x86_64-randconfig-a001-20200812 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 30c1633386e7cfb01c0a54b31ccf4c3a3873e71b)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/xen/grant-table.c:59:
include/xen/balloon.h:8:8: error: redefinition of 'balloon_stats'
struct balloon_stats {
^
include/xen/xen.h:61:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
drivers/xen/grant-table.c:59:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
include/xen/balloon.h:8:8: note: unguarded header; consider using #ifdef guards or #pragma once
struct balloon_stats {
^
In file included from drivers/xen/grant-table.c:59:
>> include/xen/balloon.h:33:20: error: redefinition of 'xen_balloon_init'
static inline void xen_balloon_init(void)
^
include/xen/balloon.h:33:20: note: previous definition is here
static inline void xen_balloon_init(void)
^
2 errors generated.
--
In file included from drivers/xen/balloon.c:70:
include/xen/balloon.h:8:8: error: redefinition of 'balloon_stats'
struct balloon_stats {
^
include/xen/xen.h:61:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
drivers/xen/balloon.c:70:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
include/xen/balloon.h:8:8: note: unguarded header; consider using #ifdef guards or #pragma once
struct balloon_stats {
^
In file included from drivers/xen/balloon.c:70:
>> include/xen/balloon.h:33:20: error: redefinition of 'xen_balloon_init'
static inline void xen_balloon_init(void)
^
include/xen/balloon.h:33:20: note: previous definition is here
static inline void xen_balloon_init(void)
^
2 errors generated.
--
In file included from drivers/xen/gntdev.c:41:
include/xen/balloon.h:8:8: error: redefinition of 'balloon_stats'
struct balloon_stats {
^
include/xen/xen.h:61:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
drivers/xen/gntdev.c:41:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
include/xen/balloon.h:8:8: note: unguarded header; consider using #ifdef guards or #pragma once
struct balloon_stats {
^
In file included from drivers/xen/gntdev.c:41:
>> include/xen/balloon.h:33:20: error: redefinition of 'xen_balloon_init'
static inline void xen_balloon_init(void)
^
include/xen/balloon.h:33:20: note: previous definition is here
static inline void xen_balloon_init(void)
^
In file included from drivers/xen/gntdev.c:48:
In file included from drivers/xen/gntdev-common.h:15:
include/linux/mman.h:134:9: warning: division by zero is undefined [-Wdivision-by-zero]
_calc_vm_trans(flags, MAP_SYNC, VM_SYNC );
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/mman.h:111:21: note: expanded from macro '_calc_vm_trans'
: ((x) & (bit1)) / ((bit1) / (bit2))))
^ ~~~~~~~~~~~~~~~~~
1 warning and 2 errors generated.
--
In file included from drivers/xen/privcmd.c:40:
include/xen/balloon.h:8:8: error: redefinition of 'balloon_stats'
struct balloon_stats {
^
include/xen/xen.h:61:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
drivers/xen/privcmd.c:40:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
include/xen/balloon.h:8:8: note: unguarded header; consider using #ifdef guards or #pragma once
struct balloon_stats {
^
In file included from drivers/xen/privcmd.c:40:
>> include/xen/balloon.h:33:20: error: redefinition of 'xen_balloon_init'
static inline void xen_balloon_init(void)
^
include/xen/balloon.h:33:20: note: previous definition is here
static inline void xen_balloon_init(void)
^
2 errors generated.
--
In file included from drivers/xen/xlate_mmu.c:43:
include/xen/balloon.h:8:8: error: redefinition of 'balloon_stats'
struct balloon_stats {
^
include/xen/xen.h:61:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
drivers/xen/xlate_mmu.c:43:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
include/xen/balloon.h:8:8: note: unguarded header; consider using #ifdef guards or #pragma once
struct balloon_stats {
^
In file included from drivers/xen/xlate_mmu.c:43:
>> include/xen/balloon.h:33:20: error: redefinition of 'xen_balloon_init'
static inline void xen_balloon_init(void)
^
include/xen/balloon.h:33:20: note: previous definition is here
static inline void xen_balloon_init(void)
^
2 errors generated.
--
In file included from drivers/xen/xenbus/xenbus_client.c:44:
In file included from include/xen/events.h:13:
In file included from arch/x86/include/asm/xen/events.h:5:
In file included from include/xen/xen.h:61:
include/xen/balloon.h:8:8: error: redefinition of 'balloon_stats'
struct balloon_stats {
^
drivers/xen/xenbus/xenbus_client.c:43:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
include/xen/xen.h:61:10: note: 'include/xen/balloon.h' included multiple times, additional include site here
#include <xen/balloon.h>
^
include/xen/balloon.h:8:8: note: unguarded header; consider using #ifdef guards or #pragma once
struct balloon_stats {
^
In file included from drivers/xen/xenbus/xenbus_client.c:44:
In file included from include/xen/events.h:13:
In file included from arch/x86/include/asm/xen/events.h:5:
In file included from include/xen/xen.h:61:
>> include/xen/balloon.h:33:20: error: redefinition of 'xen_balloon_init'
static inline void xen_balloon_init(void)
^
include/xen/balloon.h:33:20: note: previous definition is here
static inline void xen_balloon_init(void)
^
2 errors generated.
vim +/xen_balloon_init +33 include/xen/balloon.h
803eb047a28d239 Daniel De Graaf 2011-03-14 7
803eb047a28d239 Daniel De Graaf 2011-03-14 @8 struct balloon_stats {
803eb047a28d239 Daniel De Graaf 2011-03-14 9 /* We aim for 'current allocation' == 'target allocation'. */
803eb047a28d239 Daniel De Graaf 2011-03-14 10 unsigned long current_pages;
803eb047a28d239 Daniel De Graaf 2011-03-14 11 unsigned long target_pages;
1cf6a6c82918c9a David Vrabel 2015-06-25 12 unsigned long target_unpopulated;
803eb047a28d239 Daniel De Graaf 2011-03-14 13 /* Number of pages in high- and low-memory balloons. */
803eb047a28d239 Daniel De Graaf 2011-03-14 14 unsigned long balloon_low;
803eb047a28d239 Daniel De Graaf 2011-03-14 15 unsigned long balloon_high;
de5a77d8422fc7e David Vrabel 2015-06-25 16 unsigned long total_pages;
803eb047a28d239 Daniel De Graaf 2011-03-14 17 unsigned long schedule_delay;
803eb047a28d239 Daniel De Graaf 2011-03-14 18 unsigned long max_schedule_delay;
803eb047a28d239 Daniel De Graaf 2011-03-14 19 unsigned long retry_count;
803eb047a28d239 Daniel De Graaf 2011-03-14 20 unsigned long max_retry_count;
803eb047a28d239 Daniel De Graaf 2011-03-14 21 };
803eb047a28d239 Daniel De Graaf 2011-03-14 22
803eb047a28d239 Daniel De Graaf 2011-03-14 23 extern struct balloon_stats balloon_stats;
803eb047a28d239 Daniel De Graaf 2011-03-14 24
803eb047a28d239 Daniel De Graaf 2011-03-14 25 void balloon_set_new_target(unsigned long target);
b6f3067985f12d5 Konrad Rzeszutek Wilk 2011-03-15 26
81b286e0f1fe520 David Vrabel 2015-06-25 27 int alloc_xenballooned_pages(int nr_pages, struct page **pages);
b6f3067985f12d5 Konrad Rzeszutek Wilk 2011-03-15 28 void free_xenballooned_pages(int nr_pages, struct page **pages);
a50777c791031d7 Dan Magenheimer 2011-07-08 29
96edd61dcf44362 Juergen Gross 2017-07-10 30 #ifdef CONFIG_XEN_BALLOON
96edd61dcf44362 Juergen Gross 2017-07-10 31 void xen_balloon_init(void);
96edd61dcf44362 Juergen Gross 2017-07-10 32 #else
96edd61dcf44362 Juergen Gross 2017-07-10 @33 static inline void xen_balloon_init(void)
---
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: 26752 bytes --]
next prev parent reply other threads:[~2020-08-12 18:14 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-11 9:44 [PATCH v4 0/2] xen/balloon: fixes for memory hotplug Roger Pau Monne
2020-08-11 9:44 ` [PATCH v4 1/2] memremap: rename MEMORY_DEVICE_DEVDAX to MEMORY_DEVICE_GENERIC Roger Pau Monne
2020-08-11 21:07 ` David Hildenbrand
[not found] ` <20200820113741.GV828@Air-de-Roger>
2020-08-31 10:19 ` Roger Pau Monné
2020-08-31 20:56 ` Ira Weiny
2020-08-31 19:20 ` Andrew Morton
2020-08-11 9:44 ` [PATCH v4 2/2] xen: add helpers to allocate unpopulated memory Roger Pau Monne
2020-08-12 7:28 ` Jürgen Groß
2020-08-12 7:38 ` Roger Pau Monné
2020-08-12 18:13 ` kernel test robot [this message]
2020-08-13 7:33 ` Christoph Hellwig
2020-08-13 7:54 ` Roger Pau Monné
2020-08-13 9:49 ` Daniel Vetter
2020-08-13 10:02 ` Roger Pau Monné
2020-08-14 7:29 ` Christoph Hellwig
2020-08-14 9:56 ` Roger Pau Monné
2020-08-14 10:27 ` Jürgen Groß
2020-08-14 12:47 ` Roger Pau Monné
2020-08-14 12:54 ` Jürgen Groß
2020-08-14 13:35 ` Roger Pau Monné
2020-08-14 13:52 ` Jürgen Groß
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=202008130220.OXfV1fy9%lkp@intel.com \
--to=lkp@intel.com \
--cc=airlied@linux.ie \
--cc=boris.ostrovsky@oracle.com \
--cc=clang-built-linux@googlegroups.com \
--cc=daniel@ffwll.ch \
--cc=error27@gmail.com \
--cc=jgross@suse.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oleksandr_andrushchenko@epam.com \
--cc=roger.pau@citrix.com \
--cc=sstabellini@kernel.org \
--cc=wl@xen.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome