1. 12 Jun, 2020 15 commits
    • Linus Torvalds's avatar
      Merge tag 'devicetree-fixes-for-5.8-1' of git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux · 4071b856
      Linus Torvalds authored
      Pull Devicetree fixes from Rob Herring:
      
       - Another round of whack-a-mole removing 'allOf', redundant cases of
         'maxItems' and incorrect 'reg' sizes
      
       - Fix support for yaml.h in non-standard paths
      
      * tag 'devicetree-fixes-for-5.8-1' of git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux:
        dt-bindings: Remove redundant 'maxItems'
        dt-bindings: Fix more incorrect 'reg' property sizes in examples
        dt-bindings: phy: qcom: Fix missing 'ranges' and example addresses
        dt-bindings: Remove more cases of 'allOf' containing a '$ref'
        scripts/dtc: use pkg-config to include <yaml.h> in non-standard path
      4071b856
    • Linus Torvalds's avatar
      Merge tag 'nios2-v5.8-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/lftan/nios2 · 7de26c41
      Linus Torvalds authored
      Pull nios2 update from Ley Foon Tan:
       "Mark expected switch fall-through in signal handling"
      
      * tag 'nios2-v5.8-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/lftan/nios2:
        nios2: signal: Mark expected switch fall-through
      7de26c41
    • Linus Torvalds's avatar
      Merge tag 'for-linus' of git://git.kernel.org/pub/scm/virt/kvm/kvm · 52cd0d97
      Linus Torvalds authored
      Pull more KVM updates from Paolo Bonzini:
       "The guest side of the asynchronous page fault work has been delayed to
        5.9 in order to sync with Thomas's interrupt entry rework, but here's
        the rest of the KVM updates for this merge window.
      
        MIPS:
         - Loongson port
      
        PPC:
         - Fixes
      
        ARM:
         - Fixes
      
        x86:
         - KVM_SET_USER_MEMORY_REGION optimizations
         - Fixes
         - Selftest fixes"
      
      * tag 'for-linus' of git://git.kernel.org/pub/scm/virt/kvm/kvm: (62 commits)
        KVM: x86: do not pass poisoned hva to __kvm_set_memory_region
        KVM: selftests: fix sync_with_host() in smm_test
        KVM: async_pf: Inject 'page ready' event only if 'page not present' was previously injected
        KVM: async_pf: Cleanup kvm_setup_async_pf()
        kvm: i8254: remove redundant assignment to pointer s
        KVM: x86: respect singlestep when emulating instruction
        KVM: selftests: Don't probe KVM_CAP_HYPERV_ENLIGHTENED_VMCS when nested VMX is unsupported
        KVM: selftests: do not substitute SVM/VMX check with KVM_CAP_NESTED_STATE check
        KVM: nVMX: Consult only the "basic" exit reason when routing nested exit
        KVM: arm64: Move hyp_symbol_addr() to kvm_asm.h
        KVM: arm64: Synchronize sysreg state on injecting an AArch32 exception
        KVM: arm64: Make vcpu_cp1x() work on Big Endian hosts
        KVM: arm64: Remove host_cpu_context member from vcpu structure
        KVM: arm64: Stop sparse from moaning at __hyp_this_cpu_ptr
        KVM: arm64: Handle PtrAuth traps early
        KVM: x86: Unexport x86_fpu_cache and make it static
        KVM: selftests: Ignore KVM 5-level paging support for VM_MODE_PXXV48_4K
        KVM: arm64: Save the host's PtrAuth keys in non-preemptible context
        KVM: arm64: Stop save/restoring ACTLR_EL1
        KVM: arm64: Add emulation for 32bit guests accessing ACTLR2
        ...
      52cd0d97
    • Linus Torvalds's avatar
      Merge tag 'for-linus-5.8b-rc1-tag' of git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip · d2d5439d
      Linus Torvalds authored
      Pull xen updates from Juergen Gross:
      
       - several smaller cleanups
      
       - a fix for a Xen guest regression with CPU offlining
      
       - a small fix in the xen pvcalls backend driver
      
       - an update of MAINTAINERS
      
      * tag 'for-linus-5.8b-rc1-tag' of git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip:
        MAINTAINERS: Update PARAVIRT_OPS_INTERFACE and VMWARE_HYPERVISOR_INTERFACE
        xen/pci: Get rid of verbose_request and use dev_dbg() instead
        xenbus: Use dev_printk() when possible
        xen-pciback: Use dev_printk() when possible
        xen: enable BALLOON_MEMORY_HOTPLUG by default
        xen: expand BALLOON_MEMORY_HOTPLUG description
        xen/pvcalls: Make pvcalls_back_global static
        xen/cpuhotplug: Fix initial CPU offlining for PV(H) guests
        xen-platform: Constify dev_pm_ops
        xen/pvcalls-back: test for errors when calling backend_connect()
      d2d5439d
    • Rob Herring's avatar
      8440d4a7
    • Rob Herring's avatar
      dt-bindings: Remove redundant 'maxItems' · 44761570
      Rob Herring authored
      There's no need to specify 'maxItems' with the same value as the number
      of entries in 'items'. A meta-schema update will catch future cases.
      
      Cc: Shawn Guo <shawnguo@kernel.org>
      Cc: Sascha Hauer <s.hauer@pengutronix.de>
      Cc: Anson Huang <Anson.Huang@nxp.com>
      Cc: linux-clk@vger.kernel.org
      Cc: linux-pwm@vger.kernel.org
      Cc: linux-usb@vger.kernel.org
      Reviewed-by: Stephen Boyd <sboyd@kernel.org> # clk
      Acked-by: default avatarThierry Reding <thierry.reding@gmail.com>
      Signed-off-by: default avatarRob Herring <robh@kernel.org>
      44761570
    • Ley Foon Tan's avatar
      nios2: signal: Mark expected switch fall-through · 6b57fa4d
      Ley Foon Tan authored
      Mark switch cases where we are expecting to fall through.
      
      Fix the following warning through the use of the new the new
      pseudo-keyword fallthrough;
      
      arch/nios2/kernel/signal.c:254:12: warning: this statement may fall through [-Wimplicit-fallthrough=]
        254 |    restart = -2;
            |    ~~~~~~~~^~~~
      arch/nios2/kernel/signal.c:255:3: note: here
        255 |   case ERESTARTNOHAND:
            |   ^~~~
      Reported-by: default avatarChristian Brauner <christian.brauner@ubuntu.com>
      Signed-off-by: default avatarGustavo A. R. Silva <gustavoars@kernel.org>
      Signed-off-by: default avatarLey Foon Tan <ley.foon.tan@intel.com>
      6b57fa4d
    • Linus Torvalds's avatar
      Merge tag 'locking-kcsan-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · b791d1bd
      Linus Torvalds authored
      Pull the Kernel Concurrency Sanitizer from Thomas Gleixner:
       "The Kernel Concurrency Sanitizer (KCSAN) is a dynamic race detector,
        which relies on compile-time instrumentation, and uses a
        watchpoint-based sampling approach to detect races.
      
        The feature was under development for quite some time and has already
        found legitimate bugs.
      
        Unfortunately it comes with a limitation, which was only understood
        late in the development cycle:
      
           It requires an up to date CLANG-11 compiler
      
        CLANG-11 is not yet released (scheduled for June), but it's the only
        compiler today which handles the kernel requirements and especially
        the annotations of functions to exclude them from KCSAN
        instrumentation correctly.
      
        These annotations really need to work so that low level entry code and
        especially int3 text poke handling can be completely isolated.
      
        A detailed discussion of the requirements and compiler issues can be
        found here:
      
          https://lore.kernel.org/lkml/CANpmjNMTsY_8241bS7=XAfqvZHFLrVEkv_uM4aDUWE_kh3Rvbw@mail.gmail.com/
      
        We came to the conclusion that trying to work around compiler
        limitations and bugs again would end up in a major trainwreck, so
        requiring a working compiler seemed to be the best choice.
      
        For Continous Integration purposes the compiler restriction is
        manageable and that's where most xxSAN reports come from.
      
        For a change this limitation might make GCC people actually look at
        their bugs. Some issues with CSAN in GCC are 7 years old and one has
        been 'fixed' 3 years ago with a half baken solution which 'solved' the
        reported issue but not the underlying problem.
      
        The KCSAN developers also ponder to use a GCC plugin to become
        independent, but that's not something which will show up in a few
        days.
      
        Blocking KCSAN until wide spread compiler support is available is not
        a really good alternative because the continuous growth of lockless
        optimizations in the kernel demands proper tooling support"
      
      * tag 'locking-kcsan-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: (76 commits)
        compiler_types.h, kasan: Use __SANITIZE_ADDRESS__ instead of CONFIG_KASAN to decide inlining
        compiler.h: Move function attributes to compiler_types.h
        compiler.h: Avoid nested statement expression in data_race()
        compiler.h: Remove data_race() and unnecessary checks from {READ,WRITE}_ONCE()
        kcsan: Update Documentation to change supported compilers
        kcsan: Remove 'noinline' from __no_kcsan_or_inline
        kcsan: Pass option tsan-instrument-read-before-write to Clang
        kcsan: Support distinguishing volatile accesses
        kcsan: Restrict supported compilers
        kcsan: Avoid inserting __tsan_func_entry/exit if possible
        ubsan, kcsan: Don't combine sanitizer with kcov on clang
        objtool, kcsan: Add kcsan_disable_current() and kcsan_enable_current_nowarn()
        kcsan: Add __kcsan_{enable,disable}_current() variants
        checkpatch: Warn about data_race() without comment
        kcsan: Use GFP_ATOMIC under spin lock
        Improve KCSAN documentation a bit
        kcsan: Make reporting aware of KCSAN tests
        kcsan: Fix function matching in report
        kcsan: Change data_race() to no longer require marking racing accesses
        kcsan: Move kcsan_{disable,enable}_current() to kcsan-checks.h
        ...
      b791d1bd
    • Linus Torvalds's avatar
      Merge tag 'locking-urgent-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 9716e57a
      Linus Torvalds authored
      Pull atomics rework from Thomas Gleixner:
       "Peter Zijlstras rework of atomics and fallbacks. This solves two
        problems:
      
         1) Compilers uninline small atomic_* static inline functions which
            can expose them to instrumentation.
      
         2) The instrumentation of atomic primitives was done at the
            architecture level while composites or fallbacks were provided at
            the generic level. As a result there are no uninstrumented
            variants of the fallbacks.
      
        Both issues were in the way of fully isolating fragile entry code
        pathes and especially the text poke int3 handler which is prone to an
        endless recursion problem when anything in that code path is about to
        be instrumented. This was always a problem, but got elevated due to
        the new batch mode updates of tracing.
      
        The solution is to mark the functions __always_inline and to flip the
        fallback and instrumentation so the non-instrumented variants are at
        the architecture level and the instrumentation is done in generic
        code.
      
        The latter introduces another fallback variant which will go away once
        all architectures have been moved over to arch_atomic_*"
      
      * tag 'locking-urgent-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        locking/atomics: Flip fallbacks and instrumentation
        asm-generic/atomic: Use __always_inline for fallback wrappers
      9716e57a
    • Linus Torvalds's avatar
      Merge branch 'akpm' (patches from Andrew) · b1a62749
      Linus Torvalds authored
      Pull updates from Andrew Morton:
       "A few fixes and stragglers.
      
        Subsystems affected by this patch series: mm/memory-failure, ocfs2,
        lib/lzo, misc"
      
      * emailed patches from Andrew Morton <akpm@linux-foundation.org>:
        amdgpu: a NULL ->mm does not mean a thread is a kthread
        lib/lzo: fix ambiguous encoding bug in lzo-rle
        ocfs2: fix build failure when TCP/IP is disabled
        mm/memory-failure: send SIGBUS(BUS_MCEERR_AR) only to current thread
        mm/memory-failure: prioritize prctl(PR_MCE_KILL) over vm.memory_failure_early_kill
      b1a62749
    • Christoph Hellwig's avatar
      amdgpu: a NULL ->mm does not mean a thread is a kthread · 8449d150
      Christoph Hellwig authored
      Use the proper API instead.
      
      Fixes: 70539bd7 ("drm/amd: Update MEC HQD loading code for KFD")
      Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      Tested-by: default avatarJens Axboe <axboe@kernel.dk>
      Reviewed-by: default avatarFelix Kuehling <Felix.Kuehling@amd.com>
      Reviewed-by: default avatarJens Axboe <axboe@kernel.dk>
      Cc: Al Viro <viro@zeniv.linux.org.uk>
      Cc: Alex Deucher <alexander.deucher@amd.com>
      Cc: Zhenyu Wang <zhenyuw@linux.intel.com>
      Cc: Zhi Wang <zhi.a.wang@intel.com>
      Cc: Felipe Balbi <balbi@kernel.org>
      Cc: "Michael S. Tsirkin" <mst@redhat.com>
      Cc: Jason Wang <jasowang@redhat.com>
      Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
      Link: http://lkml.kernel.org/r/20200404094101.672954-1-hch@lst.de
      Link: http://lkml.kernel.org/r/20200404094101.672954-2-hch@lst.deSigned-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      8449d150
    • Dave Rodgman's avatar
      lib/lzo: fix ambiguous encoding bug in lzo-rle · b5265c81
      Dave Rodgman authored
      In some rare cases, for input data over 32 KB, lzo-rle could encode two
      different inputs to the same compressed representation, so that
      decompression is then ambiguous (i.e.  data may be corrupted - although
      zram is not affected because it operates over 4 KB pages).
      
      This modifies the compressor without changing the decompressor or the
      bitstream format, such that:
      
       - there is no change to how data produced by the old compressor is
         decompressed
      
       - an old decompressor will correctly decode data from the updated
         compressor
      
       - performance and compression ratio are not affected
      
       - we avoid introducing a new bitstream format
      
      In testing over 12.8M real-world files totalling 903 GB, three files
      were affected by this bug.  I also constructed 37M semi-random 64 KB
      files totalling 2.27 TB, and saw no affected files.  Finally I tested
      over files constructed to contain each of the ~1024 possible bad input
      sequences; for all of these cases, updated lzo-rle worked correctly.
      
      There is no significant impact to performance or compression ratio.
      Signed-off-by: default avatarDave Rodgman <dave.rodgman@arm.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      Cc: Mark Rutland <mark.rutland@arm.com>
      Cc: Dave Rodgman <dave.rodgman@arm.com>
      Cc: Willy Tarreau <w@1wt.eu>
      Cc: Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>
      Cc: Markus F.X.J. Oberhumer <markus@oberhumer.com>
      Cc: Minchan Kim <minchan@kernel.org>
      Cc: Nitin Gupta <ngupta@vflare.org>
      Cc: Chao Yu <yuchao0@huawei.com>
      Cc: <stable@vger.kernel.org>
      Link: http://lkml.kernel.org/r/20200507100203.29785-1-dave.rodgman@arm.comSigned-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      b5265c81
    • Tom Seewald's avatar
      ocfs2: fix build failure when TCP/IP is disabled · fce1affe
      Tom Seewald authored
      After commit 12abc5ee ("tcp: add tcp_sock_set_nodelay") and commit
      c488aead ("tcp: add tcp_sock_set_user_timeout"), building the kernel
      with OCFS2_FS=y but without INET=y causes it to fail with:
      
        ld: fs/ocfs2/cluster/tcp.o: in function `o2net_accept_many':
        tcp.c:(.text+0x21b1): undefined reference to `tcp_sock_set_nodelay'
        ld: tcp.c:(.text+0x21c1): undefined reference to `tcp_sock_set_user_timeout'
        ld: fs/ocfs2/cluster/tcp.o: in function `o2net_start_connect':
        tcp.c:(.text+0x2633): undefined reference to `tcp_sock_set_nodelay'
        ld: tcp.c:(.text+0x2643): undefined reference to `tcp_sock_set_user_timeout'
      
      This is due to tcp_sock_set_nodelay() and tcp_sock_set_user_timeout()
      being declared in linux/tcp.h and defined in net/ipv4/tcp.c, which
      depend on TCP/IP being enabled.
      
      To fix this, make OCFS2_FS depend on INET=y which already requires
      NET=y.
      
      Fixes: 12abc5ee ("tcp: add tcp_sock_set_nodelay")
      Fixes: c488aead ("tcp: add tcp_sock_set_user_timeout")
      Signed-off-by: default avatarTom Seewald <tseewald@gmail.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      Reviewed-by: default avatarJoseph Qi <joseph.qi@linux.alibaba.com>
      Acked-by: default avatarChristoph Hellwig <hch@lst.de>
      Cc: Sagi Grimberg <sagi@grimberg.me>
      Cc: Jason Gunthorpe <jgg@mellanox.com>
      Cc: David S. Miller <davem@davemloft.net>
      Cc: Mark Fasheh <mark@fasheh.com>
      Cc: Joel Becker <jlbec@evilplan.org>
      Cc: Junxiao Bi <junxiao.bi@oracle.com>
      Cc: Changwei Ge <gechangwei@live.cn>
      Cc: Gang He <ghe@suse.com>
      Cc: Jun Piao <piaojun@huawei.com>
      Link: http://lkml.kernel.org/r/20200606190827.23954-1-tseewald@gmail.comSigned-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      fce1affe
    • Naoya Horiguchi's avatar
      mm/memory-failure: send SIGBUS(BUS_MCEERR_AR) only to current thread · 03151c6e
      Naoya Horiguchi authored
      Action Required memory error should happen only when a processor is
      about to access to a corrupted memory, so it's synchronous and only
      affects current process/thread.
      
      Recently commit 872e9a20 ("mm, memory_failure: don't send
      BUS_MCEERR_AO for action required error") fixed the issue that Action
      Required memory could unnecessarily send SIGBUS to the processes which
      share the error memory.  But we still have another issue that we could
      send SIGBUS to a wrong thread.
      
      This is because collect_procs() and task_early_kill() fails to add the
      current process to "to-kill" list.  So this patch is suggesting to fix
      it.  With this fix, SIGBUS(BUS_MCEERR_AR) is never sent to non-current
      process/thread.
      Signed-off-by: default avatarNaoya Horiguchi <naoya.horiguchi@nec.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      Acked-by: default avatarTony Luck <tony.luck@intel.com>
      Acked-by: default avatarPankaj Gupta <pankaj.gupta.linux@gmail.com>
      Link: http://lkml.kernel.org/r/1591321039-22141-3-git-send-email-naoya.horiguchi@nec.comSigned-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      03151c6e
    • Naoya Horiguchi's avatar
      mm/memory-failure: prioritize prctl(PR_MCE_KILL) over vm.memory_failure_early_kill · 4e018b45
      Naoya Horiguchi authored
      Patch series "hwpoison: fixes signaling on memory error"
      
      This is a small patchset to solve issues in memory error handler to send
      SIGBUS to proper process/thread as expected in configuration.  Please
      see descriptions in individual patches for more details.
      
      This patch (of 2):
      
      Early-kill policy is controlled from two types of settings, one is
      per-process setting prctl(PR_MCE_KILL) and the other is system-wide
      setting vm.memory_failure_early_kill.  Users expect per-process setting
      to override system-wide setting as many other settings do, but
      early-kill setting doesn't work as such.
      
      For example, if a system configures vm.memory_failure_early_kill to 1
      (enabled), a process receives SIGBUS even if it's configured to
      explicitly disable PF_MCE_KILL by prctl().  That's not desirable for
      applications with their own policies.
      
      This patch is suggesting to change the priority of these two types of
      settings, by checking sysctl_memory_failure_early_kill only when a given
      process has the default kill policy.
      
      Note that this patch is solving a thread choice issue too.
      
      Originally, collect_procs() always chooses the main thread when
      vm.memory_failure_early_kill is 1, even if the process has a dedicated
      thread for memory error handling.  SIGBUS should be sent to the
      dedicated thread if early-kill is enabled via
      vm.memory_failure_early_kill as we are doing for PR_MCE_KILL_EARLY
      processes.
      Signed-off-by: default avatarNaoya Horiguchi <naoya.horiguchi@nec.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      Cc: Tony Luck <tony.luck@intel.com>
      Cc: Pankaj Gupta <pankaj.gupta.linux@gmail.com>
      Link: http://lkml.kernel.org/r/1591321039-22141-1-git-send-email-naoya.horiguchi@nec.com
      Link: http://lkml.kernel.org/r/1591321039-22141-2-git-send-email-naoya.horiguchi@nec.comSigned-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      4e018b45
  2. 11 Jun, 2020 25 commits
    • Linus Torvalds's avatar
      Merge tag 'io_uring-5.8-2020-06-11' of git://git.kernel.dk/linux-block · b961f8dc
      Linus Torvalds authored
      Pull io_uring fixes from Jens Axboe:
       "A few late stragglers in here. In particular:
      
         - Validate full range for provided buffers (Bijan)
      
         - Fix bad use of kfree() in buffer registration failure (Denis)
      
         - Don't allow close of ring itself, it's not fully safe. Making it
           fully safe would require making the system call more expensive,
           which isn't worth it.
      
         - Buffer selection fix
      
         - Regression fix for O_NONBLOCK retry
      
         - Make IORING_OP_ACCEPT honor O_NONBLOCK (Jiufei)
      
         - Restrict opcode handling for SQ/IOPOLL (Pavel)
      
         - io-wq work handling cleanups and improvements (Pavel, Xiaoguang)
      
         - IOPOLL race fix (Xiaoguang)"
      
      * tag 'io_uring-5.8-2020-06-11' of git://git.kernel.dk/linux-block:
        io_uring: fix io_kiocb.flags modification race in IOPOLL mode
        io_uring: check file O_NONBLOCK state for accept
        io_uring: avoid unnecessary io_wq_work copy for fast poll feature
        io_uring: avoid whole io_wq_work copy for requests completed inline
        io_uring: allow O_NONBLOCK async retry
        io_wq: add per-wq work handler instead of per work
        io_uring: don't arm a timeout through work.func
        io_uring: remove custom ->func handlers
        io_uring: don't derive close state from ->func
        io_uring: use kvfree() in io_sqe_buffer_register()
        io_uring: validate the full range of provided buffers for access
        io_uring: re-set iov base/len for buffer select retry
        io_uring: move send/recv IOPOLL check into prep
        io_uring: deduplicate io_openat{,2}_prep()
        io_uring: do build_open_how() only once
        io_uring: fix {SQ,IO}POLL with unsupported opcodes
        io_uring: disallow close of ring itself
      b961f8dc
    • Linus Torvalds's avatar
      Merge tag 'block-5.8-2020-06-11' of git://git.kernel.dk/linux-block · a58dfea2
      Linus Torvalds authored
      Pull block fixes from Jens Axboe:
       "Some followup fixes for this merge window. In particular:
      
         - Seqcount write missing preemption disable for stats (Ahmed)
      
         - blktrace fixes (Chaitanya)
      
         - Redundant initializations (Colin)
      
         - Various small NVMe fixes (Chaitanya, Christoph, Daniel, Max,
           Niklas, Rikard)
      
         - loop flag bug regression fix (Martijn)
      
         - blk-mq tagging fixes (Christoph, Ming)"
      
      * tag 'block-5.8-2020-06-11' of git://git.kernel.dk/linux-block:
        umem: remove redundant initialization of variable ret
        pktcdvd: remove redundant initialization of variable ret
        nvmet: fail outstanding host posted AEN req
        nvme-pci: use simple suspend when a HMB is enabled
        nvme-fc: don't call nvme_cleanup_cmd() for AENs
        nvmet-tcp: constify nvmet_tcp_ops
        nvme-tcp: constify nvme_tcp_mq_ops and nvme_tcp_admin_mq_ops
        nvme: do not call del_gendisk() on a disk that was never added
        blk-mq: fix blk_mq_all_tag_iter
        blk-mq: split out a __blk_mq_get_driver_tag helper
        blktrace: fix endianness for blk_log_remap()
        blktrace: fix endianness in get_pdu_int()
        blktrace: use errno instead of bi_status
        block: nr_sects_write(): Disable preemption on seqcount write
        block: remove the error argument to the block_bio_complete tracepoint
        loop: Fix wrong masking of status flags
        block/bio-integrity: don't free 'buf' if bio_integrity_add_page() failed
      a58dfea2
    • David Howells's avatar
      afs: Fix afs_store_data() to set mtime in new operation descriptor · b3597945
      David Howells authored
      Fix afs_store_data() so that it sets the mtime in the new operation
      descriptor otherwise the mtime on the server gets set to 0 when a write is
      stored to the server.
      
      Fixes: e49c7b2f ("afs: Build an abstraction around an "operation" concept")
      Reported-by: default avatarDave Botsch <botsch@cnf.cornell.edu>
      Signed-off-by: default avatarDavid Howells <dhowells@redhat.com>
      Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      b3597945
    • Linus Torvalds's avatar
      Merge tag 'x86-urgent-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 6a45a658
      Linus Torvalds authored
      Pull more x86 updates from Thomas Gleixner:
       "A set of fixes and updates for x86:
      
         - Unbreak paravirt VDSO clocks.
      
           While the VDSO code was moved into lib for sharing a subtle check
           for the validity of paravirt clocks got replaced. While the
           replacement works perfectly fine for bare metal as the update of
           the VDSO clock mode is synchronous, it fails for paravirt clocks
           because the hypervisor can invalidate them asynchronously.
      
           Bring it back as an optional function so it does not inflict this
           on architectures which are free of PV damage.
      
         - Fix the jiffies to jiffies64 mapping on 64bit so it does not
           trigger an ODR violation on newer compilers
      
         - Three fixes for the SSBD and *IB* speculation mitigation maze to
           ensure consistency, not disabling of some *IB* variants wrongly and
           to prevent a rogue cross process shutdown of SSBD. All marked for
           stable.
      
         - Add yet more CPU models to the splitlock detection capable list
           !@#%$!
      
         - Bring the pr_info() back which tells that TSC deadline timer is
           enabled.
      
         - Reboot quirk for MacBook6,1"
      
      * tag 'x86-urgent-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        x86/vdso: Unbreak paravirt VDSO clocks
        lib/vdso: Provide sanity check for cycles (again)
        clocksource: Remove obsolete ifdef
        x86_64: Fix jiffies ODR violation
        x86/speculation: PR_SPEC_FORCE_DISABLE enforcement for indirect branches.
        x86/speculation: Prevent rogue cross-process SSBD shutdown
        x86/speculation: Avoid force-disabling IBPB based on STIBP and enhanced IBRS.
        x86/cpu: Add Sapphire Rapids CPU model number
        x86/split_lock: Add Icelake microserver and Tigerlake CPU models
        x86/apic: Make TSC deadline timer detection message visible
        x86/reboot/quirks: Add MacBook6,1 reboot quirk
      6a45a658
    • Linus Torvalds's avatar
      Merge tag 'timers-urgent-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip · 92ac9712
      Linus Torvalds authored
      Pull timer fix from Thomas Gleixner:
       "A small fix for the VDSO code to force inline
        __cvdso_clock_gettime_common() so the compiler
        can't generate horrible code"
      
      * tag 'timers-urgent-2020-06-11' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
        lib/vdso: Force inlining of __cvdso_clock_gettime_common()
      92ac9712
    • Linus Torvalds's avatar
      Merge branch 'akpm' (patches from Andrew) · 623f6dc5
      Linus Torvalds authored
      Merge some more updates from Andrew Morton:
      
       - various hotfixes and minor things
      
       - hch's use_mm/unuse_mm clearnups
      
      Subsystems affected by this patch series: mm/hugetlb, scripts, kcov,
      lib, nilfs, checkpatch, lib, mm/debug, ocfs2, lib, misc.
      
      * emailed patches from Andrew Morton <akpm@linux-foundation.org>:
        kernel: set USER_DS in kthread_use_mm
        kernel: better document the use_mm/unuse_mm API contract
        kernel: move use_mm/unuse_mm to kthread.c
        kernel: move use_mm/unuse_mm to kthread.c
        stacktrace: cleanup inconsistent variable type
        lib: test get_count_order/long in test_bitops.c
        mm: add comments on pglist_data zones
        ocfs2: fix spelling mistake and grammar
        mm/debug_vm_pgtable: fix kernel crash by checking for THP support
        lib: fix bitmap_parse() on 64-bit big endian archs
        checkpatch: correct check for kernel parameters doc
        nilfs2: fix null pointer dereference at nilfs_segctor_do_construct()
        lib/lz4/lz4_decompress.c: document deliberate use of `&'
        kcov: check kcov_softirq in kcov_remote_stop()
        scripts/spelling: add a few more typos
        khugepaged: selftests: fix timeout condition in wait_for_scan()
      623f6dc5
    • Rob Herring's avatar
      dt-bindings: Fix more incorrect 'reg' property sizes in examples · 0db958b6
      Rob Herring authored
      The examples template is a 'simple-bus' with a size of 1 cell for
      had between 2 and 4 cells which really only errors on I2C or SPI type
      devices with a single cell.
      
      The easiest fix in most cases is to change the 'reg' property to 1 cell
      for address and size.
      
      Cc: "Heiko Stübner" <heiko@sntech.de>
      Cc: Ezequiel Garcia <ezequiel@collabora.com>
      Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
      Cc: Philipp Zabel <p.zabel@pengutronix.de>
      Cc: Miquel Raynal <miquel.raynal@bootlin.com>
      Cc: Richard Weinberger <richard@nod.at>
      Cc: "David S. Miller" <davem@davemloft.net>
      Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
      Cc: Kishon Vijay Abraham I <kishon@ti.com>
      Cc: Vinod Koul <vkoul@kernel.org>
      Cc: Liam Girdwood <lgirdwood@gmail.com>
      Cc: linux-rockchip@lists.infradead.org
      Cc: linux-media@vger.kernel.org
      Cc: linux-mtd@lists.infradead.org
      Cc: netdev@vger.kernel.org
      Cc: alsa-devel@alsa-project.org
      Acked-by: default avatarMark Brown <broonie@kernel.org>
      Signed-off-by: default avatarRob Herring <robh@kernel.org>
      0db958b6
    • Joerg Roedel's avatar
      alpha: Fix build around srm_sysrq_reboot_op · 5cd221e8
      Joerg Roedel authored
      The patch introducing the struct was probably never compile tested,
      because it sets a handler with a wrong function signature. Wrap the
      handler into a functions with the correct signature to fix the build.
      
      Fixes: 0f1c9688 ("tty/sysrq: alpha: export and use __sysrq_get_key_op()")
      Cc: Emil Velikov <emil.l.velikov@gmail.com>
      Signed-off-by: default avatarJoerg Roedel <jroedel@suse.de>
      Reviewed-by: default avatarGuenter Roeck <linux@roeck-us.net>
      Tested-by: default avatarGuenter Roeck <linux@roeck-us.net>
      Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      5cd221e8
    • Linus Torvalds's avatar
      Merge tag 'riscv-for-linus-5.8-mw1' of git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux · cd16ed33
      Linus Torvalds authored
      Pull more RISC-V updates from Palmer Dabbelt:
      
       - Kconfig select statements are now sorted alphanumerically
      
       - first-level interrupts are now handled via a full irqchip driver
      
       - CPU hotplug is fixed
      
       - vDSO calls now use the common vDSO infrastructure
      
      * tag 'riscv-for-linus-5.8-mw1' of git://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux:
        riscv: set the permission of vdso_data to read-only
        riscv: use vDSO common flow to reduce the latency of the time-related functions
        riscv: fix build warning of missing prototypes
        RISC-V: Don't mark init section as non-executable
        RISC-V: Force select RISCV_INTC for CONFIG_RISCV
        RISC-V: Remove do_IRQ() function
        clocksource/drivers/timer-riscv: Use per-CPU timer interrupt
        irqchip: RISC-V per-HART local interrupt controller driver
        RISC-V: Rename and move plic_find_hart_id() to arch directory
        RISC-V: self-contained IPI handling routine
        RISC-V: Sort select statements alphanumerically
      cd16ed33
    • Linus Torvalds's avatar
      Merge tag 'arm64-upstream' of git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux · 55d728b2
      Linus Torvalds authored
      Pull arm64 fixes from Will Deacon:
       "arm64 fixes that came in during the merge window.
      
        There will probably be more to come, but it doesn't seem like it's
        worth me sitting on these in the meantime.
      
         - Fix SCS debug check to report max stack usage in bytes as advertised
      
         - Fix typo: CONFIG_FTRACE_WITH_REGS => CONFIG_DYNAMIC_FTRACE_WITH_REGS
      
         - Fix incorrect mask in HiSilicon L3C perf PMU driver
      
         - Fix compat vDSO compilation under some toolchain configurations
      
         - Fix false UBSAN warning from ACPI IORT parsing code
      
         - Fix booting under bootloaders that ignore TEXT_OFFSET
      
         - Annotate debug initcall function with '__init'"
      
      * tag 'arm64-upstream' of git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux:
        arm64: warn on incorrect placement of the kernel by the bootloader
        arm64: acpi: fix UBSAN warning
        arm64: vdso32: add CONFIG_THUMB2_COMPAT_VDSO
        drivers/perf: hisi: Fix wrong value for all counters enable
        arm64: ftrace: Change CONFIG_FTRACE_WITH_REGS to CONFIG_DYNAMIC_FTRACE_WITH_REGS
        arm64: debug: mark a function as __init to save some memory
        scs: Report SCS usage in bytes rather than number of entries
      55d728b2
    • Linus Torvalds's avatar
      Merge tag 'm68knommu-for-v5.8' of git://git.kernel.org/pub/scm/linux/kernel/git/gerg/m68knommu · d3ea6934
      Linus Torvalds authored
      Pull m68knommu updates from Greg Ungerer:
      
       - casting clean up in the user access macros
      
       - memory leak on error case fix for PCI probing
      
       - update of a defconfig
      
      * tag 'm68knommu-for-v5.8' of git://git.kernel.org/pub/scm/linux/kernel/git/gerg/m68knommu:
        m68k,nommu: fix implicit cast from __user in __{get,put}_user_asm()
        m68k,nommu: add missing __user in uaccess' __ptr() macro
        m68k: Drop CONFIG_MTD_M25P80 in stmark2_defconfig
        m68k/PCI: Fix a memory leak in an error handling path
      d3ea6934
    • Rob Herring's avatar
      dt-bindings: phy: qcom: Fix missing 'ranges' and example addresses · c0eca14c
      Rob Herring authored
      The QCom QMP PHY bindings have child nodes with translatable (MMIO)
      addresses, so a 'ranges' property is required in the parent node.
      Additionally, the examples default to 1 address and size cell, so let's
      fix that, too.
      
      Fixes: ccf51c1c ("dt-bindings: phy: qcom,qmp: Convert QMP PHY bindings to yaml")
      Cc: Andy Gross <agross@kernel.org>
      Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
      Cc: Kishon Vijay Abraham I <kishon@ti.com>
      Cc: Vinod Koul <vkoul@kernel.org>
      Cc: Manu Gautam <mgautam@codeaurora.org>
      Cc: linux-arm-msm@vger.kernel.org
      Signed-off-by: default avatarRob Herring <robh@kernel.org>
      c0eca14c
    • Rob Herring's avatar
      dt-bindings: Remove more cases of 'allOf' containing a '$ref' · 086e9074
      Rob Herring authored
      Another round of 'allOf' removals that came in this cycle.
      
      json-schema versions draft7 and earlier have a weird behavior in that
      any keywords combined with a '$ref' are ignored (silently). The correct
      form was to put a '$ref' under an 'allOf'. This behavior is now changed
      in the 2019-09 json-schema spec and '$ref' can be mixed with other
      keywords. The json-schema library doesn't yet support this, but the
      tooling now does a fixup for this and either way works.
      
      This has been a constant source of review comments, so let's change this
      treewide so everyone copies the simpler syntax.
      Signed-off-by: default avatarRob Herring <robh@kernel.org>
      086e9074
    • Linus Torvalds's avatar
      Merge tag 'mailbox-v5.8' of git://git.linaro.org/landing-teams/working/fujitsu/integration · 2dca74a4
      Linus Torvalds authored
      Pull mailbox updates from Jassi Brar:
       "qcom:
         - new controller driver for IPCC
         - reorg the of_device data
         - add support for ipq6018 platform
      
        spreadtrum:
         - new sprd controller driver
      
        imx:
         - implement suspend/resume PM support
      
        misc:
         - make pcc driver struct static
         - fix return value in imx_mu_scu
         - disable clock before bailout in imx probe
         - remove duplicate error mssg in zynqmp probe
         - fix header size in imx.scu
         - check for null instead of is-err in zynqmp"
      
      * tag 'mailbox-v5.8' of git://git.linaro.org/landing-teams/working/fujitsu/integration:
        mailbox: qcom: Add ipq6018 apcs compatible
        mailbox: qcom: Add clock driver name in apcs mailbox driver data
        dt-bindings: mailbox: Add YAML schemas for QCOM APCS global block
        mailbox: imx: ONLY IPC MU needs IRQF_NO_SUSPEND flag
        mailbox: imx: Add runtime PM callback to handle MU clocks
        mailbox: imx: Add context save/restore for suspend/resume
        MAINTAINERS: Add entry for Qualcomm IPCC driver
        mailbox: Add support for Qualcomm IPCC
        dt-bindings: mailbox: Add devicetree binding for Qcom IPCC
        mailbox: zynqmp-ipi: Fix NULL vs IS_ERR() check in zynqmp_ipi_mbox_probe()
        mailbox: imx-mailbox: fix scu msg header size check
        mailbox: sprd: Add Spreadtrum mailbox driver
        dt-bindings: mailbox: Add the Spreadtrum mailbox documentation
        mailbox: ZynqMP IPI: Delete an error message in zynqmp_ipi_probe()
        mailbox: imx: Disable the clock on devm_mbox_controller_register() failure
        mailbox: imx: Fix return in imx_mu_scu_xlate()
        mailbox: imx: Support runtime PM
        mailbox: pcc: make pcc_mbox_driver static
      2dca74a4
    • Linus Torvalds's avatar
      Merge tag 'sound-fix-5.8-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound · e0154bd4
      Linus Torvalds authored
      Pull sound fixes from Takashi Iwai:
       "Here are last-minute fixes gathered before merge window close; a few
        fixes are for the core while the rest majority are driver fixes.
      
         - PCM locking annotation fixes and the possible self-lock fix
      
         - ASoC DPCM regression fixes with multi-CPU DAI
      
         - A fix for inconsistent resume from system-PM on USB-audio
      
         - Improved runtime-PM handling with multiple USB interfaces
      
         - Quirks for HD-audio and USB-audio
      
         - Hardened firmware handling in max98390 codec
      
         - A couple of fixes for meson"
      
      * tag 'sound-fix-5.8-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound: (21 commits)
        ASoC: rt5645: Add platform-data for Asus T101HA
        ASoC: Intel: bytcr_rt5640: Add quirk for Toshiba Encore WT10-A tablet
        ASoC: SOF: nocodec: conditionally set dpcm_capture/dpcm_playback flags
        ASoC: Intel: boards: replace capture_only by dpcm_capture
        ASoC: core: only convert non DPCM link to DPCM link
        ASoC: soc-pcm: dpcm: fix playback/capture checks
        ASoC: meson: add missing free_irq() in error path
        ALSA: pcm: disallow linking stream to itself
        ALSA: usb-audio: Manage auto-pm of all bundled interfaces
        ALSA: hda/realtek - add a pintbl quirk for several Lenovo machines
        ALSA: pcm: fix snd_pcm_link() lockdep splat
        ALSA: usb-audio: Use the new macro for HP Dock rename quirks
        ALSA: usb-audio: Add vendor, product and profile name for HP Thunderbolt Dock
        ALSA: emu10k1: delete an unnecessary condition
        dt-bindings: ASoc: Fix tdm-slot documentation spelling error
        ASoC: meson: fix memory leak of links if allocation of ldata fails
        ALSA: usb-audio: Fix inconsistent card PM state after resume
        ASoC: max98390: Fix potential crash during param fw loading
        ASoC: max98390: Fix incorrect printf qualifier
        ASoC: fsl-asoc-card: Defer probe when fail to find codec device
        ...
      e0154bd4
    • Linus Torvalds's avatar
      Merge tag 'drm-next-2020-06-11-1' of git://anongit.freedesktop.org/drm/drm · d4e181f2
      Linus Torvalds authored
      Pull drm fixes from Dave Airlie:
       "One sun4i fix and a connector hotplug race The ast fix is for a
        regression in 5.6, and one of the i915 ones fixes an oops reported by
        dhowells.
      
        core:
         - fix race in connectors sending hotplug
      
        i915:
         - Avoid use after free in cmdparser
         - Avoid NULL dereference when probing all display encoders
         - Fixup to module parameter type
      
        sun4i:
         - clock divider fix
      
        ast:
         - 24/32 bpp mode setting fix"
      
      * tag 'drm-next-2020-06-11-1' of git://anongit.freedesktop.org/drm/drm:
        drm/ast: fix missing break in switch statement for format->cpp[0] case 4
        drm/sun4i: hdmi ddc clk: Fix size of m divider
        drm/i915/display: Only query DP state of a DDI encoder
        drm/i915/params: fix i915.reset module param type
        drm/i915/gem: Mark the buffer pool as active for the cmdparser
        drm/connector: notify userspace on hotplug after register complete
      d4e181f2
    • Linus Torvalds's avatar
      Merge tag 'nfs-for-5.8-1' of git://git.linux-nfs.org/projects/anna/linux-nfs · a5395682
      Linus Torvalds authored
      Pull NFS client updates from Anna Schumaker:
       "New features and improvements:
         - Sunrpc receive buffer sizes only change when establishing a GSS credentials
         - Add more sunrpc tracepoints
         - Improve on tracepoints to capture internal NFS I/O errors
      
        Other bugfixes and cleanups:
         - Move a dprintk() to after a call to nfs_alloc_fattr()
         - Fix off-by-one issues in rpc_ntop6
         - Fix a few coccicheck warnings
         - Use the correct SPDX license identifiers
         - Fix rpc_call_done assignment for BIND_CONN_TO_SESSION
         - Replace zero-length array with flexible array
         - Remove duplicate headers
         - Set invalid blocks after NFSv4 writes to update space_used attribute
         - Fix direct WRITE throughput regression"
      
      * tag 'nfs-for-5.8-1' of git://git.linux-nfs.org/projects/anna/linux-nfs: (27 commits)
        NFS: Fix direct WRITE throughput regression
        SUNRPC: rpc_xprt lifetime events should record xprt->state
        xprtrdma: Make xprt_rdma_slot_table_entries static
        nfs: set invalid blocks after NFSv4 writes
        NFS: remove redundant initialization of variable result
        sunrpc: add missing newline when printing parameter 'auth_hashtable_size' by sysfs
        NFS: Add a tracepoint in nfs_set_pgio_error()
        NFS: Trace short NFS READs
        NFS: nfs_xdr_status should record the procedure name
        SUNRPC: Set SOFTCONN when destroying GSS contexts
        SUNRPC: rpc_call_null_helper() should set RPC_TASK_SOFT
        SUNRPC: rpc_call_null_helper() already sets RPC_TASK_NULLCREDS
        SUNRPC: trace RPC client lifetime events
        SUNRPC: Trace transport lifetime events
        SUNRPC: Split the xdr_buf event class
        SUNRPC: Add tracepoint to rpc_call_rpcerror()
        SUNRPC: Update the RPC_SHOW_SOCKET() macro
        SUNRPC: Update the rpc_show_task_flags() macro
        SUNRPC: Trace GSS context lifetimes
        SUNRPC: receive buffer size estimation values almost never change
        ...
      a5395682
    • Marco Elver's avatar
      compiler_types.h, kasan: Use __SANITIZE_ADDRESS__ instead of CONFIG_KASAN to decide inlining · 1f44328e
      Marco Elver authored
      Use __always_inline in compilation units that have instrumentation
      disabled (KASAN_SANITIZE_foo.o := n) for KASAN, like it is done for
      KCSAN.
      
      Also, add common documentation for KASAN and KCSAN explaining the
      attribute.
      
       [ bp: Massage commit message. ]
      Signed-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Link: https://lkml.kernel.org/r/20200521142047.169334-12-elver@google.com
      
      
      1f44328e
    • Marco Elver's avatar
      compiler.h: Move function attributes to compiler_types.h · eb73876c
      Marco Elver authored
      Cleanup and move the KASAN and KCSAN related function attributes to
      compiler_types.h, where the rest of the same kind live.
      
      No functional change intended.
      Signed-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Link: https://lkml.kernel.org/r/20200521142047.169334-11-elver@google.com
      
      
      eb73876c
    • Marco Elver's avatar
      compiler.h: Avoid nested statement expression in data_race() · 95c094fc
      Marco Elver authored
      It appears that compilers have trouble with nested statement
      expressions. Therefore, remove one level of statement expression nesting
      from the data_race() macro. This will help avoiding potential problems
      in the future as its usage increases.
      Reported-by: default avatarBorislav Petkov <bp@suse.de>
      Reported-by: default avatarNathan Chancellor <natechancellor@gmail.com>
      Signed-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Tested-by: default avatarNick Desaulniers <ndesaulniers@google.com>
      Link: https://lkml.kernel.org/r/20200520221712.GA21166@zn.tnic
      Link: https://lkml.kernel.org/r/20200521142047.169334-10-elver@google.com
      
      
      95c094fc
    • Marco Elver's avatar
      compiler.h: Remove data_race() and unnecessary checks from {READ,WRITE}_ONCE() · 44b97dcc
      Marco Elver authored
      The volatile accesses no longer need to be wrapped in data_race()
      because compilers that emit instrumentation distinguishing volatile
      accesses are required for KCSAN.
      
      Consequently, the explicit kcsan_check_atomic*() are no longer required
      either since the compiler emits instrumentation distinguishing the
      volatile accesses.
      
      Finally, simplify __READ_ONCE_SCALAR() and remove __WRITE_ONCE_SCALAR().
      
       [ bp: Convert commit message to passive voice. ]
      Signed-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Link: https://lkml.kernel.org/r/20200521142047.169334-9-elver@google.com
      
      
      44b97dcc
    • Marco Elver's avatar
      kcsan: Update Documentation to change supported compilers · 0bb9ab3a
      Marco Elver authored
      Document change in required compiler version for KCSAN, and remove the
      now redundant note about __no_kcsan and inlining problems with older
      compilers.
      Signed-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Link: https://lkml.kernel.org/r/20200521142047.169334-8-elver@google.com
      
      
      0bb9ab3a
    • Marco Elver's avatar
      kcsan: Remove 'noinline' from __no_kcsan_or_inline · e3b779d9
      Marco Elver authored
      Some compilers incorrectly inline small __no_kcsan functions, which then
      results in instrumenting the accesses. For this reason, the 'noinline'
      attribute was added to __no_kcsan_or_inline. All known versions of GCC
      are affected by this. Supported versions of Clang are unaffected, and
      never inline a no_sanitize function.
      
      However, the attribute 'noinline' in __no_kcsan_or_inline causes
      unexpected code generation in functions that are __no_kcsan and call a
      __no_kcsan_or_inline function.
      
      In certain situations it is expected that the __no_kcsan_or_inline
      function is actually inlined by the __no_kcsan function, and *no* calls
      are emitted. By removing the 'noinline' attribute, give the compiler
      the ability to inline and generate the expected code in __no_kcsan
      functions.
      Signed-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Link: https://lkml.kernel.org/r/CANpmjNNOpJk0tprXKB_deiNAv_UmmORf1-2uajLhnLWQQ1hvoA@mail.gmail.com
      Link: https://lkml.kernel.org/r/20200521142047.169334-6-elver@google.com
      
      
      e3b779d9
    • Marco Elver's avatar
      kcsan: Pass option tsan-instrument-read-before-write to Clang · d31d4d6b
      Marco Elver authored
      Clang (unlike GCC) removes reads before writes with matching addresses
      in the same basic block. This is an optimization for TSAN, since writes
      will always cause conflict if the preceding read would have.
      
      However, for KCSAN we cannot rely on this option, because we apply
      several special rules to writes, in particular when the
      KCSAN_ASSUME_PLAIN_WRITES_ATOMIC option is selected. To avoid missing
      potential data races, pass the -tsan-instrument-read-before-write option
      to Clang if it is available [1].
      
      [1] https://github.com/llvm/llvm-project/commit/151ed6aa38a3ec6c01973b35f684586b6e1c0f7eSigned-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Link: https://lkml.kernel.org/r/20200521142047.169334-5-elver@google.com
      
      
      d31d4d6b
    • Marco Elver's avatar
      kcsan: Support distinguishing volatile accesses · 75d75b7a
      Marco Elver authored
      In the kernel, the "volatile" keyword is used in various concurrent
      contexts, whether in low-level synchronization primitives or for
      legacy reasons. If supported by the compiler, it will be assumed
      that aligned volatile accesses up to sizeof(long long) (matching
      compiletime_assert_rwonce_type()) are atomic.
      
      Recent versions of Clang [1] (GCC tentative [2]) can instrument
      volatile accesses differently. Add the option (required) to enable the
      instrumentation, and provide the necessary runtime functions. None of
      the updated compilers are widely available yet (Clang 11 will be the
      first release to support the feature).
      
      [1] https://github.com/llvm/llvm-project/commit/5a2c31116f412c3b6888be361137efd705e05814
      [2] https://gcc.gnu.org/pipermail/gcc-patches/2020-April/544452.html
      
      This change allows removing of any explicit checks in primitives such as
      READ_ONCE() and WRITE_ONCE().
      
       [ bp: Massage commit message a bit. ]
      Signed-off-by: default avatarMarco Elver <elver@google.com>
      Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
      Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
      Acked-by: default avatarWill Deacon <will@kernel.org>
      Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
      Link: https://lkml.kernel.org/r/20200521142047.169334-4-elver@google.com
      
      
      75d75b7a