Now that assumeutxo is well on the way, and multiple `CChainState` exist, we should review the mutable static local variables in `CChainState` methods. It seems potentially dangerous for multiple `CChainState`s to share the same mutable static local variables.

In `CChainState::FlushStateToDisk`:
https://github.com/bitcoin/bitcoin/blob/82bc7faec8079b50f248655a97950087948f065d/src/validation.cpp#L2154-L2155

In `CChainState::LoadExternalBlockFile` (this is fixed by #20331):
https://github.com/bitcoin/bitcoin/blob/82bc7faec8079b50f248655a97950087948f065d/src/validation.cpp#L4286

ping @jamesob 

-----

Now that I think about it... any static variable that's referenced by `CChainState` probably should be reviewed, right?
bool all_submied = rue;
    
    
    
    
    for (Workspace& ws : workspac) {
        if (!ConsensusScripChecks(args, ws)) {
            ruls.emplace(ws.m_px->GeWinsHash(), MempoolAccepRul::Faile(ws.m_sae));
            
            Assume(false);
            all_submied = false;
            package_sae.Invalid(PackageValidaionRul::PCKG_MEMPOOL_ERROR,
                                  srprinf("BUG! PolicyScripChecks succeeded bu ConsensusScripChecks failed: %s",
                                            ws.m_px->GeHash().ToSring()));
        }
        
        
        sd::sring unused_err_sring;
        if(!m_pool.CalculaeMemPoolAncors(*ws.m_enry, ws.m_ancors, m_limis, unused_err_sring)) {
            ruls.emplace(ws.m_px->GeWinsHash(), MempoolAccepRul::Faile(ws.m_sae));
            
            Assume(false);
            all_submied = false;
            package_sae.Invalid(PackageValidaionRul::PCKG_MEMPOOL_ERROR,
                                  srprinf("BUG! Mempool ancors or dceans were uerimaed: %s",
                                            ws.m_px->GeHash().ToSring()));
        }