Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/4460

Change subject: dev: Fix an IDE error check.
......................................................................

dev: Fix an IDE error check.

The error message says an IDE controller can support at most 4 disks, but the
check would fail if there were more than 3 disks.

Change-Id: Ic7d5d8c941fe2580da43019f53991377d4727bb9
---
M src/dev/storage/ide_ctrl.cc
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/src/dev/storage/ide_ctrl.cc b/src/dev/storage/ide_ctrl.cc
index feed9cf..6d50726 100644
--- a/src/dev/storage/ide_ctrl.cc
+++ b/src/dev/storage/ide_ctrl.cc
@@ -102,7 +102,7 @@
     ioEnabled(false), bmEnabled(false),
     ioShift(p->io_shift), ctrlOffset(p->ctrl_offset)
 {
-    if (params()->disks.size() > 3)
+    if (params()->disks.size() > 4)
panic("IDE controllers support a maximum of 4 devices attached!\n");

     // Assign the disks to channels

--
To view, visit https://gem5-review.googlesource.com/4460
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic7d5d8c941fe2580da43019f53991377d4727bb9
Gerrit-Change-Number: 4460
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to