Re: [PATCH] drm/amdgpu: Re-enable DCN for 64-bit powerpc

From: Timothy Pearson
Date: Mon Jul 25 2022 - 15:43:33 EST




----- Original Message -----
> From: "Linus Torvalds" <torvalds@xxxxxxxxxxxxxxxxxxxx>
> To: "Michael Ellerman" <mpe@xxxxxxxxxxxxxx>
> Cc: "linuxppc-dev" <linuxppc-dev@xxxxxxxxxxxxxxxx>, "Alex Deucher" <alexdeucher@xxxxxxxxx>, "amd-gfx"
> <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>, linux@xxxxxxxxxxxx, "linux-kernel" <linux-kernel@xxxxxxxxxxxxxxx>, "Dan Horák"
> <dan@xxxxxxxx>, "Timothy Pearson" <tpearson@xxxxxxxxxxxxxxxxxxxxx>
> Sent: Monday, July 25, 2022 2:19:57 PM
> Subject: Re: [PATCH] drm/amdgpu: Re-enable DCN for 64-bit powerpc

> On Mon, Jul 25, 2022 at 5:39 AM Michael Ellerman <mpe@xxxxxxxxxxxxxx> wrote:
>>
>> Further digging shows that the build failures only occur with compilers
>> that default to 64-bit long double.
>
> Where the heck do we have 'long double' things anywhere in the kernel?
>
> I tried to grep for it, and failed miserably. I found some constants
> that would qualify, but they were in the v4l colorspaces-details.rst
> doc file.
>
> Strange.

We don't, at least not that I can see. The affected code uses standard doubles.

What I'm wondering is if the compiler is getting confused between standard and long doubles when they are both the same bit length...