[PR PATCH] mesa: switch from i915 to i915g (gallium)

[PR PATCH] mesa: switch from i915 to i915g (gallium)

[tsndqst]

[-- Attachment #1: Type: text/plain, Size: 1275 bytes --]

There is a new pull request by tsndqst against master on the void-packages repository

https://github.com/tsndqst/void-packages i915g
https://github.com/void-linux/void-packages/pull/34620

mesa: switch from i915 to i915g (gallium)
I made this change to get `sway` working on my EeePC 1005HA.  I set `WLR_NO_HARDWARE_CURSORS=1` before starting sway to get the mouse cursor to work.  More information [here](https://github.com/swaywm/wlroots/issues/2506).

#### Testing the changes
- I tested the changes in this PR: **briefly**

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration)
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->

#### Local build testing
- I built this PR locally for my native architecture, (i686-glibc, native mode with a 32bit masterdir on x86-64)


A patch file from https://github.com/void-linux/void-packages/pull/34620.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-i915g-34620.patch --]
[-- Type: text/x-diff, Size: 1057 bytes --]

From 9722b25a72d76d209bf204cd7fe5bf982b795cef Mon Sep 17 00:00:00 2001
From: Tim Sandquist <tim.sandquist@gmail.com>
Date: Fri, 10 Dec 2021 20:15:40 -0600
Subject: [PATCH] mesa: switch from i915 to i915g (gallium)

---
 srcpkgs/mesa/template | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/mesa/template b/srcpkgs/mesa/template
index eebe1f722e9e..c8b19e20fe72 100644
--- a/srcpkgs/mesa/template
+++ b/srcpkgs/mesa/template
@@ -1,7 +1,7 @@
 # Template file for 'mesa'
 pkgname=mesa
 version=21.2.6
-revision=1
+revision=2
 build_style=meson
 configure_args="-Dglvnd=true -Dshared-glapi=enabled -Dgbm=enabled -Degl=enabled
  -Dosmesa=true -Dgles1=enabled -Dgles2=enabled -Dglx=dri -Ddri3=enabled
@@ -91,9 +91,9 @@ fi
 
 if [ "$_have_intel" ]; then
 	_have_vulkan=yes
-	_gallium_drivers+=",crocus,iris"
+	_gallium_drivers+=",crocus,iris,i915"
 	_vulkan_drivers+=",intel"
-	_dri_drivers+=",i915,i965"
+	_dri_drivers+=",i965"
 	subpackages+=" mesa-vulkan-intel"
 	# transitional dummy packages
 	subpackages+=" mesa-intel-dri"

Re: mesa: switch from i915 to i915g (gallium)

[notramo]

[-- Attachment #1: Type: text/plain, Size: 355 bytes --]

New comment by notramo on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-997475257

Comment:
It should be  different package, because gallium version is more buggy than classic.
Classic is more reliable for Xorg, and gallium is only needed for wlroots which requires features that are missing from classic.

Re: mesa: switch from i915 to i915g (gallium)

[notramo]

[-- Attachment #1: Type: text/plain, Size: 397 bytes --]

New comment by notramo on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-997475257

Comment:
It should be  different package, because gallium version is more buggy than classic.
Classic is more reliable for Xorg and some less popular Wayland compositors, and gallium is only needed for wlroots which requires features that are missing from classic.

Re: mesa: switch from i915 to i915g (gallium)

[tsndqst]

[-- Attachment #1: Type: text/plain, Size: 215 bytes --]

New comment by tsndqst on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-997476729

Comment:
That makes sense. My quick and dirty change for my system won't be enough.

Re: mesa: switch from i915 to i915g (gallium)

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 390 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-997566547

Comment:
Dang, what a mess :/

https://gitlab.freedesktop.org/mesa/mesa/-/issues/5418 says it will become default at some point, but we are clearly not there yet

I'd consider potentially a build option for the Void package until then, what do you think?

Re: mesa: switch from i915 to i915g (gallium)

[q66]

[-- Attachment #1: Type: text/plain, Size: 318 bytes --]

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-997624728

Comment:
my 2c: mesa 22 retires all the classic dri drivers anyway, so there is nothing much to deal with - we could just flip the switch, turn dri drivers off entirely, and be done with it 

Re: mesa: switch from i915 to i915g (gallium)

[q66]

[-- Attachment #1: Type: text/plain, Size: 339 bytes --]

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-997624728

Comment:
my 2c: mesa 22 (coming in February) retires all the classic dri drivers anyway, so there is nothing much to deal with - we could just flip the switch, turn dri drivers off entirely, and be done with it 

Re: mesa: switch from i915 to i915g (gallium)

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 277 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-998006080

Comment:
My hope is that until February the Gallium driver will be (more) fixed, so doing the switch now would hurt a lot of users unnecessarily.

Re: mesa: switch from i915 to i915g (gallium)

[q66]

[-- Attachment #1: Type: text/plain, Size: 294 bytes --]

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-998342795

Comment:
any examples of how i915g is buggy in current releases?

(also, there's probably relatively few i915 users left at this point, since it's ancient hardware)

Re: mesa: switch from i915 to i915g (gallium)

[Anachron]

[-- Attachment #1: Type: text/plain, Size: 269 bytes --]

New comment by Anachron on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-1155159268

Comment:
Maybe we could revisit this? I'm a `i915` user and with the recent `mesa` and `linux` combination I can't boot into X anymore.

Re: mesa: switch from i915 to i915g (gallium)

[tsndqst]

[-- Attachment #1: Type: text/plain, Size: 544 bytes --]

New comment by tsndqst on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-1155176022

Comment:
I won't be much help beyond providing another data point.  I am able to run X on my EeePC 1005HA with the latest Mesa 22 package.

@Anachron it might help to provide more information about your setup.  Kernel version, `/var/log/Xorg.0.log` messages (especially if there are lines with `(EE)`) , etc.

Other folks may be able to provide more specific information about troubleshooting Mesa/X issues.

Re: mesa: switch from i915 to i915g (gallium)

[tsndqst]

[-- Attachment #1: Type: text/plain, Size: 511 bytes --]

New comment by tsndqst on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-1155184275

Comment:
@Anachron are you using any Mesa environment variables to override behavior such as `MESA_LOADER_DRIVER_OVERRIDE`?  https://docs.mesa3d.org/envvars.html#core-mesa-environment-variables

Perhaps the wrong driver is being selected.  The i915 driver is gallium-only so there is no i915g driver anymore.  i915 IS the gallium driver.  In the past there was i915 and i915g.

Re: mesa: switch from i915 to i915g (gallium)

[Anachron]

[-- Attachment #1: Type: text/plain, Size: 270 bytes --]

New comment by Anachron on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-1155994443

Comment:
Turns out I had kernel boot parameters added before which are not supported with my i915 hardware anymore. Sorry for the noise.

Re: mesa: switch from i915 to i915g (gallium)

[tsndqst]

[-- Attachment #1: Type: text/plain, Size: 236 bytes --]

New comment by tsndqst on void-packages repository

https://github.com/void-linux/void-packages/pull/34620#issuecomment-1156543826

Comment:
I'm glad you figured it out.

Mesa has been updated to 22 now so I'm going to close this PR.

Re: [PR PATCH] [Closed]: mesa: switch from i915 to i915g (gallium)

[tsndqst]

[-- Attachment #1: Type: text/plain, Size: 1126 bytes --]

There's a closed pull request on the void-packages repository

mesa: switch from i915 to i915g (gallium)
https://github.com/void-linux/void-packages/pull/34620

Description:
I made this change to get `sway` working on my EeePC 1005HA.  I set `WLR_NO_HARDWARE_CURSORS=1` before starting sway to get the mouse cursor to work.  More information [here](https://github.com/swaywm/wlroots/issues/2506).

#### Testing the changes
- I tested the changes in this PR: **briefly**

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration)
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->

#### Local build testing
- I built this PR locally for my native architecture, (i686-glibc, native mode with a 32bit masterdir on x86-64)