* [PATCH] mfd: kempld-core: don't replace resources provided by ACPI
@ 2024-02-21 17:10 Michael Brunner
2024-02-21 17:34 ` Guenter Roeck
0 siblings, 1 reply; 8+ messages in thread
From: Michael Brunner @ 2024-02-21 17:10 UTC (permalink / raw)
To: lee.jones@linaro.org
Cc: linux@roeck-us.net, mibru@gmx.de, linux-kernel@vger.kernel.org
The current implementation to retrieve ACPI resources is faulty
and may cause issues that even can lead to non-booting systems.
When adding data from an ACPI device, the resources are already
assigned to the platform device. Therefore there is no need to
retrieve the resource list from ACPI and manually assign it to
the platform device. Also there shouldn't be any BIOS in the wild
anymore, that does not have resources added to the KEMPLD ACPI
data.
In particular this fixes an issue where the retrieval of the
resource list using /proc/ioports is disturbed and does not list
the assigned resource for the kempld device or even no resources
at all.
On some distributions this also leads to problems during system
initialization (e.g. with udev) and causes the system to not
boot at all.
I have reproduced the issue with the following kernel versions:
5.10.209
5.15.148
6.1.25
6.6.17
6.7.5
6.8-rc5
The patch applies to all of those versions and seems to resolve
the issue.
Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
---
drivers/mfd/kempld-core.c | 37 -------------------------------------
1 file changed, 37 deletions(-)
diff --git a/drivers/mfd/kempld-core.c b/drivers/mfd/kempld-core.c
index 67af36a38913..5557f023a173 100644
--- a/drivers/mfd/kempld-core.c
+++ b/drivers/mfd/kempld-core.c
@@ -428,50 +428,13 @@ static int kempld_detect_device(struct kempld_device_data *pld)
#ifdef CONFIG_ACPI
static int kempld_get_acpi_data(struct platform_device *pdev)
{
- struct list_head resource_list;
- struct resource *resources;
- struct resource_entry *rentry;
struct device *dev = &pdev->dev;
- struct acpi_device *acpi_dev = ACPI_COMPANION(dev);
const struct kempld_platform_data *pdata;
int ret;
- int count;
pdata = acpi_device_get_match_data(dev);
ret = platform_device_add_data(pdev, pdata,
sizeof(struct kempld_platform_data));
- if (ret)
- return ret;
-
- INIT_LIST_HEAD(&resource_list);
- ret = acpi_dev_get_resources(acpi_dev, &resource_list, NULL, NULL);
- if (ret < 0)
- goto out;
-
- count = ret;
-
- if (count == 0) {
- ret = platform_device_add_resources(pdev, pdata->ioresource, 1);
- goto out;
- }
-
- resources = devm_kcalloc(&acpi_dev->dev, count, sizeof(*resources),
- GFP_KERNEL);
- if (!resources) {
- ret = -ENOMEM;
- goto out;
- }
-
- count = 0;
- list_for_each_entry(rentry, &resource_list, node) {
- memcpy(&resources[count], rentry->res,
- sizeof(*resources));
- count++;
- }
- ret = platform_device_add_resources(pdev, resources, count);
-
-out:
- acpi_dev_free_resource_list(&resource_list);
return ret;
}
--
2.25.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] mfd: kempld-core: don't replace resources provided by ACPI
2024-02-21 17:10 [PATCH] mfd: kempld-core: don't replace resources provided by ACPI Michael Brunner
@ 2024-02-21 17:34 ` Guenter Roeck
2024-02-21 17:50 ` Michael Brunner
0 siblings, 1 reply; 8+ messages in thread
From: Guenter Roeck @ 2024-02-21 17:34 UTC (permalink / raw)
To: Michael Brunner, lee.jones@linaro.org
Cc: mibru@gmx.de, linux-kernel@vger.kernel.org
On 2/21/24 09:10, Michael Brunner wrote:
> The current implementation to retrieve ACPI resources is faulty
> and may cause issues that even can lead to non-booting systems.
>
> When adding data from an ACPI device, the resources are already
> assigned to the platform device. Therefore there is no need to
> retrieve the resource list from ACPI and manually assign it to
> the platform device. Also there shouldn't be any BIOS in the wild
> anymore, that does not have resources added to the KEMPLD ACPI
> data.
>
> In particular this fixes an issue where the retrieval of the
> resource list using /proc/ioports is disturbed and does not list
> the assigned resource for the kempld device or even no resources
> at all.
> On some distributions this also leads to problems during system
> initialization (e.g. with udev) and causes the system to not
> boot at all.
>
> I have reproduced the issue with the following kernel versions:
> 5.10.209
> 5.15.148
> 6.1.25
> 6.6.17
> 6.7.5
> 6.8-rc5
>
> The patch applies to all of those versions and seems to resolve
> the issue.
>
> Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> ---
> drivers/mfd/kempld-core.c | 37 -------------------------------------
> 1 file changed, 37 deletions(-)
>
> diff --git a/drivers/mfd/kempld-core.c b/drivers/mfd/kempld-core.c
> index 67af36a38913..5557f023a173 100644
> --- a/drivers/mfd/kempld-core.c
> +++ b/drivers/mfd/kempld-core.c
> @@ -428,50 +428,13 @@ static int kempld_detect_device(struct kempld_device_data *pld)
> #ifdef CONFIG_ACPI
> static int kempld_get_acpi_data(struct platform_device *pdev)
> {
> - struct list_head resource_list;
> - struct resource *resources;
> - struct resource_entry *rentry;
> struct device *dev = &pdev->dev;
> - struct acpi_device *acpi_dev = ACPI_COMPANION(dev);
> const struct kempld_platform_data *pdata;
> int ret;
> - int count;
>
> pdata = acpi_device_get_match_data(dev);
> ret = platform_device_add_data(pdev, pdata,
> sizeof(struct kempld_platform_data));
> - if (ret)
> - return ret;
> -
> - INIT_LIST_HEAD(&resource_list);
> - ret = acpi_dev_get_resources(acpi_dev, &resource_list, NULL, NULL);
> - if (ret < 0)
> - goto out;
> -
> - count = ret;
> -
> - if (count == 0) {
> - ret = platform_device_add_resources(pdev, pdata->ioresource, 1);
> - goto out;
> - }
> -
> - resources = devm_kcalloc(&acpi_dev->dev, count, sizeof(*resources),
> - GFP_KERNEL);
> - if (!resources) {
> - ret = -ENOMEM;
> - goto out;
> - }
> -
> - count = 0;
> - list_for_each_entry(rentry, &resource_list, node) {
> - memcpy(&resources[count], rentry->res,
> - sizeof(*resources));
> - count++;
> - }
> - ret = platform_device_add_resources(pdev, resources, count);
> -
> -out:
> - acpi_dev_free_resource_list(&resource_list);
>
> return ret;
> }
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] mfd: kempld-core: don't replace resources provided by ACPI
2024-02-21 17:34 ` Guenter Roeck
@ 2024-02-21 17:50 ` Michael Brunner
2024-02-23 10:52 ` Lee Jones
0 siblings, 1 reply; 8+ messages in thread
From: Michael Brunner @ 2024-02-21 17:50 UTC (permalink / raw)
To: linux@roeck-us.net, lee@kernel.org
Cc: mibru@gmx.de, linux-kernel@vger.kernel.org
Added Lee Jones again, as I erroneously used an old eMail address in
my original mail.
Thanks Guenter, for the review!
On Wed, 2024-02-21 at 09:34 -0800, Guenter Roeck wrote:
>> The current implementation to retrieve ACPI resources is faulty
>> and may cause issues that even can lead to non-booting systems.
>>
>> When adding data from an ACPI device, the resources are already
>> assigned to the platform device. Therefore there is no need to
>> retrieve the resource list from ACPI and manually assign it to
>> the platform device. Also there shouldn't be any BIOS in the wild
>> anymore, that does not have resources added to the KEMPLD ACPI
>> data.
>>
>> In particular this fixes an issue where the retrieval of the
>> resource list using /proc/ioports is disturbed and does not list
>> the assigned resource for the kempld device or even no resources
>> at all.
>> On some distributions this also leads to problems during system
>> initialization (e.g. with udev) and causes the system to not
>> boot at all.
>>
>> I have reproduced the issue with the following kernel versions:
>> 5.10.209
>> 5.15.148
>> 6.1.25
>> 6.6.17
>> 6.7.5
>> 6.8-rc5
>>
>> The patch applies to all of those versions and seems to resolve
>> the issue.
>>
>> Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
>
>Reviewed-by: Guenter Roeck <linux@roeck-us.net>
>
>> ---
>> drivers/mfd/kempld-core.c | 37 -------------------------------------
>> 1 file changed, 37 deletions(-)
>>
>> diff --git a/drivers/mfd/kempld-core.c b/drivers/mfd/kempld-core.c
>> index 67af36a38913..5557f023a173 100644
>> --- a/drivers/mfd/kempld-core.c
>> +++ b/drivers/mfd/kempld-core.c
>> @@ -428,50 +428,13 @@ static int kempld_detect_device(struct kempld_device_data *pld)
>> #ifdef CONFIG_ACPI
>> static int kempld_get_acpi_data(struct platform_device *pdev)
>> {
>> - struct list_head resource_list;
>> - struct resource *resources;
>> - struct resource_entry *rentry;
>> struct device *dev = &pdev->dev;
>> - struct acpi_device *acpi_dev = ACPI_COMPANION(dev);
>> const struct kempld_platform_data *pdata;
>> int ret;
>> - int count;
>>
>> pdata = acpi_device_get_match_data(dev);
>> ret = platform_device_add_data(pdev, pdata,
>> sizeof(struct kempld_platform_data));
>> - if (ret)
>> - return ret;
>> -
>> - INIT_LIST_HEAD(&resource_list);
>> - ret = acpi_dev_get_resources(acpi_dev, &resource_list, NULL, NULL);
>> - if (ret < 0)
>> - goto out;
>> -
>> - count = ret;
>> -
>> - if (count == 0) {
>> - ret = platform_device_add_resources(pdev, pdata->ioresource, 1);
>> - goto out;
>> - }
>> -
>> - resources = devm_kcalloc(&acpi_dev->dev, count, sizeof(*resources),
>> - GFP_KERNEL);
>> - if (!resources) {
>> - ret = -ENOMEM;
>> - goto out;
>> - }
>> -
>> - count = 0;
>> - list_for_each_entry(rentry, &resource_list, node) {
>> - memcpy(&resources[count], rentry->res,
>> - sizeof(*resources));
>> - count++;
>> - }
>> - ret = platform_device_add_resources(pdev, resources, count);
>> -
>> -out:
>> - acpi_dev_free_resource_list(&resource_list);
>>
>> return ret;
>> }
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] mfd: kempld-core: don't replace resources provided by ACPI
2024-02-21 17:50 ` Michael Brunner
@ 2024-02-23 10:52 ` Lee Jones
2024-02-23 11:03 ` [PATCH RESEND] " Michael Brunner
2024-02-23 12:50 ` [PATCH] " Michael Brunner
0 siblings, 2 replies; 8+ messages in thread
From: Lee Jones @ 2024-02-23 10:52 UTC (permalink / raw)
To: Michael Brunner
Cc: linux@roeck-us.net, mibru@gmx.de, linux-kernel@vger.kernel.org
On Wed, 21 Feb 2024, Michael Brunner wrote:
> Added Lee Jones again, as I erroneously used an old eMail address in
> my original mail.
I can't take/review patches like this.
Please [RESEND].
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH RESEND] mfd: kempld-core: don't replace resources provided by ACPI
2024-02-23 10:52 ` Lee Jones
@ 2024-02-23 11:03 ` Michael Brunner
2024-02-23 12:39 ` Michael Brunner
2024-02-23 12:50 ` [PATCH] " Michael Brunner
1 sibling, 1 reply; 8+ messages in thread
From: Michael Brunner @ 2024-02-23 11:03 UTC (permalink / raw)
To: lee@kernel.org
Cc: linux@roeck-us.net, mibru@gmx.de, linux-kernel@vger.kernel.org
The current implementation to retrieve ACPI resources is faulty
and may
cause issues that even can lead to non-booting systems.
When adding data from an ACPI device, the resources are already
assigned to the platform device. Therefore there is no need to
retrieve the resource list from ACPI and manually assign it to
the platform device. Also there shouldn't be any BIOS in the wild
anymore, that does not have resources added to the KEMPLD ACPI
data.
In particular this fixes an issue where the retrieval of the
resource list using /proc/ioports is disturbed and does not list
the assigned resource for the kempld device or even no resources
at all.
On some distributions this also leads to problems during system
initialization (e.g. with udev) and causes the system to not
boot at all.
I have reproduced the issue with the following kernel versions:
5.10.209
5.15.148
6.1.25
6.6.17
6.7.5
6.8-rc5
The patch applies to all of those versions and seems to resolve
the issue.
Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
---
drivers/mfd/kempld-core.c | 37 -------------------------------------
1 file changed, 37 deletions(-)
diff --git a/drivers/mfd/kempld-core.c b/drivers/mfd/kempld-core.c
index 67af36a38913..5557f023a173 100644
--- a/drivers/mfd/kempld-core.c
+++ b/drivers/mfd/kempld-core.c
@@ -428,50 +428,13 @@ static int kempld_detect_device(struct kempld_device_data *pld)
#ifdef CONFIG_ACPI
static int kempld_get_acpi_data(struct platform_device *pdev)
{
- struct list_head resource_list;
- struct resource *resources;
- struct resource_entry *rentry;
struct device *dev = &pdev->dev;
- struct acpi_device *acpi_dev = ACPI_COMPANION(dev);
const struct kempld_platform_data *pdata;
int ret;
- int count;
pdata = acpi_device_get_match_data(dev);
ret = platform_device_add_data(pdev, pdata,
sizeof(struct kempld_platform_data));
- if (ret)
- return ret;
-
- INIT_LIST_HEAD(&resource_list);
- ret = acpi_dev_get_resources(acpi_dev, &resource_list, NULL, NULL);
- if (ret < 0)
- goto out;
-
- count = ret;
-
- if (count == 0) {
- ret = platform_device_add_resources(pdev, pdata->ioresource, 1);
- goto out;
- }
-
- resources = devm_kcalloc(&acpi_dev->dev, count, sizeof(*resources),
- GFP_KERNEL);
- if (!resources) {
- ret = -ENOMEM;
- goto out;
- }
-
- count = 0;
- list_for_each_entry(rentry, &resource_list, node) {
- memcpy(&resources[count], rentry->res,
- sizeof(*resources));
- count++;
- }
- ret = platform_device_add_resources(pdev, resources, count);
-
-out:
- acpi_dev_free_resource_list(&resource_list);
return ret;
}
--
2.25.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH RESEND] mfd: kempld-core: don't replace resources provided by ACPI
2024-02-23 11:03 ` [PATCH RESEND] " Michael Brunner
@ 2024-02-23 12:39 ` Michael Brunner
2024-02-23 17:10 ` (subset) " Lee Jones
0 siblings, 1 reply; 8+ messages in thread
From: Michael Brunner @ 2024-02-23 12:39 UTC (permalink / raw)
To: lee@kernel.org
Cc: linux@roeck-us.net, mibru@gmx.de, linux-kernel@vger.kernel.org
The current implementation to retrieve ACPI resources is faulty
and may cause issues that even can lead to non-booting systems.
When adding data from an ACPI device, the resources are already
assigned to the platform device. Therefore there is no need to
retrieve the resource list from ACPI and manually assign it to
the platform device. Also there shouldn't be any BIOS in the wild
anymore, that does not have resources added to the KEMPLD ACPI
data.
In particular this fixes an issue where the retrieval of the
resource list using /proc/ioports is disturbed and does not list
the assigned resource for the kempld device or even no resources
at all.
On some distributions this also leads to problems during system
initialization (e.g. with udev) and causes the system to not
boot at all.
I have reproduced the issue with the following kernel versions:
5.10.209
5.15.148
6.1.25
6.6.17
6.7.5
6.8-rc5
The patch applies to all of those versions and seems to resolve
the issue.
Signed-off-by: Michael Brunner <michael.brunner@kontron.com>
---
drivers/mfd/kempld-core.c | 37 -------------------------------------
1 file changed, 37 deletions(-)
diff --git a/drivers/mfd/kempld-core.c b/drivers/mfd/kempld-core.c
index 67af36a38913..5557f023a173 100644
--- a/drivers/mfd/kempld-core.c
+++ b/drivers/mfd/kempld-core.c
@@ -428,50 +428,13 @@ static int kempld_detect_device(struct kempld_device_data *pld)
#ifdef CONFIG_ACPI
static int kempld_get_acpi_data(struct platform_device *pdev)
{
- struct list_head resource_list;
- struct resource *resources;
- struct resource_entry *rentry;
struct device *dev = &pdev->dev;
- struct acpi_device *acpi_dev = ACPI_COMPANION(dev);
const struct kempld_platform_data *pdata;
int ret;
- int count;
pdata = acpi_device_get_match_data(dev);
ret = platform_device_add_data(pdev, pdata,
sizeof(struct kempld_platform_data));
- if (ret)
- return ret;
-
- INIT_LIST_HEAD(&resource_list);
- ret = acpi_dev_get_resources(acpi_dev, &resource_list, NULL, NULL);
- if (ret < 0)
- goto out;
-
- count = ret;
-
- if (count == 0) {
- ret = platform_device_add_resources(pdev, pdata->ioresource, 1);
- goto out;
- }
-
- resources = devm_kcalloc(&acpi_dev->dev, count, sizeof(*resources),
- GFP_KERNEL);
- if (!resources) {
- ret = -ENOMEM;
- goto out;
- }
-
- count = 0;
- list_for_each_entry(rentry, &resource_list, node) {
- memcpy(&resources[count], rentry->res,
- sizeof(*resources));
- count++;
- }
- ret = platform_device_add_resources(pdev, resources, count);
-
-out:
- acpi_dev_free_resource_list(&resource_list);
return ret;
}
--
2.25.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: (subset) [PATCH RESEND] mfd: kempld-core: don't replace resources provided by ACPI
2024-02-23 12:39 ` Michael Brunner
@ 2024-02-23 17:10 ` Lee Jones
0 siblings, 0 replies; 8+ messages in thread
From: Lee Jones @ 2024-02-23 17:10 UTC (permalink / raw)
To: lee, Michael Brunner; +Cc: linux, mibru, linux-kernel
On Fri, 23 Feb 2024 12:39:12 +0000, Michael Brunner wrote:
> The current implementation to retrieve ACPI resources is faulty
> and may cause issues that even can lead to non-booting systems.
>
> When adding data from an ACPI device, the resources are already
> assigned to the platform device. Therefore there is no need to
> retrieve the resource list from ACPI and manually assign it to
> the platform device. Also there shouldn't be any BIOS in the wild
> anymore, that does not have resources added to the KEMPLD ACPI
> data.
>
> [...]
Applied, thanks!
[1/1] mfd: kempld-core: don't replace resources provided by ACPI
commit: 8f4585bae2d30813e06000497905de7d4cfe6aea
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] mfd: kempld-core: don't replace resources provided by ACPI
2024-02-23 10:52 ` Lee Jones
2024-02-23 11:03 ` [PATCH RESEND] " Michael Brunner
@ 2024-02-23 12:50 ` Michael Brunner
1 sibling, 0 replies; 8+ messages in thread
From: Michael Brunner @ 2024-02-23 12:50 UTC (permalink / raw)
To: lee@kernel.org
Cc: linux@roeck-us.net, mibru@gmx.de, linux-kernel@vger.kernel.org
> On Wed, 21 Feb 2024, Michael Brunner wrote:
>
> > Added Lee Jones again, as I erroneously used an old eMail address
> > in
> > my original mail.
>
> I can't take/review patches like this.
>
> Please [RESEND].
Thanks for the hint!
I resent the patch twice, as the first resent version had the comment
garbled. Sorry for that.
Hope everything is OK now!
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2024-02-23 17:10 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-21 17:10 [PATCH] mfd: kempld-core: don't replace resources provided by ACPI Michael Brunner
2024-02-21 17:34 ` Guenter Roeck
2024-02-21 17:50 ` Michael Brunner
2024-02-23 10:52 ` Lee Jones
2024-02-23 11:03 ` [PATCH RESEND] " Michael Brunner
2024-02-23 12:39 ` Michael Brunner
2024-02-23 17:10 ` (subset) " Lee Jones
2024-02-23 12:50 ` [PATCH] " Michael Brunner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox