1. 08 4月, 2020 17 次提交
  2. 03 4月, 2020 2 次提交
  3. 30 3月, 2020 1 次提交
  4. 29 3月, 2020 2 次提交
    • M
      kbuild: add -Wall to KBUILD_HOSTCXXFLAGS · 735aab1e
      Masahiro Yamada 提交于
      Add -Wall to catch more warnings for C++ host programs.
      
      When I submitted the previous version, the 0-day bot reported
      -Wc++11-compat warnings for old GCC:
      
        HOSTCXX -fPIC scripts/gcc-plugins/latent_entropy_plugin.o
      In file included from /usr/lib/gcc/x86_64-linux-gnu/4.8/plugin/include/tm.h:28:0,
                       from scripts/gcc-plugins/gcc-common.h:15,
                       from scripts/gcc-plugins/latent_entropy_plugin.c:78:
      /usr/lib/gcc/x86_64-linux-gnu/4.8/plugin/include/config/elfos.h:102:21: warning: C++11 requires a space between string literal and macro [-Wc++11-compat]
          fprintf ((FILE), "%s"HOST_WIDE_INT_PRINT_UNSIGNED"\n",\
                           ^
      /usr/lib/gcc/x86_64-linux-gnu/4.8/plugin/include/config/elfos.h:170:24: warning: C++11 requires a space between string literal and macro [-Wc++11-compat]
             fprintf ((FILE), ","HOST_WIDE_INT_PRINT_UNSIGNED",%u\n",  \
                              ^
      In file included from /usr/lib/gcc/x86_64-linux-gnu/4.8/plugin/include/tm.h:42:0,
                       from scripts/gcc-plugins/gcc-common.h:15,
                       from scripts/gcc-plugins/latent_entropy_plugin.c:78:
      /usr/lib/gcc/x86_64-linux-gnu/4.8/plugin/include/defaults.h:126:24: warning: C++11 requires a space between string literal and macro [-Wc++11-compat]
             fprintf ((FILE), ","HOST_WIDE_INT_PRINT_UNSIGNED",%u\n",  \
                              ^
      
      The source of the warnings is in the plugin headers, so we have no
      control of it. I just suppressed them by adding -Wno-c++11-compat to
      scripts/gcc-plugins/Makefile.
      Signed-off-by: NMasahiro Yamada <masahiroy@kernel.org>
      Acked-by: NKees Cook <keescook@chromium.org>
      735aab1e
    • M
      kconfig: remove unused variable in qconf.cc · dbd35860
      Masahiro Yamada 提交于
      If this file were compiled with -Wall, the following warning would be
      reported:
      
      scripts/kconfig/qconf.cc:312:6: warning: unused variable ‘i’ [-Wunused-variable]
        int i;
            ^
      
      The commit prepares to turn on -Wall for C++ host programs.
      Signed-off-by: NMasahiro Yamada <masahiroy@kernel.org>
      Reviewed-by: NKees Cook <keescook@chromium.org>
      dbd35860
  5. 27 3月, 2020 2 次提交
    • D
      scripts/dtc: Remove redundant YYLOC global declaration · e33a814e
      Dirk Mueller 提交于
      gcc 10 will default to -fno-common, which causes this error at link
      time:
      
        (.text+0x0): multiple definition of `yylloc'; dtc-lexer.lex.o (symbol from plugin):(.text+0x0): first defined here
      
      This is because both dtc-lexer as well as dtc-parser define the same
      global symbol yyloc. Before with -fcommon those were merged into one
      defintion. The proper solution would be to to mark this as "extern",
      however that leads to:
      
        dtc-lexer.l:26:16: error: redundant redeclaration of 'yylloc' [-Werror=redundant-decls]
         26 | extern YYLTYPE yylloc;
            |                ^~~~~~
      In file included from dtc-lexer.l:24:
      dtc-parser.tab.h:127:16: note: previous declaration of 'yylloc' was here
        127 | extern YYLTYPE yylloc;
            |                ^~~~~~
      cc1: all warnings being treated as errors
      
      which means the declaration is completely redundant and can just be
      dropped.
      Signed-off-by: NDirk Mueller <dmueller@suse.com>
      Signed-off-by: NDavid Gibson <david@gibson.dropbear.id.au>
      [robh: cherry-pick from upstream]
      Cc: stable@vger.kernel.org
      Signed-off-by: NRob Herring <robh@kernel.org>
      e33a814e
    • J
      parse-maintainers: Do not sort section content by default · 5cdbec10
      Joe Perches 提交于
      Add an --order switch to control section reordering.
      Default for --order is off.
      
      Change the default ordering to a slightly more sensible:
      
      M:  Person acting as a maintainer
      R:  Person acting as a patch reviewer
      L:  Mailing list where patches should be sent
      S:  Maintenance status
      W:  URI for general information
      Q:  URI for patchwork tracking
      B:  URI for bug tracking/submission
      C:  URI for chat
      P:  URI or file for subsystem specific coding styles
      T:  SCM tree type and location
      F:  File and directory pattern
      X:  File and directory exclusion pattern
      N:  File glob
      K:  Keyword - patch content regex
      Signed-off-by: NJoe Perches <joe@perches.com>
      Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
      5cdbec10
  6. 25 3月, 2020 4 次提交
  7. 21 3月, 2020 1 次提交
  8. 19 3月, 2020 4 次提交
  9. 18 3月, 2020 1 次提交
  10. 17 3月, 2020 1 次提交
    • J
      modpost: move the namespace field in Module.symvers last · 5190044c
      Jessica Yu 提交于
      In order to preserve backwards compatability with kmod tools, we have to
      move the namespace field in Module.symvers last, as the depmod -e -E
      option looks at the first three fields in Module.symvers to check symbol
      versions (and it's expected they stay in the original order of crc,
      symbol, module).
      
      In addition, update an ancient comment above read_dump() in modpost that
      suggested that the export type field in Module.symvers was optional. I
      suspect that there were historical reasons behind that comment that are
      no longer accurate. We have been unconditionally printing the export
      type since 2.6.18 (commit bd5cbced), which is over a decade ago now.
      
      Fix up read_dump() to treat each field as non-optional. I suspect the
      original read_dump() code treated the export field as optional in order
      to support pre <= 2.6.18 Module.symvers (which did not have the export
      type field). Note that although symbol namespaces are optional, the
      field will not be omitted from Module.symvers if a symbol does not have
      a namespace. In this case, the field will simply be empty and the next
      delimiter or end of line will follow.
      
      Cc: stable@vger.kernel.org
      Fixes: cb9b55d2 ("modpost: add support for symbol namespaces")
      Tested-by: NMatthias Maennich <maennich@google.com>
      Reviewed-by: NMatthias Maennich <maennich@google.com>
      Reviewed-by: NLucas De Marchi <lucas.demarchi@intel.com>
      Signed-off-by: NJessica Yu <jeyu@kernel.org>
      Signed-off-by: NMasahiro Yamada <masahiroy@kernel.org>
      5190044c
  11. 14 3月, 2020 2 次提交
  12. 13 3月, 2020 3 次提交
    • R
      scripts/dtc: Update to upstream version v1.6.0-2-g87a656ae5ff9 · d047cd8a
      Rob Herring 提交于
      This adds the following commits from upstream:
      
      87a656ae5ff9 check: Inform about missing ranges
      73d6e9ecb417 libfdt: fix undefined behaviour in fdt_splice_()
      2525da3dba9b Bump version to v1.6.0
      62cb4ad286ff Execute tests on FreeBSD with Cirrus CI
      1f9a41750883 tests: Allow running the testsuite on already installed binary / libraries
      c5995ddf4c20 tests: Honour NO_YAML make variable
      e4ce227e89d7 tests: Properly clean up .bak file from tests
      9b75292c335c tests: Honour $(NO_PYTHON) flag from Makefile in run_tests.sh
      6c253afd07d4 Encode $(NO_PYTHON) consistently with other variables
      95ec8ef706bd tests: No need to explicitly pass $PYTHON from Make to run_tests.sh
      2b5f62d109a2 tests: Let run_tests.sh run Python tests without Makefile assistance
      76b43dcbd18a checks: Add 'dma-ranges' check
      e5c92a4780c6 libfdt: Use VALID_INPUT for FDT_ERR_BADSTATE checks
      e5cc26b68bc0 libfdt: Add support for disabling internal checks
      28fd7590aad2 libfdt: Improve comments in some of the assumptions
      fc207c32341b libfdt: Fix a few typos
      0f61c72dedc4 libfdt: Allow exclusion of fdt_check_full()
      f270f45fd5d2 libfdt: Add support for disabling ordering check/fixup
      c18bae9a4c96 libfdt: Add support for disabling version checks
      fc03c4a2e04e libfdt: Add support for disabling rollback handling
      77563ae72b7c libfdt: Add support for disabling sanity checks
      57bc6327b80b libfdt: Add support for disabling dtb checks
      464962489dcc Add a way to control the level of checks in the code
      0c5326cb2845 libfdt: De-inline fdt_header_size()
      cc6a5a071504 Revert "yamltree: Ensure consistent bracketing of properties with phandles"
      0e9225eb0dfe Remove redundant YYLOC global declaration
      cab09eedd644 Move -DNO_VALGRIND into CPPFLAGS
      0eb1cb0b531e Makefile: pass $(CFLAGS) also during dependency generation
      Signed-off-by: NRob Herring <robh@kernel.org>
      d047cd8a
    • R
      scripts/dtc: Remove unused makefile fragments · 78154212
      Rob Herring 提交于
      The Makefile.dtc and Makefile.libfdt fragments from upstream dtc aren't
      used by the kernel build, so let's remove them and stop syncing them.
      Signed-off-by: NRob Herring <robh@kernel.org>
      78154212
    • M
      kconfig: make 'imply' obey the direct dependency · 3a9dd3ec
      Masahiro Yamada 提交于
      The 'imply' statement may create unmet direct dependency when the
      implied symbol depends on m.
      
      [Test Code]
      
        config FOO
                tristate "foo"
                imply BAZ
      
        config BAZ
                tristate "baz"
                depends on BAR
      
        config BAR
                def_tristate m
      
        config MODULES
                def_bool y
                option modules
      
      If you set FOO=y, BAZ is also promoted to y, which results in the
      following .config file:
      
        CONFIG_FOO=y
        CONFIG_BAZ=y
        CONFIG_BAR=m
        CONFIG_MODULES=y
      
      This does not meet the dependency 'BAZ depends on BAR'.
      
      Unlike 'select', what is worse, Kconfig never shows the
      'WARNING: unmet direct dependencies detected for ...' for this case.
      
      Because 'imply' is considered to be weaker than 'depends on', Kconfig
      should take the direct dependency into account.
      
      For clarification, describe this case in kconfig-language.rst too.
      Signed-off-by: NMasahiro Yamada <masahiroy@kernel.org>
      Acked-by: NNicolas Pitre <nico@fluxnic.net>
      Tested-by: NGeert Uytterhoeven <geert@linux-m68k.org>
      3a9dd3ec