All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sebastian Riemer <sebastian.riemer-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
To: Vasiliy Tolstov <v.tolstov-+9FY0jupvH6HXe+LvDLADg@public.gmane.org>
Cc: Bart Van Assche <bvanassche-HInyCGIudOg@public.gmane.org>,
	Dongsu Park <dongsu.park-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	David Dillow <dillowda-1Heg1YXhbW8@public.gmane.org>,
	Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	scst-devel
	<scst-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
Subject: Re: [ANNOUNCE] SRP: ProfitBricks publishes its SRP Initiator patches
Date: Mon, 06 May 2013 10:44:41 +0200	[thread overview]
Message-ID: <51876D79.50003@profitbricks.com> (raw)
In-Reply-To: <CACaajQtQk0YJ3g0G5PyOue0qG3SzhaW3qmXj_TLqYDVqzKwJXQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Hi Vasiliy,

sorry for the late reply! I was ill last week.

The main difference so far is that my patches are much easier to
understand as I don't provide back-porting and also don't put
performance improvements in between the stability fixes. I provide full
test cases + scripts which makes it simple to check if you've got the
expected/same behavior. Everything is straight forward and completely
described in the commit messages.

My work focuses on using ib_srp without the srp_tools. Just fast IO
failing + reconnect. No performance improvements. We gradually find
compromises to get a cool solution. We've e.g. agreed on detecting
connection loss upon the QP timeout only.

Today I'll release our own automatic reconnect.

Sorry Bart, but a reconnect with just the commit message
"IB/srp: Add kernel-level transport layer recovery" and no further
description isn't very trustworthy for me. I also wonder why you need so
much locking.

Cheers,
Sebastian


On 03.05.2013 14:27, Vasiliy Tolstov wrote:
> 
> 03.05.2013 15:45 пользователь "Bart Van Assche" <bvanassche-HInyCGIudOg@public.gmane.org
> <mailto:bvanassche-HInyCGIudOg@public.gmane.org>> написал:
>>
>> On 04/30/13 09:34, Vasiliy Tolstov wrote:
>>>
>>> What is main difference between bvanassche repo and sriemer ?
>>
>>
>> Good question. As soon as I have the time I will try to find a single
> approach that works for everyone and post a new patch series for review
> on the linux-rdma mailing list such that these can be included in the
> mainline kernel.
>>
>> Bart.
> 
> Thats very good! Thans! I'm already using 3.8 with you patches and all
> works very stable.
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2013-05-06  8:44 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-12 11:19 [ANNOUNCE] SRP: ProfitBricks publishes its SRP Initiator patches Sebastian Riemer
     [not found] ` <5167EDC8.3030909-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2013-04-30  7:34   ` Vasiliy Tolstov
     [not found]     ` <CACaajQs0PswMb5KhpSkE1=_Sd1SwX2BznZP9x1FCkEk6ukJYrA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-05-03 11:45       ` Bart Van Assche
     [not found]         ` <CACaajQtQk0YJ3g0G5PyOue0qG3SzhaW3qmXj_TLqYDVqzKwJXQ@mail.gmail.com>
     [not found]           ` <CACaajQtQk0YJ3g0G5PyOue0qG3SzhaW3qmXj_TLqYDVqzKwJXQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-05-06  8:44             ` Sebastian Riemer [this message]
     [not found]               ` <51876D79.50003-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2013-05-06  9:44                 ` Bart Van Assche
     [not found]                   ` <51877B71.3080202-HInyCGIudOg@public.gmane.org>
2013-05-08 15:41                     ` Sebastian Riemer
     [not found]                       ` <518A7233.9070400-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2013-05-14 10:00                         ` Vasiliy Tolstov
     [not found]                           ` <CACaajQun61QAM0qqwadWECSFO7UxUaOrdO1-0vGLPQBjdwdMyw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-05-14 16:11                             ` Sebastian Riemer
2013-05-14 16:31                             ` Bart Van Assche
     [not found]                               ` <519266CA.2010306-HInyCGIudOg@public.gmane.org>
2013-05-15  5:12                                 ` Vasiliy Tolstov
     [not found]                                   ` <CACaajQt2V1Y-fQFmXyyAJk57KahbMwBqvEGoF7sug5F1hYyWGw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-05-15 10:47                                     ` Sebastian Riemer
2013-05-15 15:56                                     ` Bart Van Assche
     [not found]                                       ` <5193B040.5040802-HInyCGIudOg@public.gmane.org>
2013-05-16  6:51                                         ` Vasiliy Tolstov

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=51876D79.50003@profitbricks.com \
    --to=sebastian.riemer-eikl63zcoxah+58jc4qpia@public.gmane.org \
    --cc=bvanassche-HInyCGIudOg@public.gmane.org \
    --cc=dillowda-1Heg1YXhbW8@public.gmane.org \
    --cc=dongsu.park-EIkl63zCoXaH+58JC4qpiA@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=scst-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=v.tolstov-+9FY0jupvH6HXe+LvDLADg@public.gmane.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.