On Fri, 2006-03-24 at 19:56 -0800, Mike Anderson wrote: > Can I get your comment(s) on this change. This looks correct as far as it goes, but it's not complete. sas_phy_free() and sas_rphy_free() now do too many parent puts with this change (actually, two too many in each case, which looks like a bug in the original code). There also looks to be two spurious puts in the rphy (for expander and end device) allocation error paths (again, a bug in the original code). 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