Skip to content
  1. Nov 24, 2023
    • Namjae Jeon's avatar
      ksmbd: fix possible deadlock in smb2_open · 864fb5d3
      Namjae Jeon authored
      [ 8743.393379] ======================================================
      [ 8743.393385] WARNING: possible circular locking dependency detected
      [ 8743.393391] 6.4.0-rc1+ #11 Tainted: G           OE
      [ 8743.393397] ------------------------------------------------------
      [ 8743.393402] kworker/0:2/12921 is trying to acquire lock:
      [ 8743.393408] ffff888127a14460 (sb_writers#8){.+.+}-{0:0}, at: ksmbd_vfs_setxattr+0x3d/0xd0 [ksmbd]
      [ 8743.393510]
                     but task is already holding lock:
      [ 8743.393515] ffff8880360d97f0 (&type->i_mutex_dir_key#6/1){+.+.}-{3:3}, at: ksmbd_vfs_kern_path_locked+0x181/0x670 [ksmbd]
      [ 8743.393618]
                     which lock already depends on the new lock.
      
      [ 8743.393623]
                     the existing dependency chain (in reverse order) is:
      [ 8743.393628]
                     -> #1 (&type->i_mutex_dir_key#6/1){+.+.}-{3:3}:
      [ 8743.393648]        down_write_nested+0x9a/0x1b0
      [ 8743.393660]        filename_create+0x128/0x270
      [ 8743.393670]        do_mkdirat+0xab/0x1f0
      [ 8743.393680]        __x64_sys_mkdir+0x47/0x60
      [ 8743.393690]        do_syscall_64+0x5d/0x90
      [ 8743.393701]        entry_SYSCALL_64_after_hwframe+0x72/0xdc
      [ 8743.393711]
                     -> #0 (sb_writers#8){.+.+}-{0:0}:
      [ 8743.393728]        __lock_acquire+0x2201/0x3b80
      [ 8743.393737]        lock_acquire+0x18f/0x440
      [ 8743.393746]        mnt_want_write+0x5f/0x240
      [ 8743.393755]        ksmbd_vfs_setxattr+0x3d/0xd0 [ksmbd]
      [ 8743.393839]        ksmbd_vfs_set_dos_attrib_xattr+0xcc/0x110 [ksmbd]
      [ 8743.393924]        compat_ksmbd_vfs_set_dos_attrib_xattr+0x39/0x50 [ksmbd]
      [ 8743.394010]        smb2_open+0x3432/0x3cc0 [ksmbd]
      [ 8743.394099]        handle_ksmbd_work+0x2c9/0x7b0 [ksmbd]
      [ 8743.394187]        process_one_work+0x65a/0xb30
      [ 8743.394198]        worker_thread+0x2cf/0x700
      [ 8743.394209]        kthread+0x1ad/0x1f0
      [ 8743.394218]        ret_from_fork+0x29/0x50
      
      This patch add mnt_want_write() above parent inode lock and remove
      nested mnt_want_write calls in smb2_open().
      
      Fixes: 40b268d3
      
       ("ksmbd: add mnt_want_write to ksmbd vfs functions")
      Cc: stable@vger.kernel.org
      Reported-by: default avatarMarios Makassikis <mmakassikis@freebox.fr>
      Signed-off-by: default avatarNamjae Jeon <linkinjeon@kernel.org>
      Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
      864fb5d3
    • Zongmin Zhou's avatar
      ksmbd: prevent memory leak on error return · 90044481
      Zongmin Zhou authored
      When allocated memory for 'new' failed,just return
      will cause memory leak of 'ar'.
      
      Fixes: 1819a904
      
       ("ksmbd: reorganize ksmbd_iov_pin_rsp()")
      Reported-by: default avatarkernel test robot <lkp@intel.com>
      Reported-by: default avatarDan Carpenter <error27@gmail.com>
      Closes: https://lore.kernel.org/r/202311031837.H3yo7JVl-lkp@intel.com/
      
      
      Signed-off-by: default avatarZongmin <Zhou&lt;zhouzongmin@kylinos.cn>
      Acked-by: default avatarNamjae Jeon <linkinjeon@kernel.org>
      Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
      90044481
  2. Nov 20, 2023
  3. Nov 19, 2023
    • Linus Torvalds's avatar
      Merge tag 'scsi-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi · 037266a5
      Linus Torvalds authored
      Pull SCSI fixes from James Bottomley:
       "Seven small fixes, six in drivers and one in sd.
      
        The sd fix is so large because it changes a struct pointer to a struct
        but otherwise is fairly simple"
      
      * tag 'scsi-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi:
        scsi: ufs: qcom-ufs: dt-bindings: Document the SM8650 UFS Controller
        scsi: sd: Fix sshdr use in sd_suspend_common()
        scsi: scsi_debug: Delete some bogus error checking
        scsi: scsi_debug: Fix some bugs in sdebug_error_write()
        scsi: ufs: core: Fix racing issue between ufshcd_mcq_abort() and ISR
        scsi: ufs: core: Expand MCQ queue slot to DeviceQueueDepth + 1
        scsi: qla2xxx: Fix system crash due to bad pointer access
      037266a5
    • Linus Torvalds's avatar
      Merge tag 'parisc-for-6.7-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux · 2254005e
      Linus Torvalds authored
      Pull parisc fixes from Helge Deller:
       "On parisc we still sometimes need writeable stacks, e.g. if programs
        aren't compiled with gcc-14. To avoid issues with the upcoming
        systemd-254 we therefore have to disable prctl(PR_SET_MDWE) for now
        (for parisc only).
      
        The other two patches are minor: a bugfix for the soft power-off on
        qemu with 64-bit kernel and prefer strscpy() over strlcpy():
      
         - Fix power soft-off on qemu
      
         - Disable prctl(PR_SET_MDWE) since parisc sometimes still needs
           writeable stacks
      
         - Use strscpy instead of strlcpy in show_cpuinfo()"
      
      * tag 'parisc-for-6.7-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux:
        prctl: Disable prctl(PR_SET_MDWE) on parisc
        parisc/power: Fix power soft-off when running on qemu
        parisc: Replace strlcpy() with strscpy()
      2254005e
    • Linus Torvalds's avatar
      Merge tag 'xfs-6.7-fixes-1' of git://git.kernel.org/pub/scm/fs/xfs/xfs-linux · b8f1fa24
      Linus Torvalds authored
      Pull xfs fixes from Chandan Babu:
      
       - Fix deadlock arising due to intent items in AIL not being cleared
         when log recovery fails
      
       - Fix stale data exposure bug when remapping COW fork extents to data
         fork
      
       - Fix deadlock when data device flush fails
      
       - Fix AGFL minimum size calculation
      
       - Select DEBUG_FS instead of XFS_DEBUG when XFS_ONLINE_SCRUB_STATS is
         selected
      
       - Fix corruption of log inode's extent count field when NREXT64 feature
         is enabled
      
      * tag 'xfs-6.7-fixes-1' of git://git.kernel.org/pub/scm/fs/xfs/xfs-linux:
        xfs: recovery should not clear di_flushiter unconditionally
        xfs: inode recovery does not validate the recovered inode
        xfs: fix again select in kconfig XFS_ONLINE_SCRUB_STATS
        xfs: fix internal error from AGFL exhaustion
        xfs: up(ic_sema) if flushing data device fails
        xfs: only remap the written blocks in xfs_reflink_end_cow_extent
        XFS: Update MAINTAINERS to catch all XFS documentation
        xfs: abort intent items when recovery intents fail
        xfs: factor out xfs_defer_pending_abort
      b8f1fa24
    • Linus Torvalds's avatar
      Merge tag 'nfsd-6.7-1' of git://git.kernel.org/pub/scm/linux/kernel/git/cel/linux · bb28378a
      Linus Torvalds authored
      Pull nfsd fixes from Chuck Lever:
      
       - Fix several long-standing bugs in the duplicate reply cache
      
       - Fix a memory leak
      
      * tag 'nfsd-6.7-1' of git://git.kernel.org/pub/scm/linux/kernel/git/cel/linux:
        NFSD: Fix checksum mismatches in the duplicate reply cache
        NFSD: Fix "start of NFS reply" pointer passed to nfsd_cache_update()
        NFSD: Update nfsd_cache_append() to use xdr_stream
        nfsd: fix file memleak on client_opens_release
      bb28378a
    • Linus Torvalds's avatar
      Merge tag '6.7-rc1-smb3-client-fixes' of git://git.samba.org/sfrench/cifs-2.6 · 33b63f15
      Linus Torvalds authored
      Pull smb client fixes from Steve French:
      
       - multichannel fixes (including a lock ordering fix and an important
         refcounting fix)
      
       - spnego fix
      
      * tag '6.7-rc1-smb3-client-fixes' of git://git.samba.org/sfrench/cifs-2.6:
        cifs: fix lock ordering while disabling multichannel
        cifs: fix leak of iface for primary channel
        cifs: fix check of rc in function generate_smb3signingkey
        cifs: spnego: add ';' in HOST_KEY_LEN
      33b63f15
    • Helge Deller's avatar
      prctl: Disable prctl(PR_SET_MDWE) on parisc · 79383813
      Helge Deller authored
      
      
      systemd-254 tries to use prctl(PR_SET_MDWE) for it's MemoryDenyWriteExecute
      functionality, but fails on parisc which still needs executable stacks in
      certain combinations of gcc/glibc/kernel.
      
      Disable prctl(PR_SET_MDWE) by returning -EINVAL for now on parisc, until
      userspace has catched up.
      
      Signed-off-by: default avatarHelge Deller <deller@gmx.de>
      Co-developed-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      Reported-by: default avatarSam James <sam@gentoo.org>
      Closes: https://github.com/systemd/systemd/issues/29775
      
      
      Tested-by: default avatarSam James <sam@gentoo.org>
      Link: https://lore.kernel.org/all/875y2jro9a.fsf@gentoo.org/
      Cc: <stable@vger.kernel.org> # v6.3+
      79383813
    • Linus Torvalds's avatar
      Merge tag 'for-6.7/dm-fixes' of... · 05aa69b0
      Linus Torvalds authored
      Merge tag 'for-6.7/dm-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm
      
      Pull device mapper fixes from Mike Snitzer:
      
       - Various fixes for the DM delay target to address regressions
         introduced during the 6.7 merge window
      
       - Fixes to both DM bufio and the verity target for no-sleep mode,
         to address sleeping while atomic issues
      
       - Update DM crypt target in response to the treewide change that
         made MAX_ORDER inclusive
      
      * tag 'for-6.7/dm-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm:
        dm-crypt: start allocating with MAX_ORDER
        dm-verity: don't use blocking calls from tasklets
        dm-bufio: fix no-sleep mode
        dm-delay: avoid duplicate logic
        dm-delay: fix bugs introduced by kthread mode
        dm-delay: fix a race between delay_presuspend and delay_bio
      05aa69b0
    • Helge Deller's avatar
      parisc/power: Fix power soft-off when running on qemu · 6ad6e15a
      Helge Deller authored
      Firmware returns the physical address of the power switch,
      so need to use gsc_writel() instead of direct memory access.
      
      Fixes: d0c21947
      
       ("parisc/power: Add power soft-off when running on qemu")
      Signed-off-by: default avatarHelge Deller <deller@gmx.de>
      Cc: stable@vger.kernel.org # v6.0+
      6ad6e15a
    • Kees Cook's avatar
      parisc: Replace strlcpy() with strscpy() · 721d28f3
      Kees Cook authored
      strlcpy() reads the entire source buffer first. This read may exceed
      the destination size limit. This is both inefficient and can lead
      to linear read overflows if a source string is not NUL-terminated[1].
      Additionally, it returns the size of the source string, not the
      resulting size of the destination string. In an effort to remove strlcpy()
      completely[2], replace strlcpy() here with strscpy().
      
      Link: https://www.kernel.org/doc/html/latest/process/deprecated.html#strlcpy [1]
      Link: https://github.com/KSPP/linux/issues/89
      
       [2]
      Cc: "James E.J. Bottomley" <James.Bottomley@HansenPartnership.com>
      Cc: Helge Deller <deller@gmx.de>
      Cc: Azeem Shaikh <azeemshaikh38@gmail.com>
      Cc: linux-parisc@vger.kernel.org
      Signed-off-by: default avatarKees Cook <keescook@chromium.org>
      Signed-off-by: default avatarHelge Deller <deller@gmx.de>
      721d28f3
    • Linus Torvalds's avatar
      Merge tag 'i2c-for-6.7-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux · 23dfa043
      Linus Torvalds authored
      Pull i2c fixes from Wolfram Sang:
       "Revert a not-working conversion to generic recovery for PXA,
        use proper IO accessors for designware, and use proper PM level
        for ocores to allow accessing interrupt providers late"
      
      * tag 'i2c-for-6.7-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux:
        i2c: ocores: Move system PM hooks to the NOIRQ phase
        i2c: designware: Fix corrupted memory seen in the ISR
        Revert "i2c: pxa: move to generic GPIO recovery"
      23dfa043
    • Linus Torvalds's avatar
      Merge tag 'turbostat-2023.11.07' of git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux · 9ea991a5
      Linus Torvalds authored
      Pull turbostat updates from Len Brown:
      
       - Turbostat features are now table-driven (Rui Zhang)
      
       - Add support for some new platforms (Sumeet Pawnikar, Rui Zhang)
      
       - Gracefully run in configs when CPUs are limited (Rui Zhang, Srinivas
         Pandruvada)
      
       - misc minor fixes
      
      [ This came in during the merge window, but sorting out the signed tag
        took a while, so thus the late merge   - Linus ]
      
      * tag 'turbostat-2023.11.07' of git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux: (86 commits)
        tools/power turbostat: version 2023.11.07
        tools/power/turbostat: bugfix "--show IPC"
        tools/power/turbostat: Add initial support for LunarLake
        tools/power/turbostat: Add initial support for ArrowLake
        tools/power/turbostat: Add initial support for GrandRidge
        tools/power/turbostat: Add initial support for SierraForest
        tools/power/turbostat: Add initial support for GraniteRapids
        tools/power/turbostat: Add MSR_CORE_C1_RES support for spr_features
        tools/power/turbostat: Move process to root cgroup
        tools/power/turbostat: Handle cgroup v2 cpu limitation
        tools/power/turbostat: Abstrct function for parsing cpu string
        tools/power/turbostat: Handle offlined CPUs in cpu_subset
        tools/power/turbostat: Obey allowed CPUs for system summary
        tools/power/turbostat: Obey allowed CPUs for primary thread/core detection
        tools/power/turbostat: Abstract several functions
        tools/power/turbostat: Obey allowed CPUs during startup
        tools/power/turbostat: Obey allowed CPUs when accessing CPU counters
        tools/power/turbostat: Introduce cpu_allowed_set
        tools/power/turbostat: Remove PC7/PC9 support on ADL/RPL
        tools/power/turbostat: Enable MSR_CORE_C1_RES on recent Intel client platforms
        ...
      9ea991a5
  4. Nov 18, 2023
    • Linus Torvalds's avatar
      Merge tag 'bcachefs-2023-11-17' of https://evilpiepirate.org/git/bcachefs · 791c8ab0
      Linus Torvalds authored
      Pull bcachefs fixes from Kent Overstreet:
       "Lots of small fixes for minor nits and compiler warnings.
      
        Bigger items:
      
         - The six locks lost wakeup is finally fixed: six_read_trylock() was
           checking for the waiting bit before decrementing the number of
           readers - validated the fix with a torture test.
      
         - Fix for a memory reclaim issue: when needing to reallocate a key
           cache key, we now do our usual GFP_NOWAIT; unlock(); GFP_KERNEL
           dance.
      
         - Multiple deleted inodes btree fixes
      
         - Fix an issue in fsck, where i_nlink would be recalculated
           incorrectly for hardlinked files if a snapshot had ever been taken.
      
         - Kill journal pre-reservations: This is a bigger patch than I would
           normally send at this point, but it deletes code and it fixes some
           of our tests that would sporadically die with the journal getting
           stuck, and it's a performance improvement, too"
      
      * tag 'bcachefs-2023-11-17' of https://evilpiepirate.org/git/bcachefs: (22 commits)
        bcachefs: Fix missing locking for dentry->d_parent access
        bcachefs: six locks: Fix lost wakeup
        bcachefs: Fix no_data_io mode checksum check
        bcachefs: Fix bch2_check_nlinks() for snapshots
        bcachefs: Don't decrease BTREE_ITER_MAX when LOCKDEP=y
        bcachefs: Disable debug log statements
        bcachefs: Fix missing transaction commit
        bcachefs: Fix error path in bch2_mount()
        bcachefs: Fix potential sleeping during mount
        bcachefs: Fix iterator leak in may_delete_deleted_inode()
        bcachefs: Kill journal pre-reservations
        bcachefs: Check for nonce offset inconsistency in data_update path
        bcachefs: Make sure to drop/retake btree locks before reclaim
        bcachefs: btree_trans->write_locked
        bcachefs: Run btree key cache shrinker less aggressively
        bcachefs: Split out btree_key_cache_types.h
        bcachefs: Guard against insufficient devices to create stripes
        bcachefs: Fix null ptr deref in bch2_backpointer_get_node()
        bcachefs: Fix multiple -Warray-bounds warnings
        bcachefs: Use DECLARE_FLEX_ARRAY() helper and fix multiple -Warray-bounds warnings
        ...
      791c8ab0
    • Linus Torvalds's avatar
      Merge tag 'mm-hotfixes-stable-2023-11-17-14-04' of... · 12ee72fe
      Linus Torvalds authored
      Merge tag 'mm-hotfixes-stable-2023-11-17-14-04' of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm
      
      Pull misc fixes from Andrew Morton:
       "Thirteen hotfixes. Seven are cc:stable and the remainder pertain to
        post-6.6 issues or aren't considered suitable for backporting"
      
      * tag 'mm-hotfixes-stable-2023-11-17-14-04' of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm:
        mm: more ptep_get() conversion
        parisc: fix mmap_base calculation when stack grows upwards
        mm/damon/core.c: avoid unintentional filtering out of schemes
        mm: kmem: drop __GFP_NOFAIL when allocating objcg vectors
        mm/damon/sysfs-schemes: handle tried region directory allocation failure
        mm/damon/sysfs-schemes: handle tried regions sysfs directory allocation failure
        mm/damon/sysfs: check error from damon_sysfs_update_target()
        mm: fix for negative counter: nr_file_hugepages
        selftests/mm: add hugetlb_fault_after_madv to .gitignore
        selftests/mm: restore number of hugepages
        selftests: mm: fix some build warnings
        selftests: mm: skip whole test instead of failure
        mm/damon/sysfs: eliminate potential uninitialized variable warning
      12ee72fe
    • Linus Torvalds's avatar
      Merge tag 'block-6.7-2023-11-17' of git://git.kernel.dk/linux · ffd75bc7
      Linus Torvalds authored
      Pull block fix from Jens Axboe:
       "Just a single fix from Christoph/Ming, fixing a case where integrity
        IO could be called without having an appropriate queue reference"
      
      * tag 'block-6.7-2023-11-17' of git://git.kernel.dk/linux:
        blk-mq: make sure active queue usage is held for bio_integrity_prep()
      ffd75bc7
    • Linus Torvalds's avatar
      Merge tag 'io_uring-6.7-2023-11-17' of git://git.kernel.dk/linux · 0e413c2a
      Linus Torvalds authored
      Pull io_uring fix from Jens Axboe:
       "Just a single fixup for a change we made in this release, which caused
        a regression in sometimes missing fdinfo output if the SQPOLL thread
        had the lock held when fdinfo output was retrieved.
      
        This brings us back on par with what we had before, where just the
        main uring_lock will prevent that output. We'd love to get rid of that
        too, but that is beyond the scope of this release and will have to
        wait for 6.8"
      
      * tag 'io_uring-6.7-2023-11-17' of git://git.kernel.dk/linux:
        io_uring/fdinfo: remove need for sqpoll lock for thread/pid retrieval
      0e413c2a
    • Linus Torvalds's avatar
      Merge tag 'drm-fixes-2023-11-17' of git://anongit.freedesktop.org/drm/drm · e63fe2d3
      Linus Torvalds authored
      Pull drm fixes from Daniel Vetter:
       "This is a 'blast from the bast' fixes pull, because it contains a
        bunch of AGP fixes for amdgpu. Otherwise nothing out of the ordinary.
      
        Next week is back to Dave unless he's knocked out by some conference
        bug.
      
         - amdgpu: fixes all over, including a set of AGP fixes
      
         - nouvea: GSP + other bugfixes
      
         - ivpu build fix
      
         - lenovo legion go panel orientation quirk"
      
      * tag 'drm-fixes-2023-11-17' of git://anongit.freedesktop.org/drm/drm: (30 commits)
        drm/amdgpu/gmc9: disable AGP aperture
        drm/amdgpu/gmc10: disable AGP aperture
        drm/amdgpu/gmc11: disable AGP aperture
        drm/amdgpu: add a module parameter to control the AGP aperture
        drm/amdgpu/gmc11: fix logic typo in AGP check
        drm/amd/display: Fix encoder disable logic
        drm/amd/display: Change the DMCUB mailbox memory location from FB to inbox
        drm/amdgpu: add and populate the port num into xgmi topology info
        drm/amd/display: Negate IPS allow and commit bits
        drm/amd/pm: Don't send unload message for reset
        drm/amdgpu: fix ras err_data null pointer issue in amdgpu_ras.c
        drm/amd/display: Clear dpcd_sink_ext_caps if not set
        drm/amd/display: Enable fast plane updates on DCN3.2 and above
        drm/amd/display: fix NULL dereference
        drm/amd/display: fix a NULL pointer dereference in amdgpu_dm_i2c_xfer()
        drm/amd/display: Add null checks for 8K60 lightup
        drm/amd/pm: Fill pcie error counters for gpu v1_4
        drm/amd/pm: Update metric table for smu v13_0_6
        drm/amdgpu: correct chunk_ptr to a pointer to chunk.
        drm/amd/display: Fix DSC not Enabled on Direct MST Sink
        ...
      e63fe2d3
    • Chuck Lever's avatar
      NFSD: Fix checksum mismatches in the duplicate reply cache · bf51c52a
      Chuck Lever authored
      
      
      nfsd_cache_csum() currently assumes that the server's RPC layer has
      been advancing rq_arg.head[0].iov_base as it decodes an incoming
      request, because that's the way it used to work. On entry, it
      expects that buf->head[0].iov_base points to the start of the NFS
      header, and excludes the already-decoded RPC header.
      
      These days however, head[0].iov_base now points to the start of the
      RPC header during all processing. It no longer points at the NFS
      Call header when execution arrives at nfsd_cache_csum().
      
      In a retransmitted RPC the XID and the NFS header are supposed to
      be the same as the original message, but the contents of the
      retransmitted RPC header can be different. For example, for krb5,
      the GSS sequence number will be different between the two. Thus if
      the RPC header is always included in the DRC checksum computation,
      the checksum of the retransmitted message might not match the
      checksum of the original message, even though the NFS part of these
      messages is identical.
      
      The result is that, even if a matching XID is found in the DRC,
      the checksum mismatch causes the server to execute the
      retransmitted RPC transaction again.
      
      Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
      Tested-by: default avatarJeff Layton <jlayton@kernel.org>
      Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
      bf51c52a
    • Chuck Lever's avatar
      NFSD: Fix "start of NFS reply" pointer passed to nfsd_cache_update() · 1caf5f61
      Chuck Lever authored
      
      
      The "statp + 1" pointer that is passed to nfsd_cache_update() is
      supposed to point to the start of the egress NFS Reply header. In
      fact, it does point there for AUTH_SYS and RPCSEC_GSS_KRB5 requests.
      
      But both krb5i and krb5p add fields between the RPC header's
      accept_stat field and the start of the NFS Reply header. In those
      cases, "statp + 1" points at the extra fields instead of the Reply.
      The result is that nfsd_cache_update() caches what looks to the
      client like garbage.
      
      A connection break can occur for a number of reasons, but the most
      common reason when using krb5i/p is a GSS sequence number window
      underrun. When an underrun is detected, the server is obliged to
      drop the RPC and the connection to force a retransmit with a fresh
      GSS sequence number. The client presents the same XID, it hits in
      the server's DRC, and the server returns the garbage cache entry.
      
      The "statp + 1" argument has been used since the oldest changeset
      in the kernel history repo, so it has been in nfsd_dispatch()
      literally since before history began. The problem arose only when
      the server-side GSS implementation was added twenty years ago.
      
      Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
      Tested-by: default avatarJeff Layton <jlayton@kernel.org>
      Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
      1caf5f61
    • Chuck Lever's avatar
      NFSD: Update nfsd_cache_append() to use xdr_stream · 49cecd86
      Chuck Lever authored
      
      
      When inserting a DRC-cached response into the reply buffer, ensure
      that the reply buffer's xdr_stream is updated properly. Otherwise
      the server will send a garbage response.
      
      Cc: stable@vger.kernel.org # v6.3+
      Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
      Tested-by: default avatarJeff Layton <jlayton@kernel.org>
      Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
      49cecd86
    • Mahmoud Adam's avatar
      nfsd: fix file memleak on client_opens_release · bc1b5acb
      Mahmoud Adam authored
      
      
      seq_release should be called to free the allocated seq_file
      
      Cc: stable@vger.kernel.org # v5.3+
      Signed-off-by: default avatarMahmoud Adam <mngyadam@amazon.com>
      Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
      Fixes: 78599c42
      
       ("nfsd4: add file to display list of client's opens")
      Reviewed-by: default avatarNeilBrown <neilb@suse.de>
      Tested-by: default avatarJeff Layton <jlayton@kernel.org>
      Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
      bc1b5acb
    • Mikulas Patocka's avatar
      dm-crypt: start allocating with MAX_ORDER · 13648e04
      Mikulas Patocka authored
      Commit 23baf831
      
       ("mm, treewide: redefine MAX_ORDER sanely")
      changed the meaning of MAX_ORDER from exclusive to inclusive. So, we
      can allocate compound pages with up to 1 << MAX_ORDER pages.
      
      Reflect this change in dm-crypt and start trying to allocate compound
      pages with MAX_ORDER.
      
      Signed-off-by: default avatarMikulas Patocka <mpatocka@redhat.com>
      Signed-off-by: default avatarMike Snitzer <snitzer@kernel.org>
      13648e04
    • Mikulas Patocka's avatar
      dm-verity: don't use blocking calls from tasklets · 28f07f2a
      Mikulas Patocka authored
      The commit 5721d4e5
      
       enhanced dm-verity, so that it can verify blocks
      from tasklets rather than from workqueues. This reportedly improves
      performance significantly.
      
      However, dm-verity was using the flag CRYPTO_TFM_REQ_MAY_SLEEP from
      tasklets which resulted in warnings about sleeping function being called
      from non-sleeping context.
      
      BUG: sleeping function called from invalid context at crypto/internal.h:206
      in_atomic(): 1, irqs_disabled(): 0, non_block: 0, pid: 14, name: ksoftirqd/0
      preempt_count: 100, expected: 0
      RCU nest depth: 0, expected: 0
      CPU: 0 PID: 14 Comm: ksoftirqd/0 Tainted: G        W 6.7.0-rc1 #1
      Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.14.0-2 04/01/2014
      Call Trace:
       <TASK>
       dump_stack_lvl+0x32/0x50
       __might_resched+0x110/0x160
       crypto_hash_walk_done+0x54/0xb0
       shash_ahash_update+0x51/0x60
       verity_hash_update.isra.0+0x4a/0x130 [dm_verity]
       verity_verify_io+0x165/0x550 [dm_verity]
       ? free_unref_page+0xdf/0x170
       ? psi_group_change+0x113/0x390
       verity_tasklet+0xd/0x70 [dm_verity]
       tasklet_action_common.isra.0+0xb3/0xc0
       __do_softirq+0xaf/0x1ec
       ? smpboot_thread_fn+0x1d/0x200
       ? sort_range+0x20/0x20
       run_ksoftirqd+0x15/0x30
       smpboot_thread_fn+0xed/0x200
       kthread+0xdc/0x110
       ? kthread_complete_and_exit+0x20/0x20
       ret_from_fork+0x28/0x40
       ? kthread_complete_and_exit+0x20/0x20
       ret_from_fork_asm+0x11/0x20
       </TASK>
      
      This commit fixes dm-verity so that it doesn't use the flags
      CRYPTO_TFM_REQ_MAY_SLEEP and CRYPTO_TFM_REQ_MAY_BACKLOG from tasklets. The
      crypto API would do GFP_ATOMIC allocation instead, it could return -ENOMEM
      and we catch -ENOMEM in verity_tasklet and requeue the request to the
      workqueue.
      
      Signed-off-by: default avatarMikulas Patocka <mpatocka@redhat.com>
      Cc: stable@vger.kernel.org	# v6.0+
      Fixes: 5721d4e5
      
       ("dm verity: Add optional "try_verify_in_tasklet" feature")
      Signed-off-by: default avatarMike Snitzer <snitzer@kernel.org>
      28f07f2a
    • Mikulas Patocka's avatar
      dm-bufio: fix no-sleep mode · 2a695062
      Mikulas Patocka authored
      dm-bufio has a no-sleep mode. When activated (with the
      DM_BUFIO_CLIENT_NO_SLEEP flag), the bufio client is read-only and we
      could call dm_bufio_get from tasklets. This is used by dm-verity.
      
      Unfortunately, commit 450e8dee
      
       ("dm bufio: improve concurrent IO
      performance") broke this and the kernel would warn that cache_get()
      was calling down_read() from no-sleeping context. The bug can be
      reproduced by using "veritysetup open" with the "--use-tasklets"
      flag.
      
      This commit fixes dm-bufio, so that the tasklet mode works again, by
      expanding use of the 'no_sleep_enabled' static_key to conditionally
      use either a rw_semaphore or rwlock_t (which are colocated in the
      buffer_tree structure using a union).
      
      Signed-off-by: default avatarMikulas Patocka <mpatocka@redhat.com>
      Cc: stable@vger.kernel.org	# v6.4
      Fixes: 450e8dee
      
       ("dm bufio: improve concurrent IO performance")
      Signed-off-by: default avatarMike Snitzer <snitzer@kernel.org>
      2a695062
    • Mikulas Patocka's avatar
      dm-delay: avoid duplicate logic · ccadc8a2
      Mikulas Patocka authored
      
      
      This is small refactoring of dm-delay - we avoid duplicate logic in
      flush_delayed_bios and flush_delayed_bios_fast and join these two
      functions into one.
      
      We also add cond_resched() to flush_delayed_bios because the list may have
      unbounded number of entries.
      
      Signed-off-by: default avatarMikulas Patocka <mpatocka@redhat.com>
      Signed-off-by: default avatarMike Snitzer <snitzer@kernel.org>
      ccadc8a2
    • Mikulas Patocka's avatar
      dm-delay: fix bugs introduced by kthread mode · 38cfff56
      Mikulas Patocka authored
      This commit fixes the following bugs introduced by commit 70bbeb29
      ("dm delay: for short delays, use kthread instead of timers and wq"):
      
      * the function flush_worker_fn has no exit path - on unload, this
        function will just loop and consume 100% CPU without any progress
      
      * the wake-up mechanism in flush_worker_fn is racy - a wake up will be
        missed if the process adds entries to the delayed_bios list just
        before set_current_state(TASK_INTERRUPTIBLE)
      
      * flush_delayed_bios_fast submits a bio while holding a global mutex;
        this may deadlock if we have multiple stacked dm-delay devices and
        the underlying device attempts to acquire the mutex too
      
      * if the target constructor fails, it will call delay_dtr. delay_dtr
        would attempt to free dc->timer_lock without it being initialized by
        the constructor.
      
      * if the target constructor's kthread allocation fails, delay_dtr
        would crash trying to dereference dc->worker because it is non-NULL
        due to ERR_PTR.
      
      Fixes: 70bbeb29
      
       ("dm delay: for short delays, use kthread instead of timers and wq")
      Signed-off-by: default avatarMikulas Patocka <mpatocka@redhat.com>
      Signed-off-by: default avatarMike Snitzer <snitzer@kernel.org>
      38cfff56
    • Mikulas Patocka's avatar
      dm-delay: fix a race between delay_presuspend and delay_bio · 6fc45b6e
      Mikulas Patocka authored
      
      
      In delay_presuspend, we set the atomic variable may_delay and then stop
      the timer and flush pending bios. The intention here is to prevent the
      delay target from re-arming the timer again.
      
      However, this test is racy. Suppose that one thread goes to delay_bio,
      sees that dc->may_delay is one and proceeds; now, another thread executes
      delay_presuspend, it sets dc->may_delay to zero, deletes the timer and
      flushes pending bios. Then, the first thread continues and adds the bio to
      delayed->list despite the fact that dc->may_delay is false.
      
      Fix this bug by changing may_delay's type from atomic_t to bool and
      only access it while holding the delayed_bios_lock mutex. Note that we
      don't have to grab the mutex in delay_resume because there are no bios
      in flight at this point.
      
      Signed-off-by: default avatarMikulas Patocka <mpatocka@redhat.com>
      Cc: stable@vger.kernel.org
      Signed-off-by: default avatarMike Snitzer <snitzer@kernel.org>
      6fc45b6e
  5. Nov 17, 2023