Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1430241 > unrolled thread

Re: kernel-4.7 bug in Intel sound and/or ACPI

Started byWim Osterholt <wim@djo.tudelft.nl>
First post2016-06-24 01:30 +0200
Last post2016-06-30 11:50 +0200
Articles 11 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: kernel-4.7 bug in Intel sound and/or ACPI Wim Osterholt <wim@djo.tudelft.nl> - 2016-06-24 01:30 +0200
    Re: kernel-4.7 bug in Intel sound and/or ACPI Sinan Kaya <okaya@codeaurora.org> - 2016-06-24 08:10 +0200
      Re: kernel-4.7 bug in Intel sound and/or ACPI Wim Osterholt <wim@djo.tudelft.nl> - 2016-06-25 03:40 +0200
        Re: kernel-4.7 bug in Intel sound and/or ACPI okaya@codeaurora.org - 2016-06-25 11:00 +0200
          Re: kernel-4.7 bug in Intel sound and/or ACPI Wim Osterholt <wim@djo.tudelft.nl> - 2016-06-27 08:30 +0200
            Re: kernel-4.7 bug in Intel sound and/or ACPI okaya@codeaurora.org - 2016-06-27 10:30 +0200
              Re: kernel-4.7 bug in Intel sound and/or ACPI Wim Osterholt <wim@djo.tudelft.nl> - 2016-06-27 15:10 +0200
                Re: kernel-4.7 bug in Intel sound and/or ACPI okaya@codeaurora.org - 2016-06-27 23:10 +0200
                  Re: kernel-4.7 bug in Intel sound and/or ACPI Sinan Kaya <okaya@codeaurora.org> - 2016-06-29 10:40 +0200
                    Re: kernel-4.7 bug in Intel sound and/or ACPI Wim Osterholt <wim@djo.tudelft.nl> - 2016-06-30 04:40 +0200
                      Re: kernel-4.7 bug in Intel sound and/or ACPI okaya@codeaurora.org - 2016-06-30 11:50 +0200

#1430241 — Re: kernel-4.7 bug in Intel sound and/or ACPI

FromWim Osterholt <wim@djo.tudelft.nl>
Date2016-06-24 01:30 +0200
SubjectRe: kernel-4.7 bug in Intel sound and/or ACPI
Message-ID<rNmel-eU-9@gated-at.bofh.it>
On Thu, Jun 23, 2016 at 11:45:47AM -0400, Sinan Kaya wrote:
> > 
> > Sure, let me get a patch for you.
> 
> Here it is

http://webserver.djo.tudelft.nl/dmesg460+printpatch2


> I am trying to find a system with similar characteristics for debug

All from the same laptop, Dell Inspiron 4100.
The same problem arises at a Dell Inspiron 510m.
I've not seen it on a workstation Dell XW4300.



Groeten, Wim.


----- wim@djo.tudelft.nl -----

[toc] | [next] | [standalone]

#1430380

FromSinan Kaya <okaya@codeaurora.org>
Date2016-06-24 08:10 +0200
Message-ID<rNstr-4rr-1@gated-at.bofh.it>
In reply to#1430241

[Multipart message — attachments visible in raw view] — view raw

On 6/23/2016 7:25 PM, Wim Osterholt wrote:
> On Thu, Jun 23, 2016 at 11:45:47AM -0400, Sinan Kaya wrote:
>>>
>>> Sure, let me get a patch for you.
>>
>> Here it is
> 
> http://webserver.djo.tudelft.nl/dmesg460+printpatch2
> 

Thanks, this was very helpful. I was able to fix the problem by using
the values in your log.

Can you give it a try?


> 
>> I am trying to find a system with similar characteristics for debug
> 
> All from the same laptop, Dell Inspiron 4100.
> The same problem arises at a Dell Inspiron 510m.
> I've not seen it on a workstation Dell XW4300.
> 
> 
> 
> Groeten, Wim.
> 
> 
> ----- wim@djo.tudelft.nl -----
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


-- 
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

[toc] | [prev] | [next] | [standalone]

#1431043

FromWim Osterholt <wim@djo.tudelft.nl>
Date2016-06-25 03:40 +0200
Message-ID<rNKJI-7uK-3@gated-at.bofh.it>
In reply to#1430380
On Fri, Jun 24, 2016 at 02:09:15AM -0400, Sinan Kaya wrote:
> 
> Can you give it a try?

Whell, I tried to no avail.

Wether it is on 4.6 or 4.7, with or without your previous patch,
I keep getting rejected hunks.
For example, here the line to be deleted is nowhere to be found:

> diff --git a/drivers/acpi/pci_link.c b/drivers/acpi/pci_link.c
> index 714ba4d..c2f22c9 100644
> --- a/drivers/acpi/pci_link.c
> +++ b/drivers/acpi/pci_link.c
> @@ -497,7 +497,7 @@ static int acpi_irq_get_penalty(int irq)
>  	int penalty = 0;
>  
>  	if (irq < ACPI_MAX_ISA_IRQS)
> -		penalty += acpi_isa_irq_penalty[irq];
> +		return acpi_isa_irq_penalty[irq];
>  
>  	/*
>  	* Penalize IRQ used by ACPI SCI. If ACPI SCI pin attributes conflict


Regards, Wim.


----- wim@djo.tudelft.nl -----

[toc] | [prev] | [next] | [standalone]

#1431101

Fromokaya@codeaurora.org
Date2016-06-25 11:00 +0200
Message-ID<rNRBv-3iY-3@gated-at.bofh.it>
In reply to#1431043
On 2016-06-24 21:39, Wim Osterholt wrote:
> On Fri, Jun 24, 2016 at 02:09:15AM -0400, Sinan Kaya wrote:
>> 
>> Can you give it a try?
> 
> Whell, I tried to no avail.
> 
> Wether it is on 4.6 or 4.7, with or without your previous patch,
> I keep getting rejected hunks.
> For example, here the line to be deleted is nowhere to be found:
> 
>> diff --git a/drivers/acpi/pci_link.c b/drivers/acpi/pci_link.c
>> index 714ba4d..c2f22c9 100644
>> --- a/drivers/acpi/pci_link.c
>> +++ b/drivers/acpi/pci_link.c
>> @@ -497,7 +497,7 @@ static int acpi_irq_get_penalty(int irq)
>>  	int penalty = 0;
>> 
>>  	if (irq < ACPI_MAX_ISA_IRQS)
>> -		penalty += acpi_isa_irq_penalty[irq];
>> +		return acpi_isa_irq_penalty[irq];
>> 
>>  	/*
>>  	* Penalize IRQ used by ACPI SCI. If ACPI SCI pin attributes conflict
> 
> 
> Regards, Wim.
> 
> 
> ----- wim@djo.tudelft.nl -----

Please apply the patches on top of clean 4.7-rc4 tree and apply them in 
order with

git am 0001...
git am 0002...

[toc] | [prev] | [next] | [standalone]

#1431744

FromWim Osterholt <wim@djo.tudelft.nl>
Date2016-06-27 08:30 +0200
Message-ID<rOydr-502-3@gated-at.bofh.it>
In reply to#1431101
On Sat, Jun 25, 2016 at 04:51:03AM -0400, okaya@codeaurora.org wrote:
> On 2016-06-24 21:39, Wim Osterholt wrote:
> 
> Please apply the patches on top of clean 4.7-rc4 tree and apply them in 
> order with
> 
> git am 0001...
> git am 0002...

It doesn't work that way.
Beginners problems with git.
Tried all kinds of things, including a new git clone to no avail.
Took a long time to discover that in the above example these were the names
of the 'attachments'.
It didn't work. Error 'could not find out the kind of patch' or some such.
Took much longer to find out that in that mail and in the saved attachments
there were lines beginning with '>From' that were the culprit.

Still not found out how to make patch work without errors.
Anyway, by doing it with more manual intervention om a stock kernel-4.7-rc4
on my (very slow) Inspiron 4100 it seems to work like before. Hooray.
However, an earlier try on my Inspiron 510m did not work.
I'll do a clean retry later today, just to make sure.

regards, Wim.


----- wim@djo.tudelft.nl -----

[toc] | [prev] | [next] | [standalone]

#1431810

Fromokaya@codeaurora.org
Date2016-06-27 10:30 +0200
Message-ID<rOA5z-69P-7@gated-at.bofh.it>
In reply to#1431744
On 2016-06-27 02:27, Wim Osterholt wrote:
> On Sat, Jun 25, 2016 at 04:51:03AM -0400, okaya@codeaurora.org wrote:
>> On 2016-06-24 21:39, Wim Osterholt wrote:
>> 
>> Please apply the patches on top of clean 4.7-rc4 tree and apply them 
>> in
>> order with
>> 
>> git am 0001...
>> git am 0002...
> 
> It doesn't work that way.
> Beginners problems with git.
> Tried all kinds of things, including a new git clone to no avail.
> Took a long time to discover that in the above example these were the 
> names
> of the 'attachments'.
> It didn't work. Error 'could not find out the kind of patch' or some 
> such.
> Took much longer to find out that in that mail and in the saved 
> attachments
> there were lines beginning with '>From' that were the culprit.
> 
> Still not found out how to make patch work without errors.
> Anyway, by doing it with more manual intervention om a stock 
> kernel-4.7-rc4
> on my (very slow) Inspiron 4100 it seems to work like before. Hooray.
> However, an earlier try on my Inspiron 510m did not work.
> I'll do a clean retry later today, just to make sure.


Ok, let me know. I can post a clean patch series. I was trying to get 
you something to test before posting the official version.

> 
> regards, Wim.
> 
> 
> ----- wim@djo.tudelft.nl -----
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[toc] | [prev] | [next] | [standalone]

#1432031

FromWim Osterholt <wim@djo.tudelft.nl>
Date2016-06-27 15:10 +0200
Message-ID<rOEsz-DU-77@gated-at.bofh.it>
In reply to#1431810
On Mon, Jun 27, 2016 at 04:22:18AM -0400, okaya@codeaurora.org wrote:
> > However, an earlier try on my Inspiron 510m did not work.
> > I'll do a clean retry later today, just to make sure.
> 
> 
> Ok, let me know. I can post a clean patch series. I was trying to get 
> you something to test before posting the official version.

The 510m just finished compiling and now it works fine too.
Thanks.



Regards, Wim.


----- wim@djo.tudelft.nl -----

[toc] | [prev] | [next] | [standalone]

#1432318

Fromokaya@codeaurora.org
Date2016-06-27 23:10 +0200
Message-ID<rOLX4-5DX-27@gated-at.bofh.it>
In reply to#1432031
On 2016-06-27 16:04, Wim Osterholt wrote:
> On Mon, Jun 27, 2016 at 04:22:18AM -0400, okaya@codeaurora.org wrote:
>> > However, an earlier try on my Inspiron 510m did not work.
>> > I'll do a clean retry later today, just to make sure.
>> 
>> 
>> Ok, let me know. I can post a clean patch series. I was trying to get
>> you something to test before posting the official version.
> 
> The 510m just finished compiling and now it works fine too.
> Thanks.
> 

Nice, I will post the official version and let you know.

> 
> 
> Regards, Wim.
> 
> 
> ----- wim@djo.tudelft.nl -----
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[toc] | [prev] | [next] | [standalone]

#1433561

FromSinan Kaya <okaya@codeaurora.org>
Date2016-06-29 10:40 +0200
Message-ID<rPjcm-1RY-25@gated-at.bofh.it>
In reply to#1432318
On 6/27/2016 5:05 PM, okaya@codeaurora.org wrote:
> On 2016-06-27 16:04, Wim Osterholt wrote:
>> On Mon, Jun 27, 2016 at 04:22:18AM -0400, okaya@codeaurora.org wrote:
>>> > However, an earlier try on my Inspiron 510m did not work.
>>> > I'll do a clean retry later today, just to make sure.
>>>
>>>
>>> Ok, let me know. I can post a clean patch series. I was trying to get
>>> you something to test before posting the official version.
>>
>> The 510m just finished compiling and now it works fine too.
>> Thanks.
>>
> 
> Nice, I will post the official version and let you know.
> 

I posted this [PATCH V2 0/4] ACPI,PCI,IRQ: correct ISA penalty calculation

Can you test this and give me a tested-by?

Alex,
If you could also review this, it would be great.

Sinan

>>
>>
>> Regards, Wim.
>>
>>
>> ----- wim@djo.tudelft.nl -----
>>
>> -- 
>> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> -- 
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
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

[toc] | [prev] | [next] | [standalone]

#1434183

FromWim Osterholt <wim@djo.tudelft.nl>
Date2016-06-30 04:40 +0200
Message-ID<rPA3v-3Ri-1@gated-at.bofh.it>
In reply to#1433561
On Wed, Jun 29, 2016 at 04:34:14AM -0400, Sinan Kaya wrote:
> > 
> 
> I posted this [PATCH V2 0/4] ACPI,PCI,IRQ: correct ISA penalty calculation
> 
> Can you test this and give me a tested-by?


Kernel-4.7-rc5 plus this patch works like a charm on my Inspiron 4100.
Dmesg output is quite similar to the one from kernel-4.6 .

Tested-by: Wim Osterholt. <wim@djo.tudelft.nl>

[toc] | [prev] | [next] | [standalone]

#1434415

Fromokaya@codeaurora.org
Date2016-06-30 11:50 +0200
Message-ID<rPGLE-7VT-11@gated-at.bofh.it>
In reply to#1434183
On 2016-06-30 05:30, Wim Osterholt wrote:
> On Wed, Jun 29, 2016 at 04:34:14AM -0400, Sinan Kaya wrote:
>> >
>> 
>> I posted this [PATCH V2 0/4] ACPI,PCI,IRQ: correct ISA penalty 
>> calculation
>> 
>> Can you test this and give me a tested-by?
> 
> 
> Kernel-4.7-rc5 plus this patch works like a charm on my Inspiron 4100.
> Dmesg output is quite similar to the one from kernel-4.6 .
> 
> Tested-by: Wim Osterholt. <wim@djo.tudelft.nl>
> 

Thank you.


> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web