All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shahar Salzman <shahar.salzman at kaminario.com>
To: spdk@lists.01.org
Subject: Re: [SPDK] SPDK + user space appliance
Date: Mon, 23 Apr 2018 15:44:48 +0000	[thread overview]
Message-ID: <DB4PR04MB37932724317481A05A85DDF89890@DB4PR04MB379.eurprd04.prod.outlook.com> (raw)
In-Reply-To: 1517607836.2371.98.camel@intel.com

[-- Attachment #1: Type: text/plain, Size: 2836 bytes --]

Hi Ben,


Bumping this thread since I've been having some new thoughts on the issue now that we are starting integration with newer spdk versions.

Unfortunately the merge isn't as smooth as I'd like it to be since the bdev module is pretty tightly integrated into spdk, perhaps we made some false assumptions writing the module, but it seems some of the newer spdk features are complicating the integration.

My question is, if this passthrough module is useful, wouldn't it be better to maintain it as part of spdk so that we can catch issues as soon as they show up?

We would be happy to help with maintaining this module, the module with is currently part of our CI with our "frozen" spdk version, but once integrated into the newer version we choose, I'll add it to the CI our CI as well.


Shahar

________________________________
From: SPDK <spdk-bounces(a)lists.01.org> on behalf of Walker, Benjamin <benjamin.walker(a)intel.com>
Sent: Friday, February 2, 2018 11:43:58 PM
To: spdk(a)lists.01.org
Subject: Re: [SPDK] SPDK + user space appliance

On Thu, 2018-02-01 at 08:29 +0000, Shahar Salzman wrote:
> Hi Ben,
>
> Would you also like to take a look at the bdev_user module?
> It still needs some patching (as some of the stuff is still hard coded), but I
> think we can get most of it cleaned up in a couple of days.
>
> In any case, is it the intention that the user write his own bdev module, or
> would this user appliance glue be a useful generic module?

For existing storage stacks that serve block I/O, like the internals of a SAN,
the idea is that you write your own bdev module to forward I/O coming out of the
SPDK bdev layer. Then you can use the SPDK iSCSI/NVMe-oF/vhost targets mostly
as-is.

In some cases, the actual iSCSI/NVMe-oF/vhost target applications won't
integrate nicely directly into an existing storage application because they
spawn their own threads and allocate their own memory. To support that, the
libraries may be consumed directly instead of the applications (lib/iscsi,
lib/scsi, lib/nvmf, etc.). The libraries don't spawn any of their own threads,
but instead rely on SPDK's abstractions in include/spdk/io_channel.h. See

http://www.spdk.io/doc/concurrency.html

We don't currently have a way to write a custom bdev module that resides outside
of the SPDK source tree, but it's very possible to add support for that. But
beyond that inconvenience (just drop your module in lib/bdev for now), writing a
bdev module is the recommended way of interacting with the bottom end of the
SPDK bdev layer. I think that's what you really want to be doing in your code,
from what I can tell.

I hope that helps!
_______________________________________________
SPDK mailing list
SPDK(a)lists.01.org
https://lists.01.org/mailman/listinfo/spdk


[-- Attachment #2: attachment.html --]
[-- Type: text/html, Size: 4124 bytes --]

             reply	other threads:[~2018-04-23 15:44 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-23 15:44 Shahar Salzman [this message]
  -- strict thread matches above, loose matches on Subject: below --
2018-07-16 22:58 [SPDK] SPDK + user space appliance 
2018-07-12 14:34 Shahar Salzman
2018-05-13  8:46 Shahar Salzman
2018-05-11 13:11 
2018-05-10 13:33 Shahar Salzman
2018-05-10  7:36 
2018-05-10  6:34 
2018-05-09 19:45 Shahar Salzman
2018-05-09 16:52 Walker, Benjamin
2018-05-09 11:57 Shahar Salzman
2018-05-09  9:54 
2018-05-09  8:43 
2018-05-09  7:48 Shahar Salzman
2018-05-08 17:30 Walker, Benjamin
2018-05-08  7:36 Shahar Salzman
2018-05-07 18:18 Harris, James R
2018-05-07 18:02 Walker, Benjamin
2018-05-06 10:32 Shahar Salzman
2018-05-03 17:50 Verkamp, Daniel
2018-05-03  8:00 Shahar Salzman
2018-04-25  6:02 Shahar Salzman
2018-04-24 16:19 Walker, Benjamin
2018-02-02 21:43 Walker, Benjamin
2018-02-01  8:29 Shahar Salzman
2018-01-31 16:48 Walker, Benjamin
2018-01-31 16:40 Walker, Benjamin
2018-01-25 14:19 Shahar Salzman
2018-01-23  8:21 Ilan Steinberg
2018-01-23  8:20 Ilan Steinberg
2018-01-19 17:48 Walker, Benjamin
2018-01-19 17:38 Aneesh Pachilangottil
2018-01-19 17:28 Walker, Benjamin
2018-01-18 23:53 Chang, Cunyin
2018-01-14  9:09 Shahar Salzman

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=DB4PR04MB37932724317481A05A85DDF89890@DB4PR04MB379.eurprd04.prod.outlook.com \
    --to=spdk@lists.01.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 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.