Date: Fri, 30 Mar 2012 18:15:10 -0400 From: Andrew Boyer <aboyer@averesystems.com> To: freebsd-scsi@freebsd.org Subject: Missing bio_resid on error case in mfi_disk_complete() Message-ID: <F84CD4B5-8C44-4B81-A313-960AA5976E4B@averesystems.com>
next in thread | raw e-mail | index | archive | help
Hello Doug,
We noticed that the AIO system doesn't react to I/O failures that come =
from mfi disks; the commands appear to complete. The upper layers are =
expecting bio->bio_resid to be non-zero if the bio failed. The patch =
below matches what scsi_da.c does in this case and also what mfi itself =
does in mfi_disk_strategy() if the I/O is invalid.
Does this make sense? I meant to get this email sent before you started =
on the big merge.
Thanks,
Andrew
Index: sys/dev/mfi/mfi_disk.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- sys/dev/mfi/mfi_disk.c (revision 233707)
+++ sys/dev/mfi/mfi_disk.c (working copy)
@@ -274,6 +274,7 @@
hdr =3D bio->bio_driver1;
=20
if (bio->bio_flags & BIO_ERROR) {
+ bio->bio_resid =3D bio->bio_bcount;
if (bio->bio_error =3D=3D 0)
bio->bio_error =3D EIO;
disk_err(bio, "hard error", -1, 1);
--------------------------------------------------
Andrew Boyer aboyer@averesystems.com
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?F84CD4B5-8C44-4B81-A313-960AA5976E4B>