1. 22 Sep, 2023 15 commits
  2. 21 Sep, 2023 17 commits
  3. 20 Sep, 2023 8 commits
    • YuBiao Wang's avatar
      drm/amdkfd: Use gpu_offset for user queue's wptr · cc39f9cc
      YuBiao Wang authored
      Directly use tbo's start address will miss the domain start offset. Need
      to use gpu_offset instead.
      Signed-off-by: default avatarYuBiao Wang <YuBiao.Wang@amd.com>
      Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
      Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
      Cc: stable@vger.kernel.org
      cc39f9cc
    • Hamza Mahfooz's avatar
      drm/amd/display: fix the ability to use lower resolution modes on eDP · 2de19022
      Hamza Mahfooz authored
      On eDP we can receive invalid modes from dm_update_crtc_state() for
      entirely new streams for which drm_mode_set_crtcinfo() shouldn't be
      called on. So, instead of calling drm_mode_set_crtcinfo() from within
      create_stream_for_sink() we can instead call it from
      amdgpu_dm_connector_mode_valid(). Since, we are guaranteed to only call
      drm_mode_set_crtcinfo() for valid modes from that function (invalid
      modes are rejected by that callback) and that is the only user
      of create_validate_stream_for_sink() that we need to call
      drm_mode_set_crtcinfo() for (as before commit cb841d27
      ("drm/amd/display: Always pass connector_state to stream validation"),
      that is the only place where create_validate_stream_for_sink()'s
      dm_state was NULL).
      
      Cc: stable@vger.kernel.org
      Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2693
      Fixes: cb841d27 ("drm/amd/display: Always pass connector_state to stream validation")
      Tested-by: default avatarMark Broadworth <mark.broadworth@amd.com>
      Reviewed-by: default avatarHarry Wentland <harry.wentland@amd.com>
      Signed-off-by: default avatarHamza Mahfooz <hamza.mahfooz@amd.com>
      Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
      2de19022
    • Cong Liu's avatar
      drm/amdgpu: fix a memory leak in amdgpu_ras_feature_enable · f387bb57
      Cong Liu authored
      This patch fixes a memory leak in the amdgpu_ras_feature_enable() function.
      The leak occurs when the function sends a command to the firmware to enable
      or disable a RAS feature for a GFX block. If the command fails, the kfree()
      function is not called to free the info memory.
      
      Fixes: 9f051d6f ("drm/amdgpu: Free ras cmd input buffer properly")
      Reviewed-by: default avatarHawking Zhang <Hawking.Zhang@amd.com>
      Signed-off-by: default avatarCong Liu <liucong2@kylinos.cn>
      Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
      f387bb57
    • Lijo Lazar's avatar
      Revert "drm/amdgpu: Report vbios version instead of PN" · 06cce38e
      Lijo Lazar authored
      This reverts commit 7748ce5b.
      
      vbios_version sysfs node is used to identify Part Number also. Revert to
      the same so that it doesn't break scripts/software which parse this.
      Signed-off-by: default avatarLijo Lazar <lijo.lazar@amd.com>
      Reviewed-by: default avatarHawking Zhang <Hawking.Zhang@amd.com>
      Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
      06cce38e
    • Muhammad Ahmed's avatar
      drm/amd/display: Fix MST recognizes connected displays as one · 6f6583e5
      Muhammad Ahmed authored
      [What]
      MST now recognizes both connected displays
      
      Fixes: 927e784c ("drm/amd/display: Add symclk enable/disable during stream enable/disable")
      Reviewed-by: default avatarCharlene Liu <charlene.liu@amd.com>
      Acked-by: default avatarStylon Wang <stylon.wang@amd.com>
      Signed-off-by: default avatarMuhammad Ahmed <ahmed.ahmed@amd.com>
      Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
      Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
      6f6583e5
    • José Pekkarinen's avatar
      drm/virtio: clean out_fence on complete_submit · 4556b93f
      José Pekkarinen authored
      The removed line prevents the following cleanup function
      to execute a dma_fence_put on the out_fence to free its
      memory, producing the following output in kmemleak:
      
      unreferenced object 0xffff888126d8ee00 (size 128):
        comm "kwin_wayland", pid 981, jiffies 4295380296 (age 390.060s)
        hex dump (first 32 bytes):
          c8 a1 c2 27 81 88 ff ff e0 14 a9 c0 ff ff ff ff  ...'............
          30 1a e1 2e a6 00 00 00 28 fc 5b 17 81 88 ff ff  0.......(.[.....
        backtrace:
          [<0000000011655661>] kmalloc_trace+0x26/0xa0
          [<0000000055f15b82>] virtio_gpu_fence_alloc+0x47/0xc0 [virtio_gpu]
          [<00000000fa6d96f9>] virtio_gpu_execbuffer_ioctl+0x1a8/0x800 [virtio_gpu]
          [<00000000e6cb5105>] drm_ioctl_kernel+0x169/0x240 [drm]
          [<000000005ad33e27>] drm_ioctl+0x399/0x6b0 [drm]
          [<00000000a19dbf65>] __x64_sys_ioctl+0xc5/0x100
          [<0000000011fa801e>] do_syscall_64+0x5b/0xc0
          [<0000000065c76d8a>] entry_SYSCALL_64_after_hwframe+0x6e/0xd8
      unreferenced object 0xffff888121930500 (size 128):
        comm "kwin_wayland", pid 981, jiffies 4295380313 (age 390.096s)
        hex dump (first 32 bytes):
          c8 a1 c2 27 81 88 ff ff e0 14 a9 c0 ff ff ff ff  ...'............
          f9 ec d7 2f a6 00 00 00 28 fc 5b 17 81 88 ff ff  .../....(.[.....
        backtrace:
          [<0000000011655661>] kmalloc_trace+0x26/0xa0
          [<0000000055f15b82>] virtio_gpu_fence_alloc+0x47/0xc0 [virtio_gpu]
          [<00000000fa6d96f9>] virtio_gpu_execbuffer_ioctl+0x1a8/0x800 [virtio_gpu]
          [<00000000e6cb5105>] drm_ioctl_kernel+0x169/0x240 [drm]
          [<000000005ad33e27>] drm_ioctl+0x399/0x6b0 [drm]
          [<00000000a19dbf65>] __x64_sys_ioctl+0xc5/0x100
          [<0000000011fa801e>] do_syscall_64+0x5b/0xc0
          [<0000000065c76d8a>] entry_SYSCALL_64_after_hwframe+0x6e/0xd8
      [...]
      
      This memleak will grow quickly, being possible to see the
      following line in dmesg after few minutes of life in the
      virtual machine:
      
      [  706.217388] kmemleak: 10731 new suspected memory leaks (see /sys/kernel/debug/kmemleak)
      
      The patch will remove the line to allow the cleanup
      function do its job.
      Signed-off-by: default avatarJosé Pekkarinen <jose.pekkarinen@foxhound.fi>
      Fixes: e4812ab8 ("drm/virtio: Refactor and optimize job submission code path")
      Signed-off-by: default avatarDmitry Osipenko <dmitry.osipenko@collabora.com>
      Link: https://patchwork.freedesktop.org/patch/msgid/20230912060824.5210-1-jose.pekkarinen@foxhound.fi
      4556b93f
    • Linus Torvalds's avatar
      Merge tag 'media/v6.6-2' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media · 42dc8149
      Linus Torvalds authored
      Pull media fixes from Mauro Carvalho Chehab:
      
       - driver fixes due to incorrect fwnode_handle_put() call
      
       - bt8xx: bttv_risc_packed(): remove field checks
      
       - vb2: frame_vector.c: replace WARN_ONCE with a comment
      
       - imx219: a couple typo fixes and perform a full mode set
         unconditionally
      
       - uvcvideo: Fix OOB read
      
       - some dependency fixes
      
      * tag 'media/v6.6-2' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media:
        media: imx-mipi-csis: Remove an incorrect fwnode_handle_put() call
        media: vb2: frame_vector.c: replace WARN_ONCE with a comment
        media: uvcvideo: Fix OOB read
        media: bt8xx: bttv_risc_packed(): remove field checks
        media: i2c: rdacm21: Remove an incorrect fwnode_handle_put() call
        media: i2c: imx219: Perform a full mode set unconditionally
        media: i2c: imx219: Fix crop rectangle setting when changing format
        media: i2c: imx219: Fix a typo referring to a wrong variable
        media: i2c: max9286: Remove an incorrect fwnode_handle_put() call
        media: ivsc: Depend on VIDEO_DEV
        media: via: Use correct dependency for camera sensor drivers
        media: v4l: Use correct dependency for camera sensor drivers
        media: pci: ivsc: Select build dependencies
      42dc8149
    • Linus Torvalds's avatar
      Merge tag 'for-6.6-rc2-tag' of git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux · a229cf67
      Linus Torvalds authored
      Pull btrfs fixes from David Sterba:
       "A few more followup fixes to the directory listing.
      
        People have noticed different behaviour compared to other filesystems
        after changes in 6.5. This is now unified to more "logical" and
        expected behaviour while still within POSIX. And a few more fixes for
        stable.
      
         - change behaviour of readdir()/rewinddir() when new directory
           entries are created after opendir(), properly tracking the last
           entry
      
         - fix race in readdir when multiple threads can set the last entry
           index for a directory
      
        Additionally:
      
         - use exclusive lock when direct io might need to drop privs and call
           notify_change()
      
         - don't clear uptodate bit on page after an error, this may lead to a
           deadlock in subpage mode
      
         - fix waiting pattern when multiple readers block on Merkle tree
           data, switch to folios"
      
      * tag 'for-6.6-rc2-tag' of git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux:
        btrfs: fix race between reading a directory and adding entries to it
        btrfs: refresh dir last index during a rewinddir(3) call
        btrfs: set last dir index to the current last index when opening dir
        btrfs: don't clear uptodate on write errors
        btrfs: file_remove_privs needs an exclusive lock in direct io write
        btrfs: convert btrfs_read_merkle_tree_page() to use a folio
      a229cf67