diff options
author | Roger Quadros <rogerq@ti.com> | 2014-04-01 17:26:40 +0300 |
---|---|---|
committer | Tom Rini <trini@ti.com> | 2014-04-02 15:44:40 -0400 |
commit | 3f62971162370213271dc1e7a396b6b3a6d5b6c6 (patch) | |
tree | a2702d6abcf36219d1688aa5313af5fe07b42f6d /include/div64.h | |
parent | 1336e2d343f088b71ec71907855caccd1053d166 (diff) | |
download | u-boot-imx-3f62971162370213271dc1e7a396b6b3a6d5b6c6.zip u-boot-imx-3f62971162370213271dc1e7a396b6b3a6d5b6c6.tar.gz u-boot-imx-3f62971162370213271dc1e7a396b6b3a6d5b6c6.tar.bz2 |
ahci: Fix data abort on multiple scsi resets.
Commit 2faf5fb82ed6 introduced a regression that causes a data
abort when running scsi init followed by scsi reset.
There are 2 problems with the original commit
1) ALLOC_CACHE_ALIGN_BUFFER() allocates memory on the stack but is
assigned to ataid[port] and used by other functions.
2) The function ata_scsiop_inquiry() tries to free memory which was
never allocated on the heap.
Fix these problems by using tmpid as a temporary cache aligned buffer.
Allocate memory separately for ataid[port] and re-use it if required.
Fixes: 2faf5fb82ed6 (ahci: Fix cache align error messages)
Reported-by: Eli Nidam <elini@marvell.com>
Signed-off-by: Roger Quadros <rogerq@ti.com>
Diffstat (limited to 'include/div64.h')
0 files changed, 0 insertions, 0 deletions