]> err.no Git - linux-2.6/commitdiff
libata: fix boot panic with SATAPI devices on non-SFF HBAs
authorJames Bottomley <James.Bottomley@HansenPartnership.com>
Fri, 18 Apr 2008 18:18:48 +0000 (13:18 -0500)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 18 Apr 2008 18:20:42 +0000 (11:20 -0700)
The kernel now panics reliably on boot if you have a SATAPI device
connected.

The problem was introduced by the libata merge trying to pull out all
the SFF code into a separate module.  Unfortunately, if you're a satapi
device you usually need to call atapi_request_sense, which has a bare
invocation of a SFF callback which is NULL on non-SFF HBAs.  Fix this by
making the call conditional.

Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/ata/libata-scsi.c

index fedf62de94600d75fe8a683f647f3b9fd5fb1c07..c16e3cea1d28b2e941773f95bfc8368e86b774de 100644 (file)
@@ -2394,7 +2394,8 @@ static void atapi_request_sense(struct ata_queued_cmd *qc)
        memset(cmd->sense_buffer, 0, SCSI_SENSE_BUFFERSIZE);
 
 #ifdef CONFIG_ATA_SFF
-       ap->ops->sff_tf_read(ap, &qc->tf);
+       if (ap->ops->sff_tf_read)
+               ap->ops->sff_tf_read(ap, &qc->tf);
 #endif
 
        /* fill these in, for the case where they are -not- overwritten */