xfree86: unexport xf86Info field and xf86InfoRec type

Not used by any external driver, so no need to keep it in public SDK.
The "xf86Info" field still needs to be _X_EXPORT'ed, since glamoregl
module referencing it.

In the future, we should consider splitting this huge struct into
individual fields - there's no practical value in having some of the
globals inside a struct, while having others separately.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult
2026-01-13 19:16:54 +01:00
parent 746f830727
commit 0df39aaed2
14 changed files with 51 additions and 71 deletions

View File

@@ -33,7 +33,7 @@
#include <fcntl.h>
#include <sys/ioctl.h>
#include <errno.h>
#include <xf86.h>
#include "xf86_priv.h"
#include <xf86Priv.h>
#include <xf86drm.h>
#define EGL_DISPLAY_NO_X_MESA

View File

@@ -32,7 +32,7 @@
#include "os/osdep.h"
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Parser_priv.h"
#include "xf86tokens.h"
#include "xf86Config.h"

View File

@@ -96,18 +96,14 @@ xf86InfoRec xf86Info = {
.vtno = -1,
.lastEventTime = -1,
.vtRequestsPending = FALSE,
#ifdef __sun
.vtPendingNum = -1,
#endif
.dontVTSwitch = FALSE,
.autoVTSwitch = TRUE,
.ShareVTs = FALSE,
.dontZap = FALSE,
.dontZoom = FALSE,
.currentScreen = NULL,
#ifdef CSRG_BASED
.consType = -1,
#endif
.allowMouseOpenFail = FALSE,
.vidModeEnabled = TRUE,
.vidModeAllowNonLocal = FALSE,

View File

@@ -43,7 +43,6 @@ extern _X_EXPORT int xf86Depth;
/* Other parameters */
extern _X_EXPORT xf86InfoRec xf86Info;
extern _X_EXPORT serverLayoutRec xf86ConfigLayout;
extern _X_EXPORT DriverPtr *xf86DriverList;

View File

@@ -42,63 +42,6 @@ typedef enum {
XF86_GlxVisualsAll,
} XF86_GlxVisuals;
/*
* xf86InfoRec contains global parameters which the video drivers never
* need to access. Global parameters which the video drivers do need
* should be individual globals.
*/
typedef struct {
int consoleFd;
int vtno;
/* event handler part */
int lastEventTime;
Bool vtRequestsPending;
#ifdef __sun
int vtPendingNum;
#endif
Bool dontVTSwitch;
Bool autoVTSwitch;
Bool ShareVTs;
Bool dontZap;
Bool dontZoom;
/* graphics part */
ScreenPtr currentScreen;
#if defined(CSRG_BASED) || defined(__FreeBSD_kernel__)
int consType; /* Which console driver? */
#endif
/* Other things */
Bool allowMouseOpenFail;
Bool vidModeEnabled; /* VidMode extension enabled */
Bool vidModeAllowNonLocal; /* allow non-local VidMode
* connections */
Bool pmFlag;
MessageType iglxFrom;
XF86_GlxVisuals glxVisuals;
MessageType glxVisualsFrom;
Bool useDefaultFontPath;
Bool ignoreABI;
Bool forceInputDevices; /* force xorg.conf or built-in input devices */
Bool autoAddDevices; /* Whether to succeed NIDR, or ignore. */
Bool autoEnableDevices; /* Whether to enable, or let the client
* control. */
Bool dri2;
MessageType dri2From;
Bool autoAddGPU;
const char *debug;
Bool autoBindGPU;
Bool singleDriver; /* Only the first successfully probed driver adds primary screens,
* others may add GPU secondary screens only */
} xf86InfoRec, *xf86InfoPtr;
/* ISC's cc can't handle ~ of UL constants, so explicitly type cast them. */
#define XLED1 ((unsigned long) 0x00000001)
#define XLED2 ((unsigned long) 0x00000002)

View File

@@ -8,6 +8,8 @@
#include "os/osdep.h"
#include "xf86.h"
#include "xf86Privstr.h"
extern Bool xf86DoConfigure;
extern Bool xf86DoConfigurePass1;
extern Bool xf86ProbeIgnorePrimary;
@@ -102,4 +104,43 @@ void xf86ExtensionInit(void);
void DoConfigure(void) _X_NORETURN;
void DoShowOptions(void) _X_NORETURN;
/*
* global parameters only inside DDX itself, not accessible by drivers
* (should split them out to separate fields)
*/
typedef struct {
int consoleFd;
int vtno;
int lastEventTime;
bool vtRequestsPending;
int vtPendingNum;
Bool dontVTSwitch;
bool autoVTSwitch;
bool ShareVTs;
Bool dontZap;
Bool dontZoom;
ScreenPtr currentScreen;
int consType;
bool allowMouseOpenFail;
bool vidModeEnabled;
bool vidModeAllowNonLocal;
bool pmFlag;
MessageType iglxFrom;
XF86_GlxVisuals glxVisuals;
MessageType glxVisualsFrom;
bool useDefaultFontPath;
Bool ignoreABI;
bool forceInputDevices;
Bool autoAddDevices;
Bool autoEnableDevices;
bool dri2;
MessageType dri2From;
Bool autoAddGPU;
const char *debug;
Bool autoBindGPU;
Bool singleDriver;
} xf86InfoRec;
extern _X_EXPORT /* internal only */ xf86InfoRec xf86Info;
#endif /* _XSERVER_XF86_PRIV_H */

View File

@@ -26,7 +26,7 @@
#include <X11/X.h>
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"
#include "xf86_OSlib.h"

View File

@@ -30,7 +30,7 @@
#include <termios.h>
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"
#include "xf86_OSlib.h"

View File

@@ -29,7 +29,7 @@
#include "os/cmdline.h"
#include "compiler.h"
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"
#include "xf86_OSlib.h"

View File

@@ -27,7 +27,7 @@
#include <sys/mman.h>
#include <X11/X.h>
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"
#include "xf86_OSlib.h"

View File

@@ -29,7 +29,7 @@
#include "compiler.h"
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"
#include "xf86_OSlib.h"

View File

@@ -27,6 +27,7 @@
#include <sys/ioctl.h>
#include <linux/kd.h>
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"

View File

@@ -26,7 +26,7 @@
#include "os/osdep.h"
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"
#include "xf86_OSlib.h"

View File

@@ -24,7 +24,7 @@
#include <X11/X.h>
#include "xf86.h"
#include "xf86_priv.h"
#include "xf86Priv.h"
#include "xf86_os_support.h"
#include "xf86_OSlib.h"