linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Brian King <brking@us.ibm.com>
To: jgarzik@pobox.com
Cc: linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: [PATCH 0/4] libata: Support SATA attached via SAS
Date: Wed, 28 Jun 2006 08:29:47 -0500	[thread overview]
Message-ID: <44A2844B.7010002@us.ibm.com> (raw)

Jeff,

Here are the remaining patches to my patchset to allow SAS HBAs to
use libata to control attached SATA devices. You had two concerns
in the last round of patches, which I have tried to address here.

1. Per device max command length checking
   - You were concerned with adding an additional check to the
     hot path. To address this I moved the check so that it is
     now only made for SAS users, which is really the only users that
     need it since existing SATA users have host->max_cmd_len
     protecting them.

2. Add ata_host_flags
   - You thought there might be a need to carry some cross port state.
     To resolve this I added a new API: ata_host_set_init. Callers
     of ata_sas_port_alloc must now pass their host_set pointer as the
     first argument. This removes the need for the patch you objected to
     and allows for a cross port state if needed.



-- 
Brian King
eServer Storage I/O
IBM Linux Technology Center

                 reply	other threads:[~2006-06-28 13:29 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=44A2844B.7010002@us.ibm.com \
    --to=brking@us.ibm.com \
    --cc=jgarzik@pobox.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).