Sandipan Das has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/16645
Change subject: arch-power: Remove redundant system call operand
......................................................................
arch-power: Remove redundant system call operand
This removes the redundant definition of GPR0 as an operand
specifically meant for use with the system call instruction
in order to denote the system call number. This information
is also accessible by reading the value of GPR0 through the
execution context. Hence, having another pseudo-operand for
this is redundant.
Change-Id: I171cb2c750321eef0e22ba8d5b6e867aebcebc68
Signed-off-by: Sandipan Das <[email protected]>
---
M src/arch/power/isa/decoder.isa
M src/arch/power/isa/operands.isa
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/arch/power/isa/decoder.isa b/src/arch/power/isa/decoder.isa
index 2257b8d..db5cc46 100644
--- a/src/arch/power/isa/decoder.isa
+++ b/src/arch/power/isa/decoder.isa
@@ -130,8 +130,12 @@
}
}
- 17: IntOp::sc({{ xc->syscall(R0, &fault); }},
- [ IsSyscall, IsNonSpeculative, IsSerializeAfter ]);
+ format IntOp {
+ 17: sc({{
+ uint64_t syscallNum = xc->tcBase()->readIntReg(0);
+ xc->syscall(syscallNum, &fault);
+ }}, [ IsSyscall, IsNonSpeculative, IsSerializeAfter ]);
+ }
format LoadDispOp {
34: lbz({{ Rt = Mem_ub; }});
diff --git a/src/arch/power/isa/operands.isa
b/src/arch/power/isa/operands.isa
index 4219668..faab3a9 100644
--- a/src/arch/power/isa/operands.isa
+++ b/src/arch/power/isa/operands.isa
@@ -77,7 +77,4 @@
'Rsv': ('IntReg', 'uw', 'INTREG_RSV', 'IsInteger', 9),
'RsvLen': ('IntReg', 'uw', 'INTREG_RSV_LEN', 'IsInteger', 9),
'RsvAddr': ('IntReg', 'ud', 'INTREG_RSV_ADDR', 'IsInteger', 9),
-
- # Hack for non-full-system syscall emulation
- 'R0': ('IntReg', 'ud', '0', None, 1),
}};
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/16645
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: I171cb2c750321eef0e22ba8d5b6e867aebcebc68
Gerrit-Change-Number: 16645
Gerrit-PatchSet: 1
Gerrit-Owner: Sandipan Das <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
