summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2 hoursxf86_OSlib.h: Don't need to include Solaris keyboard headers hereHEADmasterAlan Coopersmith1-11/+0
Left over from the builtin keyboard driver removed from the Xorg server in 2006 (commit 3eeb62e8f587732e6b433c2b9c6879eb26a3f1b4). Requires xorg/driver/xf86-input-keyboard@40ef7d9e24987eb6708d822d0ea070 to build the current keyboard driver with this change. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1541>
6 hoursos: drop duplicate include of misc.hEnrico Weigelt, metux IT consult1-1/+0
Need to include it only once. Fixes: be4c8444eb0cc9c96682e2650d45f028b9a13b6c Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1542>
9 hoursos: drop remains of STREAMSCONNEnrico Weigelt, metux IT consult1-4/+4
STREAMSCONN has been removed about a decade ago, but some comments have been forgotten to be cleaned. Fixes: b3e9c534e2b0dc2c9acd2fe9b942e1fc5227339b Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1538>
17 hoursxwayland: Move XRandR emulation to the ResizeWindow hookOlivier Fourdan1-5/+2
This restores the handling of the XRandR emulation for Xwayland rootless where it was before commit fa7b1c20. Some compositors may trigger a protocol error if the viewport source is larger than the actual window size, having that handled in the window resize hook makes sure we do not regress. Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1521>
17 hoursxwayland: Handle rootful resize in ResizeWindowOlivier Fourdan1-11/+16
Commit fa7b1c20 ("xwayland: Use ConfigNotify screen hook instead of ResizeWindow") replaced the WindowResize hook with ConfigNotify. However, that's breaking rootful Xwayland with libdecor because the root window size is already set so the libdecor size is not updated, and the root size will be reverted back as soon as the focus changes. Reinstate the rootful size change in ResizeWindow to avoid that issue. Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1669 Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1671 Fixes: fa7b1c20 - xwayland: Use ConfigNotify screen hook instead of ResizeWindow Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1521>
17 hoursxwayland: Restore the ResizeWindow handlerOlivier Fourdan4-0/+25
For now it just chains to ResizeWindow hook. This is preparation work for the next commit, no functional change. Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1521>
3 daysxwayland: Send ei_device_frame on device_scroll_discreteJoshua Ashton1-0/+1
This fixes the scroll action in Steam Input in SteamOS/Gamescope when using the new libeis backend. Fixes: a13333427072dfb27792c5c9ff22ed145ede0f0a ("xwayland: Add XTEST support using EIS") Signed-off-by: Joshua Ashton <joshua@froggi.es> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1531>
3 daysdix: fix misspelled header guardEnrico Weigelt, metux IT consult1-1/+1
Fix FTBS on BSD: In file included from ../dix/colormap.c:57: ../dix/colormap_priv.h:5:9: warning: '_XSERVER_DIX_COLORMAP_PRIV_H' is used as a header guard here, followed by #define of a different macro [-Wheader-guard] ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../dix/colormap_priv.h:6:9: note: '_XSERVER_DIX_COLORMAP_PRIv_H' is defined here; did you mean '_XSERVER_DIX_COLORMAP_PRIV_H'? ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ _XSERVER_DIX_COLORMAP_PRIV_H 1 warning generated. [7/531] Compiling C object dix/liblibxserver_dix.a.p/devices.c.o In file included from ../dix/devices.c:62: ../dix/ptrveloc_priv.h:20:18: warning: redefinition of typedef 'PointerAccelerationProfileFunc' is a C11 feature [-Wtypedef-redefinition] typedef double (*PointerAccelerationProfileFunc) ^ ../include/ptrveloc.h:50:18: note: previous definition is here typedef double (*PointerAccelerationProfileFunc) ^ In file included from ../dix/devices.c:62: ../dix/ptrveloc_priv.h:33:3: warning: redefinition of typedef 'MotionTracker' is a C11 feature [-Wtypedef-redefinition] } MotionTracker, *MotionTrackerPtr; ^ ../include/ptrveloc.h:54:31: note: previous definition is here typedef struct _MotionTracker MotionTracker, *MotionTrackerPtr; ^ In file included from ../dix/devices.c:62: ../dix/ptrveloc_priv.h:33:19: warning: redefinition of typedef 'MotionTrackerPtr' is a C11 feature [-Wtypedef-redefinition] } MotionTracker, *MotionTrackerPtr; ^ ../include/ptrveloc.h:54:47: note: previous definition is here typedef struct _MotionTracker MotionTracker, *MotionTrackerPtr; ^ 3 warnings generated. [8/531] Compiling C object dix/liblibxserver_dix.a.p/main.c.o In file included from ../dix/main.c:86: ../dix/callback_priv.h:10:31: warning: redefinition of typedef 'CallbackListPtr' is a C11 feature [-Wtypedef-redefinition] typedef struct _CallbackList *CallbackListPtr; ^ ../include/callback.h:60:31: note: previous definition is here typedef struct _CallbackList *CallbackListPtr; /* also in misc.h */ ^ 1 warning generated. [9/531] Compiling C object dix/liblibxserver_dix.a.p/dispatch.c.o In file included from ../dix/dispatch.c:111: ../dix/screenint_priv.h:11:25: warning: redefinition of typedef 'ScreenPtr' is a C11 feature [-Wtypedef-redefinition] typedef struct _Screen *ScreenPtr; ^ ../include/screenint.h:55:25: note: previous definition is here typedef struct _Screen *ScreenPtr; ^ 1 warning generated. [10/531] Compiling C object dix/liblibxserver_dix.a.p/dixutils.c.o In file included from ../dix/dixutils.c:90: ../dix/callback_priv.h:10:31: warning: redefinition of typedef 'CallbackListPtr' is a C11 feature [-Wtypedef-redefinition] typedef struct _CallbackList *CallbackListPtr; ^ Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1530>
4 daysmodesetting: Fix invalid identity CTM on 32-bit.Trevor Davenport1-3/+3
On 32-bit, the shifts used to initialized the identity CTM overflow and result in zero instead of the intended 1. This results in a broken display (on at least i915) when using the modesetting xorg driver. Fix the invalid CTM by using ULL suffix on the shifts. Fixes:4e670f1281ad75c5673b6ac75645036d810da8d9 Signed-off-by: Trevor Davenport <trevor_davenport@selinc.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1526>
6 daysos: unexport ddx callbacksEnrico Weigelt, metux IT consult18-21/+50
The DDX callbacks (where core/DIX calls into DDX) aren't supposed to be called by drivers directly, so unexport them. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1332>
6 daysinclude: xkbfile: clean up forgotten unused declarationsEnrico Weigelt, metux IT consult1-35/+11
Back in 2008, when cleaning out unused/undefined stuff, some pieces have been forgotten in commit fb22d4d928bc5d1a893494a059359da2ed2ac639. This was accidentially introduced by 68bd7ac1930b5cffb6657b8d5f5bf8ae58eae8d9. Fixes: fb22d4d928bc5d1a893494a059359da2ed2ac639 Fixes: 68bd7ac1930b5cffb6657b8d5f5bf8ae58eae8d9 Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1473>
6 daysxquartz: Remove invalid Unicode sequenceOlivier Fourdan1-1/+1
This is flagged by the automatic scanning tools. Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1673 Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1524>
7 daysxwayland: Use correct xwl_window lookup function in xwl_set_shapeVlad Zahorodnii1-1/+1
In xwl_set_shape(), xwl_window_set_input_region() should be called only when the input shape of the toplevel window changes. However, given that xwl_window_from_window() is going to walk the ancestor tree until it finds an xwl_window, that lookup function cannot be used. Instead, xwl_window_get() should be used. It's going to return a valid xwl_window object iff the specified window has one associated with it. Fixes: a4ed100c0 - xwayland: Set wl_surface input region Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1672 Reviewed-by: Michel Dänzer <mdaenzer@redhat.com> Signed-off-by: Vlad Zahorodnii <vlad.zahorodnii@kde.org> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1516>
12 daysxwayland/present: Drop vblank->flip_ready assignmentMichel Dänzer1-1/+0
Xwayland hasn't used the flip_ready field's value for anything since e1f16fb1aca5 ("xwayland: don't scrap pending present requests"). Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1511>
12 daysxwayland/present: Skip queued flip when a new one becomes readyMichel Dänzer1-0/+19
If multiple flips become ready for the same MSC, we would previously execute them all sequentially, one per MSC for sync flips. This could result in an unbounded flip queue and corresponding memory consumption. With implicit sync, leave the mailbox handling to the compositor for async flips though. v2: * Use present_vblank_rec::sync_flip. Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1664 Fixes: e1f16fb1aca5 ("xwayland: don't scrap pending present requests") Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1511>
12 dayspresent: signal explicit sync release point in present_vblank_scrapErik Kurzinger1-1/+8
If a present request using explicit sync is scrapped, instead of sending a PresentIdleNotify event we should signal the release point. Signed-off-by: Erik Kurzinger <ekurzinger@nvidia.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1511>
14 daysxwayland: use write fence in xwl_glamor_dmabuf_import_sync_fileErik Kurzinger1-1/+1
The functions xwl_glamor_dmabuf_import_sync_file and xwl_glamor_dmabuf_export_sync_file are used to ensure proper synchronization between clients using PresentPixmapSynced and compositors that do not support the wp_linux_drm_syncobj_v1 protocol when presenting by flipping. The acquire point's fence will be imported as the DMA-BUF's implicit fence before handing it off to the compositor, and then, after the DMA-BUF has been released, its new implicit fence will be exported and become the release point's fence which the client is expected to wait for before re-using the buffer. Both functions currently set the flags arguments of their respective ioctls to DMA_BUF_SYNC_READ. When importing a sync file, this means that any subsequent implicitly synchronized reads from the buffer will not wait for the fence, and when exporting a sync file it means that the returned fence may be signaled before preceeding reads from the buffer have completed. While this is correct for xwl_glamor_dmabuf_export_sync_file since the compositor will never write to the buffer, it is incorrect for xwl_glamor_dmabuf_import_sync_file. To avoid corruption, we need any reads from the buffer by the compositor to wait on the acquire point's fence. As a fix, instead of setting the DMA_BUF_SYNC_READ flag in xwl_glamor_dmabuf_import_sync_file, we set the DMA_BUF_SYNC_WRITE flag. This *does* provide the necessary guarantees. Signed-off-by: Erik Kurzinger <ekurzinger@nvidia.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1509>
14 daysxwayland: Set wl_surface input regionVlad Zahorodnii4-0/+61
Some applications that use client side decorations usually set custom input shape in order to prevent drop shadows stealing pointer events from windows below. Currently, the only way to get it is to use some XFixes APIs. On the other hand, plenty of wayland compositors use solely the wl_surface input region to decide what view can receive pointer input, which results in some pointer input issues around client side drop shadows because Xwayland doesn't set wl_surface.input_region. See-also: https://bugs.kde.org/show_bug.cgi?id=448119 Signed-off-by: Vlad Zahorodnii <vlad.zahorodnii@kde.org> Reviewed-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1510>
14 daysxwayland: Do not remove output on withdraw if leasedOlivier Fourdan2-2/+31
On DRM lease connector withdrawn event, Xwayland would free the corresponding xwl_output offered for lease. However, the pointer is still referenced from the rrLease->outputs[], meaning that trying to clean up the RANDR DRM leases as done with commit ef181265 (xwayland: Clean up drm lease when terminating) would cause a use after free and random crashes. To avoid that issue, on the connector withdraw event, set the connector withdrawn flag but do not to remove (i.e. free) the xwayland output if its is offered for lease. Then, once the lease is terminated, check for the xwl_outputs with a withdrawn connector and remove them (once we have no use for them anymore. Note that we cannot do that cleanup from xwl_randr_terminate_lease() as removing the xwl_output will free the RRcrtc resources, which checks for leases in XRANDR, and calls RRTerminateLease(), which chains back to xwl_randr_terminate_lease(). v2: Use a "withdrawn_connector" flag to mark outputs to remove (Xaver) Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Reviewed-by: Xaver Hugl <xaver.hugl@kde.org> fixes: ef181265 - xwayland: Clean up drm lease when terminating See-also: https://gitlab.freedesktop.org/xorg/xserver/-/issues/946 See-also: https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1130 Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1482>
14 daysxwayland: Check for outputs before lease devicesOlivier Fourdan1-7/+7
In xwl_randr_request_lease(), the code checks first for leased device, and then checks for existing output for lease. The former assumes there are outputs for lease whereas the latter checks for the output, connector and lease. So if there is any existing rrLease->outputs[]->devPrivate unset, the code would crash on a NULL pointer dereference on the first sanity check before having a chance to reach the second check that would have caught the problem. Invert the sanity checks so that we would catch this first and return a BadValue instead of possibly segfaulting. Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Reviewed-by: Xaver Hugl <xaver.hugl@kde.org> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1482>
2024-04-30include: move private definitions out of exevents.hEnrico Weigelt, metux IT consult39-248/+246
Public server module API shouldn't be clobbered with private definitions, thus move them out to private header. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1289>
2024-04-30include: move private definitions out of extinit.hEnrico Weigelt, metux IT consult35-78/+77
Public server module API shouldn't be clobbered with private definitions, thus move them out to extinit_priv.h. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1289>
2024-04-30include: move private defs to dixstruct_priv.hEnrico Weigelt, metux IT consult24-143/+169
Public server module API shouldn't be clobbered with private definitions, thus move them out to dixstruct_priv.h Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1289>
2024-04-30include: split out non-exported prototypes to dix_priv.hEnrico Weigelt, metux IT consult25-231/+227
Public server module API shouldn't be clobbered with private definitions, thus move them out to dix-intern.h Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1289>
2024-04-28xkb: make XkbInternAtom() staticEnrico Weigelt, metux IT consult2-7/+1
It's only used in the same .c file, so make it static. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1506>
2024-04-25xwayland/glamor: Handle depth 15 in gbm_format_for_depthMichel Dänzer2-0/+4
Prevents Xwayland with glamor from logging unexpected depth: 15 to stderr many times when running rendercheck -t blend -o clear Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1507>
2024-04-23xwayland: Use the connector name for XRANDR leasesOlivier Fourdan1-6/+11
Use the connector name as basis for the Xwayland output name in XRANDR, similar to what we do for regular outputs, instead of the generic "XWAYLAND<n>" name which changes every time the output is leased. Prefix the actual name with "lease-" to distinguish from duplicate names from the regular outputs. v2: avoid duplicate names (Simon) v3: Move the check for duplicates to xwl_output_set_name() (Simon) Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1492>
2024-04-23xwayland: Check for duplicate output namesOlivier Fourdan1-0/+21
Even though the name provided by either xdg-output or wl_output are guaranteed to be unique, that might not be the case with output names between different protocols, such as the one offered for DRM lease. To avoid running into name conflicts, check that no other existing output of the same name exists prior to changing the output name. Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1492>
2024-04-23xwayland: Make xwl_output_set_name() publicOlivier Fourdan2-1/+3
No functional change, this is preparation work for the next commit. Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1492>
2024-04-23xwayland: Define MAX_OUTPUT_NAME in the headerOlivier Fourdan2-2/+2
So that other parts of the Xwayland code can use it. Signed-off-by: Olivier Fourdan <ofourdan@redhat.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1492>
2024-04-23dbe: unexport dbestruct.hEnrico Weigelt, metux IT consult2-14/+0
This include isn't used by any (known) driver nor included by any other public header, so no need to keep it in the public module API. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1422>
2024-04-23xkb: move *_TIMER defines into xkbAccessX.cEnrico Weigelt, metux IT consult2-6/+6
These are only used in this file (and also not any external driver), so no need to have them in a public header file. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1459>
2024-04-23include: move dbus-core.h to configEnrico Weigelt, metux IT consult5-4/+7
This header is internal (not installed) and holds definitions for sources in config/, thus it fells more clean moving it to config/, too. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1357>
2024-04-23include: colormap.h: move internal typedef to dix/colormap_priv.hEnrico Weigelt, metux IT consult4-1/+15
Moving the internal EntryType typedef from "colormap.h" into newly added internal-only header "dix/colormap_priv.h" Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1283>
2024-04-23xnest: fix segfault in miCreateScreenResources()Enrico Weigelt, metux IT consult3-2/+26
With aa3f5023e3fae0df74039702b6c8218bc14dc679, pScreen->devPrivate now is initialized only once, which uncovered a silent bug in xnestOpenScreen: It's NULL'ing the pScreen->devPrivate pointer which already had been initialized by previous miScreenDevPrivateInit() call. Fixes: aa3f5023e3fae0df74039702b6c8218bc14dc679 Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1495>
2024-04-22include: drop obsolete check for typeof operatorEnrico Weigelt, metux IT consult3-24/+0
It really seems that compilers on all our supported platforms (including Solaris) supporting `typeof`, so we can always use it and drop hacks with undefind behaviour entirely. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1432>
2024-04-22xfree86: sdksyms: drop errornous check for mifillarc.hEnrico Weigelt, metux IT consult1-1/+0
This file had been dropped from public API a decade ago, but sdksyms.sh yet had been forgotten to be fixed. Fixes: 707965407a3c907058b89610e73e02989fd0b552 Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1497>
2024-04-18xfree86: x86emu: fix warning on unneccessary abs()Enrico Weigelt, metux IT consult1-2/+2
fix warning: > In file included from ../hw/xfree86/int10/x86emu.c:11: > ../hw/xfree86/x86emu/prim_ops.c:2478:9: warning: taking the absolute value of unsigned type 'x86emuu32' (aka 'unsigned int') has no effect [-Wabsolute-value] > if (abs(div) > 0xff) { > ^ > ../hw/xfree86/x86emu/prim_ops.c:2478:9: note: remove the call to 'abs' since unsigned values cannot be negative > if (abs(div) > 0xff) { > ^~~ > ../hw/xfree86/x86emu/prim_ops.c:2502:9: warning: taking the absolute value of unsigned type 'x86emuu32' (aka 'unsigned int') has no effect [-Wabsolute-value] > if (abs(div) > 0xffff) { > ^ > ../hw/xfree86/x86emu/prim_ops.c:2502:9: note: remove the call to 'abs' since unsigned values cannot be negative > if (abs(div) > 0xffff) { > ^~~ Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1428>
2024-04-18xfree86: x86emu: drop unused stq_u()Enrico Weigelt, metux IT consult1-16/+0
fix warning on unused function: > ../hw/xfree86/x86emu/sys.c:87:1: warning: unused function 'stq_u' [-Wunused-function] > stq_u(u64 val, u64 * p) > ^ Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1428>
2024-04-18xfree86: x86emu: drop unused ldq_u()Enrico Weigelt, metux IT consult1-17/+0
fix warning on unused function: > ../hw/xfree86/x86emu/sys.c:69:1: warning: unused function 'ldq_u' [-Wunused-function] > ldq_u(u64 * p) > ^ > ../hw/xfree86/x86emu/sys.c:95:1: warning: unused function 'stq_u' [-Wunused-function] > stq_u(u64 val, u64 * p) > ^ Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1428>
2024-04-18xfree86: os-support: bsd: fix missing prototypesEnrico Weigelt, metux IT consult1-1/+0
Fix warnings on missing prototypes: > ../hw/xfree86/os-support/bsd/bsd_VTsw.c:56:20: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86VTSwitchPending() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_VTsw.c:67:17: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86VTSwitchAway() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_VTsw.c:82:15: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86VTSwitchTo() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_init.c:155:16: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86OpenConsole() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_init.c:322:15: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86OpenPccons() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_init.c:347:16: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86OpenSyscons() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_init.c:453:13: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86OpenPcvt() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_init.c:596:17: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86CloseConsole() > ^ > void > ../hw/xfree86/os-support/bsd/bsd_init.c:673:11: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes] > xf86UseMsg() > ^ > void Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1428>
2024-04-18xfree86: modes: drop unused xf86_driver_has_show_cursor()Enrico Weigelt, metux IT consult1-6/+0
Fix warning on unused function: > ../hw/xfree86/modes/xf86Cursors.c:212:1: warning: unused function 'xf86_driver_has_show_cursor' [-Wunused-function] > xf86_driver_has_show_cursor(xf86CrtcPtr crtc) > ^ Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1428>
2024-04-18composite: fix duplicate typedef of ScreenPtrEnrico Weigelt, metux IT consult1-1/+1
fix warning on duplicate typedef: > ../composite/compositeext_priv.h:11:25: warning: redefinition of typedef 'ScreenPtr' is a C11 feature [-Wtypedef-redefinition] > typedef struct _Screen *ScreenPtr; > ^ > ../include/screenint.h:55:25: note: previous definition is here > typedef struct _Screen *ScreenPtr; > ^ Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1430>
2024-04-18xfree86: os-support: move hidden Solaris-specific symbols out of public headerEnrico Weigelt, metux IT consult4-7/+10
These aren't exported at all, so no need to have it in public header. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1456>
2024-04-18xfree86: os-support: move xf86FatalError macro out of public headerEnrico Weigelt, metux IT consult2-13/+15
This macro is only used inside xfree86's os-support layer, not by any (known) drivers. Thus no need to have it exported in the public API. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1456>
2024-04-18xfree86: os-support: move unexported stuff out of xf86_OSproc.hEnrico Weigelt, metux IT consult5-9/+12
It's cleaner to have public headers only holding public stuff and xf86_os_support seems to be much more appropriate place for this. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1456>
2024-04-18xfree86: os-support: move VidMemInfo xf86_os_support.hEnrico Weigelt, metux IT consult2-41/+4
This type is only used inside the os-support layer of xfree86, so it fits better into xf86_os_support.h, whose purpose is being the primary entry point into os-support layer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1456>
2024-04-18xfree86: os-support: move xf86OSInitVidMem() to xf86_os_support.hEnrico Weigelt, metux IT consult15-18/+19
This function isn't exported at all, so it better fits into xf86_os_support.h, whose purpose is being the primary entry point into os-support layer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1456>
2024-04-18xfree86: os-support: unexport xf86RemoveSIGIOHandlerEnrico Weigelt, metux IT consult5-1/+5
It's not used by any driver, so no need to export it. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1456>
2024-04-18xfree86: os-support: unexport xf86DeallocateGARTMemoryEnrico Weigelt, metux IT consult5-1/+4
It's not used in any drivers, so no need to export it. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1456>
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-14 01:24:10 +0000