Re: [PATCH V2 1/4] ACPI,PCI,IRQ: factor in PCI possible

2016-06-29 Thread Sinan Kaya
On 6/29/2016 5:19 PM, Rafael J. Wysocki wrote: > On Wed, Jun 29, 2016 at 8:47 PM, Sinan Kaya wrote: >> On 6/29/2016 9:13 AM, Rafael J. Wysocki wrote: >>> On Wed, Jun 29, 2016 at 10:27 AM, Sinan Kaya wrote: The change introduced in commit 103544d86976 ("ACPI,PCI,IRQ: reduce resource requ

Re: [PATCH V2 1/4] ACPI,PCI,IRQ: factor in PCI possible

2016-06-29 Thread Rafael J. Wysocki
On Wed, Jun 29, 2016 at 11:21 PM, Sinan Kaya wrote: > On 6/29/2016 5:19 PM, Rafael J. Wysocki wrote: >> On Wed, Jun 29, 2016 at 8:47 PM, Sinan Kaya wrote: >>> On 6/29/2016 9:13 AM, Rafael J. Wysocki wrote: On Wed, Jun 29, 2016 at 10:27 AM, Sinan Kaya wrote: > The change introduced in co

Re: [PATCH V2 1/4] ACPI,PCI,IRQ: factor in PCI possible

2016-06-29 Thread Rafael J. Wysocki
On Wed, Jun 29, 2016 at 11:21 PM, Sinan Kaya wrote: > On 6/29/2016 5:19 PM, Rafael J. Wysocki wrote: >> On Wed, Jun 29, 2016 at 8:47 PM, Sinan Kaya wrote: >>> On 6/29/2016 9:13 AM, Rafael J. Wysocki wrote: On Wed, Jun 29, 2016 at 10:27 AM, Sinan Kaya wrote: > The change introduced in co

Re: [PATCH V2 1/4] ACPI,PCI,IRQ: factor in PCI possible

2016-06-29 Thread Rafael J. Wysocki
On Wed, Jun 29, 2016 at 8:47 PM, Sinan Kaya wrote: > On 6/29/2016 9:13 AM, Rafael J. Wysocki wrote: >> On Wed, Jun 29, 2016 at 10:27 AM, Sinan Kaya wrote: >>> The change introduced in commit 103544d86976 ("ACPI,PCI,IRQ: reduce >>> resource requirements") omitted the initially assigned POSSIBLE pe

Re: [PATCH V2 1/4] ACPI,PCI,IRQ: factor in PCI possible

2016-06-29 Thread Sinan Kaya
On 6/29/2016 9:13 AM, Rafael J. Wysocki wrote: > On Wed, Jun 29, 2016 at 10:27 AM, Sinan Kaya wrote: >> The change introduced in commit 103544d86976 ("ACPI,PCI,IRQ: reduce >> resource requirements") omitted the initially assigned POSSIBLE penalty >> when the IRQ is active. > > It would be good to

Re: [PATCH V2 1/4] ACPI,PCI,IRQ: factor in PCI possible

2016-06-29 Thread Rafael J. Wysocki
On Wed, Jun 29, 2016 at 10:27 AM, Sinan Kaya wrote: > The change introduced in commit 103544d86976 ("ACPI,PCI,IRQ: reduce > resource requirements") omitted the initially assigned POSSIBLE penalty > when the IRQ is active. It would be good to say what can go wrong with that here. > The original c

[PATCH V2 1/4] ACPI,PCI,IRQ: factor in PCI possible

2016-06-29 Thread Sinan Kaya
The change introduced in commit 103544d86976 ("ACPI,PCI,IRQ: reduce resource requirements") omitted the initially assigned POSSIBLE penalty when the IRQ is active. The original code would assign the POSSIBLE value divided by the number of possible IRQs during initialization. Later, if the IRQ is