diff options
author | Philip J Kelleher <pjk1939@linux.vnet.ibm.com> | 2013-03-26 12:03:07 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2013-03-26 16:48:11 -0400 |
commit | 4dcaf47258d59010802bd0eda933f69ee7d98cc7 (patch) | |
tree | b1787a3dcdd4a1ce5d59d0ba0ccff811fc30f268 /drivers/block/rsxx/rsxx_priv.h | |
parent | d8d595dfce7925627de78b9eecc8598a6ffda610 (diff) |
rsxx: enable error return of rsxx_eeh_save_issued_dmas()
Commit d8d595df introduced a bug where we did not check for a NULL
return from kmalloc(). Make rsxx_eeh_save_issued_dmas() return an
error for that case, and make the callers handle that.
Signed-off-by: Philip J Kelleher <pjk1939@linux.vnet.ibm.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block/rsxx/rsxx_priv.h')
-rw-r--r-- | drivers/block/rsxx/rsxx_priv.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/rsxx/rsxx_priv.h b/drivers/block/rsxx/rsxx_priv.h index 8a7ac87f1dc5..382e8bf5c03b 100644 --- a/drivers/block/rsxx/rsxx_priv.h +++ b/drivers/block/rsxx/rsxx_priv.h | |||
@@ -381,7 +381,7 @@ int rsxx_dma_queue_bio(struct rsxx_cardinfo *card, | |||
381 | rsxx_dma_cb cb, | 381 | rsxx_dma_cb cb, |
382 | void *cb_data); | 382 | void *cb_data); |
383 | int rsxx_hw_buffers_init(struct pci_dev *dev, struct rsxx_dma_ctrl *ctrl); | 383 | int rsxx_hw_buffers_init(struct pci_dev *dev, struct rsxx_dma_ctrl *ctrl); |
384 | void rsxx_eeh_save_issued_dmas(struct rsxx_cardinfo *card); | 384 | int rsxx_eeh_save_issued_dmas(struct rsxx_cardinfo *card); |
385 | void rsxx_eeh_cancel_dmas(struct rsxx_cardinfo *card); | 385 | void rsxx_eeh_cancel_dmas(struct rsxx_cardinfo *card); |
386 | int rsxx_eeh_remap_dmas(struct rsxx_cardinfo *card); | 386 | int rsxx_eeh_remap_dmas(struct rsxx_cardinfo *card); |
387 | 387 | ||