summaryrefslogtreecommitdiff
path: root/compiler/utils
diff options
context:
space:
mode:
authorMax Bolingbroke <batterseapower@hotmail.com>2011-07-29 12:05:46 +0100
committerMax Bolingbroke <batterseapower@hotmail.com>2011-07-29 12:54:23 +0100
commit0e765db44229aed591f9f423ef909b5f76696de0 (patch)
treeea3dfabe0073b43f476864bbd0109b25b505a3c2 /compiler/utils
parent969383ba17493d67664b2c0faaec481561401b18 (diff)
downloadhaskell-0e765db44229aed591f9f423ef909b5f76696de0.tar.gz
Add CoreMonad.reinitializeGlobals so plugins can work around linker issues
When a plugin is loaded, it currently gets linked against a *newly loaded* copy of the GHC package. This would not be a problem, except that the new copy has its own mutable state that is not shared with that state that has already been initialized by the original GHC package. This leads to loaded plugins calling GHC code which pokes the static flags, and then dying with a panic because the static flags *it* sees are uninitialized. There are two possible solutions: 1. Export the symbols from the GHC executable from the GHC library and link against this existing copy rather than a new copy of the GHC library 2. Carefully ensure that the global state in the two copies of the GHC library matches I tried 1. and it *almost* works (and speeds up plugin load times!) except on Windows. On Windows the GHC library tends to export more than 65536 symbols (see #5292) which overflows the limit of what we can export from the EXE and causes breakage. (Note that if the GHC exeecutable was dynamically linked this wouldn't be a problem, because we could share the GHC library it links to.) We are going to try 2. instead. Unfortunately, this means that every plugin will have to say `reinitializeGlobals` before it does anything, but never mind. I've threaded the cr_globals through CoreM rather than giving them as an argument to the plugin function so that we can turn this function into (return ()) without breaking any plugins when we eventually get 1. working.
Diffstat (limited to 'compiler/utils')
-rw-r--r--compiler/utils/Util.lhs10
1 files changed, 3 insertions, 7 deletions
diff --git a/compiler/utils/Util.lhs b/compiler/utils/Util.lhs
index ea46b28334..c5f1c0c2ed 100644
--- a/compiler/utils/Util.lhs
+++ b/compiler/utils/Util.lhs
@@ -74,7 +74,7 @@ module Util (
doesDirNameExist,
modificationTimeIfExists,
- global, consIORef, globalMVar, globalEmptyMVar,
+ global, consIORef, globalM,
-- * Filenames and paths
Suffix,
@@ -99,7 +99,6 @@ import Data.Data
import Data.IORef ( IORef, newIORef, atomicModifyIORef )
import System.IO.Unsafe ( unsafePerformIO )
import Data.List hiding (group)
-import Control.Concurrent.MVar ( MVar, newMVar, newEmptyMVar )
#ifdef DEBUG
import FastTypes
@@ -857,11 +856,8 @@ consIORef var x = do
\end{code}
\begin{code}
-globalMVar :: a -> MVar a
-globalMVar a = unsafePerformIO (newMVar a)
-
-globalEmptyMVar :: MVar a
-globalEmptyMVar = unsafePerformIO newEmptyMVar
+globalM :: IO a -> IORef a
+globalM ma = unsafePerformIO (ma >>= newIORef)
\end{code}
Module names: