[Mesa-dev] [PATCH 2/4] configure/r600: Only require libdrm_amdgpu if CL is enabled

Aaron Watry awatry at gmail.com
Tue Jun 6 15:04:43 UTC 2017


On Tue, Jun 6, 2017 at 8:51 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>
> On 5 June 2017 at 21:31, Aaron Watry <awatry at gmail.com> wrote:
> > Otherwise r600g will fail to build when the amdgpu drm library is missing
> >
> And the code that pulls amdgpu is not used by r600g, which makes this
> misplaced duck tape.
> As you/Jan don't have the time to properly address this, I'll try to
> find some later today.


Thank you.

You're right that I don't have a lot of time.  If I'm lucky, I get 30
minutes in the average evening to poke at any of my own hobbies,
including this. I'd love to be able to spend the whole day ripping the
amdgpu dependency out of the parts of amd/common that r600 uses, but
that's just not possible this week (or next week), hence the bandaid.

Combine that with a lack of understanding of autotools and you can see
how we ended up here.

What Jan and I sent gets the r600 build working again, without losing
functionality, and buys us time to deal with the amdgpu dependency in
the amd/common and gallium/radeon parts, which honestly is all I had
time to do at the moment without taking a day off of work to rip
amd/common apart to fix and test it properly.

--Aaron

>
>
> -Emil


More information about the mesa-dev mailing list