Re: [PATCH] parisc: DMA API: return error instead of BUG_ON for dma ops on non dma devs

From	Helge Deller <deller@gmx.de>
Newsgroups	linux.kernel
Subject	Re: [PATCH] parisc: DMA API: return error instead of BUG_ON for dma ops on non dma devs
Date	2017-07-05 23:00 +0200
Message-ID	<tZZyV-2US-3@gated-at.bofh.it> (permalink)
References	<tZ5n4-78c-23@gated-at.bofh.it> <tZaPP-2Pz-89@gated-at.bofh.it> <tZXGO-1Dt-23@gated-at.bofh.it> <tZYjw-2dl-21@gated-at.bofh.it> <tZZfA-2Op-21@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 05.07.2017 22:33, Christoph Hellwig wrote:
> On Wed, Jul 05, 2017 at 09:36:09PM +0200, Helge Deller wrote:
>> On 05.07.2017 20:57, Christoph Hellwig wrote:
>>> I've got a tree pending that removes DMA_ERROR_CODE, and it's been
>>> in linux-next for a while.
>>
>> I had the parisc patch in for-next as well and didn't received
>> any warnings.
> 
> There are no warnings, as the macro just won't be used by common code
> anymore.

And that's the reason why I didn't checked.
The generic DMA_ERROR_CODE has been there since 3.13 (or something
like that), so I don't think anybody would have assumed that it's
planned to vanish.
On the other side the patch fixed a real kernel crash on parisc,
and the patch applies as-is down to 3.13.
With that in mind, technically it's good that the parisc patch went in 
before yours.

> But the commit is from July 3rd, and the pull request to Linus for
> it was merged on the same day.  I can't see how it could have been
> in linux-next for long.  It certainly wasn't in the last linux-next
> release that I looked at before the long weekend (the July 30 one)

True.
Anyway, let's get your patch in now.

>> Can we please change this to something like 
>> "Prepare parisc to get rid of DMA_ERROR_CODE tree-wide" ?
>> It's somewhat more neutral.
> 
> Fine with me.

I did not changed it.

Helge

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] parisc: DMA API: return error instead of BUG_ON for dma  ops on non dma devs Helge Deller <deller@gmx.de> - 2017-07-03 16:50 +0200
  Re: [PATCH] parisc: DMA API: return error instead of BUG_ON for dma  ops on non dma devs Christoph Hellwig <hch@infradead.org> - 2017-07-05 21:00 +0200
    Re: [PATCH] parisc: DMA API: return error instead of BUG_ON for dma  ops on non dma devs Helge Deller <deller@gmx.de> - 2017-07-05 21:40 +0200
      Re: [PATCH] parisc: DMA API: return error instead of BUG_ON for dma  ops on non dma devs Christoph Hellwig <hch@infradead.org> - 2017-07-05 22:40 +0200
        Re: [PATCH] parisc: DMA API: return error instead of BUG_ON for dma  ops on non dma devs Helge Deller <deller@gmx.de> - 2017-07-05 23:00 +0200

csiph-web