On Fri, Nov 12, 2010 at 05:42:06PM -0800, Linus Torvalds wrote: > And the sad thing is that this could _trivially_ have been fixed while > actually making the patch no bigger. Make the new function look like > > int func(struct Scsi_Host *shost, struct scsi_cmnd *cmd, void > (*done)(struct scsi_cmnd *)); Actually, it'd be even better if we took away an argument. int queuecommand(struct scsi_cmnd *cmd); Every single driver sets cmd->scsi_done = done; (some don't do it immediately, but they all do it eventually. Do it in the single caller, and eventually remove the assignments from the drivers). -- Matthew Wilcox Intel Open Source Technology Centre "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." -- To unsubscribe from this list: 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