[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1031021: marked as done (xserver-xorg-video-amdgpu: Mouse cursor is invisible with KMS on X11, when using RDNA3 GPU - 7900XTX)



Your message dated Sun, 26 Feb 2023 09:35:43 +0000
with message-id <E1pWDRX-00GAoh-Jo@fasolo.debian.org>
and subject line Bug#1031021: fixed in xserver-xorg-video-amdgpu 23.0.0-1
has caused the Debian Bug report #1031021,
regarding xserver-xorg-video-amdgpu: Mouse cursor is invisible with KMS on X11, when using RDNA3 GPU - 7900XTX
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1031021: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1031021
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: xserver-xorg-video-amdgpu
Version: 22.0.0-3
Severity: important
Tags: patch
X-Debbugs-Cc: andamu@posteo.net

Dear Maintainer,

Using the new Radeon RX 7900XTX GPU, the mouse cursor is invisible in
X11 sessions, with xserver-xorg-video-amdgpu.

The patch found here, fixes it:
https://cgit.freedesktop.org/xorg/driver/xf86-video-amdgpu/patch/?id=9c959fac3af28d191105f63236096ad456dca614

I confirm the patch applies cleanly on current xserver-xorg-video-amdgpu
in Debian Bookworm. Maybe you could include it please?

VGA-compatible devices on PCI bus:
----------------------------------
0a:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 31 [Radeon RX 7900 XT/7900 XTX] [1002:744c] (rev c8)

Kernel version (/proc/version):
-------------------------------
Linux version 6.1.0-3-amd64 (debian-kernel@lists.debian.org) (gcc-12 (Debian 12.2.0-14) 12.2.0, GNU ld (GNU Binutils for Debian) 2.40) #1 SMP PREEMPT_DYNAMIC Debian 6.1.8-1 (2023-01-29)

DRM Information from dmesg:
---------------------------

-- System Information:
Debian Release: bookworm/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 6.1.0-3-amd64 (SMP w/32 CPU threads; PREEMPT)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE=en_US:en
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages xserver-xorg-video-amdgpu depends on:
ii  libc6                                  2.36-8
ii  libdrm-amdgpu1                         2.4.114-1
ii  libgbm1                                22.3.3-1
ii  libudev1                               252.5-2
ii  xserver-xorg-core [xorg-video-abi-25]  2:21.1.6-1

xserver-xorg-video-amdgpu recommends no packages.

Versions of packages xserver-xorg-video-amdgpu suggests:
ii  firmware-amd-graphics  20221214-5

-- no debconf information
>From 9c959fac3af28d191105f63236096ad456dca614 Mon Sep 17 00:00:00 2001
From: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Date: Thu, 29 Sep 2022 16:42:09 +0200
Subject: Use DRM_CAP_CURSOR_WIDTH/HEIGHT if possible

There's no need to hardcode the cursor size if the kernel can
report the value it wants.
---
 src/amdgpu_kms.c | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/src/amdgpu_kms.c b/src/amdgpu_kms.c
index 9364d17..1f049c9 100644
--- a/src/amdgpu_kms.c
+++ b/src/amdgpu_kms.c
@@ -1501,6 +1501,25 @@ static Bool AMDGPUCreateWindow_oneshot(WindowPtr pWin)
 	return ret;
 }
 
+static void amdgpu_determine_cursor_size(int fd, AMDGPUInfoPtr info)
+{
+	uint64_t value;
+
+	if (drmGetCap(fd, DRM_CAP_CURSOR_WIDTH, &value) == 0)
+		info->cursor_w = value;
+	else if (info->family < AMDGPU_FAMILY_CI)
+		info->cursor_w = CURSOR_WIDTH;
+	else
+		info->cursor_w = CURSOR_WIDTH_CIK;
+
+	if (drmGetCap(fd, DRM_CAP_CURSOR_HEIGHT, &value) == 0)
+		info->cursor_h = value;
+	else if (info->family < AMDGPU_FAMILY_CI)
+		info->cursor_h = CURSOR_HEIGHT;
+	else
+		info->cursor_h = CURSOR_HEIGHT_CIK;
+}
+
 /* When the root window is mapped, set the initial modes */
 void AMDGPUWindowExposures_oneshot(WindowPtr pWin, RegionPtr pRegion
 #if XORG_VERSION_CURRENT < XORG_VERSION_NUMERIC(1,16,99,901,0)
@@ -1684,13 +1703,7 @@ Bool AMDGPUPreInit_KMS(ScrnInfoPtr pScrn, int flags)
 	else
 		pAMDGPUEnt->HasCRTC2 = TRUE;
 
-	if (info->family < AMDGPU_FAMILY_CI) {
-		info->cursor_w = CURSOR_WIDTH;
-		info->cursor_h = CURSOR_HEIGHT;
-	} else {
-		info->cursor_w = CURSOR_WIDTH_CIK;
-		info->cursor_h = CURSOR_HEIGHT_CIK;
-	}
+	amdgpu_determine_cursor_size(pAMDGPUEnt->fd, info);
 
 	amdgpu_query_heap_size(pAMDGPUEnt->pDev, AMDGPU_GEM_DOMAIN_GTT,
 				&heap_size, &max_allocation);
-- 
cgit v1.2.1


--- End Message ---
--- Begin Message ---
Source: xserver-xorg-video-amdgpu
Source-Version: 23.0.0-1
Done: Timo Aaltonen <tjaalton@debian.org>

We believe that the bug you reported is fixed in the latest version of
xserver-xorg-video-amdgpu, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 1031021@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Timo Aaltonen <tjaalton@debian.org> (supplier of updated xserver-xorg-video-amdgpu package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sun, 26 Feb 2023 11:22:20 +0200
Source: xserver-xorg-video-amdgpu
Built-For-Profiles: noudeb
Architecture: source
Version: 23.0.0-1
Distribution: unstable
Urgency: medium
Maintainer: Debian X Strike Force <debian-x@lists.debian.org>
Changed-By: Timo Aaltonen <tjaalton@debian.org>
Closes: 1031021
Changes:
 xserver-xorg-video-amdgpu (23.0.0-1) unstable; urgency=medium
 .
   * New upstream release. (Closes: #1031021)
   * upstream: Add signing key from Shashank Sharma.
   * patches: Dropped, all upstream.
   * control: Fix gl-dev build-dep.
Checksums-Sha1:
 b8b50ef1b73455dc91c8b0c44a0135011ffde5ee 2513 xserver-xorg-video-amdgpu_23.0.0-1.dsc
 aab9e2e01c49f3c347c5a458947f556e97236133 558278 xserver-xorg-video-amdgpu_23.0.0.orig.tar.gz
 b029c3fae839c6b68776281b214fca650c12211a 659 xserver-xorg-video-amdgpu_23.0.0.orig.tar.gz.asc
 7129e25b9a9b2c3cb4e0f23ea1cb7926ab66a8d3 19309 xserver-xorg-video-amdgpu_23.0.0-1.diff.gz
 65a477e16049f9eec711bf3b5915abe201fa5a46 9751 xserver-xorg-video-amdgpu_23.0.0-1_source.buildinfo
Checksums-Sha256:
 243d1f380591d778d4cb70f46c97b2aeac0b441da62dd9b826537cec9d6acca8 2513 xserver-xorg-video-amdgpu_23.0.0-1.dsc
 08c38287d39b999fd61ecb6e7b23d5079762e2b4b2179b3567973ed9aaf71222 558278 xserver-xorg-video-amdgpu_23.0.0.orig.tar.gz
 bbb29a715eddd8cc921379e5055e0edb1c2e1791de7f4810641d6c24554cc15c 659 xserver-xorg-video-amdgpu_23.0.0.orig.tar.gz.asc
 588f50126cb64c94be7fea42a4082b31e9d83d7077a6744276422cd06aac1e44 19309 xserver-xorg-video-amdgpu_23.0.0-1.diff.gz
 2c53dcd3ee1ff99e1d1f1af3079cc08141546af8720e2e0226594ff1cb8c7aa3 9751 xserver-xorg-video-amdgpu_23.0.0-1_source.buildinfo
Files:
 db95907bc3147a8c2f9da3fbaa3ce39e 2513 x11 optional xserver-xorg-video-amdgpu_23.0.0-1.dsc
 7c4d8c22dc0c5e59360224ff56215554 558278 x11 optional xserver-xorg-video-amdgpu_23.0.0.orig.tar.gz
 e6ff415c673bd49e6ada4c797ed5c57e 659 x11 optional xserver-xorg-video-amdgpu_23.0.0.orig.tar.gz.asc
 46e1090df9917ecbca0cb1989a0e0467 19309 x11 optional xserver-xorg-video-amdgpu_23.0.0-1.diff.gz
 2730106e2d3ee2178bc3ba3dd1b7faba 9751 x11 optional xserver-xorg-video-amdgpu_23.0.0-1_source.buildinfo

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEdS3ifE3rFwGbS2Yjy3AxZaiJhNwFAmP7JNsACgkQy3AxZaiJ
hNzYcQ//S0eRkj7yRZYLpLgbsNxIPUBIhh75sXPzoFuGsMKeDx7qFQMh859qYZBF
EPK/trPG4lXKcz2UNxIN+khwdaLUnnLd8Q4H4jxaCC05ewnvv03iN85jAL10wsNw
InCZ5tSGJqatqoHcUR8rjKsWp9yuC75PXNel4XBqcFyfVasNrtpMU1FpumTFL+9y
L0bFC7butNenO6wNS/mF5+lHF8keGyPlU3ClvZnHiEl+uFmZJGxJBgF8lubVR/Q6
67jRiRXAnbhpXGry2pt4+xWGOCjiH/9qFKEysT1dSMGTFw9FzvrlGqr/0cgL5Atx
tspt2bhrWpBgXF51FQroEX6waYr4NfJ2YQvrCLx8CdFdDE4hD5kGNLHMoTOAxuBd
sARFf2pk3HmBRNa8DcXdHU0pZo0np6je8BaT8e8lWc2lJMy/envtfSA4gF6Ot80P
k3Qffr7LFGItONvPY8uP2ow4fK+lBHqPjBs9/VMKlWqPTwfz/Md+eL6prsehs3zl
wTElIAsipe/syAqHV6d/KjzUxXyEOH1LwADAnvayIdJyqEwxr2AMBoeZQJhUXL0k
+jw4qc8v3ak/2ECNMsIFXss+PjJq+JvRb+Cz9osUItkhGzV9uUN7oXmq7z5r92np
+PvqIhdZByHOwGakAyIvUrD/KY6rS+JKKnh2nc+jQ4i62Um/Zcs=
=o8Bm
-----END PGP SIGNATURE-----

--- End Message ---

Reply to: