* [PATCH] drm/panfrost: fix GENERIC_ATOMIC64 dependency
@ 2023-01-17 16:44 Arnd Bergmann
2023-01-18 11:09 ` Steven Price
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2023-01-17 16:44 UTC (permalink / raw)
To: Rob Herring, Tomeu Vizoso
Cc: Arnd Bergmann, Steven Price, Alyssa Rosenzweig, David Airlie,
Daniel Vetter, Adrián Larumbe, dri-devel, linux-kernel
From: Arnd Bergmann <arnd@arndb.de>
On ARMv5 and earlier, a randconfig build can still run into
WARNING: unmet direct dependencies detected for IOMMU_IO_PGTABLE_LPAE
Depends on [n]: IOMMU_SUPPORT [=y] && (ARM [=y] || ARM64 || COMPILE_TEST [=y]) && !GENERIC_ATOMIC64 [=y]
Selected by [y]:
- DRM_PANFROST [=y] && HAS_IOMEM [=y] && DRM [=y] && (ARM [=y] || ARM64 || COMPILE_TEST [=y] && !GENERIC_ATOMIC64 [=y]) && MMU [=y]
Rework the dependencies to always require a working cmpxchg64.
Fixes: db594ba3fcf9 ("drm/panfrost: depend on !GENERIC_ATOMIC64 when using COMPILE_TEST")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/gpu/drm/panfrost/Kconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/panfrost/Kconfig b/drivers/gpu/drm/panfrost/Kconfig
index 079600328be1..e6403a9d66ad 100644
--- a/drivers/gpu/drm/panfrost/Kconfig
+++ b/drivers/gpu/drm/panfrost/Kconfig
@@ -3,7 +3,8 @@
config DRM_PANFROST
tristate "Panfrost (DRM support for ARM Mali Midgard/Bifrost GPUs)"
depends on DRM
- depends on ARM || ARM64 || (COMPILE_TEST && !GENERIC_ATOMIC64)
+ depends on ARM || ARM64 || COMPILE_TEST
+ depends on !GENERIC_ATOMIC64 # for IOMMU_IO_PGTABLE_LPAE
depends on MMU
select DRM_SCHED
select IOMMU_SUPPORT
--
2.39.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] drm/panfrost: fix GENERIC_ATOMIC64 dependency
2023-01-17 16:44 [PATCH] drm/panfrost: fix GENERIC_ATOMIC64 dependency Arnd Bergmann
@ 2023-01-18 11:09 ` Steven Price
2023-01-18 11:19 ` Robin Murphy
0 siblings, 1 reply; 3+ messages in thread
From: Steven Price @ 2023-01-18 11:09 UTC (permalink / raw)
To: Arnd Bergmann, Rob Herring, Tomeu Vizoso
Cc: Arnd Bergmann, Alyssa Rosenzweig, David Airlie, Daniel Vetter,
Adrián Larumbe, dri-devel, linux-kernel, Robin Murphy
On 17/01/2023 16:44, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> On ARMv5 and earlier, a randconfig build can still run into
>
> WARNING: unmet direct dependencies detected for IOMMU_IO_PGTABLE_LPAE
> Depends on [n]: IOMMU_SUPPORT [=y] && (ARM [=y] || ARM64 || COMPILE_TEST [=y]) && !GENERIC_ATOMIC64 [=y]
> Selected by [y]:
> - DRM_PANFROST [=y] && HAS_IOMEM [=y] && DRM [=y] && (ARM [=y] || ARM64 || COMPILE_TEST [=y] && !GENERIC_ATOMIC64 [=y]) && MMU [=y]
>
> Rework the dependencies to always require a working cmpxchg64.
>
> Fixes: db594ba3fcf9 ("drm/panfrost: depend on !GENERIC_ATOMIC64 when using COMPILE_TEST")
Looking at db594ba3fcf9 - it states:
> Since panfrost has a 'select' on IOMMU_IO_PGTABLE_LPAE we must depend on
> the same set of flags. Otherwise IOMMU_IO_PGTABLE_LPAE will be forced on
> even though it cannot build (no support for cmpxchg64).
And at the time the dependencies on IOMMU_IO_PGTABLE_LPAE were exactly
these.
However d286a58bc8f4 ("iommu: Tidy up io-pgtable dependencies")
(currently in the iommu tree) changed the depends to split the
!GENERIC_ATOMIC64 out. So we could argue that really that's the commit
that should be blamed in the fixes line.
However there's no harm in this being backported further than it
strictly needs to be, and it's clearly better having the
!GENERIC_ATOMIC64 split out. So I'll merge this to drm-misc-fixes.
Reviewed-by: Steven Price <steven.price@arm.com>
Thanks!
Steve
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> drivers/gpu/drm/panfrost/Kconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/panfrost/Kconfig b/drivers/gpu/drm/panfrost/Kconfig
> index 079600328be1..e6403a9d66ad 100644
> --- a/drivers/gpu/drm/panfrost/Kconfig
> +++ b/drivers/gpu/drm/panfrost/Kconfig
> @@ -3,7 +3,8 @@
> config DRM_PANFROST
> tristate "Panfrost (DRM support for ARM Mali Midgard/Bifrost GPUs)"
> depends on DRM
> - depends on ARM || ARM64 || (COMPILE_TEST && !GENERIC_ATOMIC64)
> + depends on ARM || ARM64 || COMPILE_TEST
> + depends on !GENERIC_ATOMIC64 # for IOMMU_IO_PGTABLE_LPAE
> depends on MMU
> select DRM_SCHED
> select IOMMU_SUPPORT
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] drm/panfrost: fix GENERIC_ATOMIC64 dependency
2023-01-18 11:09 ` Steven Price
@ 2023-01-18 11:19 ` Robin Murphy
0 siblings, 0 replies; 3+ messages in thread
From: Robin Murphy @ 2023-01-18 11:19 UTC (permalink / raw)
To: Steven Price, Arnd Bergmann, Rob Herring, Tomeu Vizoso
Cc: Arnd Bergmann, Alyssa Rosenzweig, David Airlie, Daniel Vetter,
Adrián Larumbe, dri-devel, linux-kernel
On 2023-01-18 11:09, Steven Price wrote:
> On 17/01/2023 16:44, Arnd Bergmann wrote:
>> From: Arnd Bergmann <arnd@arndb.de>
>>
>> On ARMv5 and earlier, a randconfig build can still run into
>>
>> WARNING: unmet direct dependencies detected for IOMMU_IO_PGTABLE_LPAE
>> Depends on [n]: IOMMU_SUPPORT [=y] && (ARM [=y] || ARM64 || COMPILE_TEST [=y]) && !GENERIC_ATOMIC64 [=y]
>> Selected by [y]:
>> - DRM_PANFROST [=y] && HAS_IOMEM [=y] && DRM [=y] && (ARM [=y] || ARM64 || COMPILE_TEST [=y] && !GENERIC_ATOMIC64 [=y]) && MMU [=y]
>>
>> Rework the dependencies to always require a working cmpxchg64.
>>
>> Fixes: db594ba3fcf9 ("drm/panfrost: depend on !GENERIC_ATOMIC64 when using COMPILE_TEST")
>
> Looking at db594ba3fcf9 - it states:
>
>> Since panfrost has a 'select' on IOMMU_IO_PGTABLE_LPAE we must depend on
>> the same set of flags. Otherwise IOMMU_IO_PGTABLE_LPAE will be forced on
>> even though it cannot build (no support for cmpxchg64).
>
> And at the time the dependencies on IOMMU_IO_PGTABLE_LPAE were exactly
> these.
>
> However d286a58bc8f4 ("iommu: Tidy up io-pgtable dependencies")
> (currently in the iommu tree) changed the depends to split the
> !GENERIC_ATOMIC64 out. So we could argue that really that's the commit
> that should be blamed in the fixes line.
Oh bum... indeed this is entirely my fault for forgetting about our one
"foreign" io-pgtable user in that commit, sorry about that.
> However there's no harm in this being backported further than it
> strictly needs to be, and it's clearly better having the
> !GENERIC_ATOMIC64 split out. So I'll merge this to drm-misc-fixes.
Thanks both!
Robin.
>
> Reviewed-by: Steven Price <steven.price@arm.com>
>
> Thanks!
>
> Steve
>
>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>> ---
>> drivers/gpu/drm/panfrost/Kconfig | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/panfrost/Kconfig b/drivers/gpu/drm/panfrost/Kconfig
>> index 079600328be1..e6403a9d66ad 100644
>> --- a/drivers/gpu/drm/panfrost/Kconfig
>> +++ b/drivers/gpu/drm/panfrost/Kconfig
>> @@ -3,7 +3,8 @@
>> config DRM_PANFROST
>> tristate "Panfrost (DRM support for ARM Mali Midgard/Bifrost GPUs)"
>> depends on DRM
>> - depends on ARM || ARM64 || (COMPILE_TEST && !GENERIC_ATOMIC64)
>> + depends on ARM || ARM64 || COMPILE_TEST
>> + depends on !GENERIC_ATOMIC64 # for IOMMU_IO_PGTABLE_LPAE
>> depends on MMU
>> select DRM_SCHED
>> select IOMMU_SUPPORT
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-01-18 12:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-17 16:44 [PATCH] drm/panfrost: fix GENERIC_ATOMIC64 dependency Arnd Bergmann
2023-01-18 11:09 ` Steven Price
2023-01-18 11:19 ` Robin Murphy
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox