Message-ID: <552654F1.6020300@hurleysoftware.com>
Date: Thu, 09 Apr 2015 06:31:13 -0400
From: Peter Hurley <peter@hurleysoftware.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0
MIME-Version: 1.0
To: Maxime Ripard <maxime.ripard@free-electrons.com>
CC: Vinod Koul <vinod.koul@intel.com>, Felipe Balbi <balbi@ti.com>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        Linux kernel <linux-kernel@vger.kernel.org>
Subject: Re: [4.0-rc+] WARNING: CPU: 0 PID: 1 at ./drivers/dma/dmaengine.c:863
 dma_async_device_register+0xe0/0x540()
References: <54F87075.5020105@hurleysoftware.com> <20150305161236.GG2613@intel.com> <55254EFD.10700@hurleysoftware.com> <20150408155945.GG26727@lukather> <55255232.40700@hurleysoftware.com> <20150409070927.GI26727@lukather>
In-Reply-To: <20150409070927.GI26727@lukather>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1747
Lines: 42

On 04/09/2015 03:09 AM, Maxime Ripard wrote:
> On Wed, Apr 08, 2015 at 12:07:14PM -0400, Peter Hurley wrote:
>> On 04/08/2015 11:59 AM, Maxime Ripard wrote:
>>> On Wed, Apr 08, 2015 at 11:53:33AM -0400, Peter Hurley wrote:
>>>> On 03/05/2015 11:12 AM, Vinod Koul wrote:
>>>>> On Thu, Mar 05, 2015 at 10:04:21AM -0500, Peter Hurley wrote:
>>>>>>> First boot of 4.0-rc+ gave me [1] on a Beaglebone Black due to
>>>>>>>
>>>>> The warning is intentional to get driver fixed and give the right behaviour
>>>>>
>>>>> Felipe had sent a patch for this but that one needs an update
>>>>
>>>> Vinod,
>>>>
>>>> Felipe's patch never made it to mainline, and this warning is 
>>>> still happening on 4.0-rc7 so please revert for 4.0-final.
>>>
>>> That patch isn't introducing any regression. I fail to see how that
>>> justifies a revert so late in the release cycle.
>>
>> Arguably it never should have been introduced in the first place
> 
> Upon which arguments? So far, I've seen none but "my boot logs look
> bad now".

1. Every subsystem has drivers that don't fully conform to a desired
   interface. If even a small fraction did this, everyone's logs would
   be cluttered with this garbage. For that matter, if just the serial
   subsystem alone did this, logs would be awash in warnings.

2. Now _every_ dmaengine driver "fix" for this will need to be pushed
   to -stable.

>> and this was at least known since Mar 5 (and likely earlier);
>> arguing that it's late in the cycle is a bit disingenuous.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/