00:08jenatali: Dunno though, when we registered the driver name it's a Mesa driver name prefix instead of MSFT
00:16anholt: my understanding is: I would go with msft, spi should be for mesa submissions where there's no other adopter responsible for it.
00:16Kayden: yeah
00:16Kayden: I think it depends on who's doing the submission, mostly
00:17Kayden: anv submissions are done by Intel employees, radv submissions typically aren't done by AMD folks
00:18Kayden: if jenatali's involved it probably makes sense just to do MSFT
00:19jenatali: Yeah makes sense
00:20jenatali: We still need to talk to Khronos to define what conformance requires from a layered VK driver...
00:20karolherbst: jenatali: isn't it like to run on two drivers?
00:21jenatali: Except our legal group bars us from participating in working group meetings
00:21karolherbst: ehh.. pass conformance on two
00:21karolherbst: wat...
00:21jenatali: karolherbst: it's defined for GL and CL that way
00:21jenatali: VK isn't defined yet AFAIK
00:21karolherbst: mhhh
00:21jenatali: (I expect it'll end up the same, just needs to get into docs)
00:21karolherbst: but you are not allowed to do that?
00:22karolherbst: or maybe just submit the MR?
00:22jenatali: Last time around we got special permission to set up meetings outside of the IP zone
00:22karolherbst: uhhh
00:22karolherbst: why is legal like this
00:22jenatali: Yeah
00:24Kayden: :(
00:26jenatali: And then once I finish on WARP I need to get a second driver to conformance. Hopefully that's easy but who knows
01:45karolherbst: dcbaker: soo.. in regards to that isystem situation with bindgen. Do you agree that the bindgen stuff needs to do something similar to what CLikeCompilerArgs.to_native does in regards to isystem arguments?
02:48mareko: Venemo: can you please Rb https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22306 ?
10:30Venemo: mareko: I'll take another look at it!
12:00karolherbst: dcbaker: I hate it, but doing it properly looks like a serious amount of work: https://github.com/mesonbuild/meson/pull/11733 Any ideas on that?
13:49Company: I need a GL/EGL expert, because versions of eglCreateContext() confuse me.
13:49Company: if I want a GL - or actually: GLES - context as new as possible buit at least version 2, what do I pass to eglCreateContext()?
13:50Company: because apparently, if I pass 2.0, ANGLE hands out 2.0 and Mesa hands out a 3.2 context
13:50Company: even though they both support GLES 3
13:52HdkR: Company: Make sure you eglBindAPI before eglCreateContext
13:53Company: the code does that
13:53linkmauve: Company, what I’ve seen programs do is request the most recent version they want to support, then one before that, then one before that, until the earliest one they support.
13:54Company: it passes MAJOR_VERSION 2 attrib, but I thought that'd automatically give me 3 if available
13:54HdkR: That too, you usually work your way backward from newest supported to oldest in your application. Changing the ctx_attrib flags as you go, and potentially channging the config you got from eglChooseConfig
13:55HdkR: Since eglChooseConfig has an `EGL_RENDERABLE_TYPE` where you specify GL or ES
13:55Company: yeah, I know that
13:55Company: I'm just confused about versions and min vs max vs actual
13:56Company: so there is no way to set a min/max and you're actually meant to try them all
13:56HdkR: Also yes, if you're passing in a version ot eglCreateContext, you're not guaranteed to be automatically promoted to a newer version
13:56linkmauve: Company, when you request a version, the driver can give you anything from that version up to the most recent compatible version.
13:56Company: right, but it's the driver's choice?
13:56HdkR: Which Angle might be respecting your decision when you chose version 2, but mesa EGL promotes
13:57HdkR: That is yes
13:57Company: because I want the most recent one
13:58HdkR: Then you'll be wanting to pass in EGL_CONTEXT_MAJOR_VERSION, EEGL_CONTEXT_MINOR_VERSION, and if you are also wanting GL, EGL_CONTEXT_OPENGL_FORWARD_COMPATIBLE
13:58HdkR: It's a big song and dance
13:59HdkR: `The context returned must be the specified version, or a later version which is backwards compatible with that version. Even if a later version is returned, the specified version must correspond to a defined version of the client API`
13:59Company: does that work the same with GLX and Apple and Windows GL, too?
13:59HdkR: EGL Spec wording around this promotion
14:01HdkR: Pretty sure
14:01Company: looks like I'm gonna write some code then
14:01Company: thx
14:06HdkR: Luckily GL/ES is a slow lumbering beast at this point. So even if you hardcode every version you care about, it is unlikely to suddenly have a GL 5.0 or an ES 4.0 :)
14:21Company: technically I think I could add code that queries the version via glGetString() and goes from there, but I'm not gonna do that, because then I need to figure out if that'a actually supported
15:16Venemo: mareko: I've taken another look at the MR but it seems to me that there are still issues and now I agree with alyssa, it needs more consideration. also I would appreciate some input from gfxstrand
16:16javierm: tzimmermann: I'm surprised that nobody ever cared about the bug you are fixing in patch #1
16:38tzimmermann: javierm, IGT triggers it while writing near the EOF. i guess that all applications write near the beginning of the framebuffer
16:38tzimmermann: and it also shows how this is all badly maintained and obsolete :(
16:38gfxstrand: karolherbst, jenatali: Have either of you seen this problem with LLVM 16:
16:39gfxstrand: : CommandLine Error: Option 'use-dbg-addr' registered more than once!
16:39tzimmermann: thanks for reviewing, BTW
16:39jenatali: gfxstrand: Yeah I saw some discussion about it a few days ago
16:39jenatali: It's a Fedora 38 problem IIRC
16:39gfxstrand: *sigh*
16:40jenatali: gfxstrand: https://bugzilla.redhat.com/show_bug.cgi?id=2187824
16:41gfxstrand: Hey, look! An LLVM update!
16:43HdkR: LLVM updated and broke GPUs again. Surprised? :P
16:44gfxstrand: Broke the mesa build, actually.
16:44gfxstrand: But, yeah
16:44HdkR: Nice
16:51javierm: tzimmermann: you are welcome. Awesome cleanup tbh
16:51javierm: I haven't noticed that there was that much duplicated code there
17:05tzimmermann: javierm, i've got to leave. bye
17:06MrCooper: gfxstrand: assuming /usr/lib64/libLLVMSPIRVLib.so is involved, should be fixed in current F38
17:07MrCooper: (see https://bugzilla.redhat.com/show_bug.cgi?id=2187824)
17:08gfxstrand: MrCooper: Yeah, I pulled an update and it's okay now.
17:08gfxstrand: Somehow it got fixed between when I did my F38 upgrade yesterday late afternoon and this morning.
17:08zmike: have to assume it was karolherbst working late at the pub
17:38gfxstrand: Ugh... now my rust isn't compiling.
17:38gfxstrand: What did this Fedora update do?!?
17:39psykose: 'Option 'use-dbg-addr' registered more than once!' means the process loaded multiple llvm versions
17:40psykose: i.e. you have one library linked to 15 and another to 16, linked to the same process, etc
17:40karolherbst: zmike: I never touched that piece of software in my life
17:40zmike: while sober*
17:41penguin42: gfxstrand: Yeh there's a fix just went in
17:41penguin42: gfxstrand: https://bugzilla.redhat.com/show_bug.cgi?id=2187824
17:42HdkR: zmike: Some software just can't be worked on while sober
17:42zmike: F A C T S
17:43ccr: standardized required level of inebriation for working on X
17:44gfxstrand:looks at chanel topic
17:56gfxstrand: Gotta love how I have to update bindgen after an LLVM update....
17:57karolherbst: I'd wished we could get rid of LLVM :)
18:01gfxstrand: Ok, now that I've updated to F38 and fixed all the LLVM fallout, I feel like I should get a new CTS baseline.
19:32anholt: should vk_common_GetPipelineCacheData be setting a nonzero count?
19:33gfxstrand: should be
19:34gfxstrand: Ugh... Looks like no one's incrementing count. :facepalm:
19:34gfxstrand: I'm surprised CTS didn't at least warn on this.
19:34gfxstrand: That's an old bug
19:35anholt: we don't expose cts warnings from the deqp runner, and maybe we should be.
19:35gfxstrand: IDK if there is a warning
19:36anholt: (been staring at this function for https://gitlab.freedesktop.org/mesa/mesa/-/issues/8868)