search

FireBurn / Overlay

Gentoo Overly
GNU General Public License v3.0
21 stars 6 forks source link

Drivers which are not needed are built #13

Closed hvenev closed 11 years ago

hvenev commented 11 years ago

For example, if I want i965 and radeonsi with opencl, opencl => r600-llvm-compiler => video_cards_radeon video_cards_radeon && classic => build r100 and r200 video_cards_radeon && !video_cards_r300 && !video_cards_r600 => build r300 and r600

This way all radeon drivers are built. What I propose is not to enable video cards automatically and only depend on r600-llvm-compiler when video_cards_r600 is enabled.

And an indentation fix.

FireBurn commented 11 years ago

I have this in /etc/portage/package.use:

media-libs/mesa -video_cards_radeon -video_cards_intel

This stops r300g and i915(g) being built

hvenev commented 11 years ago

opencl => r600-llvm-compiler => video_cards_radeon

FireBurn commented 11 years ago

I don't understand the point you're trying to make

I currently have opencl r600-llvm-compiler and only the r600g driver - what would your merge request bring that's not already available?

hvenev commented 11 years ago

The ability to use OpenCL with radeonsi without r600g. If I don't have video_cards_r600 and video_cards_r300 but have video_cards_radeon (required by r600-llvm-compiler required by opencl), the ebuild will build r600 and r300. Oh and why the **\ do I need r600-llvm-compiler (required by opencl) if I don't use r600g?

FireBurn commented 11 years ago

OK it sounds like you're using the radeonsi driver. This has to use the r600-llvm-compiler. If you think you've disabled it chances are mesa has just forced it on. Its for r600+ cards including southern islands. If you can't compile radeonsi without the other cards that's a bug. If you can't compile radeonsi without r600-llvm-compiler then its working as designed.

FireBurn commented 11 years ago

OK I've applied a fix so radeonsi can be compiled on it's own without r600 or radeon being set - which I think fixes the issue you've been having. It's much simpler than the commit you've provided

I'd recommend you let the folks that look after the x11 overlay know about it too (and explain a little bit more concisely) as their overlay feeds directly into portage when releases are ready

FireBurn commented 11 years ago

No response for over a week so I'm going to close this issue - if you're still experiencing problems since my fix then please reopen this