[lkml]   [2010]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [GIT PULL] SCSI queuecommand API change for 2.6.37-rc1
    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."

     \ /
      Last update: 2010-11-13 05:31    [W:0.030 / U:1.256 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site