lkml.org 
[lkml]   [2010]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] dma-mapping: Remove WARN_ON in dma_free_coherent
Matthew Wilcox wrote:
> On Mon, Mar 29, 2010 at 08:09:21PM +0200, Stefan Bader wrote:
>> But it seems modifying that driver would be a bigger effort and maybe
>> not really worthwile as it is a rather old driver. On the other side
>> I was told [1] that this WARN_ON applies only to ARM which could sleep
>> in the free path and it is just bogus for X86. As the code is in
>> arch/x86 it would never be used for anything else than X86 and if
>> its truely bogus on X86, couldn't it get removed?
>
> My suggestion was that you remove it from the Ubuntu kernel, not submit
> it upstream.
>
No, I did not want to give that impression. But my rational was, that if this is
really a warning that has no meaning on X86, then why should it stay in code
that is x86 only. And then why should that not go upstream.
When I tried following the code, it looked like this was carried forward for
quite a while and maybe it just stayed because nobody noticed it.
But with KVM using that scsi driver, other distros might see the same bogus
warning and then it would be beneficial for all to just remove it.
If Ingo or hpa say it is needed, ok. But its at least worth a go.

-Stefan


\
 
 \ /
  Last update: 2010-03-29 22:53    [W:0.038 / U:0.764 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site