Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 21 Jun 2022 16:19:24 +0200
From:   Christoph Hellwig <hch@lst.de>
To:     Mark Hounschell <markh@compro.net>
Cc:     Christoph Hellwig <hch@lst.de>,
        Robin Murphy <robin.murphy@arm.com>,
        Linux-kernel <linux-kernel@vger.kernel.org>,
        dmaengine@vger.kernel.org
Subject: Re: [BUG] dma-mapping: remove CONFIG_DMA_REMAP
Message-ID: <20220621141924.GA8348@lst.de>
References: <c32d2da1-9122-66bd-12fc-916be79b33fd@compro.net> <20220621134837.GA8025@lst.de> <9de341bc-fe8d-1820-187a-46455e4b9bf2@compro.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <9de341bc-fe8d-1820-187a-46455e4b9bf2@compro.net>
User-Agent: Mutt/1.5.17 (2007-11-01)
Precedence: bulk

On Tue, Jun 21, 2022 at 10:06:59AM -0400, Mark Hounschell wrote:
> On 6/21/22 09:48, Christoph Hellwig wrote:
>> On Tue, Jun 21, 2022 at 09:43:18AM -0400, Mark Hounschell wrote:
>>> Revert that commit and all works like normal. This commit breaks user land.
>>
>> No.  We had that discussion before.  It exposeѕ how broken your out of
>> tree driver is, which you don't bother to fix despite Robin even taking
>> the pains to explain you how.
>
> No, this is not the original issue and we never actually had a discussion. 
> That original issue was about using Set/ClearPageReserved. You nor Robin 
> even tried to explain why it was wrong to use it. It was never an issue in 
> previous kernels. Why now? In any case I have removed that code. This is 
> what happens now.
>
> What is it you think I am doing wrong. Except for using 
> Set/ClearPageReserved you have not explained anything to me.

Which part of "you must not call virt_to_page on the result that is
very clearly stated in the documentation and has been explained to
you repeatly" is still not clear to you?

Which part of "if your of tree modules stops working, this does not
constitute userspace breakage" is not clear to you?

I'm done with this, please stop bothering me.