TUD-OS / NRE

NOVA runtime environment (official branch)
GNU General Public License v2.0
33 stars 12 forks source link

Incorrect sector count calculation in AHCI driver #37

Closed parthy closed 11 years ago

parthy commented 11 years ago

The shifting value in HostAHCIDevice.cc is wrong (should be 9 for 512 bytes). On my machine, this results in an overflow interrupt from the controller at the very first DMA command. Interestingly enough, this is not a problem in qemu, so it seems that qemu doesn't implement the overflow interrupt.

Nils-TUD commented 11 years ago

hm...that seems to have happened during the switching between size and sector-count. Thanks for reporting! Could you test it again please?

parthy commented 11 years ago

I already tested it with the correct shift before reporting, the disk test now works fine on my machine.