03:48airlied: jekstrand: got smoke passing on new uapi, even found a small nvk bug :-P
03:49airlied: dakr: ^ the fault I saw wasn't preexisting nvk bug
03:51airlied: jekstrand: !147 when you drive by :0
03:58dakr: airlied: ouch, which commit are you using? I just pushed two fixes that might be relevant.
04:02airlied: dakr: I was on drm/nouveau: sched: add support for timeline syncobj
04:03airlied: just grabbed the fixes, will rebuild
04:10airlied: dakr: lols one of your fixes, broke the userspace driver :-), the userspace driver is wrong though it's all good
04:12airlied: jekstrand: the simple submit is kinda a bad idea in the vma world
04:12airlied: creates a bo with no good lifetime
04:19dakr: airlied: :D interesting, wonder what userspace can do wrong to make this happen..
04:20dakr: since one patch was fixing a double free in the err path of synchronous binds..
04:21dakr: and the other one is fixing unbind ioctls to actually unmap the vma rather than just destroying the uvma entry.
04:23airlied: dakr: it was destroying a bo before it was synced
04:23airlied: in hte old code is never got unbound due to the bug you fixed
04:25dakr: ah, makes sense.
05:27airlied: dakr: also need to protect unbind against out of range addrs, I've got an error case that passes it garbage and kernel blowsu p
05:30airlied: probably should pull the checks out of nouveau_uvma_new and move them into the submit checking
05:33airlied: probably also need to validate the addr/range to make sure it's outside the managed zone
05:49HdkR: airlied: Is the new UAPI available for pushing through my struct validator?
05:50airlied: kinda premature
05:50airlied: like months premature
05:50HdkR: ah, alright
06:24airlied:runs in the first vm/bo split problem
14:39dakr: airlied: yeah, there might be more such cases, I'm just about to handle them by coming up with some igt test cases feeding all kinds of nonsense into the UAPI trying to blow things up.
14:40dakr: regarding split/merge, I'm planning to come up with something this week. :-)
16:31fdobridge: <jekstrand> Pulled it in.
17:37airlied: jekstrand: so in the descriptor set code where it adds the maxsets * ubo min alignment
17:37airlied: should that be a multiply?
18:19fdobridge: <airlied> @jekstrand dEQP-VK.texture.filtering.2d.formats.r16g16b16a16_sfloat.nearest fails along with a whole lot of other ones, that was my best guess at a fix
18:19fdobridge: <airlied> - bo_size += NVK_MIN_UBO_ALIGNMENT * pCreateInfo->maxSets;
18:19fdobridge: <airlied> + bo_size *= NVK_MIN_UBO_ALIGNMENT * pCreateInfo->maxSets;
18:19fdobridge: <marysaka> Does nvk/main requires any patches on the kernel side? Currently getting a null pointer dereference in kernel space when trying to run vulkaninfo with an RTX 3060 Ti
18:35fdobridge: <airlied> @jekstrand created !148 in case i'm right
18:36fdobridge: <airlied> @marysaka for turing it shouldn't for basic use, for stability it does
18:37fdobridge: <jekstrand> That doesn't look right
18:41fdobridge: <karolherbst🐧🦀> ehhh...
18:41fdobridge: <karolherbst🐧🦀> @jekstrand that's ampere...
18:41fdobridge: <karolherbst🐧🦀> huh...
18:41fdobridge: <karolherbst🐧🦀> sounds like a kernel bug
18:41fdobridge: <karolherbst🐧🦀> if I don't forget I can check that out, but ampere isn't supported atm, unless you use bens branch
18:41fdobridge: <marysaka> I see
18:42fdobridge: <marysaka> it's crashing inside ``nvkm_gr_units`` from the little I looked
18:43fdobridge: <marysaka> Well I guess I will boot my laptop I guess
18:43fdobridge: <marysaka> Well I guess I will boot my laptop (edited)
18:47fdobridge: <karolherbst🐧🦀> huh.. that shouldn't crash. odd
18:58fdobridge: <marysaka> It's because gr is NULL
18:58fdobridge: <marysaka> https://github.com/torvalds/linux/blob/master/drivers/gpu/drm/nouveau/nvkm/engine/device/base.c#L2628-L2647
18:59fdobridge: <karolherbst🐧🦀> mhhh... I guess nouveau doesn't verify all the ioctls.. is that on the device_info stuff?
18:59fdobridge: <karolherbst🐧🦀> ehh.. wait, that's getparam with `NOUVEAU_GETPARAM_GRAPH_UNITS`
18:59fdobridge: <marysaka> https://github.com/torvalds/linux/blob/b44f2fd87919b5ae6e1756d4c7ba2cbba22238e1/drivers/gpu/drm/nouveau/nouveau_abi16.c#L237
19:00fdobridge: <marysaka> yeah
19:00fdobridge: <karolherbst🐧🦀> okay, I see
19:00fdobridge: <karolherbst🐧🦀> nouveau doesn't check if the firmware got loaded or the device is even supported
19:00dakr: airlied: just pushed some patches to validate bind/unbind ops, hope this covers the error case you run into.
19:00fdobridge: <karolherbst🐧🦀> guess we should catch that in nvk as well, but the kernel should also not hit a null pointer here
21:44airlied: dakr: yes that should catch the garbage I was passing in yesterday
23:09airlied: dakr: just fyi pushing a patch onto wip branch
23:25airlied: looks like my discord a/c got disabled again
23:25karolherbst: yo the hell
23:26karolherbst: why does this happen to you and danvet :D
23:26HdkR: airlied: Did you have 2FA enabled?
23:26karolherbst: and here I am like with neither of those things :D
23:26karolherbst: probably should enable 2FA tho
23:27HdkR: As far as I'm aware if you have 2FA enabled then you are less likely to be flagged as a bot
23:27karolherbst: I did connect my steam/github/twitter acc tho
23:27HdkR: neobrain also had their account banned+deleted because of their autobot crap
23:27karolherbst: yeah heh..
23:28HdkR: The original FEX-Emu discord server creator also had it happen, but they managed to get it unbanned.
23:28karolherbst: the only real alternative I am currently consider as an IRC replacement or addition would be mattermost atm.... but ughghhhuhduahsd...
23:28danvet: karolherbst, btw still not yet unbanned :-/
23:28danvet: airlied, so you're in discord purgatory too?
23:28karolherbst: guess it remains a memeing server then
23:28danvet: well not without me
23:28danvet: unless you folks all stepped up your game a bit meanwhile
23:29karolherbst: the moment you were gone, we posted like 10000 memes
23:29danvet: I have memes for decades in the queue for this
23:29danvet: HdkR, so does 2fa help or hinder?
23:29HdkR: prior to ban I think it helps
23:30airlied: danvet: I got banned last week and unbanned, not banned again
23:30danvet: lol, first disabling was before I even managed to create an account
23:30airlied: HdkR: I didn't even see the option to turn 2fa on, I should looke closer
23:30HdkR: So far I haven't heard of anyone getting bot-banned that have 2FA enabled
23:30airlied: danvet: yeah I that as well
23:30danvet: airlied, lol
23:30airlied: I was just trying to log in, and it banned me
23:30HdkR: User Settings-> 2FA on the first page :D
23:30danvet: airlied, so you're 1 ban ahead of me?
23:31airlied: I see on /. they are talking big about killing spam acocunts
23:31airlied: so I suspect they were probably just trying to show up twitter or something and got a bit over zealous
23:31danvet: yeah the bot detect seems utter garbage
23:31danvet: like I consistently fail their captcha too
23:31danvet: when it bothers showing up
23:31karolherbst: soo.. 2FA enabled
23:32airlied:wonders why they don't force 2FA if they are disabling accounts without it
23:32airlied: doesn't seem like the discussion place to bring new people in
23:32danvet: yeah it's pretty good fail, at least rn it seems
23:32karolherbst: airlied: well.. I don't see how 2FA would prevent bots tbh...
23:33karolherbst: like bots can just read qr codes and pass the test...
23:33HdkR: Hopefully effort to care :P
23:33karolherbst: I'd say linking github/whatever accounts helps more :P
23:33karolherbst: like who else would have a steam account with thou...... ehm.. a lot of games
23:34danvet: airlied, where do you see a discussion on /.?
23:34danvet: I can't even find a search on that ....
23:34airlied: Discord Bans 68,000 Servers, 55 Million Accounts
23:34karolherbst: there was some news on this
23:34airlied: news article on /. frontpage
23:35danvet: uh maybe it's annoyed about the cookie/adblock settings here
23:35karolherbst: huh.. would be weird
23:35karolherbst: because I do have adblock as well
23:36karolherbst: also... which ads
23:36karolherbst: or do you have DNS based adblock in your network?
23:36danvet: nah that not I think
23:36danvet: but yeah maybe too strict cookie settings and it gets mad
23:36karolherbst: I also have a more or less aggressive ublock in chromium and that also always worked
23:36karolherbst: just link your github and twitter accounts :P
23:36danvet: hm yeah ublock was more my worry, dunno
23:38karolherbst: but I am mainly using the client though
23:38karolherbst: support should deal with it.. *sigh*
23:38danvet: yeah mostly webapp here, maybe this is the reason
23:39danvet: also had no idea you could even link twitter and shit like that
23:39danvet: I kinda don't want to link everything
23:39karolherbst: you don't have to make it public tho
23:40danvet: ah maybe I should try then
23:41karolherbst: the only permission discord wants from twitter is "Display your Twitter account on your Discord profile" btw
23:41karolherbst: and my github acc is linked as an oauth provider
23:41karolherbst: steam dunno.. too lazy to check
23:43karolherbst: could also pay 10€/m :P I am sure that won't get you banned
23:48danvet: tbh they're good enough at banning I'm not sure they'd try that
23:48danvet: *I'd try that
23:48danvet: anyway I really should be sleeping instead of ranting about meme servers
23:48karolherbst: yeah well, I'd ban bots if they give me money, so why would discord :P
23:48karolherbst: meme servers are important
23:49danvet: not when I can't be there
23:49karolherbst: that's not my problem
23:49danvet: also way too many memes along the lines of "your meme game so strong discord had to ban you to give all the other noobs reprieve" in the queue :-P
23:50karolherbst: I'll be waiting
23:50karolherbst: *skeleton watiing on bench meme*
23:51danvet: hey I used that one already
23:51danvet: and the pablo escobar one too
23:52karolherbst: you wasted it you mean
23:52danvet: good memes are never wasted
23:52danvet: they get stronger with every misuse pretty much
23:53danvet: in dire situations I reach back to 10 year old weapons grade classics :-)
23:53karolherbst: do it
23:53danvet: also I kinda did already
23:54karolherbst: well... could create a freedesktop-meme channel and bridge it
23:55danvet: wow, success kid is 15 years old
23:56danvet: karolherbst, yeah but where's the fun without emotes
23:56danvet: nor inline meme display
23:57karolherbst: maybe we should just come up with actual competent IRC clients
23:57karolherbst: opening URL ain't that hard
23:57danvet: yeah, unless you think remote exploits and tracking are uncool
23:57karolherbst: danvet: funny though, the bridge displays the meme on discord :D
23:58danvet: ^^ pls lots of headpats on this one, I'm very sad
23:58karolherbst: so you see you could meme
23:59danvet: not that old really
23:59karolherbst: became a classic one real quick