From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751577AbaI3Xtw (ORCPT ); Tue, 30 Sep 2014 19:49:52 -0400 Received: from mailout32.mail01.mtsvc.net ([216.70.64.70]:42387 "EHLO n23.mail01.mtsvc.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751004AbaI3Xtt (ORCPT ); Tue, 30 Sep 2014 19:49:49 -0400 Message-ID: <542B4194.8070604@hurleysoftware.com> Date: Tue, 30 Sep 2014 19:49:40 -0400 From: Peter Hurley User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.2 MIME-Version: 1.0 To: Thomas Gleixner CC: Akinobu Mita , LKML , Andrew Morton , Marek Szyprowski , Konrad Rzeszutek Wilk , David Woodhouse , Don Dutile , Ingo Molnar , "H. Peter Anvin" , Andi Kleen , x86@kernel.org, iommu@lists.linux-foundation.org, Greg KH Subject: Re: [PATCH v3 0/5] enhance DMA CMA on x86 References: <1397567329-3771-1-git-send-email-akinobu.mita@gmail.com> <5426CA0A.7000806@hurleysoftware.com> <54294C0B.1060705@hurleysoftware.com> <542ABF77.1020402@hurleysoftware.com> In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Authenticated-User: 990527 peter@hurleysoftware.com X-MT-ID: 8FA290C2A27252AACF65DBC4A42F3CE3735FB2A4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/30/2014 07:45 PM, Thomas Gleixner wrote: > On Tue, 30 Sep 2014, Peter Hurley wrote: >> I read the UFS Unified Memory Extension v1.0 (JESD220-1) specification and >> it is not clear to me that using DMA mapping is the right approach to >> supporting UM, at least on x86. >> >> And without a mainline user, the merits of this approach are not evident. >> I cannot even find a production x86 UFS controller, much less one that >> supports UME. >> >> The only PCI UFS controller I could find (and that mainline supports) is >> Samsung's x86 FPGA-based test unit for developing UFS devices in a x86 test >> environment, and not a production x86 design. > > And how is that relevant? That device exists and you have no reason to > deny it to be supported just because you are not interested in it. > >> Unless there's something else I've missed, I don't think these patches >> belong in mainline. > > You missed that there is no reason WHY such a device should not be > supported in mainline. Mainline already supports this card right now without these patches. >> Samsung's own roadmap >> (http://www.slideshare.net/linaroorg/next-gen-mobilestorageufs) >> mentions nothing about bringing UFS to x86 designs. > > And that's telling you what? > > - That we should deny Samsung proper support for their obviously > x86 based test card > > - That we should ignore a JEDEC Standard which is obviously never > going to hit x86 land just because you decide it? > > Your argumentation is just ass backwards. Linux wants to support the > full zoo of hardware including this particular PCI card. Period. > > Whether the proposed patchset is the correct solution to support it is > a completely different question. And there is currently no way to determine that because there is no user in mainline that requires this support. Which you would understand if you had read more carefully. Regards, Peter Hurley