65f9e7a3c7
Fix a hang that occasionally occurred during commands such as move.
(A bug introduced in 10.20220927, in
commit 6a3bd283b8
)
The restage.log was kept locked while running a complex index refresh
action. In an unusual situation, that action could need to write to the
restage log, which caused a deadlock.
The solution is a two-stage process. First the restage.log is moved to a
work file, which is done with the lock held. Then the content of the work
file is read and processed, which happens without the lock being held.
This is all done in a crash-safe manner.
Note that streamRestageLog may not be fully safe to run concurrently
with itself. That's ok, because restagePointerFiles uses it with the
index lock held, so only one can be run at a time.
streamRestageLog does delete the restage.old file at the end without
locking. If a calcRestageLog is run concurrently, it will either see the
file content before it was deleted, or will see it's missing. Either is
ok, because at most this will cause calcRestageLog to report more
work remains to be done than there is.
Sponsored-by: Dartmouth College's Datalad project
159 lines
5.2 KiB
Haskell
159 lines
5.2 KiB
Haskell
{- git-annex log files
|
|
-
|
|
- Copyright 2018-2022 Joey Hess <id@joeyh.name>
|
|
-
|
|
- Licensed under the GNU AGPL version 3 or higher.
|
|
-}
|
|
|
|
{-# LANGUAGE BangPatterns #-}
|
|
|
|
module Logs.File (
|
|
writeLogFile,
|
|
withLogHandle,
|
|
appendLogFile,
|
|
modifyLogFile,
|
|
streamLogFile,
|
|
streamLogFileUnsafe,
|
|
checkLogFile,
|
|
calcLogFile,
|
|
calcLogFileUnsafe,
|
|
) where
|
|
|
|
import Annex.Common
|
|
import Annex.Perms
|
|
import Annex.LockFile
|
|
import Annex.ReplaceFile
|
|
import Utility.Tmp
|
|
|
|
import qualified Data.ByteString.Lazy as L
|
|
import qualified Data.ByteString.Lazy.Char8 as L8
|
|
|
|
-- | Writes content to a file, replacing the file atomically, and
|
|
-- making the new file have whatever permissions the git repository is
|
|
-- configured to use. Creates the parent directory when necessary.
|
|
writeLogFile :: RawFilePath -> String -> Annex ()
|
|
writeLogFile f c = createDirWhenNeeded f $ viaTmp writelog (fromRawFilePath f) c
|
|
where
|
|
writelog tmp c' = do
|
|
liftIO $ writeFile tmp c'
|
|
setAnnexFilePerm (toRawFilePath tmp)
|
|
|
|
-- | Runs the action with a handle connected to a temp file.
|
|
-- The temp file replaces the log file once the action succeeds.
|
|
withLogHandle :: RawFilePath -> (Handle -> Annex a) -> Annex a
|
|
withLogHandle f a = do
|
|
createAnnexDirectory (parentDir f)
|
|
replaceGitAnnexDirFile (fromRawFilePath f) $ \tmp ->
|
|
bracket (setup tmp) cleanup a
|
|
where
|
|
setup tmp = do
|
|
setAnnexFilePerm (toRawFilePath tmp)
|
|
liftIO $ openFile tmp WriteMode
|
|
cleanup h = liftIO $ hClose h
|
|
|
|
-- | Appends a line to a log file, first locking it to prevent
|
|
-- concurrent writers.
|
|
appendLogFile :: RawFilePath -> RawFilePath -> L.ByteString -> Annex ()
|
|
appendLogFile f lck c =
|
|
createDirWhenNeeded f $
|
|
withExclusiveLock lck $ do
|
|
liftIO $ withFile f' AppendMode $
|
|
\h -> L8.hPutStrLn h c
|
|
setAnnexFilePerm (toRawFilePath f')
|
|
where
|
|
f' = fromRawFilePath f
|
|
|
|
-- | Modifies a log file.
|
|
--
|
|
-- If the function does not make any changes, avoids rewriting the file
|
|
-- for speed, but that does mean the whole file content has to be buffered
|
|
-- in memory.
|
|
--
|
|
-- The file is locked to prevent concurrent writers, and it is written
|
|
-- atomically.
|
|
modifyLogFile :: RawFilePath -> RawFilePath -> ([L.ByteString] -> [L.ByteString]) -> Annex ()
|
|
modifyLogFile f lck modf = withExclusiveLock lck $ do
|
|
ls <- liftIO $ fromMaybe []
|
|
<$> tryWhenExists (L8.lines <$> L.readFile f')
|
|
let ls' = modf ls
|
|
when (ls' /= ls) $
|
|
createDirWhenNeeded f $
|
|
viaTmp writelog f' (L8.unlines ls')
|
|
where
|
|
f' = fromRawFilePath f
|
|
writelog lf b = do
|
|
liftIO $ L.writeFile lf b
|
|
setAnnexFilePerm (toRawFilePath lf)
|
|
|
|
-- | Checks the content of a log file to see if any line matches.
|
|
checkLogFile :: RawFilePath -> RawFilePath -> (L.ByteString -> Bool) -> Annex Bool
|
|
checkLogFile f lck matchf = withSharedLock lck $ bracket setup cleanup go
|
|
where
|
|
setup = liftIO $ tryWhenExists $ openFile f' ReadMode
|
|
cleanup Nothing = noop
|
|
cleanup (Just h) = liftIO $ hClose h
|
|
go Nothing = return False
|
|
go (Just h) = do
|
|
!r <- liftIO (any matchf . L8.lines <$> L.hGetContents h)
|
|
return r
|
|
f' = fromRawFilePath f
|
|
|
|
-- | Folds a function over lines of a log file to calculate a value.
|
|
calcLogFile :: RawFilePath -> RawFilePath -> t -> (L.ByteString -> t -> t) -> Annex t
|
|
calcLogFile f lck start update =
|
|
withSharedLock lck $ calcLogFileUnsafe f start update
|
|
|
|
-- | Unsafe version that does not do locking.
|
|
calcLogFileUnsafe :: RawFilePath -> t -> (L.ByteString -> t -> t) -> Annex t
|
|
calcLogFileUnsafe f start update = bracket setup cleanup go
|
|
where
|
|
setup = liftIO $ tryWhenExists $ openFile f' ReadMode
|
|
cleanup Nothing = noop
|
|
cleanup (Just h) = liftIO $ hClose h
|
|
go Nothing = return start
|
|
go (Just h) = go' start =<< liftIO (L8.lines <$> L.hGetContents h)
|
|
go' v [] = return v
|
|
go' v (l:ls) = do
|
|
let !v' = update l v
|
|
go' v' ls
|
|
f' = fromRawFilePath f
|
|
|
|
-- | Streams lines from a log file, passing each line to the processor,
|
|
-- and then empties the file at the end.
|
|
--
|
|
-- If the processor is interrupted or throws an exception, the log file is
|
|
-- left unchanged.
|
|
--
|
|
-- There is also a finalizer, that is run once all lines have been
|
|
-- streamed. It is run even if the log file does not exist. If the
|
|
-- finalizer throws an exception, the log file is left unchanged.
|
|
--
|
|
-- Locking is used to prevent writes to to the log file while this
|
|
-- is running.
|
|
streamLogFile :: FilePath -> RawFilePath -> Annex () -> (String -> Annex ()) -> Annex ()
|
|
streamLogFile f lck finalizer processor =
|
|
withExclusiveLock lck $ do
|
|
streamLogFileUnsafe f finalizer processor
|
|
liftIO $ writeFile f ""
|
|
setAnnexFilePerm (toRawFilePath f)
|
|
|
|
-- Unsafe version that does not do locking, and does not empty the file
|
|
-- at the end.
|
|
streamLogFileUnsafe :: FilePath -> Annex () -> (String -> Annex ()) -> Annex ()
|
|
streamLogFileUnsafe f finalizer processor = bracketOnError setup cleanup go
|
|
where
|
|
setup = liftIO $ tryWhenExists $ openFile f ReadMode
|
|
cleanup Nothing = noop
|
|
cleanup (Just h) = liftIO $ hClose h
|
|
go Nothing = finalizer
|
|
go (Just h) = do
|
|
mapM_ processor =<< liftIO (lines <$> hGetContents h)
|
|
liftIO $ hClose h
|
|
finalizer
|
|
|
|
createDirWhenNeeded :: RawFilePath -> Annex () -> Annex ()
|
|
createDirWhenNeeded f a = a `catchNonAsync` \_e -> do
|
|
-- Most of the time, the directory will exist, so this is only
|
|
-- done if writing the file fails.
|
|
createAnnexDirectory (parentDir f)
|
|
a
|