Jason Lowe-Power has submitted this change and it was merged. (
https://gem5-review.googlesource.com/2863 )
Change subject: dev: Add ATA command used in recent Linux kernels
......................................................................
dev: Add ATA command used in recent Linux kernels
Add a case for the ATA command ATAPI_IDENTIFY_DEVICE.
This avoids the panic: Unsupported ATA command when booting a recent Linux
kernel. This was tested on 4.8.13.
Change-Id: Ib297a2c02da0730d8698c59801254dd0f5ee9f7f
Signed-off-by: Jason Lowe-Power <[email protected]>
Reviewed-on: https://gem5-review.googlesource.com/2863
Reviewed-by: Andreas Sandberg <[email protected]>
Reviewed-by: Gabe Black <[email protected]>
---
M src/dev/storage/ide_disk.cc
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Andreas Sandberg: Looks good to me, approved
Gabe Black: Looks good to me, approved
Jason Lowe-Power: Looks good to me, approved
diff --git a/src/dev/storage/ide_disk.cc b/src/dev/storage/ide_disk.cc
index e4b4e62..de0986e 100644
--- a/src/dev/storage/ide_disk.cc
+++ b/src/dev/storage/ide_disk.cc
@@ -657,6 +657,7 @@
// Supported PIO data-in commands
case WDCC_IDENTIFY:
+ case ATAPI_IDENTIFY_DEVICE:
cmdBytes = cmdBytesLeft = sizeof(struct ataparams);
devState = Prepare_Data_In;
action = ACT_DATA_READY;
@@ -852,7 +853,8 @@
status |= STATUS_DRQ_BIT;
// copy the data into the data buffer
- if (cmdReg.command == WDCC_IDENTIFY) {
+ if (cmdReg.command == WDCC_IDENTIFY ||
+ cmdReg.command == ATAPI_IDENTIFY_DEVICE) {
// Reset the drqBytes for this block
drqBytesLeft = sizeof(struct ataparams);
--
To view, visit https://gem5-review.googlesource.com/2863
To unsubscribe, visit https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib297a2c02da0730d8698c59801254dd0f5ee9f7f
Gerrit-Change-Number: 2863
Gerrit-PatchSet: 3
Gerrit-Owner: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
