Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 717930 - media-libs/mesa lm-sensors and gallium use flags
Summary: media-libs/mesa lm-sensors and gallium use flags
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-17 18:05 UTC by jorge
Modified: 2020-04-30 17:35 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description jorge 2020-04-17 18:05:15 UTC
The bug report https://bugs.gentoo.org/663586 was followed by the commit https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=040ff87a62f80ba7b77edcc3595be8ce878b0d3b 

This commit adds flag USE="gallium" as a dependency for USE="lm-sensors".

For example, if the user sets flag for lm-sensors in media-libs/mesa, gallium is pulled automatically (no support for either lm-sensors or gallium before this operation):

-----------------------------------------------------------------
emerge --ask --newuse --deep @world

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R    ] media-libs/mesa-19.3.5  USE="gallium* lm-sensors*" 

Would you like to merge these packages? [Yes/No]
------------------------------------------------------------------

gallium is probably pulled by other dependencies as well. In my case I have enabled VIDEO_CARDS="i965" in my make.conf. But my GPU is an intel Gen7.5 and the gallium driver only applies to Gen8+. Intel classic drivers from Gen4 and up use the moniker "i965". The info on the lm-sensors USE flag of the media-libs/mesa (https://packages.gentoo.org/packages/media-libs/mesa) does not help: "Add linux lm-sensors (hardware sensors) support" a more appropiate info can be "Add linux hardware sensors support (gallium drivers only)".

So, for the end user enabling sensors is reasonable but enabling this is pointless for Gen7.5- GPU's, as it only applies to modern intel Gen8+ GPU's. The info popup does not state this and as such, the gallium driver installs itself if the user is not aware enough.

Thanks to fturco for the info https://forums.gentoo.org/viewtopic-t-1111568-highlight-.html
Comment 1 Matt Turner gentoo-dev 2020-04-19 16:48:15 UTC
What is it that you want changed, exactly? To update the description in metadata.xml?
Comment 2 jorge 2020-04-19 23:20:02 UTC
I am sorry for being vague. When I mentioned the USE flag descriptions, I was referring to the page https://packages.gentoo.org/packages/media-libs/mesa 
When you mouse over the lm-sensors flag, a popup appears with the legend: "Add linux lm-sensors (hardware sensors) support". Unwary folks like myself can enable this flag but, it is pointless the hardware support isn't there (in the case of Intel, Gen8 or greater).

The simplest solution would be to offer an updated USE flag description. If so, I apologize because this is not a bug and should not be reported here. The other solution is to automatically check if the user has the appropriate hardware support.
Comment 3 Matt Turner gentoo-dev 2020-04-20 05:51:41 UTC
No no, this is a good bug report. I just didn't understand :)
Comment 4 Larry the Git Cow gentoo-dev 2020-04-30 17:35:37 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9bed562f2971f41406c3b0807d11962f7074fa94

commit 9bed562f2971f41406c3b0807d11962f7074fa94
Author:     Matt Turner <mattst88@gentoo.org>
AuthorDate: 2020-04-30 17:24:54 +0000
Commit:     Matt Turner <mattst88@gentoo.org>
CommitDate: 2020-04-30 17:34:41 +0000

    media-libs/mesa: Add a local description of IUSE=lm-sensors
    
    Closes: https://bugs.gentoo.org/717930
    Signed-off-by: Matt Turner <mattst88@gentoo.org>

 media-libs/mesa/metadata.xml | 1 +
 1 file changed, 1 insertion(+)