Message-ID: <553760A8.2090802@profitbricks.com>
Date: Wed, 22 Apr 2015 10:49:44 +0200
From: Michael Wang <yun.wang@profitbricks.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0
MIME-Version: 1.0
To: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
CC: Roland Dreier <roland@kernel.org>, Sean Hefty <sean.hefty@intel.com>,
        linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org,
        hal@dev.mellanox.co.il, Tom Tucker <tom@opengridcomputing.com>,
        Steve Wise <swise@opengridcomputing.com>,
        Hoang-Nam Nguyen <hnguyen@de.ibm.com>,
        Christoph Raisch <raisch@de.ibm.com>,
        Mike Marciniszyn <infinipath@intel.com>, Eli Cohen <eli@mellanox.com>,
        Faisal Latif <faisal.latif@intel.com>,
        Jack Morgenstein <jackm@dev.mellanox.co.il>,
        Or Gerlitz <ogerlitz@mellanox.com>, Haggai Eran <haggaie@mellanox.com>,
        Ira Weiny <ira.weiny@intel.com>, Tom Talpey <tom@talpey.com>,
        Doug Ledford <dledford@redhat.com>
Subject: Re: [PATCH v5 22/27] IB/Verbs: Use management helper cap_ipoib()
References: <5534B8C9.506@profitbricks.com> <5534BBC2.4010902@profitbricks.com> <20150422054058.GB29609@obsidianresearch.com>
In-Reply-To: <20150422054058.GB29609@obsidianresearch.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 917
Lines: 28


On 04/22/2015 07:40 AM, Jason Gunthorpe wrote:
> On Mon, Apr 20, 2015 at 10:41:38AM +0200, Michael Wang wrote:
>  
>> Introduce helper cap_ipoib() to help us check if the port of an
>> IB device support IP over Infiniband.
> 
> I thought we were dropping this in favor of listing the actual
> features the ULP required unconditionally? One of my messages had the
> start of a list..

Shall we drop it now or wait until the mechanism introduced?

Just wondering the requirement of ULP could be similar to the
requirement of management, isn't it? if the device can tell
which ULP it support, then may be a cap_XX() make sense in here?

Regards,
Michael Wang

> 
> Jason
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/