Date: Mon, 16 Feb 2009 10:25:58 -0600 (CST) From: Wes Morgan <morganw@chemikals.org> To: Scott Long <scottl@samsco.org> Cc: scsi@freebsd.org Subject: Re: HEADS UP: More CAM fixes. Message-ID: <alpine.BSF.2.00.0902161018540.30796@ibyngvyr.purzvxnyf.bet> In-Reply-To: <499981AF.9030204@samsco.org> References: <499981AF.9030204@samsco.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 16 Feb 2009, Scott Long wrote: > FWI. I need lots of testing on this. Only real SCSI controllers, please, > not RAID controllers (except for MPT-SCSI with integrated mirroring). So > Adaptec, LSI, Symbios, Buslogic, Tekram, SME, etc, > users, please try this and get back to me. The patch should apply > to FreeBSD 7 as well. FreeBSD 6 is only affected by this problem > when CAM_NEW_TRAN_CODE is enabled. I have a motherboard with an LSI SAS1068 8-port SAS controller, with SATA devices connected, running -current. All the drives are in passthrough mode, so none of the IM features are being used. Should this setup be affected, and if so, what should I look for? > > Scott > > > -------- Original Message -------- > Subject: svn commit: r188671 - head/sys/cam > Date: Mon, 16 Feb 2009 14:57:15 +0000 (UTC) > From: Scott Long <scottl@FreeBSD.org> > To: src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, > svn-src-head@FreeBSD.org > > Author: scottl > Date: Mon Feb 16 14:57:15 2009 > New Revision: 188671 > URL: http://svn.freebsd.org/changeset/base/188671 > > Log: > Fix parallel SCSI negotiation in the CAM_NEW_TRAN_CODE world order. > Overzealous sanity checks were locking the sync_rate and offset values to > zero, thanks to a twisty maze of recursive code. > > Modified: > head/sys/cam/cam_xpt.c > > Modified: head/sys/cam/cam_xpt.c > ============================================================================== > --- head/sys/cam/cam_xpt.c Mon Feb 16 14:38:52 2009 (r188670) > +++ head/sys/cam/cam_xpt.c Mon Feb 16 14:57:15 2009 (r188671) > @@ -6679,9 +6679,7 @@ xpt_set_transfer_settings(struct ccb_tra > if (((device->flags & CAM_DEV_INQUIRY_DATA_VALID) != 0 > && (inq_data->flags & SID_Sync) == 0 > && cts->type == CTS_TYPE_CURRENT_SETTINGS) > - || ((cpi.hba_inquiry & PI_SDTR_ABLE) == 0) > - || (spi->sync_offset == 0) > - || (spi->sync_period == 0)) { > + || ((cpi.hba_inquiry & PI_SDTR_ABLE) == 0)) { > /* Force async */ > spi->sync_period = 0; > spi->sync_offset = 0; > @@ -6729,7 +6727,8 @@ xpt_set_transfer_settings(struct ccb_tra > if (spi->bus_width == 0) > spi->ppr_options = 0; > > - if ((spi->flags & CTS_SPI_FLAGS_DISC_ENB) == 0) { > + if ((spi->valid & CTS_SPI_VALID_DISC) > + && ((spi->flags & CTS_SPI_FLAGS_DISC_ENB) == 0)) { > /* > * Can't tag queue without disconnection. > */ > _______________________________________________ > freebsd-stable@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-stable > To unsubscribe, send any mail to "freebsd-stable-unsubscribe@freebsd.org" >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.0902161018540.30796>