/srv/irclogs.ubuntu.com/2013/07/24/#ubuntu-mir.txt

robotfuelkgunn: it doesn't save that information in the logs00:03
robotfuelkgunn: I will rerun the tests to get that info, it goes to stdout00:03
robotfuelkgunn: I don't notice any tearing, but open arena is always moving it would be hard to see00:06
robotfuelkgunn: I'll watch again and get back to you. brb00:07
robotfuelkgunn: openarena runs too fast to tell if it's tearing00:27
Sarvattrun non xmir openarena with vblank_mode=0 openarena?00:31
robotfuelxorg had a segfault on intel with xmir http://10.97.2.10:8080/job/utah-xmir-troubleshooting/ws/results/Xorg.0.log00:31
tvoss_RAOF, ping05:08
sgx1hi. now there're two mir PPAs, which one to use? installing_prebuilt_mir_on_pc.md suggests using system-compositor-testing ppa. but i find packages in Mir staging ppa are newer.05:27
tvoss_sgx1, yup, but staging revs too fast and system-compositor-testing is the save alternative so far :)05:28
tvoss_sgx1, we cherry-pick over to system-compositor-testing on a regular basis05:28
RAOFtvoss_: Pong.05:28
robert_ancellRAOF, duflu, alf__, hikiko, racarr, kdub, kgunn, meeting time!05:58
robert_ancellthomi too!05:58
hikikohi05:58
hikikojoining in a sec robert_ancell05:59
kgunnkdub: you around?06:06
tvoss_didrocks, ping06:25
didrockstvoss_: pong06:29
tvoss_didrocks, got a branch for you to test on the ati machine :)06:29
didrockstvoss_: yeah! there are two ways: building it locally (but I don't have any i386 machines ready)06:30
didrockstvoss_: or pushing that to the ppa manually06:30
didrocksI'm in favor of #206:30
tvoss_didrocks, so what do you need from me? just the branch?06:31
didrockstvoss_: yep, the branch is fine06:31
tvoss_didrocks, lp: ~thomas-voss/mir/default-to-null-input-by-default06:32
didrockstvoss_: branching, pushing, building06:32
didrocksthen will rerun06:32
tvoss_didrocks, ack06:32
tvoss_didrocks, status?07:05
tvoss_didrocks, I need to step out at 10 my time (in ~55 minute) to run some errands07:06
didrockstvoss_: well, it's building in the ppa right now07:06
didrockstvoss_: and as there is an ABI break, regarding to latest version, I have to rebuild u-s-c as well07:06
didrocksso waiting for mir to publish first07:07
tvoss_didrocks, ack ... which ppa are you using?07:07
didrockstvoss_: daily-build-next07:09
tvoss_didrocks, ack and thx07:09
didrocksyw ;)07:09
didrockswill keep you posted07:09
tvoss_didrocks, stepping out now, back in ~1 hour07:38
didrockstvoss_: ok, u-s-c is building now07:38
tvoss_didrocks, ack07:38
dholbachgood morning07:46
mlankhorstdidrocks: ping08:03
didrocksmlankhorst: pong08:05
mlankhorstdidrocks: on that non-working machine, could you post the full logs from /var/log/lightdm and full dmesg?08:05
didrocksmlankhorst: I are running a rerun from tvoss first08:06
didrocksmlankhorst: so wait for it I would say ;)08:06
didrockscurrently running, so shold have results soon08:06
mlankhorstyeah my own test machine fails with std::exception::what: Failed to set the current VT mode08:07
* mlankhorst doesn't have plymouth or splash enabled, might be why08:07
mlankhorsthm seems to be https://bugs.launchpad.net/mir/+bug/119550908:13
ubot5Launchpad bug 1195509 in Unity System Compositor "System compositor fails to start - Failed to set the current VT mode: Input/output error (5)" [Critical,Triaged]08:13
=== dholbach_ is now known as dholbach
didrocksmlankhorst: do you start it from lightdm?08:31
mlankhorstyeah08:31
didrocksweird08:31
=== seb128_ is now known as seb128
mlankhorstdoes lightdm switch to the console before letting unity-system-compositor configure it?08:33
mlankhorstand can I use strace on ust somehow, to figure this one out08:37
didrocksmlankhorst: I can answer on the second one08:37
didrocksmlankhorst: /etc/lightdm/lightdm.conf.d/08:38
didrocksyou have a file shipped by usc08:38
didrocksyou can put a command08:38
didrockslike: system-compositor=strace unity-system-compositor08:38
mlankhorstyeah08:40
mlankhorsthm of course that works..08:40
mlankhorstdidrocks: who spawns the xserver, unity-system-compositor or lightdm?08:49
didrocksmlankhorst: ligthdm AFAIK08:51
didrocksbut I can be wrong08:52
mlankhorsthm seems to be a plymouth race :P08:54
tvossdidrocks, ping09:38
didrockstvoss: wb!09:38
didrockstvoss: so passed \o/09:38
didrockstvoss: please file a MP ;)09:39
didrocks(I'm still rerunning to ensure we don't get any race, but should be good)09:39
tvossdidrocks, \o/09:40
tvossRAOF, any more insight? any handover tasks I should look into?09:40
mlankhorstlooks like it's still failing locally, I'll try to figure out why :p09:40
RAOFtvoss: I'm still hooking up exposing the buffer id to the client. Once that's done, I'll instrument everything and give it a run.09:41
tvossRAOF, awesome, thank you :)09:41
tvossgreyback, ping09:43
tvossalf__, didrocks https://code.launchpad.net/~thomas-voss/mir/default-to-null-input-by-default/+merge/17663809:45
greybacktvoss: pong09:48
alf__tvoss: what's the driving force behind default-to-null-input-by-default ?10:00
tvossalf__, first: it saves us a bunch of wakeups per second in the system compositor scenario. Second: it solves issues with booting up on certain ATI HW where creating a bo for the hw cursor results in an exception10:01
alf__tvoss: can't the system compositor turn this off, instead of making it the default?10:02
tvossalf__, possible, too10:02
didrockstvoss: should I rebuild Mir without your patch and be ready to take the new system-compositor in?10:20
didrockstvoss: ran 4 times, with full test suit and pass10:20
mlankhorstI think I understand the bug, but not the fix yet :P10:23
tvossmlankhorst, which one exactly? the ati one?10:23
mlankhorsthttps://bugs.launchpad.net/mir/+bug/119550910:23
ubot5Launchpad bug 1195509 in Unity System Compositor "System compositor fails to start - Failed to set the current VT mode: Input/output error (5)" [Critical,Triaged]10:23
tvossmlankhorst, \o/ I have been looking at that, too10:23
tvossdidrocks, yup, but might take me some time, an hour or so10:23
didrockstvoss: ok, I can rebuild Mir meanwhile10:24
didrocksto not loose time10:24
didrockstvoss: or will you have Mir changes?10:24
didrocks(to expose an API or so on…)10:24
mlankhorsttvoss: well it's a dumb bug10:24
tvossdidrocks, most likely no mir changes, at least I hope so10:25
didrockstvoss: ok, let me do a preventive build then :)10:25
RAOFmlankhorst: Something involving plymouth is a dumb bug? Say it ain't so!10:27
mlankhorstI don't get why plymouth is killed by lightdm *after* xserver started..10:28
mlankhorstor mir10:28
didrocksmlankhorst: I think for smooth transitioning? (just a guess)10:31
mlankhorstanyway the io comes from the TTY having hung up10:35
RAOFNo, that shouldn't be it.10:35
mlankhorstI'm guessing it is being opened too early, while plymouth still has control over it10:35
RAOFmlankhorst: How's that mir-plymouth branch goin? :)10:36
mlankhorstRAOF: blocking on the same thing as I want to block Xmir for, the separate thread is ugly10:36
RAOFmlankhorst: Indeed it is; bug tvoss about that :)10:37
mlankhorstbut I did a control WARN_ON(1), and I did get that unity-system-compositor had a WARN_ONn hung_up_tty_ioctl+0x27/0x5010:37
mlankhorstso it looks like this is the issue here10:38
tvossmlankhorst, RAOF happy to adjust the separate thread, but can we fix the bug without me on the critical path?10:38
mlankhorsttvoss: erm sure, just postpone opening the fd or patch lightdm some more10:39
tvossmlankhorst, cool10:39
mlankhorstI can do the latter10:39
RAOFtvoss: Yeah, it's fixable. It's just that there wouldn't be an opportunity for this bug to exist if we weren't doing the stupidly fragile plymouth dance.10:39
tvossRAOF, help me understand, what is the fragile plymouth dance? :)10:40
mlankhorsttty handover from plymouth10:40
RAOFPlymouth starts, grabs the display. We then ask plymouth to die but hide its dying, and then X starts up, grabs the display, and takes the contents of the framebuffer.10:41
mlankhorstwell it looks like a workaround here might be to use VT_SETACTIVE instead of VT_ACTIVATE10:42
mlankhorsterm VT_SETACTIVATE *10:42
* duflu goes to find food10:43
* tvoss loves those subtle differences for VT_* ;)10:43
tvossduflu, good luck10:43
* duflu rolls eyes at those identifiers10:43
mlankhorstwe'd snatch the terminal away from plymouth, if it's still active :P10:43
tvossRAOF, so in an ideal world, plymouth would just be a mir client, right?10:43
RAOFCorrect!10:43
mlankhorstmeh lets see if it works..10:44
RAOFmlankhorst: Can we just nuke the whole VT subsystem?10:44
tvossRAOF, would disconnecting from the controlling tty for Mir help here?10:48
mlankhorstlets see if this works...10:49
mlankhorsttvoss: when touching the modes of the controlling tty, you want to not be attached to the tty?10:49
tvossmlankhorst, random idea :)10:49
mlankhorstI guess not touching the tty at all could work though :p10:52
* tvoss mutters something about process group leaders and setsid10:53
tvossalf__, I set the status of https://code.launchpad.net/~thomas-voss/mir/fix_process_waitpid_approach_to_correctly_setup_tracing/+merge/17636711:05
tvossback to needs review11:05
tvossI'm quite sure I can fix the heisenbug :)11:05
mlankhorsttvoss: actually it might need setsid for this to work correctly..11:09
mlankhorstis unity-system-compositor run as root?11:11
tvossmlankhorst, I think so11:11
alf__tvoss: as you like, I figured that since this MP was not related to the problem that cause the failure, we might as well merge it11:12
RAOFmlankhorst: Yes, u-s-c is run as root.11:13
mlankhorstblergh, looks like we need to do setpgid, setsid, before opening console11:13
mlankhorstafter that when /dev/tty is opened it automatically becomes ours, or in the worst case we need to claim it for ourselves11:14
tvossalf__, fair point, I will repropose under a different name11:15
mlankhorstI now know more about job control than I cared for :p11:25
RAOFtvoss: I'm off to bed; I've not yet managed to run a fully instrumented stack, so I'm not sure what I'll find.11:41
tvossRAOF, ack, thanks11:44
mlankhorstmeh got a fix, I think11:45
mlankhorsthttp://paste.debian.net/18230/11:46
mlankhorstthe TIOCSCTTY needs to be uncommented if the setsid is not enough..11:47
mlankhorstbut it's kind of rude since it means snatching the tty away from someone else :p11:50
tvossmlankhorst, well, that's the consequence of our plymouth dance, right? :)11:54
mlankhorstI guess so, what is the sleep for btw11:54
mlankhorstit seems to work fine without11:54
tvossmlankhorst, didrocks was seeing some races iirc :) didrocks ^?11:55
mlankhorstapart from some gpu stall everything works fine I guess11:58
mlankhorstand the stall might be related to changes in my own tree11:58
mlankhorstmeh the remainder seems to be related to my card being flaky12:02
tvossmlankhorst, \o/12:05
tvossmlankhorst, wanna come up with an mp?12:05
mlankhorstwhat branch is base?12:07
tvossmlankhorst, trunk, bzr branch lp:mir12:08
mlankhorstok12:08
mlankhorstlets see if this works12:12
tvossalf__, didrocks https://code.launchpad.net/~thomas-voss/mir/fix_process_waitpid_approach_to_correctly_setup_tracing/+merge/17636712:17
mlankhorsthm, I need a better git-bzr12:17
=== JoseeAntonioR is now known as jose
tvossmzanetti, could you retrigger CI for https://code.launchpad.net/~thomas-voss/mir/fix_process_waitpid_approach_to_correctly_setup_tracing/+merge/17636712:30
tvoss?12:30
tvossdidrocks, you around?12:30
mzanettitvoss: sure. If you want I can give you permissions to retrigger jobs12:30
tvossmzanetti, I have, still need to setup my vpn :)12:31
tvossagain :)12:31
mzanettitvoss: actually, in this case its enough to re-set the MR to "Approved" and Jenkins will pick it up again12:31
tvossmzanetti, top-state?12:32
mzanettitvoss: yeah12:32
didrockstvoss: back now12:32
tvossdidrocks, https://code.launchpad.net/~thomas-voss/mir/fix_process_waitpid_approach_to_correctly_setup_tracing/+merge/17636712:32
tvossdidrocks, that's not the usc adjustment12:33
didrocksok, looking for the additional commit12:33
didrockstvoss: ok, a little bit complex. What I would suggest is that we get the ATI fix first, then, having that running (with the workaround) a couple of day, and then, removing the sleep workaround, thoughts?12:35
tvossdidrocks, not sure I do get you :) the mr I pasted is not connected to the ati issue ;)12:36
didrockstvoss: right, what I meant is:12:36
didrockslet's not try to remove the race workaround for now12:36
didrocksyour fix is to avoid this issue, right? ^12:37
tvossdidrocks, where do you see that I remove it?12:37
tvossnope :)12:37
mlankhorstlook I don't think that this is ready for universe if it requires a sleep workaround..12:37
tvossdidrocks, mlankhorst might have a fix for the sleep workaround12:37
didrockstvoss: ah, I thought the waitpid would help that case as well, but maybe not enough context in the diff and it would be better than I download the whole file :)12:37
didrocksah good ;)12:38
tvossdidrocks, yeah, this should fix the armhf races12:38
didrocksbut I would suggest that we keep the sleep for a couple of days (even if we have the fix)12:38
didrocksto get things flowing12:38
didrocksand then, trying to removing it and testing several rounds12:38
tvossdidrocks, for disabling the input: can we pass additional command-line arguments to usc via lightdm?12:39
didrockstvoss: it seems to be possible, you just can't use shell expansion or && for instance12:40
didrocksbut command --option seems to work12:40
didrocks(as "strace unity-system-compositor" worked)12:40
tvossdidrocks, okay, then you can just pass --enable-input=false to usc :)12:41
didrockstvoss: this is already implemented?12:42
tvossdidrocks, yup12:42
didrockshum, grep isn't my friend12:42
tvossdidrocks, usc uses mir's DefaultServerConfiguration, which in turn respects that command-line option12:42
didrocksah ok ;)12:42
tvossdidrocks, you need to fgrep in mir12:43
didrocksyeah, makes sense12:43
tvosssrc/server/default_server_configuration.cpp12:43
didrocksindeed, saw it :)12:43
didrockstvoss: https://code.launchpad.net/~didrocks/unity-system-compositor/fix-for-ati/+merge/17667412:45
didrocksI'll then rebuild once in trunk and ensure with multiple test runs it's reliable12:45
mlankhorstand again that needs fixing too :P12:46
mlankhorsthttps://code.launchpad.net/~mlankhorst/mir/setsid/+merge/17667612:48
tvossdidrocks, mind filing a bug against usc to make sure that we reenable hw pointer functioanlity shortly?12:50
didrockstvoss: sure12:50
tvossdidrocks, thx12:50
mlankhorstI don't think papering around it is a good thing though, that bug should be fixed first12:51
didrocksmlankhorst: we have deadlines as well, we can't be idealistic when working on time constraints12:51
didrocksas long as bugs are logged in, I don't think we should hold off12:51
didrocksif we know that the workaround is reliably working12:52
mlankhorstyeah but you want to land a random git snapshot of mesa, some xorg patches and some other stuff and then put mir in universe?12:52
didrocksmlankhorst: mir will be in main I guess12:52
didrocksto be able to build xorg12:52
didrocksas it's a build-dep, right?12:52
mlankhorstyes12:52
didrocksso Mir in main, but not enabled by default12:52
didrocksjust a technology snapshot to get people trying it12:53
mlankhorstand if it's in main it means I have to support it, including the mir bits. So open bugs that are not understood become a lot more important to me then..12:53
robotfuelDoes anyone know how I change the resolution in xmir? xrandr doesn't work12:54
didrocksmlankhorst: you're welcome to give a hand to properly resolving them promptly consequently12:54
didrockstvoss: https://bugs.launchpad.net/mir/+bug/120450512:54
ubot5Launchpad bug 1204505 in Mir "unity-system-compositor needs to have hw pointer functionality enabled back" [Undecided,New]12:54
tvossdidrocks, thx12:55
didrockstvoss: not sure how you want to prioritize it12:55
tvossdidrocks, not sure yet, put it high, please12:55
didrocksdoing so12:55
robotfuelSarvatt: I tried openarena with vblank_mode=0 in xmir, it still was a lot faster than x13:11
tvossbregma, ping13:12
bregmatvoss, hi13:12
kgunntvoss: thank for all your help overnight!13:14
tvossbregma, I looked into the acceptance tests, input-related ones specifically, and came up with an MP13:15
tvosshttps://code.launchpad.net/~thomas-voss/mir/fix_process_waitpid_approach_to_correctly_setup_tracing/+merge/17636713:15
kgunnrobotfuel: if that weird "better on xmir than x" only happening on ati gpu ? or e.g. on my intel i did not see this...i saw a 10% drop13:15
tvossbregma, it contains a pipe-based cross-process sync appraoch and adds a timeout to wait_for_termination13:15
robotfuelkgunn: yes this is only ati13:15
kgunnrobotfuel: wondering if its related to ati video driver13:15
bregmatvoss, will it handle the case where a server child process is waiting for a client child process and the client child process exits without sending a synch message?13:16
* kgunn wonders if intel is vsync'ing properly...and ati is unhinge from vsync13:16
robotfuelkgunn: we also can't to xmir benchmarks at 800x600 until this bug is fixed https://bugs.launchpad.net/mir/+bug/119623913:16
ubot5Launchpad bug 1196239 in Mir "Cannot change display resolution" [Critical,Triaged]13:16
robotfuelkgunn: we can get the default resolution for the monitor though13:17
tvossrobotfuel, that's fine for now13:18
kgunnrobotfuel: hmmm & that bug is only on ati as well correct ?...again, i get different #s on my intel when i run 600x800 vs native res, meaning...my intel numbers all look sane13:19
robotfuelkgunn: it might be, intel isn't working for me right now, it has the driver bug13:20
robotfuelkgunn: can you delete xserver-xorg-video-intel package version 2:2.21.9+xmir5870-1~saucy1 from the mir-team/system-compositor-testing ppa?13:21
robotfuelkgunn: the 0~saucy1 driver is suppose to work13:22
kgunnrobotfuel: lemme see13:23
kgunntvoss: so, robotfuel asked for me to roll back intel driver to 0~saucy1....but before i venture off, do you know why raof manually updated it ? do you have some context ? (is this for sna ?)13:28
tvosskgunn, I actually copied it over, it's the driver with sna enabled13:29
tvossrobotfuel, are you sure that you have the ppa pinned? the driver is working perfectly fine here13:29
kgunnrobotfuel: confirmed...its working on mine for past few days as well13:31
kgunni was tvoss guinea pig & just left it installed13:31
robotfueltvoss: kgunn  https://bugs.launchpad.net/xmir/+bug/1203776 a lot of people have a problem with the 1~saucy driver13:42
ubot5Launchpad bug 1203776 in XMir "X crashes w/ latest xserver intel driver from System Compositor Testing PPA" [Undecided,New]13:42
tvossrobotfuel, are we sure that those people did pin the ppa correctly?13:43
tvossrobotfuel, can you try the intel driver from staging real quick on the machine?13:43
robotfueltvoss: I'll try the one from the staging ppa, RAOF's looked at the bug and verified it.13:44
tvossrobotfuel, ack and thx13:45
didrockstvoss: ok, unity-system-compositor starts and hang (remember that it's with Mir trunk rebuilt from the last couple of hours)13:45
didrockstvoss: dm_connection_start13:45
didrocksFailed to read header13:45
tvossdidrocks, dm_connection talks to lightdm13:46
tvossrobotfuel, if your issue is fixed with the intel driver, I will copy it over to the testing-ppa13:46
tvosskgunn, ^13:46
didrockstvoss: new xorg-server in distro…13:47
didrocksI bet that's it13:47
* didrocks looks at the version13:47
tvossargh13:47
robotfueltvoss: didrocks http://10.97.2.10:8080/job/utah-xmir-troubleshooting/3/artifact/results/dpkg-list.log has the versions of everything installed when the xorg crash happens13:49
didrockstvoss: yeah, I bet it's the xorg issue13:49
didrockstvoss: I'll just apt-get source the old one13:49
tvossdidrocks, for the dm_connection thingy?13:49
didrocksand update with it13:49
didrockstvoss: yeah, it's a consequence13:49
didrocksxorg doesn't have -mir option13:50
tvossdidrocks, ack13:50
tvossdidrocks, ah ...13:50
didrocksso, /me apt-get source, fake a bump changelog and reupload to daily-buid-next ppa13:50
didrocksmlankhorst: would be nice to coordinate with us and not make the whole thing more painful :p13:51
kgunndidrocks: so we're not dead yet, once  you update daily-build-next ppa we get another shot at tvoss "disable input" for the ati bug?13:53
didrockskgunn: right13:54
didrockskgunn: I have faith, his fix in mir directly worked with multiple runs13:54
didrockskgunn: I think if the option in unity-system-compositor is correctly wired up, it shouldn't make a difference and should pass as well13:54
kgunndidrocks: that's good news...13:54
kgunnthanks13:54
xjuniorMorning edge people13:55
didrocksmlankhorst: actually, it was doko on that one…13:57
tvossrobotfuel, did you install the driver, yet?14:00
robotfueltvoss: no not yet, I'll do it now.14:01
tvossrobotfuel, ack14:02
greybackalf__: ping14:07
robotfueltvoss: FYI the drivers in the latest drivers staging ppa failed to build https://launchpad.net/~mir-team/+archive/staging/+packages14:10
alf__greyback: pong14:12
greybackalf__: hey, the snapshot API is working great, thank you14:12
alf__greyback: Great news! Thanks for letting me know.14:14
greybackalf__: I've one question however. I grab a snapshot just after the application surface is created, which is a touch too early as I guess the application hasn't drawn to it yet (i.e. I get black snapshot). Would there be a way to detect if an app has actually drawn to it's surface14:14
greybackalf__: this is nothing at all major (for now I just delay snapshot with a timer), but would be nice for later14:14
tvossgreyback, I think we should postpone that discussion until we have cleaned the scenegraph-conglomerat within mir14:15
robotfueltvoss: the lastest successful build from the staging ppa is already installed on the system. It doesn't work14:16
alf__greyback: We have such a mechanism internally, but I don't think it is exposed in any way to the shell.14:16
greybackalf__: I see. I'll keep it in mind. I might log a bug for it. Thanks!14:22
alf__greyback: np14:23
robotfueltvoss: I have some other intel systems that didn't have an xorg crash14:38
robotfueltvoss: I am retrying to see if it's a race condition.14:38
tvossrobotfuel, ack14:38
robotfuelkgunn: resolution changes do work on intel, not on radeon. :)14:45
kgunnrobotfuel: i am so frustrated with the ati...14:46
kgunn:-/14:46
tvossdidrocks, status?14:59
didrockstvoss: 3 runs until now, it's all green14:59
robotfueltvoss: the xmir crash on intel is race condition14:59
didrockstvoss: I'm waiting for the 4th to finish14:59
tvossrobotfuel, where?14:59
didrocksto send an email :)14:59
tvossdidrocks, ack and thx14:59
didrockstvoss: yw! thanks to you!14:59
robotfueltvoss: https://bugs.launchpad.net/xmir/+bug/120377615:01
ubot5Launchpad bug 1203776 in XMir "X crashes w/ latest xserver intel driver from System Compositor Testing PPA" [Undecided,New]15:01
tvossrobotfuel, no, I meant where in the code ;)15:03
robotfueltvoss: I haven't looked in to it.15:03
tvossrobotfuel, okay, thx15:05
qenghoHi all. I maintain Chromium browser, and I received a pointer from upstream to a doc about their efforts to abstract some of the display interface.  We may find it interesting. https://sites.google.com/a/chromium.org/dev/developers/design-documents/ozone15:05
tvossqengho, hey there :)15:08
qenghotvoss: yo.15:08
tvossqengho, I recently read it, quite interesting :)15:11
hikiko_alf__, revno 881 works better: I see the mouse pointer but I can't move it15:12
qenghotvoss: Thanks.15:15
tvossqengho, thanks for keeping us posted15:16
tvossbregma, ping16:15
bregmatvoss, pong16:26
tvossbregma, so my local testing reveals some issues under valgrind, still trying to digest all of the information16:26
bregma....16:28
tvossbregma, thought I would handover, as your mail suggested to me that you are looking into the issue, too16:28
bregmasure16:29
=== om26er is now known as om26er_afk
=== om26er_afk is now known as om26er
kdubkgunn, ping17:46
kgunnkdub: pong17:48
racarrlol...whoops18:53
racarrI just found a unit-test file which is missing from cMakeLists.txt18:53
racarrafter spending 5 minutes like oh god how does this compile18:53
racarrdo I even understand C++18:53
racarrwhat is happening18:53
racarretc18:53
kdubhaha18:53
racarr(missing from CMakeLists in trunk)18:54
racarrfor who knows how long18:54
racarrbecause the API it uses is quite old18:54
racarrthats ok though its a lame test18:54
racarr*makes new branch*18:54
racarrI wonder if I should write a failing test that all our test files are built :p18:55
racarromg19:04
racarrmore dead files19:04
racarrthis is embarassing -.-19:04
kdubwhich tests?19:05
racarrkdub: test_android_input_registrar19:08
racarrand mock_input_configuration.h which it includes19:08
kdubwhat19:08
racarrits been this way for at least19:08
racarr3 months19:08
racarrbecause Ir emember I rebuilt the input configuration19:08
racarrwhen I still lived at my old place19:08
racarrwe could use bzr blame but it's almost certainly my fault so19:08
racarr:p19:08
kdubah, i thought you said another registrar test that I was sure was working lately :)19:09
racarrI hate using droidinput::Vector19:59
racarrs/hate/have a strong distaste19:59
racarrhttps://code.launchpad.net/~robertcarr/mir/support-monitor-input-channels/+merge/176775 whee20:11
racarrgood news, unity is done20:12
racarr*closes eyes and begins to sleep*20:12
racarr:p20:12
kdubracarr, droidinput::Vector can't be as bad as writing new classes that support android::sp :)20:16
racarrkdub: Ah, android::sp20:17
racarryou know the input stack still isnt the most beautiful code20:18
racarrbut we have come a long way since20:18
racarrandroid::sp in DefaultServerConfiguration20:18
racarrXD20:18
kduboh yeah, much better now :)20:36
kdubmir on the whole is much better20:36
kdubremember back in the day, when we wrapped std::thread/mutex/chrono with boost? :)20:36
racarrkdub: Oh god20:56
racarrthat led to the most fucked up compiler errors.20:56
racarrI just remember, coming on to the Mir team, in the past I had been a perpetual C++ skeptic but I was like20:57
racarrok well I guess if they are doing C++ carefully and the right way it seems pretty good and well lets try it20:57
racarrthen I ran in to some stuff from the std::thread stuff and mir::std and I was just like20:57
racarroh god....20:57
racarrBack soon21:00
=== racarr is now known as racarrlunch
ollirobert_ancell, ping21:24
robert_ancellolli, hello21:27
ollihey robert_ancell21:27
olliquick q, I am trying to understand the new dashboard gema announced recently21:27
ollifrom there it looks like we are building u-s-c without running any tests21:28
ollihttp://91.189.93.67/staging/merger/unity-system-compositor/21:28
ollikgunn, ^21:29
robert_ancellolli, u-s-c doesn't have any tests currently21:30
ollirobert_ancell, how come, I thought anything mir has tons of tests21:30
ollithat's what I keep getting told at least21:31
olli:)21:31
robert_ancellolli, not u-s-c. :(21:31
ollihow do we intend to land that in distro if it doesn't have tests21:32
=== racarrlunch is now known as racarr
ollirobert_ancell, ^21:35
ollibrb21:35
robert_ancellolli, it doesn't have much code over top of existing Mir so it is quite well tested. We will have integration tests that ensure that the whole stack works together. But yes, it should have some local tests too21:37
racarrbecause of the structure we use kind of like...21:43
racarr1. Mir component. 2. default implementation in tree for demo-server and system-compositor21:43
racarr3. another implementation for unity21:43
racarrI agree USC is really pretty well tested21:43
racarreven if it has no in tree tests21:43
ollirobert_ancell, what's the plan to get tests in place21:51
robert_ancellolli, file bugs and prioritize them21:54
ollithat's a reaction to my ping but apparently not a long standing plan21:55
robert_ancellkernel panic, please repeat anything directed at me since " robert_ancell, what's the plan to get tests in place"21:57
olli<robert_ancell> olli, file bugs and prioritize them21:59
olli<olli> that's a reaction to my ping but apparently not a long standing plan21:59
robert_ancellolli, the long plan is the same as Mir - new functionality needs new tests. At the moment we have very little functionalty22:00
robert_ancellracarr, regarding https://code.launchpad.net/~robertcarr/mir/session-transactions/+merge/175669 alan is away22:17
robert_ancellthis week22:17
robert_ancellSo, get a second opinion or you have to wait if you need his response22:17
racarrrobert_ancell: Hoping for a second opinion :)22:22
racarrnot in the, need someone to approve it sense, but in the actually trying to figure out if its a good idea sense22:22
kgunnthomi: question about http://10.97.2.10:8080/job/mir-saucy-amd64-ci/22:46
kgunnam i reading that right...are the number of tests dramatically dropping?22:47
kgunnor did they migrate...http://10.97.2.10:8080/job/mir-vm-ci-build/22:48
kgunn?22:48
* thomi looks22:53
thomikgunn: that graph seems odd - I'm sure we don't have 50000 tests :-/22:56
thomikgunn: looks like the results are being repeated a bunch of times: http://10.97.2.10:8080/job/mir-saucy-amd64-ci/216/testReport/%28root%29/SurfaceCreation/22:57
robert_ancell23 outstanding MPs! We need to land some code here23:28
robert_ancellkdub, can you check if you're happy with https://code.launchpad.net/~afrantzis/mir/rectangle-closed-open/+merge/17634823:29
kdubrobert_ancell, sure23:29
kdubi think i am, just wanted to get the input take on the matter23:29
robert_ancellkdub, also, if you've fixed the issues duflu pointed out https://code.launchpad.net/~kdub/mir/add-pf-query/+merge/176451 can land23:31
racarrgreyback: https://code.launchpad.net/~robertcarr/mir/support-monitor-input-channels/+merge/17677523:35
racarra present for you23:35
greybackracarr: aww geee, you shouldn't have!23:35
racarrgreyback: Don't worry, I left a bag of coal in the Qt scene graph thread so it's balanced23:36
greybackracarr: :)23:36
racarrthe impl is a little different than how it was on SF but I think as long as23:36
racarryou are using the surface InputRegioni23:36
racarrint he same way the trap windows were used before23:37
racarrit should work23:37
racarrI even read some QML code23:37
greybackracarr: goodness, you have been busy23:39
=== qengho_ is now known as qengho
racarrgreyback: Nah. it's just a carefully crafted illusion23:43
racarr*kicks up feet*23:43
racarrHappy to see your video last night before I went to sleep :)23:43
racarrdid you guys get my email to scene graph thread because I didnt get my own email23:44
racarrbut normally do...23:45
greybackracarr: it looks like it does all I need. I'll try integrating it tomorrow. Note I'm on holiday Friday, so tomorrow is gonna be busy for me23:45
racarrif it's simple, i.e. surface needs to become a monitor surface and X needs to be hooked up to Y and bla and bla23:46
racarrI have some time23:46
racarrwell not tonight probably anyway23:47
racarrbut if you cant finish it tomorrow, maybe send me a brain dump if it makes sense andI can try and run with it23:47
greybackracarr: yep, got it. I didn't read it as rough-edged23:47
greybacksure23:47
greybackit's what I'll attempt first thing. As right now, cannot actually interact with apps!23:48
racarrpft23:48
racarrgreyback: So we just got it working and now already first thing feature bloat23:48
greybackracarr: XD23:48
kdubooo android 4.323:51
racarrkdub: Oooo23:53
kdubyay, looks like the HAL layer is the same23:53
kdubfor graphics23:53
kdubracarr, time to start merging their new input stack!23:53
kdub:P23:53
racarrkdub: I was about 3 second ahead of you there23:53
racarrkdub: "Improved window buffer allocation results in a faster image buffer allocation for your apps, reducing the time taken to start rendering when you create a window."23:53
racarrthat ones for you ;)23:53
kdubwhere did you see that?23:54
racarrkdub: http://developer.android.com/about/versions/jelly-bean.html?4323:54
racarrVideo encoding from a surface23:55
racarrStarting in Android 4.3 you can use a surface as the input to a video encoder. For example, you can now direct a stream from an OpenGL ES surface to the encoder, rather than having to copy between buffers.23:55
racarr"On-screen GPU profiling23:55
racarr"23:55
racarronly new SF stuff23:55
kdubyeah, seems like all new SF stuff23:55
racarrwell23:55
racarrand drivers are23:55
racarropen gles 3 now?!23:55
kdubprobably just an option though?23:56
kdublots of new bluetooth hals though23:56
RAOFWhy does xmir suddenly infinite loop in xf86DeleteDriver now that I'm trying to debug?!23:57
kdubthank goodness we're not a bluetooth server23:57
racarrkdub: yeah they just say the nexus devices have23:58
racarropengles323:58
racarrI cant even think of one clever thing you could do with mir and opengles3 XD23:58
racarrIm sure its good for clients23:59

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!