On Mon, 2006-07-31 at 12:20 -0500, Michael Reed wrote: > mptscsih_slave_destroy issues an unconditional synchronize cache. A) mptscsih_slave_destroy shouldn't be doing this ... it should be relying on the functionality in the mid-layer. > It is not necessary for fibre channel devices for which the rport > has been deleted. This patch creates a new function, > mptfc_slave_destroy(), which implements the appropriate test. So, this is really code that shouldn't be in a driver, it should be higher up. The other three patches look fine. James - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html