Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/49429 )
Change subject: scons: Stop importing SimObjects into src/SConscript.
......................................................................
scons: Stop importing SimObjects into src/SConscript.
Get rid of the actual imports, and all the machinery which supports it.
Everything that had been using them is now handled using helper scripts
and/or the gem5py_m5 utility binary.
Change-Id: I079e50bdabef6d8d199caa80b589319d6419c4ba
---
M src/SConscript
1 file changed, 0 insertions(+), 121 deletions(-)
diff --git a/src/SConscript b/src/SConscript
index 31ff5b8..0a470be 100644
--- a/src/SConscript
+++ b/src/SConscript
@@ -37,12 +37,8 @@
# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-import bisect
import collections
import distutils.spawn
-import importlib
-import importlib.machinery
-import importlib.util
import itertools
import os
import os.path
@@ -92,8 +88,6 @@
class PySource(SourceFile):
'''Add a python source file to the named package'''
- modules = {}
-
def __init__(self, package, source, tags=None, add_tags=None):
'''specify the python package, the source file, and any tags'''
super(PySource, self).__init__(source, tags, add_tags)
@@ -115,11 +109,7 @@
if not os.path.exists(abspath):
abspath = self.tnode.abspath
- self.modname = modname
self.modpath = modpath
- self.abspath = abspath
-
- PySource.modules[modpath] = self
cpp = File(self.filename + '.cc')
@@ -142,9 +132,6 @@
'''Add a SimObject python file as a python source object and add
it to a list of sim object modules'''
- fixed = False
- modnames = []
-
sim_objects = dict()
enums = dict()
@@ -153,14 +140,10 @@
'''Specify the source file and any tags (automatically in
the m5.objects package)'''
super(SimObject, self).__init__('m5.objects', source, tags,
add_tags)
- if self.fixed:
- raise AttributeError("Too late to call SimObject now.")
SimObject.sim_objects[self.modpath] = sim_objects
SimObject.enums[self.modpath] = enums
- bisect.insort_right(SimObject.modnames, self.modname)
-
# This regular expression is simplistic and assumes that the import takes
up
# the entire line, doesn't have the keyword "public", uses double quotes,
has
@@ -507,110 +490,6 @@
env.Command('config/the_gpu_isa.hh', [],
MakeAction(makeTheGPUISA, Transform("CFG ISA", 0)))
-########################################################################
-#
-# Prevent any SimObjects from being added after this point, they
-# should all have been added in the SConscripts above
-#
-SimObject.fixed = True
-
-class SimpleModuleLoader(importlib.abc.Loader):
- '''A simple wrapper which delegates setting up a module to a
function.'''
- def __init__(self, executor):
- super(SimpleModuleLoader, self).__init__()
- self.executor = executor
- def create_module(self, spec):
- return None
-
- def exec_module(self, module):
- self.executor(module)
-
-class M5MetaPathFinder(importlib.abc.MetaPathFinder):
- def __init__(self, modules):
- super(M5MetaPathFinder, self).__init__()
- self.modules = modules
- self.installed = set()
-
- def unload(self):
- import sys
- for module in self.installed:
- del sys.modules[module]
- self.installed = set()
-
- def find_spec(self, fullname, path, target=None):
- spec = None
-
- # If this isn't even in the m5 package, ignore it.
- if fullname.startswith('m5.'):
- if fullname.startswith('m5.objects'):
- # When imported in this context, return a spec for a dummy
- # package which just serves to house the modules within it.
- # This is subtley different from "import * from m5.objects"
- # which relies on the __init__.py in m5.objects. That in
turn
- # indirectly relies on the c++ based _m5 package which
doesn't
- # exist yet.
- if fullname == 'm5.objects':
- dummy_loader = SimpleModuleLoader(lambda x: None)
- spec = importlib.machinery.ModuleSpec(
- name=fullname, loader=dummy_loader,
- is_package=True)
- spec.loader_state = self.modules.keys()
-
- # If this is a module within the m5.objects package,
return a
- # spec that maps to its source file.
- elif fullname in self.modules:
- source = self.modules[fullname]
- spec = importlib.util.spec_from_file_location(
- name=fullname, location=source.abspath)
-
- # The artificial m5.defines subpackage.
- elif fullname == 'm5.defines':
- def build_m5_defines(module):
- module.__dict__['buildEnv'] = dict(build_env)
-
- spec = importlib.util.spec_from_loader(name=fullname,
- loader=SimpleModuleLoader(build_m5_defines))
-
- # If we're handling this module, write it down so we can unload it
- # later.
- if spec is not None:
- self.installed.add(fullname)
-
- return spec
-
-import m5.SimObject
-import m5.params
-
-m5.SimObject.clear()
-m5.params.clear()
-
-# install the python importer so we can grab stuff from the source
-# tree itself. We can't have SimObjects added after this point or
-# else we won't know about them for the rest of the stuff.
-importer = M5MetaPathFinder(PySource.modules)
-sys.meta_path[0:0] = [ importer ]
-
-import_globals = globals().copy()
-# import all sim objects so we can populate the all_objects list
-# make sure that we're working with a list, then let's sort it
-for modname in SimObject.modnames:
- exec('from m5.objects import %s' % modname, import_globals)
-
-# we need to unload all of the currently imported modules so that they
-# will be re-imported the next time the sconscript is run
-importer.unload()
-sys.meta_path.remove(importer)
-
-sim_objects = m5.SimObject.allClasses
-all_enums = m5.params.allEnums
-
-########################################################################
-#
-# calculate extra dependencies
-#
-module_depends = ["m5", "m5.SimObject", "m5.params"]
-depends = [ PySource.modules[dep].snode for dep in module_depends ]
-depends.sort(key = lambda x: x.name)
########################################################################
#
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/49429
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I079e50bdabef6d8d199caa80b589319d6419c4ba
Gerrit-Change-Number: 49429
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
