Andreas Sandberg has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/16422
Change subject: scons: Marshal Python sources using the same Python as gem5
......................................................................
scons: Marshal Python sources using the same Python as gem5
We currently use the Python version used by scons to marshal Python
code. This doesn't work when building gem5 with Python 3 support since
scons typically runs in Python 2.7. Add a custom marshal helper that
links with the same library as gem5 to generate byte code that is
guaranteed to work in gem5's Python interpreter.
Change-Id: I665b0f2078726d4c055d74a3e668a580fc613b59
Signed-off-by: Andreas Sandberg <andreas.sandb...@arm.com>
---
M src/SConscript
A src/marshall.cc
2 files changed, 76 insertions(+), 6 deletions(-)
diff --git a/src/SConscript b/src/SConscript
index a99624b..ab18494 100644
--- a/src/SConscript
+++ b/src/SConscript
@@ -1126,6 +1126,9 @@
Transform("VER TAGS")))
env.AlwaysBuild(tags)
+py_marshall = env.Program(target='marshall',
+ source=[File('marshall.cc'), ])
+
# Embed python files. All .py files that have been indicated by a
# PySource() call in a SConscript need to be embedded into the M5
# library. To do that, we compile the file to byte code, marshal the
@@ -1137,15 +1140,15 @@
return "0"
return '"%s"' % string
+ from m5.util import compareVersions, readCommand
+
'''Action function to compile a .py into a code object, marshal
it, compress it, and stick it into an asm file so the code appears
as just bytes with a label in the data section'''
- src = file(str(source[0]), 'r').read()
-
+ helper = File(py_marshall)
pysource = PySource.tnodes[source[0]]
- compiled = compile(src, pysource.abspath, 'exec')
- marshalled = marshal.dumps(compiled)
+ marshalled = readCommand([helper[0].abspath, str(source[0])])
compressed = zlib.compress(marshalled)
data = compressed
sym = pysource.symname
@@ -1174,8 +1177,9 @@
code.write(str(target[0]))
for source in PySource.all:
- env.Command(source.cpp, source.tnode,
- MakeAction(embedPyFile, Transform("EMBED PY")))
+ c = env.Command(source.cpp, source.tnode,
+ MakeAction(embedPyFile, Transform("EMBED PY")))
+ env.Depends(c, py_marshall)
Source(source.cpp, tags=source.tags, add_tags='python')
########################################################################
diff --git a/src/marshall.cc b/src/marshall.cc
new file mode 100644
index 0000000..2c9a753
--- /dev/null
+++ b/src/marshall.cc
@@ -0,0 +1,66 @@
+/*
+ * Copyright (c) 2019 ARM Limited
+ * All rights reserved
+ *
+ * The license below extends only to copyright in the software and shall
+ * not be construed as granting a license to any other intellectual
+ * property including but not limited to intellectual property relating
+ * to a hardware implementation of the functionality of the software
+ * licensed hereunder. You may use the software subject to the license
+ * terms below provided that you ensure that this notice is replicated
+ * unmodified and in its entirety in all distributions of the software,
+ * modified or unmodified, in source code or in binary form.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: Andreas Sandberg
+ */
+
+#include <pybind11/embed.h>
+
+#include <iostream>
+
+namespace py = pybind11;
+using namespace pybind11::literals;
+
+int
+main(int argc, char **argv) {
+ py::scoped_interpreter guard{};
+
+ if (argc != 2) {
+ std::cerr << "Usage: marshall PYSOURCE\n" << std::endl;
+ exit(1);
+ }
+
+ auto locals = py::dict("source"_a=argv[1]);
+
+ py::exec(
+ "import sys, marshal\n"
+ "with open(source, 'r') as f: src = f.read()\n"
+ "compiled = compile(src, source, 'exec')\n"
+ "sys.stdout.buffer.write(marshal.dumps(compiled))\n",
+ py::globals(), locals);
+
+ return 0;
+}
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/16422
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I665b0f2078726d4c055d74a3e668a580fc613b59
Gerrit-Change-Number: 16422
Gerrit-PatchSet: 1
Gerrit-Owner: Andreas Sandberg <andreas.sandb...@arm.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev