From 7533fa9bd5a4a0f7743d553be186514d684308c8 Mon Sep 17 00:00:00 2001 From: Alan Coopersmith Date: Thu, 21 Feb 2019 14:51:22 -0800 Subject: [PATCH] os-support/solaris: Drop ExtendedEnabled global variable Keeping track of kernel state in user space doesn't buy us anything, and introduces bugs, as we were keeping global state but the Solaris kernel tracks IOPL per thread. Signed-off-by: Alan Coopersmith --- hw/xfree86/os-support/solaris/sun_vid.c | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/hw/xfree86/os-support/solaris/sun_vid.c b/hw/xfree86/os-support/solaris/sun_vid.c index 25f76181c..9601d039b 100644 --- a/hw/xfree86/os-support/solaris/sun_vid.c +++ b/hw/xfree86/os-support/solaris/sun_vid.c @@ -73,22 +73,14 @@ xf86OSInitVidMem(VidMemInfoPtr pVidMem) /* I/O Permissions section */ /***************************************************************************/ -#if defined(__i386__) || defined(__i386) || defined(__x86) -static Bool ExtendedEnabled = FALSE; -#endif - Bool xf86EnableIO(void) { #if defined(__i386__) || defined(__i386) || defined(__x86) - if (ExtendedEnabled) - return TRUE; - if (sysi86(SI86V86, V86SC_IOPL, PS_IOPL) < 0) { xf86Msg(X_WARNING, "xf86EnableIOPorts: Failed to set IOPL for I/O\n"); return FALSE; } - ExtendedEnabled = TRUE; #endif /* i386 */ return TRUE; } @@ -97,11 +89,6 @@ void xf86DisableIO(void) { #if defined(__i386__) || defined(__i386) || defined(__x86) - if (!ExtendedEnabled) - return; - sysi86(SI86V86, V86SC_IOPL, 0); - - ExtendedEnabled = FALSE; #endif /* i386 */ }