All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Wise <swise-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
To: Jason Gunthorpe
	<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
Cc: Roland Dreier <rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>,
	Alekseys Senin <alekseys-smomgflXvOZWk0Htik3J/w@public.gmane.org>,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Eli Cohen <eli-VPRAkNaXOzVS1MOuV/RT9w@public.gmane.org>,
	"ewg-G2znmakfqn7U1rindQTSdQ@public.gmane.org"
	<ewg-G2znmakfqn7U1rindQTSdQ@public.gmane.org>,
	Moni Shoua <monis-smomgflXvOZWk0Htik3J/w@public.gmane.org>
Subject: Re: [ewg] [PATCH v4] IB Core: RAW ETH support
Date: Wed, 16 Jun 2010 12:12:06 -0500	[thread overview]
Message-ID: <4C1905E6.2000304@opengridcomputing.com> (raw)
In-Reply-To: <20100616170234.GA2932-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>

Jason Gunthorpe wrote:
> On Wed, Jun 16, 2010 at 09:09:59AM -0500, Steve Wise wrote:
>
>   
>> Granted our dev process may not be documented, but I always assumed the 
>> general idea was to get changes accepted upstream, then pull into ofed. 
>> OFED is just a mechanism to make top-of-tree linux work on distro 
>> kernels. There are some exceptions, but this stuff shouldn't be an 
>> exception.
>>     
>
> That is what many people wish for, me included, but it is not at all
> what generally happens :(
>
> In my observation the typical flow is:
>  - A patch is written, it may or may not be sent to the list
>  - 'business drivers' get it slammed into OFED right away
>  - A patch is finally sent for proper review
>  - It is not merged, there are comments..
>  - Interest in doing anything is lost because it is already in
>    OFED and that is all that matters, right?
>  - People complain.
>
> For instance, the iWarp thingy we were just discussing fits this
> process rather well.
>
>   

You're wrong.  I started that iWARP change in 2007 on LKLM.  I proposed 
a few ideas and show the pros/cons of each.  And it was NAKed 100% by 
mister miller.    It was then included in OFED as a last resort only 
because I couldn't get any help with trying to add this upstream in any 
form.  I even spent a few weeks developing a way to administor "iwarp 
only" ipaddresses, but Roland didn't like that scheme for various 
reasons.  So please don't mention that particular patch as a "bad 
process" unless you want to argue with me some more about it.

Also, the chelsio iWARP driver has 100% been upstream first, then 
ofed.   Some of us are indeed trying to do the right thing.

<steps off soap box>


--
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:[~2010-06-16 17:12 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E113D394D7C5DB4F8FF691FA7EE9DB4439A76E8632@MTLMAIL.mtl.com>
     [not found] ` <1276513450.30132.51.camel@alst60.voltaire.com>
     [not found]   ` <1276513450.30132.51.camel-uOVkuFIEnOODI2cvxHXf6UEOCMrvLtNR@public.gmane.org>
2010-06-15 21:14     ` [ewg] [PATCH v4] IB Core: RAW ETH support Roland Dreier
     [not found]       ` <aday6eguhfn.fsf-BjVyx320WGW9gfZ95n9DRSW4+XlvGpQz@public.gmane.org>
2010-06-16 14:09         ` Steve Wise
     [not found]           ` <4C18DB37.6050501-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
2010-06-16 15:47             ` Moni Shoua
     [not found]               ` <4C18F21D.3050502-hKgKHo2Ms0F+cjeuK/JdrQ@public.gmane.org>
2010-06-16 15:54                 ` Steve Wise
     [not found]                   ` <4C18F39D.3040609-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
2010-06-16 15:59                     ` Tung, Chien Tin
2010-06-16 17:02             ` Jason Gunthorpe
     [not found]               ` <20100616170234.GA2932-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-06-16 17:12                 ` Steve Wise [this message]
     [not found]                   ` <4C1905E6.2000304-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
2010-06-16 17:52                     ` Jason Gunthorpe
     [not found]                       ` <20100616175259.GF4630-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-06-16 18:35                         ` Steve Wise
     [not found]                           ` <4C19195B.1080003-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
2010-06-17 13:58                             ` Doug Ledford
2010-06-17 14:10                     ` Doug Ledford
2010-06-16 14:53         ` [ewg] " Moni Shoua
     [not found]           ` <4C18E557.5050804-hKgKHo2Ms0F+cjeuK/JdrQ@public.gmane.org>
2010-06-23 17:31             ` Roland Dreier
     [not found]               ` <ada4ogtvenr.fsf-BjVyx320WGW9gfZ95n9DRSW4+XlvGpQz@public.gmane.org>
2010-06-24 19:44                 ` Moni Shoua

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=4C1905E6.2000304@opengridcomputing.com \
    --to=swise-7bpotxp6k4+p2yhjcf5u+vpxobypeauw@public.gmane.org \
    --cc=alekseys-smomgflXvOZWk0Htik3J/w@public.gmane.org \
    --cc=eli-VPRAkNaXOzVS1MOuV/RT9w@public.gmane.org \
    --cc=ewg-G2znmakfqn7U1rindQTSdQ@public.gmane.org \
    --cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=monis-smomgflXvOZWk0Htik3J/w@public.gmane.org \
    --cc=rdreier-FYB4Gu1CFyUAvxtiuMwx3w@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.