linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Boaz Harrosh <bharrosh@panasas.com>
To: Vladislav Bolkhovitin <vst@vlnb.net>
Cc: James Bottomley <James.Bottomley@suse.de>,
	John Chandy <john.chandy@uconn.edu>,
	open-osd <osd-dev@open-osd.org>,
	linux-scsi <linux-scsi@vger.kernel.org>
Subject: Re: [PATCH 4/4 ver2] libosd: write/read_sg_kern API
Date: Sun, 24 Oct 2010 11:55:30 +0200	[thread overview]
Message-ID: <4CC40292.3090500@panasas.com> (raw)
In-Reply-To: <4CC1CA91.1080003@vlnb.net>

On 10/22/2010 07:32 PM, Vladislav Bolkhovitin wrote:
> Boaz Harrosh, on 10/19/2010 06:13 PM wrote:
>> This is a trivial addition to the SG API that can receive kernel
>> pointers. It is only used by the out-of-tree test module. So
>> it's immediate need is questionable. For maintenance ease it might
>> just get in, as it's very small.
> 
> This patch doesn't handle queue's restrictions (DMA, etc.).
> 

Please for give me. I know that the name is misleading. The
Scatter-List here is the osd scatter-list. Which describes
extents on the object, not of the memory. The proper name would
be extents API. The reason we named it SG is because this is the
way the OSD standard names it. We keep very close with the STD
naming convention, so it ended up this way.

> Here is a complete patch for reading/writing SG lists from the kernel:
> http://lkml.org/lkml/2010/9/14/201
> 
> Vlad

Boaz

  reply	other threads:[~2010-10-24  9:55 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-19 12:18 [PATCHSET 0/4] libosd: scatter gather commands and stuff for 2.6.37 Boaz Harrosh
2010-10-19 12:20 ` [PATCH 1/4] libosd: Fix bug in attr_page handling Boaz Harrosh
2010-10-19 12:21 ` [PATCH 2/4] libosd: Free resources in reverse order of allocation Boaz Harrosh
2010-10-19 12:22 ` [PATCH 3/4] libosd: Support for scatter gather write/read commands Boaz Harrosh
2010-10-19 12:22 ` [PATCH 4/4] libosd: write/read_sg_kern API Boaz Harrosh
2010-10-19 14:13   ` Boaz Harrosh
2010-10-19 14:13   ` [PATCH 4/4 ver2] " Boaz Harrosh
2010-10-22 17:32     ` Vladislav Bolkhovitin
2010-10-24  9:55       ` Boaz Harrosh [this message]
2010-10-25 18:50         ` Vladislav Bolkhovitin
2010-10-26 10:07           ` Boaz Harrosh
2010-10-20 14:28   ` [PATCH 4/4] " John Chandy
2010-10-21 12:13     ` Boaz Harrosh
2010-11-01 16:13   ` Christoph Hellwig
2010-11-02  8:33     ` Boaz Harrosh

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=4CC40292.3090500@panasas.com \
    --to=bharrosh@panasas.com \
    --cc=James.Bottomley@suse.de \
    --cc=john.chandy@uconn.edu \
    --cc=linux-scsi@vger.kernel.org \
    --cc=osd-dev@open-osd.org \
    --cc=vst@vlnb.net \
    /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).