From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from exil.voltaire.com (fwil.voltaire.com [193.47.165.2]) by ozlabs.org (Postfix) with ESMTP id 5CE1EDDEA0 for ; Wed, 12 Dec 2007 23:26:41 +1100 (EST) Message-ID: <475FD0A1.3090304@voltaire.com> Date: Wed, 12 Dec 2007 14:14:25 +0200 From: Or Gerlitz MIME-Version: 1.0 To: Joachim Fenkes Subject: Re: [ewg] Re: [PATCH] IB/ehca: Serialize HCA-related hCalls on POWER5 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Cc: Arnd Bergmann , Roland Dreier , LKML , linuxppc-dev@ozlabs.org, Christoph Raisch , OF-General , Stefan Roscher List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Joachim Fenkes wrote: > Roland Dreier wrote on 10.12.2007 22:47:37: >> It's an optional device feature, so this should be OK >> (although the iSER driver currently seems to depend on a device >> supporting FMRs, which is probably going to be a problem with iWARP >> support in the future anyway). > I don't feel very well with removing code from the driver that iSER seems > to depend on. Are there plans to fix this in iSER? What is the fix you suggest, to add a device query that tells you for which verbs the documentation does not apply? or enhance the code of the map_phys_fmr verb within the ehca driver to return error if called from non-sleepable context? Or.