All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Christie <michaelc@cs.wisc.edu>
To: scameron@beardog.cce.hp.com
Cc: linux-scsi@vger.kernel.org
Subject: Re: If abort request comes in for command not known to LLD?
Date: Fri, 02 Mar 2012 15:10:02 -0600	[thread overview]
Message-ID: <4F51372A.8080309@cs.wisc.edu> (raw)
In-Reply-To: <20120302154438.GE17975@beardog.cce.hp.com>

On 03/02/2012 09:44 AM, scameron@beardog.cce.hp.com wrote:
> 
> What should the LLD do if an abort request comes into the
> abort error handler from the midlayer for a command which is
> not known to the LLD?
> 
> I see aic7xxx_osm.c handles it in this way in ahc_linux_queue_recovery_cmd():
> 
> no_cmd:
>         /*
>          * Our assumption is that if we don't have the command, no
>          * recovery action was required, so we return success.  Again,
>          * the semantics of the mid-layer recovery engine are not
>          * well defined, so this may change in time.
>          */
>         retval = SUCCESS;
> 
> Is that the right thing to do?  Seems a bit weird, but if that's
> the right thing to do, I can do that too.
> 

How do you hit this case?

I think it is ok. The reasons I have seen drivers hit it this is that
race where the driver is completing a command while the timer code is
starting to go off, or the cmd has timed out then the driver completes
the command before the abort code is run.

In those cases the driver has cleaned up its internal accounting because
the command has completed. At that point there is not much it can do
even if it wanted to. It does not have away to look up things like
internal tags/ids for the command.

  reply	other threads:[~2012-03-02 21:13 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-02 15:44 If abort request comes in for command not known to LLD? scameron
2012-03-02 21:10 ` Mike Christie [this message]
2012-03-02 23:01   ` scameron
2012-03-04 10:25     ` Mike Christie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F51372A.8080309@cs.wisc.edu \
    --to=michaelc@cs.wisc.edu \
    --cc=linux-scsi@vger.kernel.org \
    --cc=scameron@beardog.cce.hp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.