diff options
| author | Al Viro <viro@www.linux.org.uk> | 2005-04-24 15:28:34 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-04-24 15:28:34 -0400 |
| commit | bc86120a85ebf553180946e8854d1993f074e9f5 (patch) | |
| tree | d2d84a28a50328d8d910176b4031482ee308bc55 /drivers/scsi/scsi_error.c | |
| parent | df6c6804ce1d383c878ab071bce8125125d96abc (diff) | |
[PATCH] SCSI GFP fixes
Somebody forgot that | has higher priority than ?:. As the result,
allocation is done with bogus flags - instead of GFP_ATOMIC + possibly
GFP_DMA we always get GFP_DMA and no GFP_ATOMIC.
Signed-off-by: Al Viro <viro@parcelfarce.linux.theplanet.co.uk>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/scsi/scsi_error.c')
| -rw-r--r-- | drivers/scsi/scsi_error.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c index 1a135f38e78d..2bf1ee2b47b6 100644 --- a/drivers/scsi/scsi_error.c +++ b/drivers/scsi/scsi_error.c | |||
| @@ -584,7 +584,7 @@ static int scsi_request_sense(struct scsi_cmnd *scmd) | |||
| 584 | 584 | ||
| 585 | memcpy(scmd->cmnd, generic_sense, sizeof(generic_sense)); | 585 | memcpy(scmd->cmnd, generic_sense, sizeof(generic_sense)); |
| 586 | 586 | ||
| 587 | scsi_result = kmalloc(252, GFP_ATOMIC | (scmd->device->host->hostt->unchecked_isa_dma) ? __GFP_DMA : 0); | 587 | scsi_result = kmalloc(252, GFP_ATOMIC | ((scmd->device->host->hostt->unchecked_isa_dma) ? __GFP_DMA : 0)); |
| 588 | 588 | ||
| 589 | 589 | ||
| 590 | if (unlikely(!scsi_result)) { | 590 | if (unlikely(!scsi_result)) { |