From: kbuild test robot <lkp@intel.com>
To: Harshad Shirwadkar <harshadshirwadkar@gmail.com>
Cc: kbuild-all@01.org, linux-ext4@vger.kernel.org,
Harshad Shirwadkar <harshadshirwadkar@gmail.com>
Subject: Re: [PATCH 05/11] jbd2: fast-commit commit path new APIs
Date: Tue, 23 Jul 2019 01:45:35 +0800 [thread overview]
Message-ID: <201907230139.mzhuV4KQ%lkp@intel.com> (raw)
In-Reply-To: <20190722040011.18892-5-harshadshirwadkar@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 8824 bytes --]
Hi Harshad,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[cannot apply to v5.3-rc1 next-20190722]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Harshad-Shirwadkar/ext4-add-handling-for-extended-mount-options/20190723-001855
config: i386-defconfig (attached as .config)
compiler: gcc-7 (Debian 7.4.0-10) 7.4.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All error/warnings (new ones prefixed by >>):
In file included from include/linux/linkage.h:7:0,
from include/linux/kernel.h:8,
from include/linux/list.h:9,
from include/linux/module.h:9,
from fs/jbd2/journal.c:22:
>> include/linux/export.h:81:20: error: redefinition of '__kstrtab_jbd2_complete_transaction'
static const char __kstrtab_##sym[] \
^
>> include/linux/export.h:120:25: note: in expansion of macro '___EXPORT_SYMBOL'
#define __EXPORT_SYMBOL ___EXPORT_SYMBOL
^~~~~~~~~~~~~~~~
>> include/linux/export.h:124:2: note: in expansion of macro '__EXPORT_SYMBOL'
__EXPORT_SYMBOL(sym, "")
^~~~~~~~~~~~~~~
>> fs/jbd2/journal.c:839:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(jbd2_complete_transaction);
^~~~~~~~~~~~~
include/linux/export.h:81:20: note: previous definition of '__kstrtab_jbd2_complete_transaction' was here
static const char __kstrtab_##sym[] \
^
>> include/linux/export.h:120:25: note: in expansion of macro '___EXPORT_SYMBOL'
#define __EXPORT_SYMBOL ___EXPORT_SYMBOL
^~~~~~~~~~~~~~~~
>> include/linux/export.h:124:2: note: in expansion of macro '__EXPORT_SYMBOL'
__EXPORT_SYMBOL(sym, "")
^~~~~~~~~~~~~~~
fs/jbd2/journal.c:812:1: note: in expansion of macro 'EXPORT_SYMBOL'
EXPORT_SYMBOL(jbd2_complete_transaction);
^~~~~~~~~~~~~
vim +/__kstrtab_jbd2_complete_transaction +81 include/linux/export.h
7290d58095712a8 Ard Biesheuvel 2018-08-21 76
f50169324df4ad9 Paul Gortmaker 2011-05-23 77 /* For every exported symbol, place a struct in the __ksymtab section */
f235541699bcf14 Nicolas Pitre 2016-01-22 78 #define ___EXPORT_SYMBOL(sym, sec) \
f50169324df4ad9 Paul Gortmaker 2011-05-23 79 extern typeof(sym) sym; \
f50169324df4ad9 Paul Gortmaker 2011-05-23 80 __CRC_SYMBOL(sym, sec) \
f50169324df4ad9 Paul Gortmaker 2011-05-23 @81 static const char __kstrtab_##sym[] \
7290d58095712a8 Ard Biesheuvel 2018-08-21 82 __attribute__((section("__ksymtab_strings"), used, aligned(1))) \
94e58e0ac31284f Masahiro Yamada 2018-05-09 83 = #sym; \
7290d58095712a8 Ard Biesheuvel 2018-08-21 84 __KSYMTAB_ENTRY(sym, sec)
f50169324df4ad9 Paul Gortmaker 2011-05-23 85
f922c4abdf76485 Ard Biesheuvel 2018-08-21 86 #if defined(__DISABLE_EXPORTS)
f922c4abdf76485 Ard Biesheuvel 2018-08-21 87
f922c4abdf76485 Ard Biesheuvel 2018-08-21 88 /*
f922c4abdf76485 Ard Biesheuvel 2018-08-21 89 * Allow symbol exports to be disabled completely so that C code may
f922c4abdf76485 Ard Biesheuvel 2018-08-21 90 * be reused in other execution contexts such as the UEFI stub or the
f922c4abdf76485 Ard Biesheuvel 2018-08-21 91 * decompressor.
f922c4abdf76485 Ard Biesheuvel 2018-08-21 92 */
f922c4abdf76485 Ard Biesheuvel 2018-08-21 93 #define __EXPORT_SYMBOL(sym, sec)
f922c4abdf76485 Ard Biesheuvel 2018-08-21 94
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 95 #elif defined(CONFIG_TRIM_UNUSED_KSYMS)
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 96
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 97 #include <generated/autoksyms.h>
c1a95fda2a40ae8 Nicolas Pitre 2016-01-22 98
c1a95fda2a40ae8 Nicolas Pitre 2016-01-22 99 /*
c1a95fda2a40ae8 Nicolas Pitre 2016-01-22 100 * For fine grained build dependencies, we want to tell the build system
c1a95fda2a40ae8 Nicolas Pitre 2016-01-22 101 * about each possible exported symbol even if they're not actually exported.
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 102 * We use a symbol pattern __ksym_marker_<symbol> that the build system filters
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 103 * from the $(NM) output (see scripts/gen_ksymdeps.sh). These symbols are
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 104 * discarded in the final link stage.
c1a95fda2a40ae8 Nicolas Pitre 2016-01-22 105 */
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 106 #define __ksym_marker(sym) \
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 107 static int __ksym_marker_##sym[0] __section(".discard.ksym") __used
f235541699bcf14 Nicolas Pitre 2016-01-22 108
f235541699bcf14 Nicolas Pitre 2016-01-22 109 #define __EXPORT_SYMBOL(sym, sec) \
bbda5ec671d3fe6 Masahiro Yamada 2018-11-30 110 __ksym_marker(sym); \
6023d2369ba7b82 Masahiro Yamada 2016-06-14 111 __cond_export_sym(sym, sec, __is_defined(__KSYM_##sym))
f235541699bcf14 Nicolas Pitre 2016-01-22 112 #define __cond_export_sym(sym, sec, conf) \
f235541699bcf14 Nicolas Pitre 2016-01-22 113 ___cond_export_sym(sym, sec, conf)
f235541699bcf14 Nicolas Pitre 2016-01-22 114 #define ___cond_export_sym(sym, sec, enabled) \
f235541699bcf14 Nicolas Pitre 2016-01-22 115 __cond_export_sym_##enabled(sym, sec)
f235541699bcf14 Nicolas Pitre 2016-01-22 116 #define __cond_export_sym_1(sym, sec) ___EXPORT_SYMBOL(sym, sec)
f235541699bcf14 Nicolas Pitre 2016-01-22 117 #define __cond_export_sym_0(sym, sec) /* nothing */
f235541699bcf14 Nicolas Pitre 2016-01-22 118
f235541699bcf14 Nicolas Pitre 2016-01-22 119 #else
f235541699bcf14 Nicolas Pitre 2016-01-22 @120 #define __EXPORT_SYMBOL ___EXPORT_SYMBOL
f235541699bcf14 Nicolas Pitre 2016-01-22 121 #endif
f235541699bcf14 Nicolas Pitre 2016-01-22 122
f50169324df4ad9 Paul Gortmaker 2011-05-23 123 #define EXPORT_SYMBOL(sym) \
f50169324df4ad9 Paul Gortmaker 2011-05-23 @124 __EXPORT_SYMBOL(sym, "")
f50169324df4ad9 Paul Gortmaker 2011-05-23 125
f50169324df4ad9 Paul Gortmaker 2011-05-23 126 #define EXPORT_SYMBOL_GPL(sym) \
f50169324df4ad9 Paul Gortmaker 2011-05-23 127 __EXPORT_SYMBOL(sym, "_gpl")
f50169324df4ad9 Paul Gortmaker 2011-05-23 128
f50169324df4ad9 Paul Gortmaker 2011-05-23 129 #define EXPORT_SYMBOL_GPL_FUTURE(sym) \
f50169324df4ad9 Paul Gortmaker 2011-05-23 130 __EXPORT_SYMBOL(sym, "_gpl_future")
f50169324df4ad9 Paul Gortmaker 2011-05-23 131
f50169324df4ad9 Paul Gortmaker 2011-05-23 132 #ifdef CONFIG_UNUSED_SYMBOLS
f50169324df4ad9 Paul Gortmaker 2011-05-23 133 #define EXPORT_UNUSED_SYMBOL(sym) __EXPORT_SYMBOL(sym, "_unused")
f50169324df4ad9 Paul Gortmaker 2011-05-23 134 #define EXPORT_UNUSED_SYMBOL_GPL(sym) __EXPORT_SYMBOL(sym, "_unused_gpl")
f50169324df4ad9 Paul Gortmaker 2011-05-23 135 #else
f50169324df4ad9 Paul Gortmaker 2011-05-23 136 #define EXPORT_UNUSED_SYMBOL(sym)
f50169324df4ad9 Paul Gortmaker 2011-05-23 137 #define EXPORT_UNUSED_SYMBOL_GPL(sym)
f50169324df4ad9 Paul Gortmaker 2011-05-23 138 #endif
f50169324df4ad9 Paul Gortmaker 2011-05-23 139
f50169324df4ad9 Paul Gortmaker 2011-05-23 140 #endif /* __GENKSYMS__ */
f50169324df4ad9 Paul Gortmaker 2011-05-23 141
f50169324df4ad9 Paul Gortmaker 2011-05-23 142 #else /* !CONFIG_MODULES... */
f50169324df4ad9 Paul Gortmaker 2011-05-23 143
f50169324df4ad9 Paul Gortmaker 2011-05-23 144 #define EXPORT_SYMBOL(sym)
f50169324df4ad9 Paul Gortmaker 2011-05-23 145 #define EXPORT_SYMBOL_GPL(sym)
f50169324df4ad9 Paul Gortmaker 2011-05-23 146 #define EXPORT_SYMBOL_GPL_FUTURE(sym)
f50169324df4ad9 Paul Gortmaker 2011-05-23 147 #define EXPORT_UNUSED_SYMBOL(sym)
f50169324df4ad9 Paul Gortmaker 2011-05-23 148 #define EXPORT_UNUSED_SYMBOL_GPL(sym)
f50169324df4ad9 Paul Gortmaker 2011-05-23 149
f50169324df4ad9 Paul Gortmaker 2011-05-23 150 #endif /* CONFIG_MODULES */
b92021b09df70c1 Rusty Russell 2013-03-15 151 #endif /* !__ASSEMBLY__ */
f50169324df4ad9 Paul Gortmaker 2011-05-23 152
f50169324df4ad9 Paul Gortmaker 2011-05-23 153 #endif /* _LINUX_EXPORT_H */
:::::: The code at line 81 was first introduced by commit
:::::: f50169324df4ad942e544386d136216c8617636a module.h: split out the EXPORT_SYMBOL into export.h
:::::: TO: Paul Gortmaker <paul.gortmaker@windriver.com>
:::::: CC: Paul Gortmaker <paul.gortmaker@windriver.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 28082 bytes --]
next prev parent reply other threads:[~2019-07-22 17:53 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-22 4:00 [PATCH 01/11] ext4: add handling for extended mount options Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 02/11] jbd2: add fast commit fields to journal_s structure Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 03/11] jbd2: fast commit setup and enable Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 04/11] jbd2: fast-commit commit path changes Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 05/11] jbd2: fast-commit commit path new APIs Harshad Shirwadkar
2019-07-22 17:45 ` kbuild test robot [this message]
2019-07-22 21:02 ` kbuild test robot
2019-07-22 4:00 ` [PATCH 06/11] jbd2: fast-commit recovery path changes Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 07/11] ext4: add fields that are needed to track changed files Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 08/11] ext4: track changed files for fast commit Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 09/11] ext4: fast-commit commit range tracking Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 10/11] ext4: fast-commit commit path changes Harshad Shirwadkar
2019-07-22 4:00 ` [PATCH 11/11] ext4: fast-commit recovery " Harshad Shirwadkar
2019-07-22 21:34 ` kbuild test robot
2019-07-22 18:15 ` [PATCH 01/11] ext4: add handling for extended mount options Andreas Dilger
2019-07-22 21:02 ` Theodore Y. Ts'o
2019-07-22 21:36 ` harshad shirwadkar
2019-07-23 21:59 ` Andreas Dilger
2019-07-24 6:03 ` harshad shirwadkar
2019-07-24 6:12 ` Darrick J. Wong
2019-07-24 16:07 ` Theodore Y. Ts'o
2019-07-24 16:56 ` Darrick J. Wong
2019-07-24 18:14 ` harshad shirwadkar
2019-07-25 3:46 ` Theodore Y. Ts'o
2019-07-25 20:03 ` Andreas Dilger
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=201907230139.mzhuV4KQ%lkp@intel.com \
--to=lkp@intel.com \
--cc=harshadshirwadkar@gmail.com \
--cc=kbuild-all@01.org \
--cc=linux-ext4@vger.kernel.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;
as well as URLs for NNTP newsgroup(s).