Date: Wed, 11 Jul 2012 12:16:18 -0700 From: Sean Bruno <seanbru@yahoo-inc.com> To: Scott Long <scott4long@yahoo.com> Cc: "freebsd-scsi@freebsd.org" <freebsd-scsi@freebsd.org>, "ambrisko@FreeBSD.ORG" <ambrisko@freebsd.org>, "scottl@freebsd.org" <scottl@freebsd.org> Subject: Re: [patch] MFI should set bio_resid on command failure Message-ID: <1342034178.9662.8.camel@powernoodle.corp.yahoo.com> In-Reply-To: <1341950347.63294.YahooMailNeo@web45701.mail.sp1.yahoo.com> References: <909AAC62-7BB4-43E7-B04B-27466B038A07@averesystems.com> <1341340916.3370.6.camel@powernoodle.corp.yahoo.com> <2936EBFD-CA1E-4EC4-9790-80D1A5DC5567@averesystems.com> <1341938629.2573.7.camel@powernoodle.corp.yahoo.com> <1341950347.63294.YahooMailNeo@web45701.mail.sp1.yahoo.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> >> >> When an MFI command fails, the driver needs to set > > bio->bio_resid so that the upper levels notice. Otherwise we see commands > > silently failing leading to data corruption. This mirrors dadone(). > >> >> > >> >> -Andrew > >> >> > >> >> Index: sys/dev/mfi/mfi_disk.c > >> >> > > =================================================================== > >> >> --- sys/dev/mfi/mfi_disk.c (revision 238071) > >> >> +++ sys/dev/mfi/mfi_disk.c (working copy) > >> >> @@ -298,6 +298,7 @@ > >> >> hdr = bio->bio_driver1; > >> >> > >> >> if (bio->bio_flags & BIO_ERROR) { > >> >> + bio->bio_resid = bio->bio_bcount; > >> >> if (bio->bio_error == 0) > >> >> bio->bio_error = EIO; > >> >> disk_err(bio, "hard error", -1, 1); > >> >> > >> >> -------------------------------------------------- > >> >> Andrew Boyer aboyer@averesystems.com Committed at svn r238371 Sean
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1342034178.9662.8.camel>