Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/55698 )

Change subject: arch-x86,dev: Make the I8259::getVector method protected.
......................................................................

arch-x86,dev: Make the I8259::getVector method protected.

Now that the I8259's vector is reported using a special memory access,
the getVector method doesn't need to be accessible outside of the class.
It's still useful internally though, since it nicely encapsulates what
should happen when an INTA signal is received.

Change-Id: I7da7c1f18fac97ffc62c965978f53fb4c5430de3
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/55698
Reviewed-by: Gabe Black <[email protected]>
Maintainer: Gabe Black <[email protected]>
Tested-by: kokoro <[email protected]>
---
M src/dev/x86/i8259.hh
1 file changed, 20 insertions(+), 1 deletion(-)

Approvals:
  Gabe Black: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass




diff --git a/src/dev/x86/i8259.hh b/src/dev/x86/i8259.hh
index 485664d..7bb319d 100644
--- a/src/dev/x86/i8259.hh
+++ b/src/dev/x86/i8259.hh
@@ -83,6 +83,8 @@
     void requestInterrupt(int line);
     void handleEOI(int line);

+    int getVector();
+
   public:
     using Params = I8259Params;

@@ -119,7 +121,6 @@
     void signalInterrupt(int line);
     void raiseInterruptPin(int number);
     void lowerInterruptPin(int number);
-    int getVector();

     void serialize(CheckpointOut &cp) const override;
     void unserialize(CheckpointIn &cp) override;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/55698
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: I7da7c1f18fac97ffc62c965978f53fb4c5430de3
Gerrit-Change-Number: 55698
Gerrit-PatchSet: 13
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Bradford Beckmann <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Matt Sinclair <[email protected]>
Gerrit-Reviewer: Matthew Poremba <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to