[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <576BF86E.8060302@codeaurora.org>
Date: Thu, 23 Jun 2016 10:55:42 -0400
From: Sinan Kaya <okaya@...eaurora.org>
To: wim@....tudelft.nl
Cc: Bjorn Helgaas <helgaas@...nel.org>,
"Rafael J. Wysocki" <rjw@...ysocki.net>,
linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
perex@...ex.cz, linux-pci@...r.kernel.org,
Takashi Iwai <tiwai@...e.com>, linux-pci-owner@...r.kernel.org
Subject: Re: kernel-4.7 bug in Intel sound and/or ACPI
On 6/23/2016 10:12 AM, Wim Osterholt wrote:
> On Wed, Jun 22, 2016 at 11:54:39PM -0400, okaya@...eaurora.org wrote:
>> On 2016-06-21 18:13, Wim Osterholt wrote:
>>>>
>>>> pr_info("%s:%d irq = %d penalty = %d\n", __func__, __LINE__, irq,
>>>> penalty);
>>>>
>>>
>>> This produced some 60 lines extra....
>>
>> Thanks, let's go back to 4.6 and add a very similar printf to every
>> single place where the array is modified and also right before the
>> enabled message.
>>
>
> I don't get this right.
> Assuming that you're still talking about the same file, I find a few
> instances of 'enabled', most of them in if-statements and one where it might
> be set, so it looks. However, that's already in a printk statement.
> I don't know about arrays and even less where these are set. Even worse, I
> don't know what to put in a 'similar' line if you don't mean 'exactly the
> same'.
> So please state file and line numbers and the line to be inserted.
>
Sure, let me get a patch for you. I was hoping to do it yesterday.
I ran out of time. I typed the message from my phone.
>
>
> Groeten, Wim.
>
>
> ----- wim@....tudelft.nl -----
>
--
Sinan Kaya
Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
Powered by blists - more mailing lists