[gem5-dev] Change in gem5/gem5[master]: mips: Add an object file loader for linux.
Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/c/public/gem5/+/18586 ) Change subject: mips: Add an object file loader for linux. .. mips: Add an object file loader for linux. Change-Id: Icae6430a210076117cf2ceadce52d6efbe58a5f3 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/18586 Reviewed-by: Gabe Black Maintainer: Gabe Black Tested-by: kokoro --- M src/arch/mips/linux/process.cc 1 file changed, 31 insertions(+), 0 deletions(-) Approvals: Gabe Black: Looks good to me, approved; Looks good to me, approved kokoro: Regressions pass diff --git a/src/arch/mips/linux/process.cc b/src/arch/mips/linux/process.cc index b1c09a5..71d20ea 100644 --- a/src/arch/mips/linux/process.cc +++ b/src/arch/mips/linux/process.cc @@ -34,6 +34,7 @@ #include "arch/mips/isa_traits.hh" #include "arch/mips/linux/linux.hh" +#include "base/loader/object_file.hh" #include "base/trace.hh" #include "cpu/thread_context.hh" #include "debug/SyscallVerbose.hh" @@ -47,6 +48,36 @@ using namespace std; using namespace MipsISA; +namespace +{ + +class MipsLinuxObjectFileLoader : public ObjectFile::Loader +{ + public: +Process * +load(ProcessParams *params, ObjectFile *obj_file) override +{ +if (obj_file->getArch() != ObjectFile::Mips) +return nullptr; + +auto opsys = obj_file->getOpSys(); + +if (opsys == ObjectFile::UnknownOpSys) { +warn("Unknown operating system; assuming Linux."); +opsys = ObjectFile::Linux; +} + +if (opsys != ObjectFile::Linux) +return nullptr; + +return new MipsLinuxProcess(params, obj_file); +} +}; + +MipsLinuxObjectFileLoader loader; + +} // anonymous namespace + /// Target uname() handler. static SyscallReturn unameFunc(SyscallDesc *desc, int callnum, Process *process, -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/18586 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: Icae6430a210076117cf2ceadce52d6efbe58a5f3 Gerrit-Change-Number: 18586 Gerrit-PatchSet: 4 Gerrit-Owner: Gabe Black Gerrit-Reviewer: Andreas Sandberg Gerrit-Reviewer: Brandon Potter Gerrit-Reviewer: Gabe Black Gerrit-Reviewer: Jason Lowe-Power Gerrit-Reviewer: kokoro Gerrit-MessageType: merged ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev
[gem5-dev] Change in gem5/gem5[master]: mips: Add an object file loader for linux.
Hello Andreas Sandberg, Brandon Potter, Jason Lowe-Power, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/18586 to look at the new patch set (#2). Change subject: mips: Add an object file loader for linux. .. mips: Add an object file loader for linux. Change-Id: Icae6430a210076117cf2ceadce52d6efbe58a5f3 --- M src/arch/mips/linux/process.cc 1 file changed, 31 insertions(+), 0 deletions(-) -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/18586 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: Icae6430a210076117cf2ceadce52d6efbe58a5f3 Gerrit-Change-Number: 18586 Gerrit-PatchSet: 2 Gerrit-Owner: Gabe Black Gerrit-Reviewer: Andreas Sandberg Gerrit-Reviewer: Brandon Potter Gerrit-Reviewer: Gabe Black Gerrit-Reviewer: Jason Lowe-Power Gerrit-MessageType: newpatchset ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev
[gem5-dev] Change in gem5/gem5[master]: mips: Add an object file loader for linux.
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/18586 Change subject: mips: Add an object file loader for linux. .. mips: Add an object file loader for linux. Change-Id: Icae6430a210076117cf2ceadce52d6efbe58a5f3 --- M src/arch/mips/linux/process.cc 1 file changed, 31 insertions(+), 0 deletions(-) diff --git a/src/arch/mips/linux/process.cc b/src/arch/mips/linux/process.cc index b1c09a5..51c1101 100644 --- a/src/arch/mips/linux/process.cc +++ b/src/arch/mips/linux/process.cc @@ -34,6 +34,7 @@ #include "arch/mips/isa_traits.hh" #include "arch/mips/linux/linux.hh" +#include "base/loader/loader.hh" #include "base/trace.hh" #include "cpu/thread_context.hh" #include "debug/SyscallVerbose.hh" @@ -47,6 +48,36 @@ using namespace std; using namespace MipsISA; +namespace +{ + +class MipsLinuxObjectFileLoader : public ObjectFileLoader +{ + public: +Process * +load(ProcessParams *params, ObjectFile *obj_file) override +{ +if (obj_file->getArch() != ObjectFile::Mips) +return nullptr; + +auto opsys = obj_file->getOpSys(); + +if (opsys == ObjectFile::UnknownOpSys) { +warn("Unknown operating system; assuming Linux."); +opsys = ObjectFile::Linux; +} + +if (opsys != ObjectFile::Linux) +return nullptr; + +return new MipsLinuxProcess(params, obj_file); +} +}; + +MipsLinuxObjectFileLoader loader; + +} // anonymous namespace + /// Target uname() handler. static SyscallReturn unameFunc(SyscallDesc *desc, int callnum, Process *process, -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/18586 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: Icae6430a210076117cf2ceadce52d6efbe58a5f3 Gerrit-Change-Number: 18586 Gerrit-PatchSet: 1 Gerrit-Owner: Gabe Black Gerrit-MessageType: newchange ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev