MIME-Version: 1.0
In-Reply-To: <20170627173640.GA375@lst.de>
References: <1498468741-12020-1-git-send-email-vladimir.murzin@arm.com>
 <20170627152753.GA29186@lst.de> <20170627165416.GP4902@n2100.armlinux.org.uk> <20170627173640.GA375@lst.de>
From: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Date: Wed, 28 Jun 2017 09:33:33 +0200
Message-ID: <CA+M3ks6zBzh6oZ-LoVoFEu81vEeH_q=bhWnGrYAgDSpB+kfxyQ@mail.gmail.com>
Subject: Re: [PATCH v6 0/7] ARM: Fix dma_alloc_coherent() and friends for NOMMU
To: Christoph Hellwig <hch@lst.de>
Cc: Russell King - ARM Linux <linux@armlinux.org.uk>,
        Vladimir Murzin <vladimir.murzin@arm.com>,
        linux-arm-kernel@lists.infradead.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        =?UTF-8?B?U3plbXrFkSBBbmRyw6Fz?= <sza@esh.hu>,
        Arnd Bergmann <arnd@arndb.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        Alexandre Torgue <alexandre.torgue@st.com>, robin.murphy@arm.com,
        kbuild-all@01.org, Marek Szyprowski <m.szyprowski@samsung.com>,
        Joerg Roedel <jroedel@suse.de>,
        Christian Borntraeger <borntraeger@de.ibm.com>,
        Michal Nazarewicz <mina86@mina86.com>,
        Alan Stern <stern@rowland.harvard.edu>,
        Yoshinori Sato <ysato@users.sourceforge.jp>,
        Rich Felker <dalias@libc.org>, Roger Quadros <rogerq@ti.com>,
        Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>,
        Doug Ledford <dledford@redhat.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1113
Lines: 24

2017-06-27 19:36 GMT+02:00 Christoph Hellwig <hch@lst.de>:
> On Tue, Jun 27, 2017 at 05:54:17PM +0100, Russell King - ARM Linux wrote:
>> On Tue, Jun 27, 2017 at 05:27:53PM +0200, Christoph Hellwig wrote:
>> > I've applied patches 1, 3 and 4 to the dma-mapping tree.  2 will need
>> > a respin for less code duplication.  I'd be happy to take 5,6 and 7 if
>> > Russell is fine with that.
>>
>> If Vladimir is okay with those going in without patch 2, then I'm
>> happy - I seem to remember a mail from Vladimir stating that they
>> depended on some of the previous patches, but I don't know which.
>>
>> Acked-by: Russell King <rmk+kernel@armlinux.org.uk>
>
> Thanks, I'll add them.  Patch 2 only adds optional mmap functionality,
> so it's not really required.  Although I'd really like to see a fixed
> up version that I can merge anyway.

Without mmap we will have a regression for nommu platform which are using
dma_alloc_coherent and specialy in drm/kms where dumb buffer are allocated
like that.
Even with this problem I think the patches should be merged but we should have
a quick fix for mmap.

Benjamin