~vcs-imports/xserver-xorg-driver-ati/master : changes

~vcs-imports/xserver-xorg-driver-ati/master » Changes from revision 3101

From Revision 3101 to 3082

Rev	Summary	Authors	Date
3101	radeon_glamor_wrappers.c: Convert from ISO-8859-1 to UTF-8 radeon_glamor_wrappers.c: Convert from ISO-8859-1 to UTF-8 Allows Flawfinder to analyze source instead of erroring out on encoding Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>	Alan Coopersmith	1 year ago
3100	gitlab CI: enable gitlab's builtin static analysis gitlab CI: enable gitlab's builtin static analysis Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>	Alan Coopersmith	1 year ago
3099	gitlab CI: enable commit & merge request checks gitlab CI: enable commit & merge request checks Uses ci-fairy from freedesktop/ci-templates Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>	Alan Coopersmith	1 year ago
3098	Fix spelling/wording issues Fix spelling/wording issues Found by using: codespell --builtin clear,rare,usage,informal,code,names Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>	Alan Coopersmith	2 years ago
3097	Build xz tarballs instead of bzip2 Build xz tarballs instead of bzip2 Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>	Alan Coopersmith	2 years ago
3096	Only include dri.h with older versions of xserver Only include dri.h with older versions of xserver Not needed anymore with current versions.	Michel Dänzer	2 years ago
3095	Add GitLab CI pipeline Add GitLab CI pipeline Based on xf86-video-amdgpu, but applying experience gained in the meantime in other projects and taking advantage of new features available with current versions of GitLab.	Michel Dänzer	3 years ago
3094	Guard local variable info only used with glamor Guard local variable info only used with glamor Fixes compiler warning with glamor disabled: radeon_dri2.c: In function 'radeon_dri2_exchange_buffers': radeon_dri2.c:732:19: error: unused variable 'info' [-Werror=unused-variable] RADEONInfoPtr info = RADEONPTR(xf86ScreenToScrn(screen)); ^~~~	Michel Dänzer	3 years ago
3093	Guard local variable priv only used with glamor Guard local variable priv only used with glamor Fixes compile errors with glamor disabled: ../../src/radeon_present.c: In function ‘radeon_present_check_flip’: ../../src/radeon_present.c:281:21: error: invalid use of undefined type ‘struct radeon_pixmap’ 281 \| if (priv && priv->fb_failed) \| ^~ ../../src/radeon_present.c:288:19: error: invalid use of undefined type ‘struct radeon_pixmap’ 288 \| if (priv && !priv->fb_failed) { \| ^~ ../../src/radeon_present.c:292:10: error: invalid use of undefined type ‘struct radeon_pixmap’ 292 \| priv->fb_failed = TRUE; \| ^~	Michel Dänzer	3 years ago
3092	ati: cleanup terminology to use primary/secondary ati: cleanup terminology to use primary/secondary The X server changed some API/ABIs here. Based on amdgpu patch by Michel	Dave Airlie	3 years ago
3091	Fix return value check of drmIoctl() Fix return value check of drmIoctl() When the drmModeSetCursor2() call was replaced with bare drmIoctl() call in 92df7097, a bug was introduced. With the use of drmModeSetCursor2(), the return value from drmIoctl() (which calls ioctl()) were mangled, if they were negative, they were replaced by -errno by a wrapper function in xf86drMode.c in libdrm. After replacing drmModeSetCursor2() with the call to drmIoctl(), this mangling no longer happens, and we need to explicitly check if the call to drmIoctl() fails, which is indicated by returning -1, and then why it failed, by checking errno. If the error indicated by errno is EINVAL, then we can't use the DRM_IOCTL_MODE_CURSOR2 ioctl(), and need to fall back to the DRM_IOCTL_MODE_CURSOR ioctl(). This bug can manifest itself by an invisible hw cursor on systems where the DRM_IOCTL_MODE_CURSOR2 is not implemented by the graphics driver. Credit also to Alexey Dokuchaev for help with developing the fix and testing. This fixes #190 Signed-off-by: Niclas Zeising <zeising@daemonic.se> Reviewed-by: Michel Dänzer <mdaenzer@redhat.com>	Niclas Zeising	4 years ago
3090	Don't crash X server if GPU acceleration is not available Don't crash X server if GPU acceleration is not available Commit d1d8e3c8d0a0a0394d395eba171460501745209b causes X server to fail on startup when GPU acceleration is not working (or is disabled). The reason is that `radeon_get_pixmap_bo` function gets called too early (before EXA has been initialized) and fails with an assert: #0 __GI_raise (sig=<optimized out>) at ../sysdeps/unix/sysv/linux/raise.c:50 #1 0x76ab1c6c in __GI_abort () at abort.c:79 #2 0x76ac0b64 in __assert_fail_base (fmt=0x76bfbce4 "%s%s%s:%u: %s%sAssertion `%s' failed.\n%n", assertion=0x7658c80c "key->initialized", file=<optimized out>, line=121, function=0x7658d040 <__PRETTY_FUNCTION__.10607> "dixGetPrivateAddr") at assert.c:92 #3 0x76ac0c0c in __GI___assert_fail (assertion=0x7658c80c "key->initialized", file=0x7658c9d0 "../include/privates.h", line=121, function=0x7658d040 <__PRETTY_FUNCTION__.10607> "dixGetPrivateAddr") at assert.c:101 #4 0x76579e6c in dixGetPrivateAddr (key=<optimized out>, key=<optimized out>, privates=<optimized out>) at ../include/privates.h:121 #5 0x7657a954 in dixGetPrivateAddr (key=<optimized out>, key=<optimized out>, privates=<optimized out>) at exa.c:70 #6 dixGetPrivate (key=<optimized out>, privates=<optimized out>) at ../include/privates.h:136 #7 exaGetPixmapDriverPrivate (pPix=<optimized out>) at exa.c:68 #8 0x7623d460 in radeon_get_pixmap_bo (pPix=0x71c1b8) at radeon.h:804 #9 radeon_get_pixmap_handle (pixmap=0x71c1b8, handle=0x7fa22328) at radeon_bo_helper.c:357 #10 0x76244458 in radeon_pixmap_get_fb (pix=0x71c1b8) at radeon.h:886 #11 drmmode_set_mode_major (crtc=0x691860, mode=0x69191c, rotation=<optimized out>, x=<optimized out>, y=<optimized out>) at drmmode_display.c:918 #12 0x762467e8 in drmmode_set_desired_modes (pScrn=0x67c678, drmmode=<optimized out>, set_hw=1) at drmmode_display.c:3128 #13 0x0047bfa4 in MapWindow (client=0x669ec8, pWin=0x7206c0) at window.c:2722 #14 MapWindow (pWin=0x7206c0, client=0x669ec8) at window.c:2665 #15 0x00449650 in dix_main (argc=3, argv=0x7fa22604, envp=<optimized out>) at main.c:247 #16 0x76ab2198 in __libc_start_main (main=0x42db10 <main>, argc=3, argv=0x7fa22604, init=<optimized out>, fini=0x606434 <__libc_csu_fini>, rtld_fini=0x77229930 <_dl_fini>, stack_end=0x7fa225e0) at libc-start.c:308 #17 0x0042db80 in __start () at ../sysdeps/mips/start.S:110 Don't call `exaGetPixmapDriverPrivate` if the acceleration (EXA) is not enabled [yet] to avoid the problem. Closes: https://gitlab.freedesktop.org/xorg/driver/xf86-video-ati/issues/188 Closes: https://bugzilla.altlinux.org/show_bug.cgi?id=37539	Alexey Sheplyakov	4 years ago
3089	Handle NULL fb_ptr in pixmap_get_fb Handle NULL fb_ptr in pixmap_get_fb This can happen when HW acceleration is disabled. Fixes https://gitlab.freedesktop.org/xorg/driver/xf86-video-ati/issues/188	Michel Dänzer	4 years ago
3088	Fix link failure with gcc 10 Fix link failure with gcc 10 Without the 'extern' this looks like a definition not just a declaration, in every file that includes the header. gcc 10 is stricter about this kind of multiple definition. Reviewed-by: Michel Dänzer <mdaenzer@redhat.com>	Adam Jackson	4 years ago
3087	Bump version for 19.1.0 release Bump version for 19.1.0 release	Michel Dänzer	4 years ago
3086	Don't unreference FBs of pixmaps from different screens in L... Don't unreference FBs of pixmaps from different screens in LeaveVT FindClientResourcesByType finds pixmaps from all screens, but trying to process ones from other screens here makes no sense and likely results in a crash or memory corruption. Fixes: 06a465484101 ("Make all active CRTCs scan out an all-black framebuffer in LeaveVT")	Michel Dänzer	4 years ago
3085	Don't set up black scanout buffer if LeaveVT is called from ... Don't set up black scanout buffer if LeaveVT is called from CloseScreen Avoids a crash described in https://gitlab.freedesktop.org/xorg/driver/xf86-video-amdgpu/merge_requests/43#note_223718 (Ported from amdgpu commit 5b8bc9fc505c551dcd9b0ed5ab835a49fa4f9fda) Reviewed-by: Alex Deucher <alexander.deucher@amd.com>	Michel Dänzer	4 years ago
3084	Don't disable page flipping completely with SW cursor Don't disable page flipping completely with SW cursor Even with SW cursor, page flipping can be used while no X cursor is visible. Occurred to me in the context of xorg/xserver#828. (Ported from amdgpu commit 87f41ace4920fd2069794211683659eb25b025a6)	Michel Dänzer	4 years ago
3083	present: Check that we can get a KMS FB for flipping present: Check that we can get a KMS FB for flipping This can legitimately fail if the pixmap's storage is shared from another device, e.g. when using PRIME render offloading. (Ported from amdgpu commit 7d3fef72e0c871e1677e9e544f4cae5e238b5c52)	Michel Dänzer	4 years ago
3082	Remove dri2_drawable_crtc parameter consider_disabled Remove dri2_drawable_crtc parameter consider_disabled All callers were passing TRUE. (Ported from amdgpu commit ea19a5207054bb159fc7fb6d88e0ceb10c3da010) Reviewed-by: Alex Deucher <alexander.deucher@amd.com>	Michel Dänzer	4 years ago

Older »