lkml.org 
[lkml]   [2014]   [Jul]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 03/14] scsi: centralize command re-queueing in scsi_dispatch_fn
    On Tue, Jul 08, 2014 at 08:51:30PM +0000, Elliott, Robert (Server Storage) wrote:
    > In scsi_lib.c, both scsi_done() and scsi_mq_done() always call
    > trace_scsi_dispatch_cmd_done(), so trace_scsi_dispatch_cmd_start()
    > should be called before scsi_done() is called. That way the
    > trace will always have a submission to match each completion.
    >
    > That means trace should be called before the sdev_state check
    > (which calls scsi_done()).
    >
    > I don't know about the scsi_device_blocked check (which just
    > returns). Should the trace record multiple submissions with
    > one completion? Maybe both trace_scsi_dispatch_cmd_start()
    > and trace_scsi_dispatch_cmd_done() should both be called?

    trace_scsi_dispatch_cmd_start is maybe a little misnamed as it traces
    the command submission to the driver. So getting a done trace without
    this one sounds perfectly fine. Adding another trace for an error
    before submission could be done if you care about pairing. The *_BUSY
    returns don't fit this scheme at all.

    But none of this really is in this patch. Hannes has some plans to clean
    up the logging and tracing mess in scsi, and it might be a good idea
    to incorporate it there.



    \
     
     \ /
      Last update: 2014-07-09 09:21    [W:4.424 / U:0.356 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site