[PATCH] arm64: defconfig: Increase CMA size for VC4

[PATCH] arm64: defconfig: Increase CMA size for VC4

[Stefan Wahren]

The VC4 needs more memory than the default setting (16 MB):

  vc4-drm soc:gpu: swiotlb: coherent allocation failed, size=16777216
  [drm:vc4_bo_create [vc4]] *ERROR* Failed to allocate from CMA:
  vc4_v3d 3fc00000.v3d: Failed to allocate memory for tile binning: -12.
    You may need to enable CMA or give it more memory.
  vc4-drm soc:gpu: failed to bind 3fc00000.v3d (ops vc4_v3d_ops [vc4]): -12
  vc4-drm soc:gpu: master bind failed: -12
  vc4-drm: probe of soc:gpu failed with error -12

So increase the value to 32 MB and fix this issue.

Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
---

Hi,
i known it's a little bit late for 4.18 or should i rebase for 4.17?

Regards

 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 7ca583e..0e1f6d7 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -172,6 +172,7 @@ CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
 CONFIG_DEVTMPFS=y
 CONFIG_DEVTMPFS_MOUNT=y
 CONFIG_DMA_CMA=y
+CONFIG_CMA_SIZE_MBYTES=32
 CONFIG_MTD=y
 CONFIG_MTD_BLOCK=y
 CONFIG_MTD_M25P80=y
-- 
2.7.4

[PATCH] arm64: defconfig: Increase CMA size for VC4

[Eric Anholt]

Stefan Wahren <stefan.wahren@i2se.com> writes:

> The VC4 needs more memory than the default setting (16 MB):
>
>   vc4-drm soc:gpu: swiotlb: coherent allocation failed, size=16777216
>   [drm:vc4_bo_create [vc4]] *ERROR* Failed to allocate from CMA:
>   vc4_v3d 3fc00000.v3d: Failed to allocate memory for tile binning: -12.
>     You may need to enable CMA or give it more memory.
>   vc4-drm soc:gpu: failed to bind 3fc00000.v3d (ops vc4_v3d_ops [vc4]): -12
>   vc4-drm soc:gpu: master bind failed: -12
>   vc4-drm: probe of soc:gpu failed with error -12
>
> So increase the value to 32 MB and fix this issue.
>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>

If there was a way to check how big the CMA pool is, then we could
probably have V3D disable itself when the pool was too small.  It's not
like V3D rendering is going to actually *work* with just a 32mb pool to
play in.

That said, I'd also be fine with this patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180503/04dd6dce/attachment.sig>

[PATCH] arm64: defconfig: Increase CMA size for VC4

[Stefan Wahren]

Hi Florian,

> Eric Anholt <eric@anholt.net> hat am 3. Mai 2018 um 23:20 geschrieben:
> 
> 
> Stefan Wahren <stefan.wahren@i2se.com> writes:
> 
> > The VC4 needs more memory than the default setting (16 MB):
> >
> >   vc4-drm soc:gpu: swiotlb: coherent allocation failed, size=16777216
> >   [drm:vc4_bo_create [vc4]] *ERROR* Failed to allocate from CMA:
> >   vc4_v3d 3fc00000.v3d: Failed to allocate memory for tile binning: -12.
> >     You may need to enable CMA or give it more memory.
> >   vc4-drm soc:gpu: failed to bind 3fc00000.v3d (ops vc4_v3d_ops [vc4]): -12
> >   vc4-drm soc:gpu: master bind failed: -12
> >   vc4-drm: probe of soc:gpu failed with error -12
> >
> > So increase the value to 32 MB and fix this issue.
> >
> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> 
> If there was a way to check how big the CMA pool is, then we could
> probably have V3D disable itself when the pool was too small.  It's not
> like V3D rendering is going to actually *work* with just a 32mb pool to
> play in.
> 
> That said, I'd also be fine with this patch.

should i rebase my patch?

[PATCH] arm64: defconfig: Increase CMA size for VC4

[Florian Fainelli]

On May 12, 2018 2:10:26 AM PDT, Stefan Wahren <stefan.wahren@i2se.com> wrote:
>Hi Florian,
>
>> Eric Anholt <eric@anholt.net> hat am 3. Mai 2018 um 23:20
>geschrieben:
>> 
>> 
>> Stefan Wahren <stefan.wahren@i2se.com> writes:
>> 
>> > The VC4 needs more memory than the default setting (16 MB):
>> >
>> >   vc4-drm soc:gpu: swiotlb: coherent allocation failed,
>size=16777216
>> >   [drm:vc4_bo_create [vc4]] *ERROR* Failed to allocate from CMA:
>> >   vc4_v3d 3fc00000.v3d: Failed to allocate memory for tile binning:
>-12.
>> >     You may need to enable CMA or give it more memory.
>> >   vc4-drm soc:gpu: failed to bind 3fc00000.v3d (ops vc4_v3d_ops
>[vc4]): -12
>> >   vc4-drm soc:gpu: master bind failed: -12
>> >   vc4-drm: probe of soc:gpu failed with error -12
>> >
>> > So increase the value to 32 MB and fix this issue.
>> >
>> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
>> 
>> If there was a way to check how big the CMA pool is, then we could
>> probably have V3D disable itself when the pool was too small.  It's
>not
>> like V3D rendering is going to actually *work* with just a 32mb pool
>to
>> play in.
>> 
>> That said, I'd also be fine with this patch.
>
>should i rebase my patch?

I was expecting Eric to pick it up, but I can do this as well sorry about the miscommunication here. Let me queue this up and update the defconfig-arm64 pull request with this commit.

-- 
Florian

[PATCH] arm64: defconfig: Increase CMA size for VC4

[Florian Fainelli]

Le 05/02/18 ? 14:31, Stefan Wahren a ?crit?:
> The VC4 needs more memory than the default setting (16 MB):
> 
>   vc4-drm soc:gpu: swiotlb: coherent allocation failed, size=16777216
>   [drm:vc4_bo_create [vc4]] *ERROR* Failed to allocate from CMA:
>   vc4_v3d 3fc00000.v3d: Failed to allocate memory for tile binning: -12.
>     You may need to enable CMA or give it more memory.
>   vc4-drm soc:gpu: failed to bind 3fc00000.v3d (ops vc4_v3d_ops [vc4]): -12
>   vc4-drm soc:gpu: master bind failed: -12
>   vc4-drm: probe of soc:gpu failed with error -12
> 
> So increase the value to 32 MB and fix this issue.
> 
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>

Applied to defconfig-arm64/next, thanks!
-- 
Florian

[PATCH] arm64: defconfig: Increase CMA size for VC4

[Stefan Wahren]

> Florian Fainelli <f.fainelli@gmail.com> hat am 13. Mai 2018 um 18:58 geschrieben:
> 
> 
> Le 05/02/18 ? 14:31, Stefan Wahren a ?crit?:
> > The VC4 needs more memory than the default setting (16 MB):
> > 
> >   vc4-drm soc:gpu: swiotlb: coherent allocation failed, size=16777216
> >   [drm:vc4_bo_create [vc4]] *ERROR* Failed to allocate from CMA:
> >   vc4_v3d 3fc00000.v3d: Failed to allocate memory for tile binning: -12.
> >     You may need to enable CMA or give it more memory.
> >   vc4-drm soc:gpu: failed to bind 3fc00000.v3d (ops vc4_v3d_ops [vc4]): -12
> >   vc4-drm soc:gpu: master bind failed: -12
> >   vc4-drm: probe of soc:gpu failed with error -12
> > 
> > So increase the value to 32 MB and fix this issue.
> > 
> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> 
> Applied to defconfig-arm64/next, thanks!
> -- 
> Florian

Thank you, i will add a note next time.

Stefan