/srv/irclogs.ubuntu.com/2013/08/20/#ubuntu-mir.txt

racarrkgunn: image fails00:24
racarrare there older images00:25
kgunnracarr: thanks... rsalveti how far back can we go ? ^00:25
racarrI have a copy of00:25
racarrjuly 26th00:25
racarron my desktop it seems00:25
racarrthats kind of far though XD00:26
kgunnso 12th would be 1 week...and the bug describes it popping up ~2 weeks ago00:26
kgunnracarr: question...is the hwcomposer.h etc matching from the 12th to current  ?00:27
racarrno hwcomposer.h on the image00:27
racarrlets check hardare.h though00:28
kgunnracarr: yeah...that's what i meant00:28
kgunngralloc and fb as well00:28
racarryep00:29
racarrthe same00:29
racarrok im going to september 26th because00:29
racarrerr00:29
racarruly 26th00:29
racarrbecause I dont have anything else to do :p00:29
racarrkgunn: maybe check your ~/Downloads/phablet-flash00:31
racarrfor images?00:31
racarrthough00:31
racarri e dont have the images elsewhere00:31
kgunnracarr: will do00:31
racarrthen bisecting probably ont help much00:31
kgunnalso...as you're building, check the ubuntu/hybris/hybris/include/android/hardware/ and see how those headers compare00:32
kgunn26th vs current00:32
racarrubuntu/hybrys/hybris/hat00:34
racarr /what?00:34
kgunnracarr: i have a jul 25th :)00:34
racarrI have been looking in /usr/include/android00:34
kgunnoops...nvmd...i was looking in source i downloaded00:35
kgunnthat's where they are before image creation00:35
racarrXD00:35
kgunn./ubuntu/hybris/hybris/include/android/hardware/fb.h00:35
kgunnetc00:35
racarrthats a hell of a path00:35
racarrI realized00:35
racarre can rule in our out the headers00:35
racarrin an easy ay00:35
racarrwell that whole theory00:36
racarrdid surface flinger change?00:36
racarrI guess that doesnt necessarily rule it out00:36
kgunnwhathcya mean?00:36
racarrI mean we have this theory that libhardware changed00:37
racarrand it broke mir because mir has the static in tree headers00:37
racarrbut if surface flinger didn't change00:37
racarrthat seems unlikely00:37
kgunnracarr: right...but isn't it truly in tree for surface flinger (e.g. he's not gonna get headers outta sync)00:42
kgunnsince its all just part of android00:42
racarrkgunn: Right I am just saying if e literally haven't updated00:46
racarrthe surfaceflinger binary00:46
racarrhavent tested this image yet but want to go get pizza before the bakery closes00:52
racarrback soon00:52
kgunnracarr: actually....i think i just found a jul 29 & aug 9 images00:53
racarrI am trying to build mir on this image...but00:54
racarrbuild deps are pulling in a hybris upgrade00:54
racarrso it would be not so surprising if it didn't work?00:54
racarrghsdflKSDfljLKsdgjldkfjglkdjglkejrg;lakerjgl;kejrgt00:55
racarrjust a little keyboard mashing :p00:55
racarrbrb pizza00:55
RAOFAh, yes.01:09
RAOFIt's important not to leak fds.01:09
RAOFHuh. Something's broken in our resource cache it seems.01:14
racarrback01:32
kgunnracarr: dont we  rebuild surfaceflinger from source everytime ? (i thot we built the whole android file system...for what we have source for)01:35
kgunnstepping out...biab01:35
racarrkgunn: yeah I guess its probably never the same01:37
kgunnracarr: just comparing bins of libsurfflinger they are diff between jul 25 & latest01:37
kgunnbut not surprising01:37
kgunnok...really stepping out01:37
dufluracarr: is surface flinger relevant to the Nexus4 bug? We intentionally disable it to run Mir (usually)01:41
racarrkgunn: july 26th works!01:45
racarrduflu: No but one of the theories is that there is a libhardware mismatch01:45
racarrbut I didn't relize we rebuilt surface flinger everytime01:45
dufluI see01:45
racarrduflu: Want to check your ~/Downloads/phablet-flash and see if you have any images beteen01:46
racarrjuly 26th and august 18th01:46
racarrand can verify mir orks/doesn't work on them01:46
racarrbisecting ;)01:46
racarrbut I am out of images01:46
racarrerr august 12th sorry01:46
racarrricmm: rsalveti: Any way we can get images from before august 12th01:47
dufluracarr: Only July 26th (works for me I think) and August 8th (broken)01:48
racarrAugust 8th broken. Awesome01:49
racarrkgunn: ^01:49
dufluracarr: At least that appears to be the latest one I have. And the latest broke my N401:50
dufluThough surely we can get image dates/versions from the devices?01:50
racarrduflu: The problem right now is cdimages.ubuntu.com01:51
racarronly has back to august 12th :p01:51
racarrso I am trying to bisect01:51
dufluracarr: Yeah Albert logged the bug on 13th a few days after he first mentioned it. Because it took me a few days to upgrade and confirm01:51
racarrbut the range is uly 26th to august 8th now01:51
racarrbut there are no images01:52
racarrinbeteen01:52
racarrween01:52
racarrSpill on keyboard:(01:52
dufluPork and beans?01:52
dufluArgh01:52
dufluThat would be weezer01:52
dufluI suck01:52
racarrI guess I will start01:58
racarrreading changeloads01:58
dufluracarr: Actually 20130808 may not be accurate. The download date for that was 20130813 and I remember I did an update immediately after01:58
dufluWill have to reflash it without updates to confirm01:59
racarrAh! thanks.01:59
racarrlxc-android-config (0.67) saucy; urgency=low01:59
racarr  * allow devices that use EFS as label for the factory partition to mount it01:59
racarr    under /efs instead of /factory01:59
racarrthat means anything01:59
racarr -- Oliver Grawert <ogra@ubuntu.com>  Sun, 04 Aug 2013 12:24:23 +020002:00
racarrI wonder if02:00
racarrI have nothing in /factory on the working image so probably not02:00
racarrI dont know what a /factory is though02:00
racarrim going to bisect02:00
racarrlxc-android-config02:00
racarrbecause permissions are suspect02:01
racarroh I can't really do that as easily as I want to02:01
racarrok I upgraded lxc-android-config02:03
racarrand it still works02:03
racarrim going to upgrade packages one at a time lol02:03
racarrthat was kind of silly :p not even sure what to try next02:05
racarrno hybris updates02:06
dufluN4 sez: Not enough charge to turn on yet. This seems to happen a lot02:06
racarrlets upgrade lxc02:07
racarrduflu: Happens to me a lot too :( and you can02:07
racarruse more poer than you get off low power USB compiling02:08
racarrmir02:08
dufluracarr: Yeah I put it on mains for a while02:08
* duflu wonders if the USB 3 port would have more power than USB 202:09
racarrMaybe02:09
racarrnot lxc! let's upgrade apparmor02:11
racarrim staring at a list of like 200 packages trying to pick ones that could be relevant lol02:11
dufluracarr: Maybe consider those that Mir depends on02:13
racarrduflu: Mm. should just start with that02:16
racarrall the build deps02:21
racarrare newest version02:21
racarras are the dependencies02:22
racarrexcept or02:22
racarrmultiarch support02:22
racarr*tests*02:22
dufluracarr: phablet-flash has no options to specify an image file(s) any more. Any clue?02:24
racarrduflu: You have to use clockwork mod02:24
racarrduflu: You flash the armel+dev first then the armhf02:24
dufluGreat. Later then02:24
racarryou have to use adb push file /sdcard/02:24
racarrinstead of02:24
racarr /sdcard02:24
racarror it ust silently fails ater taking02:24
racarrforever02:24
racarrlol02:24
racarraw not multiarch support either02:25
RAOFGrr overlapping outputs.02:28
racarrok I think I narrowed it don to a set of 20 packages02:34
racarris there an easy way to reverse my last02:35
racarrapt transaction02:35
racarrhttp://pastebin.ubuntu.com/6004993/02:36
racarrsomething in here killed it'02:36
racarrhttp://pastebin.ubuntu.com/6004994/02:37
racarrthis is suspiscious02:37
racarrreverting back to working image to narrow the list02:44
racarrSetting up initramfs-tools-ubuntu-touch (0.34) ...02:45
racarrseems sort of sketchy02:45
racarrcompared to everything else at least02:45
smspillazracarr: arrrround?03:21
racarrsmspillaz: Enthusiastically!03:27
=== chihchun_afk is now known as chihchun
kgunnracarr: you're awsome...so down to 20 packages, somewhere between jul 26 & aug 803:33
racarrkgunn: Actually just got a working system again and clicking, so more like 19 packages lol03:35
racarrreally only like 4 or 5 that seem reasonable we should know super soon03:35
kgunnracarr: cool03:36
smspillazduflu: so I don't know if something I'm seeing is a bug in the mir buffer swapper or a bug in my code03:37
kgunnduflu: ping03:37
smspillazduflu: basically if I do draw1 -> swap1 -> draw2 -> swap2 -> draw3 -> swap303:38
smspillazI seem to see something like03:38
smspillazafter swap1 : nothing after swap2 : draw1 after swap3: draw203:38
smspillaz(using mir_surface_swap_buffers directly)03:38
duflusmspillaz: Do you have a very recent server? That was probably fixed in lp:mir r95503:39
smspillazduflu: bzr pulled as of yesterday03:39
duflukgunn: Hi03:39
duflusmspillaz: Yeah that's new enough03:40
smspillazhmm ok I'll try again03:40
smspillazmaybe I was just an idiot and forgot to set my path right or something last night03:40
duflusmspillaz: For manual testing that kind of thing I use fingerpaint (in which single clicks generate single swaps and should generate immediate output)03:41
dufluThough we have automated tests for it of course03:41
smspillazah you know what03:41
smspillazit could be that stupid thing with sudo not working with LD_LIBRARY_PATH03:41
duflukgunn: pong?03:41
smspillazwell not stupid, but a gotcha nonetheless03:41
smspillazso I was probably still running the one in /usr/bin03:42
duflusmspillaz: Yeah you have to set it after sudo03:42
smspillazyou guys really need something like weston-launch03:42
duflusmspillaz: You mean a dumb and simple shell?03:42
smspillazduflu: not really. Its a setuid thing that talks to pam to get the invoking env, handles all of the vt switching and drmSetMaster/drmDropMaster stuff and then runs weston as user03:44
duflusmspillaz: Oh right. I have thought about similar03:44
dufluSomeone just has to do it03:44
smspillazlol03:44
smspillazit makes life a lot easier :)03:45
racarrsmspillaz: Ill take care of it on Yotday, the imaginary day I just invented inbetween03:45
racarrnow and featufre  freeze03:45
racarr:p03:45
smspillazthough admittedly not as useful anymore because logind no longer allows arbitrary processes other than init to spawn new sessions03:45
smspillazso it has to start on the same vt you launched it on as opposed to starting on a new one03:46
RAOFAs long as you don't want VT switching you should be able to run mir_demo_server_shell as your user directly.03:46
smspillazRAOF: don't you need root for input?03:47
smspillazI seem to recall that's what half of what weston-launch was about03:47
RAOFsudo chown smspillaz /dev/input/**/*03:47
smspillazor maybe that was to do with vt-switching and input03:47
smspillazRAOF: :/03:48
RAOFsmspillaz: Hey, it's harmless¹03:48
RAOF¹: As long as you trust everything on your box03:48
smspillazI guess my alternative solution of doing source /home/smspillaz/.bashrc was probably not better03:49
smspillaz(so that I didn't have to re-type every bloody path again)03:49
racarrNooooooooooooooooooooooooooooooooo03:50
racarrbisection failed03:50
racarrlife03:50
racarrno03:50
racarrjust bisection03:50
smspillazduflu: :/ I'm still getting the one-frame lag03:57
smspillaz(correct paths and all)03:57
racarrerr03:59
racarr...I retract my err04:00
smspillazthough oddly its not happening on the demo clients which use mir_surface_swap_buffers_sync!?!04:01
RAOFHrm04:02
smspillazattack of the threads!04:03
duflusmspillaz: Not sure then. We have automated and easily manual testing of this issue. If you can figure out the scenario exactly that would be helpful04:06
smspillazduflu: I'm just doing a dump of the image surface to a png to figure out what's going on for now04:07
smspillazreeeeecompiling04:07
kgunnracarr: bisection failed as in ...you added everything back one by one to a failing image that no longer fails but actually boots with unity-mir ?04:12
racarrkgunn: I think I made a istake in thinking I hd found a broken set of packges04:12
racarrin that it just didnt ork once04:12
racarrmaybe I didnt use sudo and forgot because adb history was fucked up or something04:12
racarrI dunno04:12
racarrno I am still on a orking images04:12
racarrand adding more and more packages04:12
racarrabout 20 at a time then rebooting and testing04:12
racarrlol04:13
racarrthis last one pulls in a bunch of stuff so I have a feeling this will break it but then to bisect this smaller list I need to go all the way back04:13
racarrand reflash04:13
racarr  ubuntu-touch-generic-initrd ubuntu-touch-session04:14
racarrthese are the most suspsicious packages that I havent ruled out yet04:14
racarrall the mir dependencies and build dependencies04:14
racarrare ruled out04:14
kgunnracarr: can you pastebin the remaining packages ?04:23
kgunnjust updating the bug04:24
robert_ancellthomi, is jenkins screwed up again? (https://jenkins.qa.ubuntu.com/job/lightdm-saucy-amd64-ci/85/console)04:24
robert_ancelland http://s-jenkins:8080/job/lightdm-ci/168/04:24
kgunnjust in case you don't find the offender04:24
thomiit wouldn't surprise me, let me see04:24
racarrkgunn: http://pastebin.ubuntu.com/6005189/04:24
racarralmost through a bunch though04:24
thomirobert_ancell: the public jenkins has been up and down all day today04:25
thomirobert_ancell: private jenkins should be working though04:25
robert_ancellthomi, also, do you happen to know who I can poke to make a PPA have a high priority?04:25
thomirobert_ancell: #launchpad-ops04:26
olli_robert_ancell, rcimm & rsalveti04:26
racarrkgunn: http://pastebin.ubuntu.com/6005191/04:26
racarrI ust ran mir succesfully04:26
racarrafter a reboot04:26
racarrand those are the only remaining packages04:26
racarrin upgrade04:26
racarrgoing to eat some ice cream then come back to it04:26
robert_ancellricmm, rsalveti, you can bump PPA priorities? Can you set ppa:mir-team/qa-testing to the same priority as ppa:mir-team/staging?04:27
thomirobert_ancell: you mean build priority in the lp build queue, right?04:27
racarrgoing to eat icecream and revisit04:27
racarroh04:28
racarrwow04:28
robert_ancellthomi, yeah04:28
racarrim going around in workspace switching loops04:28
thomirobert_ancell: I thought so, I've always had to ask launchpad people to tweak that for me, but maybe others have more permissons than I do04:28
kgunnracarr: icecream can only help04:29
smspillazkgunn: so I'm working at the australian election and our counting how-to guide give the example of replacing the candidates name with "ice cream"04:40
smspillazas an example of an informal vote04:40
smspillazI really don't know who or why somebody would want to vote for ice cream04:41
racarrso close04:42
racarrps daily show had coverage of australlian elections it as hilarious04:42
smspillazracarr: I saw that!04:42
smspillazahh good ol' cameron diaz04:42
racarrdoes anyone have time to revie client-focus-notifications while I am still awake?04:44
racarrlibopenobex1 libpam-systemd libpixman-1-0 libplymouth2 libpolkit-agent-1-0 libpolkit-backend-1-0 libpolkit-gobject-1-0 libsystemd-daemon0 libsystemd-login0 libupower-glib1 libusb-1.0-0 libwaudio1 login multiarch-support packagekit-plugin-click plymouth policykit-1 powerd systemd-services telepathy-gabble telepathy-ofono tzdata04:47
racarrone is evil04:48
RAOFracarr: Sure. I want that branch landed :)04:49
racarrRAOF: It feels like today is the day04:49
RAOFhttps://code.launchpad.net/~robertcarr/mir/client-focus-notifications/+merge/179801 is the current one, right?04:50
racarrI'm looking for the words to say...I don't think I can hide...what I'm feeling inside...04:50
racarrRAOF: https://code.launchpad.net/~robertcarr/mir/client-focus-notifications/+merge/18090304:50
racarrdidn't you check the moon charts?04:50
racarr  libpam-systemd libplymouth2 libpolkit-agent-1-0 libpolkit-backend-1-004:51
racarr  libpolkit-gobject-1-0 libsystemd-daemon0 libusb-1.0-0 libwaudio104:51
racarr  multiarch-support packagekit-plugin-click plymouth policykit-1 powerd04:51
racarr  systemd-services04:51
racarrhmmm04:51
racarrim trying to eliminate ones which seem really unlikely to break mir04:52
racarrfirst so when I finally break it (and have to start the whole thing over again)04:52
racarrthe pool is super small and I can just guess04:52
racarrlibpam-systemd libplymouth2 libsystemd-daemon0 plymouth policykit-1 powerd systemd-services04:54
racarrany bets? :p04:55
racarrsystemd plymouth policykit or powerd one of them broke mir on the nexus 404:55
* RAOF lays a bet on "systemd"04:58
racarrseems likely04:58
racarrit wasn't powerd04:58
racarrseeing if plymouth is the surprise offender now04:58
racarrdamnit, ran demo ithout root, 30 second penalty as phone hangs.04:59
racarr:p04:59
* RAOF should really get around to submitting a branch that fixes the build with clang 3.405:00
racarrplymouth, hitelisted05:01
racarrwhat05:04
racarrI am fully upgraded05:04
racarrand just ran mir05:04
racarrufahsufashfasha05:04
RAOFHuzzah!05:04
RAOFYou've fixed it!05:04
racarrwhat if it fixed itself in the archive05:05
racarrwhile ive been doing this all day05:05
racarrive been using05:08
racarrthe same mir05:08
racarrbinary05:08
racarr...maybe a header changed05:08
racarrugh05:08
racarrok literally no headers changed05:08
racarrim flashing a latest image and upgrading to see if it really did just05:10
racarr...fix itself05:10
tvossgood morning05:15
tvossRAOF, we have exclusive access to the ati machine :)05:15
RAOFExcellent.05:20
* RAOF is making coffee for a moment.05:20
tvossRAOF, I'm back in ~1 hour05:23
racarrtvoss: So I've been bisecting images, until I got to august 12th broken july 26th orks then I ran out of images so I started05:26
racarrinstalling packages to upgrade, about 10 at a time05:26
racarrand rebooting and testing05:26
racarrat one point it seemed to break, so I bisected those packages from the working image05:26
racarrbut then it always orked so I started over05:27
racarrover about 30 reboots I saw maybe 2 failures here I had to reboot, but then it alays worked until I was05:27
racarrfully ugpraded, and it still worked!05:27
tvossracarr, wtf05:27
racarrand then I verified I asn't crazy, and the fully upgraded binary was working05:27
racarrand cmake, didn't detect any thing as touched, i.e. mir needing rebuild, in fact none of the mir05:27
tvossracarr, could you reboot like 20 times to rule out a race?05:27
racarrbuild depdencies were ever upgraded05:28
racarrtvoss: I have, it seems either it always fails, or05:28
racarrjust occasionally it fails in this one way that a reboot fix05:28
racarrbut always works otherwise05:28
racarrwhat I am doing now is a clean image to see i it was05:28
racarrmagically fixed in the archive05:28
tvossokay05:28
tvossracarr, can you compile that into an email?05:29
racarrtvoss: Yeah I am about 5 minutes away from the result on the05:30
racarrclean image then I will write things up05:30
racarrit doesn't really make any sense as far as I can see, but I was pretty careful05:31
racarrbut I dunno, I haven't really zoomed back out yet05:31
racarrive just been...in robot mode05:31
racarrugh05:34
racarrit didn't work05:34
didrockshey robert_ancell! Just a quick note on lightdm packaging, you took my cleanup branch, right? (https://code.launchpad.net/~didrocks/lightdm/packaging-cleanup)05:35
didrocksit was merged in the /unity one, not sure you took it for trunk05:35
robert_ancelldidrocks, yeah, it's a mix of a few different packaging branches :)05:36
robert_ancelli.e. what was in the archive + the changes you made + the test changes made for CI05:36
robert_ancellwill be nice to be down to one branch :)05:36
didrocksrobert_ancell: indeed ;) thanks!05:37
* didrocks can close a tab then \o/05:37
racarrtvoss: OK I commented on the bug05:39
racarrbrain basically off now05:44
RAOFdidrocks: Yo!05:45
didrocks RAOF hey!05:45
RAOFBah. Why aren't all the rdeps of xmir magically rebuilt when I break the ABI? :)05:46
RAOFdidrocks: By the way, this patch http://paste.ubuntu.com/5983202/ worked fine; it was when we turned on logging for the msg-processor and frontend that the race disappeared.05:48
RAOFAnd by ‘worked fine’ I obviously mean “hung like a bastard”05:48
didrocksRAOF: was the rdepends a question for me?05:49
didrocksRAOF: ah, so you got all the info you needed?05:50
RAOFNo :)05:50
RAOFdidrocks: Well, we got all the information that patch provided. :)05:50
didrocksheh05:50
RAOFWhich was "yes, XMir really is writing the request to the socket and is not receiving a reply"05:50
didrocksRAOF: the ATI machine is completely deprovisionned btw05:52
RAOFSo I hear.05:52
didrocks(just done)05:52
didrocksif you need help to setup the environment, do not hesitate05:53
RAOFI think others are taking the lead on this bug for a while, while I get multihead for XMir done.05:53
didrocksok05:53
alf__racarr: dpms ping06:14
RAOFI seem to be leaking ALL THE FDs06:22
RAOFBut I don't think that's my fault.06:23
smspillazthe awkward moment when you have been writing C++ for so long that you forgot how to write GObjects06:24
RAOFI hear the correct way to do that is to write a DSL.06:24
smspillazRAOF: DSL?06:25
RAOFDomain Specific Language; ie: Vala06:26
smspillazlol06:26
smspillazactually06:26
smspillazI think this marks the first time this year06:26
smspillazI will have written a GObject06:26
duflu\o/06:26
smspillaznot a cause for celebration06:26
dufluor /o\ depending on how you look at it06:27
smspillaz/o\ most certainly06:27
duflugcc. How I hate you and your insane C++ error messages06:43
RAOFYes06:46
dholbachgood morning06:53
RAOFHrm.07:41
tvossRAOF, ?07:42
RAOFI think we might leak fds for clients with more than one window07:42
RAOFOr, at least, we leak ALL THE FDs when XMir has two surfaces, one for each of my displays, and we don't leak fds when XMir is only driving one display.07:43
RAOFThe problem could be in the XMir code, but I thought that mirclient should handle the fds sent over the wire.07:49
dufluRAOF: Is this related?... 119453407:51
duflubug 119453407:51
ubot5bug 1194534 in Mir "mir_connection_release does not free associated surfaces" [Medium,Triaged] https://launchpad.net/bugs/119453407:51
RAOFduflu: I don't think so; I don't call mir_connection_release07:51
dufluEver?07:52
RAOFEver07:52
RAOFAt the moment. I guess I should at some point, but it's not hugely important.07:53
dufluRAOF: So you *should* get leaks of some sort anyway :)07:53
RAOFWhat *is* important is that Mir doesn't leave in excess of 1024 file descriptors open, causing X to fail to accept any new clients.07:53
RAOFduflu: Oh, sure. It just shouldn't be “there are more than 1024 file descriptors leaked by the RPC code”07:54
dufluThat is indeed a lot07:54
tvossalf__, ping08:25
alf__tvoss: pong08:26
alan_ghikiko: how is it going?08:38
hikikohi alan_g08:39
hikikoI started a new branch here: https://code.launchpad.net/~hikiko/mir/mir.native-platform-functions and I am now replacing the functions in native platform08:40
alan_ghikiko: by "native platform" you mean *nested* platform?08:45
hikikono08:45
hikikoI started filling the:08:45
hikikoget_ipc_package, create_internal_client, create_buffer_allocator08:46
alan_gfor android and gbm?08:48
alan_gLet's first rewrite the nested functions that can be written "return native_platform->xxxxxxx();" and get that onto trunk.08:48
hikikowithout implementing the xxxx() you mean?08:51
alan_ghttps://code.launchpad.net/~hikiko/mir/mir.native-platform-functions already has implementations (stubs)08:52
hikikoyes08:52
hikikoso I ll just call the stub functions08:52
hikikoyou don't want me to implement the stubs as well08:52
alan_gIf we get the nested code landed (with stubs) then we can separate work on the GBM and Android support08:53
hikikocool08:54
* alan_g likes to keep as much code on trunk as possible. (I.e. small incremental merges)08:54
alan_ghikiko: like: https://code.launchpad.net/~alan-griffiths/mir/display-is-not-output/+merge/180914 (which you could review once you've MP'd the above)08:57
RAOFOh, that's right. Callbacks are non-optional.09:25
dufluThat sucks. My thinkpad has pinned itself to 800MHz even though I'm building on both cores09:29
dufluUsually that only happens on low battery or high temperature...09:29
RAOFIs there some philosophical reason why all our async functions *require* valid callbacks to be passed in?09:40
dufluRAOF: I was wondering that at the start of the year. I think I proposed making them optional and it was rejected. Can't remember09:45
hikikoalan_g, https://code.launchpad.net/~hikiko/mir/mir.nested-platform-functions/+merge/18100109:45
hikikoI'll review the display-is-not-output now09:46
RAOFduflu: I guess we could have a no-op callback in mirclient?09:46
alan_ghikiko: OK (I'll review yours after I finish on session_lifecycle)09:46
RAOFBecause I really *don't care* when this surface gets released09:47
dufluRAOF: It's a bit ugly and definitely suboptimal09:47
dufluRAOF: I'd prefer support for NULL09:47
dufluNULL is definitely a valid callback in some cases09:47
RAOFAn explicit no-op callback is probably more self-documenting, but NULL is more idiomatic.09:48
dufluRAOF: Then to solve both: const CallbackType *noop = NULL;09:48
hikikosure alan_g take your time09:48
dufluRAOF: Or just #define NO_CALLBACK NULL09:49
RAOFBut now you need to explicitly handle NULL in mirclient!09:49
dufluRAOF: So...?09:49
* RAOF is not really arguing against NULL09:50
mpttvoss, katie: When we say surfaces aren't aware of their absolute position, what's the one-sentence reason why?10:04
hikikoalan_g, what does the POC mean here? // TODO as a POC just use the first active display10:08
alan_g"Proof Of Concept" (note to self: don't abbreviate)10:09
tvossmpt, as seen in the past, it leads to assumptions regarding the phyiscal layout of screens for example.10:09
hikikoso, alan_g this branch just modifies the display to match the new Display interface with the multimonitor support or I am missing something?10:15
hikikono, there's the displayoutput as well10:16
alan_ghikiko: yep10:16
mpttvoss, when there are multiple displays, do apps know even what display each surface is on?10:19
tvossmpt, the plan is to only communicate dpi changes10:20
=== chihchun is now known as chihchun_afk
mpttvoss, thanks. Next question: If an app has three windows open, A, B, and C, and A is focused, does the app have knowledge that B is above C or vice versa?10:21
tvossmpt, the app knows which surface is focused10:22
mptYes, but that's A. :-)10:22
tvossmpt, sure :) but under the constraint that stacking only changes when focus changes, an app knows if B is over C or vice versa10:23
mptah, right10:23
mptSo it couldn't interrogate, but it could remember10:23
hikikoalan_g, output == context?10:23
alan_ghikiko: ?10:23
tvossmpt, exactly. our focus policy makes sure that stacking per app only ever changes if focus changes (at least for regular surfaces)10:24
mpttvoss, unless a window manager decides to go Amiga Intuition 1.x style and add a "Send to Back" button to every title bar. :-)10:25
hikikoI mean, a display's output (eg the nested display's output) is the sum of variables that describe the context we render in the display (the state)?10:25
tvossmpt, hmmm, good point ;)10:25
hikikoand we ll have a different context for every display and a different display for every monitor?10:26
alan_ghikiko: there's one Display and an output for each active monitor. (At least that's what's in the native GBM code)10:27
alan_gBut that's not what I've implemented for nested (yet)10:28
hikikoso each monitor is a different "buffer"/window?10:28
alan_gack10:28
hikikoyes, I understand what you did and it looks good to me, I am just trying to get sure I understand the whole concept10:28
hikiko+can I top approve? it has 4 positive reviews10:30
alan_ghikiko: I think it should be a customization point - and am still thinking. (E.g. I may want a nested mir with a single window that doesn't match a monitor)10:30
alan_ghikiko: you can top approve10:31
alan_g(And you don't need to ask permission)10:31
katietvoss, another question for you.. is it Mir or the application that should remember whether a surface is fullscreen/maximised etc?10:31
tvosskatie, for the surface state, it should be the application10:32
hikikoalan_g, if this 1 window can be splitted to as many buffers as the monitors and can have the size of the virt. screen size there shouldn't be a problem (I am not 100% sure though)10:34
mpttvoss, katie: If Mir can remember its absolute position, why can't Mir remember its state too?10:34
tvossmpt, fair question: we could add a function: restore previous state10:35
mpttvoss, katie: I think we're missing a description of how an app tells Mir the identity of a surface it's reopening. That would be the place to describe that Mir restores not just the previous position and size but the previous state too.10:36
alan_ghikiko: it isn't a problem for now - but I was thinking more of a window smaller than a monitor. (So I could run up a server as a "normal" application)10:36
tvossmpt, ah, that's a good point. so the app is responsible for "tagging" a surface10:37
mpttvoss, when an app opens a surface, does it have to provide a preferred size?10:53
tvossmpt, yup, an initial one at least10:53
mpttvoss, but what if I develop a word processor, and the preferred size for new document windows depends on the physical size of the current display, up to a reasonable limit? Should all apps contain that kind of logic?10:55
tvossmpt, why would it depend on the physical size? you mean in terms of percentage?10:56
mpttvoss, katie suggests that in that case, an app could just ask for a preferred size of, say, 8" by 12". On any screen smaller than that (e.g. phone, tablet), it would take up full screen. On any screen larger (e.g. Cinema Display), I'd get that actual size.10:57
tvossmpt, yup, that sounds like a good idea. How do you want to handle aspect ratios?10:58
mpttvoss, if you reopened a surface on a display that was still wide enough but not tall enough, katie and I think it would only get shorter, not narrower.11:01
tvossmpt, so we do not preserve aspect ratio?11:03
mpttvoss, right11:03
tvossmpt, okay. What if an app needs that functionality?11:03
mpttvoss, unless you want to back that into window management (it seems a bit niche), the app would need to notice its new height and narrow itself proportionately.11:05
mpt*back -> bake11:05
tvossmpt, the one thing that concerns me is the feedback cycle required ...11:06
tvossmpt, user resizes window, app receives resize notification, resizes its frame11:06
mpttvoss, well there are other cases where a window wants to set strict limits on how it can be resized ... The most complicated example I've seen is floating toolbars, where the exact combination of width and height depends on what buttons and other controls are present.11:08
mptPalettes is an even better example.11:08
tvossmpt, ack, so why don't we say an app can request the following flow: about-to-be-resized(old_size, newly_requested_size) -> new_size_as_calculated_by_app11:10
tvossand only then we resize11:10
mptHmm, I thought the Gimp toolbox would be an example, but it isn't, because of that dire "You can drop dockable dialogues [sic] here"11:12
mpttvoss, what use is old_size there?11:15
tvossmpt, hmmm, to express the user requested change ... although the application should know that anyways11:18
mpttvoss, ideally the app would animate internal reshuffling of a window's contents, but that isn't helped by knowing the old size of the entire window.11:20
mptFor that it needs to know the old position of individual elements inside.11:21
tvossmpt, which it does anyway11:23
=== tvoss is now known as tvoss|lunch
Saviqgreyback, any idea how do we stand on screen blanking in unity8@Mir?11:42
=== hikiko is now known as hikiko|lunch
greybackSaviq: I've not attempted it. Right now it doesn't blank.11:44
Saviqgreyback, right, another one to the list of regressions11:45
Saviqgreyback, did you write it in the end or shall I?11:47
greybackSaviq: http://sketchpad.cc/zYRXs1hnWu11:49
Saviqgreyback, added some more and their blueprint-like status, how's it look?11:52
greybackSaviq: looks ok11:53
=== tvoss|lunch is now known as tvoss
tvossalan_g, got a nexus 4 handy?12:08
alan_gyes (but lunch is calling)12:08
tvossalan_g, can you ping me after lunch then?12:09
alan_gtvoss: Sure. Should I leave it installing anything?12:10
tvossalan_g, would be cool if you could phablet flash the latest image12:10
alan_gtvoss: ack12:11
alan_gRats!! (It is in "I don't have enough charge to charge mode" again.)12:14
=== alan_g is now known as alan_g|lunch
tvossolli_, ping12:15
olli_tvoss, pong12:16
=== hikiko|lunch is now known as hikiko
=== alan_g|lunch is now known as alan_g
alan_gtvoss: back (but flashing failed) "ERROR:phablet-flash:Command 'adb shell mount /sdcard/' returned non-zero exit status 255"13:38
tvossalan_g, please ask in #ubuntu-touch for help13:41
alan_gtvoss: I'll ping you when I've got it working again13:42
tvossalan_g, ack13:42
alan_ghikiko: how are you doing?13:54
tvosssil2100, didrocks lxc-start on the ati machine fails, I can briefly see the login prompt, then the container shuts down13:55
tvosssil2100, didrocks lxc-attach fails with lxc-attach: failed to get the init pid13:56
didrockstvoss: do you restart the Mir environment?13:56
didrocksthe right run?13:56
alan_gtvoss: OK I've got a newly flashed N413:57
ricmmthomi: did you guys get the prio sorted?13:57
ricmmI obv cant push it, I just ask whoever is on vanguard in webops13:57
tvossricmm, thomi is unlikely to be online13:57
ricmmright13:59
tvossdidrocks, how do I do that?13:59
didrockstvoss: that's what I told, we need to reset the Mir environment14:00
tvossdidrocks, mind telling me how? :)14:00
didrocksbut I don't have time for that (taking 15 minutes and hoping on meetings)14:00
didrockstvoss: I can recreate quickly a blank one rather14:00
tvossdidrocks, ack14:00
didrockswhere you have a persistency daily image14:00
didrocksthen you add the ppa14:00
didrocksand the packages14:00
didrockssounds ok?14:00
hikikoalan_g, in this case the window could be resized to be fullscreen when necessary14:01
hikikoso, why not14:01
didrockslet's take that as a yes :)14:01
alan_ghikiko: Don't worry about that use case yet14:02
=== chihchun_afk is now known as chihchun
alan_ghikiko: what are you doing currently?14:04
hikikoI am looking at the display14:05
didrockstvoss: ok done14:05
didrockslet me pm the instructions14:05
hikikothere are at about 8 functions with todo14:05
hikikoI think that maybe I should start from there14:06
=== jono is now known as Guest90491
hikikoif you agree14:06
hikiko(alan_g)14:06
=== greyback is now known as greyback|needsfo
=== greyback|needsfo is now known as greyback|food
alan_ghikiko: Don't try to do too much at once.  I'm looking at the mulit-monitor interactions for NestedDisplay (I think alf__  is busy writing the stuff I need for it)14:09
hikikoalan_g, what do you suggest I do as a next step then?14:09
alan_gso maybe the best place to for you to start is in the NativeGBMPlatform14:09
alan_gBTW Don't worry about duplicating code with GBMPlatform - we can refactor that after we have things working.14:10
alan_ghikiko: ^^14:10
hikikook then :) I ll start adding the functions there!14:11
alan_ghikiko: don't go too fast - just get the test to get through one more function call14:12
alan_g(Before an exception gets thrown)14:13
hikikoalan_g, what do you mean get the test?14:15
rsalvetitvoss: racarr: can start mir_demo_server successfully with aug 5th14:15
rsalvetitvoss: which other test can I run?14:15
rsalvetihm, seems it crashed now14:16
rsalvetiE/Adreno200-GSL( 1724): <ioctl_kgsl_driver_entry:269>: open(/dev/kgsl-3d0) failed: errno 2. No such file or directory14:16
alan_ghikiko: there's only one TestNestedMir test currently - I mean that one14:16
hikikoyou mean to run this test each time I add a new function?14:17
alan_ghikiko: I mean change the system so that the exception caught comes from the next not-implemented function that gets called, update the test for the new exception and MP those changes.14:18
hikikoalan_g, ok :) I might bother you when I have to modify the test because it will be my first one... but ok, I'll do the change first :)14:23
=== greyback|food is now known as greyback
alan_galf__: @place-surface-in-output - how does a client select an output? I don't see any client-side API for it - what have I missed.14:41
alf__alan_g: it's a new field in surface creation parameters .output_id14:43
alan_galf__: thanks14:44
=== alan_g is now known as alan_g|tea
=== alan_g|tea is now known as alan_g
tvossracarr, ping15:10
hikikobye!15:15
tsdgeosolli_: is it ok if instead of 19.2 I test image 20?15:17
tsdgeosphablet-flash cdimage-touch --pending is giving me 20 instead of 19.215:17
tvosstsdgeos, please do 19.215:17
tsdgeostvoss: then i'm going to need help on how to get it15:17
tvossogra_, how to get 19.2 with phablet-flash?15:18
ogra_tvoss, dunno, i never use phablet-flash ... i think you need to give it a --url parameter15:21
ogra_tvoss, reading teeh help i would guess: phablet-flash cdimage-touch -d mako --ubuntu-path=http://cdimage.ubuntu.com/ubuntu-touch/daily-preinstalled/20130819.2/15:23
tvosstsdgeos, ^15:23
tsdgeosogra_: the help confuses me15:24
ogra_ phablet-flash cdimage-touch -h15:24
tsdgeoswhat's the difference between --device-path and --ubuntu-path15:24
tsdgeosogra_: that is what is confusing me, yes i can get there ;-)15:24
ogra_dunno, ask sergiuiens :)15:24
ogra_he invents those options all the time :)15:25
rsalvetidevice-path is the android zip file15:32
rsalvetiubuntu-path is the ubuntu zip file15:32
rsalvetiyou need both15:32
ogra_oh ?15:32
tsdgeoscan't i just say "i want 20130819.2" easily?15:33
ogra_i thought there was a patch that you only need one with the latest phablet-flash15:33
ogra_tsdgeos, depends how well your voice input works15:33
ogra_:P15:33
bschaefertvoss, ping15:49
tvossbschaefer, otp15:49
bschaefertvoss, alright15:49
=== tvoss is now known as tvoss|afk
=== chihchun is now known as chihchun_afk
alan_gHave a good day everybody!16:58
=== alan_g is now known as alan_g|EOD
kgunnracarr: you on?17:16
racarrkgunn: Yeah just recently17:17
kgunnnp...feel free to coffee up17:17
kgunnbasically....the current request is to retest using 19.2 with -b (to make it completely fresh)17:18
kgunnrsalveti was able to get it working17:18
kgunnin case the image moved....use this command line17:18
rsalvetiI didn't get the crash, but just noticed I'm also getting a black screen17:18
rsalvetithe issue when allocating memory is due a race in udev/ueventd, which I'm working on atm17:18
kgunnphablet-flash cdimage-touch --ubuntu-path http://cdimages.ubuntu.com/ubuntu-touch/daily-preinstalled/20130819.2/saucy-preinstalled-touch-armhf.zip --device-path http://cdimages.ubuntu.com/ubuntu-touch/daily-preinstalled/20130819.2/saucy-preinstalled-touch-armel+[device].zip -b17:18
racarr19.2?17:18
rsalvetiracarr: which was the latest issue that you were having? a crash or just a black screen?17:19
racarrrsalveti: Ah yeah black screen is all I get17:19
racarrcrash is rare17:19
racarrany time I have ever seen the crash17:19
racarra reboot has made it go aay and it17:19
racarrreverted to black screen17:19
rsalvetiright, then let me flash the older image again, which was working without black screen17:19
racarrerr so wait should I try this17:20
racarrphblet-flash kgunn gave me?17:20
rsalvetiplease try17:20
rsalvetiseems it worked fine for ev17:20
kgunnrsalveti: "worked fine" means unity8 came up on mir ? & no black screen ?17:20
racarrErr17:21
rsalvetiunity coming afaik17:21
racarrflashing17:22
racarrok installing mir ppa on image17:39
bschaeferkgunn, hey, sooo Im suppose to help work on an ATI bug? Would you happen to know something about that?17:40
kgunnbschaefer: ok....so, it seems to be corrected by the latest kernel (updated to 3.11 y'day ~2pm my time)17:40
bschaeferthrough didrocks machine, and I missed tvoss a bit ago...soo I figured you might know something!17:40
bschaeferkgunn, o really?17:40
bschaeferkgunn, soo do we need to confirm this, or is it already?17:40
kgunntvoss|afk: & didrocks rebuilt and tried reboot ad-nauseum17:41
kgunnit never failed17:41
kgunnthere were some kernel changes (e.g. mutex related) that certainly could've effected17:41
kgunnso anyway...result was to place didrocks machine back "on the line"17:42
bschaefersweet, well lets hope it stays fixed :)17:42
kgunnhe will start pushing updates tomorrow morning his tmie17:42
kgunntime17:42
kgunnyep17:42
bschaefercool! Well that was fun to work on :)17:42
kgunnwe are going to pull his machine again in a couple of weeks during bug squash week to root cause17:42
kgunnseriously the pressure is off....but if you wanted to attempt some root cause effort that would be great17:43
bschaeferthat would be a good idea, don't want this coming up later on17:43
kgunnaltho you'd need to coordinate with him....as its back in service17:43
kgunnpainful part will be trying to stay on 3.10 kernel to test17:43
bschaeferkgunn, I can try, I think i remember trying to reproduce this last week on my ati machine17:43
kgunnbschaefer: were you ever successful ?17:44
bschaeferright, yeah if he comes back and wants some more testing i my time zone Ill go with it!17:44
bschaeferkgunn, nope :(17:44
bschaeferkgunn, it was only didrocks machine17:44
kgunnbschaefer: yeah...that's the thing....his machine was kinda golden17:44
kgunneven raof's debug add ons changed timing and bug disappeared....17:44
bschaeferyeeah, didrocks should get a new machine :)17:44
kgunnlike i say mutex changes coming in 3.11 kernel make me wonder17:44
bschaeferbut it is nice it did catch something like that, would have been hard to figure out later on...17:45
bschaeferkgunn, in a bad way?17:45
bschaeferor rather the mutex fixes are the fix?17:45
kgunnb/c xmir would always start ok...it only ever stalled when compiz was trying to init gl for unity17:45
kgunnin a good way17:45
kgunnyes...mutex fixes17:45
bschaeferright, I ran into that on my intel machine17:45
bschaeferbut then i purged all of my libegl* stuff and it never came back...17:46
kgunn<mlankhorst> mir fixes, ww_mutex changes, dpm support17:46
bschaeferkgunn, then my laptop died :(...17:46
kgunn<mlankhorst> maybe uvd fixes17:46
bschaeferawesome17:46
kgunnthat was the response i got for kernel changes that might've effected this17:47
bschaeferwell IIRC it was thought to be a deadlock right?17:47
bschaeferor a race for this ATI problem?17:47
racarrback on DPMS mode :)17:49
racarrphone almost upgraded17:49
kgunnracarr: so did 19.2 work for you ?17:49
kgunnbschaefer: well...deadlock only occured when they tried subsequent start of x after the compiz stall on gl load17:50
kgunnbschaefer: so the 1st symptom was always failure to load gl from compiz (even tho mir was fine)17:50
bschaeferkgunn, oo interesting, well when tvoss|afk gets back I can play around on the machine again to try and get the issue or dig some more into it17:51
racarrkgunn: It's installing mir ppa17:51
racarrit takes me 5-6 minutes to run apt-get update alone :(17:51
racarrphablet is so slo on my wireless17:51
rsalvetiracarr: 19.2 always gives black screen here, but aug 5th works fine, will test some other images17:51
rsalvetidoesn't crash, but gets a black screen17:51
kgunnracarr: rsalveti ....ok, i tried earlier, got black screen....just reflashed and _only_ installed the mir ppa (e.g. didn;t do a dist-upgrade rather install 1by1)17:51
kgunnand still got a black screen17:51
racarrWait then why am I trying17:52
racarrbecause isnt this hat I did17:52
racarryesterday17:52
rsalvetiright, I first thought the issue was related with the crash that happens sometimes17:52
racarrno17:52
racarrthe crash just happens XD17:52
rsalvetihahah, right17:52
racarrI think it's different17:52
racarrrsalveti: Ok so yesterday17:52
rsalvetilet me try to isolate the black screen then17:52
racarrI started from a working image17:52
racarrjuly 26th17:52
racarrand started upgrading packages in groups of 1017:52
kgunnrsalveti: also...for racarr's sanity....didn't you see unity boot (no black screen) when apparmor=0 ?17:52
racarra few times I saw the crash, but reboot always fixed it and it orks multiple times over17:52
racarreventually I ully ugpraded17:52
kgunnor was that just a red herring17:52
racarrand it worked too!17:53
rsalvetiapparmor=0 doesn't have any effect here17:53
racarrNever had any effect here either:(17:53
kgunnrsalveti: thanks...i'll stop saying that then :)17:53
racarrrsalveti: Anyay what this means (i.e. if someone can confirm that upgrading uly 26th works while installing the latest image fails)17:53
rsalvetiracarr: right, will try with some other images here, if it's android related I should be able to see what changed easily17:53
racarrthen we can narrow it down to things in the image process17:53
racarrgreat17:53
racarrI was wondering if it could have to do17:53
racarrdidn't the images change17:53
rsalvetiracarr: just tested from aug 5th, and worked fine17:53
racarrto multiuser17:53
rsalvetibut didn't dist-upgrade17:53
rsalvetilet me do that now17:53
rsalvetiwill download 100mb =\17:53
racarrtested what?17:54
racarri.e. mir/unity, etc all run fine (from which source)17:54
rsalvetiimage from aug5th17:54
racarron august 5th image?17:54
rsalvetiyup17:54
racarrthe mir image? or17:54
racarrimage + mir ppa or whatever17:54
racarrwhat*17:54
rsalvetidefault image, and default mir from ppa (just testing the egltriangle client example)_17:54
rsalvetifor now17:54
rsalvetiwhich is already enough to reproduce the black screen issue17:54
rsalvetisorry, default mir from archive17:54
rsalvetilet me dist-upgrade now to check17:55
kgunnrsalveti: you sure about that?...i thot racarr was getting black screen , but then could subsequently run mir demo client fine17:55
kgunnracarr: ^ ?17:55
rsalvetilol, so many issues at the same time it seems :-)17:55
rsalvetibut I'm getting black screen with mir_demo_client_egltriangle17:56
kgunn:)~17:56
racarrYes I can run the client17:57
racarrfine17:57
racarrand it reports getting FPS, etc17:57
racarr(frequently it gets stuck at half FPS though not always)17:57
racarrbut there is never anything on the screen17:57
rsalvetithen why the hell even the client is getting a black screen here17:57
racarrrsalveti: No, black screen17:57
rsalvetishowing fps and such, but black screen17:57
racarrit ust reports working17:57
racarryes17:57
rsalvetiright17:57
rsalvetithen it's the same behavior here17:57
racarrOn any image where I have ever seen that17:57
racarrI have never seen17:58
racarranything on screen  from mir :)17:58
racarrbut occasionally have seen the crash17:58
racarron any image where I have seen things on screen from mir17:58
racarrI have also occasionally seen the crash17:58
racarrbut never seen17:58
racarrjust black screen17:58
racarrso I think it's really just two issues17:58
rsalvetiright17:58
rsalvetithe crash is something that should be fixed once I fix the udev/ueventd races17:58
rsalvetinow we need to isolate this black screen one, got quite a few images to test here17:59
rsalvetiafter doing dist-upgrade17:59
racarrok makes sense that the crash is a race17:59
racarrim glad you hve the images :)18:00
racarr19.2 is still broken for me18:00
racarrust for the record18:00
rsalvetiok, should have enough to keep me busy for the next hour18:01
racarrOk. Let me know if I can help18:01
kgunnrsalveti: let me know if i can be of help as well....18:01
rsalvetisure18:01
racarrkgunn: I am considering that maybe I should try and land dpms with the ineffeciency that I was talking about yesterday18:02
racarrbecause it will be easy to fix after FF18:02
kgunni'm not the fastest....but i follow instructions well18:02
kgunnracarr: yes18:02
kgunnagreed...18:02
rsalvetiracarr: worked fine after dist-upgrade18:08
rsalvetinow to try some other images18:08
racarrSo sstrange18:10
racarrits going to be something awfully small in the end haha18:10
rsalveticould be on the android side18:12
rsalvetilike the migration to 4.2.218:12
racarroh didnt realize that happened18:14
rsalvetibut the interesting thing is that the binaries we were using were already from 4.2.218:16
racarrso strange :/18:19
rsalvetiyeah, image from 07 already gives me a black screen18:21
rsalvetinow let me revert the android side18:21
rsalvetiyeah, just flashing the android zip from 05 made 07 to work again18:23
rsalvetiso the different is on the android side, will investigate that a bit more18:24
racarrAha! close18:25
* rsalveti takes a break, late lunch18:26
racarr:) enoy18:26
racarrLunch for me18:52
=== racarr is now known as racarr|lunch
tvoss_racarr, ping19:01
kgunnrsalveti: so 07 vs 05...that's android config as of August 7 and August 5 respectively ?19:19
rsalvetikgunn: yes19:19
rsalvetiaugust 5 was still based on 4.2.1, >= august 6 is based on 4.2.219:20
kgunnrsalveti: got it...19:20
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== racarr|lunch is now known as racarr
racarrhoops back19:46
racarrwhoops*19:46
racarrtvoss_: pong19:52
thomimorning20:12
racarrdpms is almost done :)20:17
racarrall done except the GBM impl hich looks really simple20:18
racarrim a little concerned about backlight control20:18
racarr                if (drmIoctl(sna->kgem.fd, DRM_IOCTL_MODE_GETPROPERTY, &prop))20:24
racarrdrm API is a pain to mock20:24
racarrnot sure how to mock the20:25
racarrDPMS property index id integer:p20:25
rsalvetiracarr: ok, so the change is part of the updated display stack for qcom (for 4.2.2)20:34
rsalvetireverted it and was able to get that to work with the latest image, now to find out the right commit :-)20:34
kgunnrsalveti: can you share the android git tree & relative history (if easily at hand)20:43
kgunnrsalveti: just curious...i mean, we're likely going to have to update to "new android stack model"20:43
kgunnwould be my guess20:43
kgunnrsalveti: p.s. thanks for the bisecting help20:43
racarrrsalveti: Great to hear :)20:46
rsalvetikgunn: http://phablet.ubuntu.com/gitweb?p=CyanogenMod/android_hardware_qcom_display.git;a=summary20:48
rsalvetithis is what I reverted, basically20:48
rsalvetifrom the phablet-saucy branch to phablet-10.120:48
rsalvetinow will try to bisect in more details20:49
racarrImplemented durning off the display but turning it back on20:53
racarris having some issues20:53
racarrrevno: 982 [merge]21:32
racarrwhat does this do21:32
racarror is it just refactoring21:32
racarrit breaks unity 821:32
racarrand the easy answer is revert it :p21:33
racarr...lol at xf86-video-modesetting21:42
racarr        /* bonghits in the randr 1.2 - uses dpms to disable crtc - bad buzz */21:43
racarr        if (mode == DPMSModeOff) {21:43
racarr        }21:43
racarr...bad buzz indeed.21:43
bschaeferanyone else getting this error with only this mir demo: http://paste.ubuntu.com/6008072/21:44
bschaeferthe multiwin, it seems when ever I try to do any kind of "get mir region" it fails :(21:44
racarrbschaefer: THERE ARE NO MORE B UGS LALALALALALALALA21:45
racarrNo I have not seen that21:45
bschaeferracarr, :)21:45
bschaeferhaha21:45
racarrand actually just built and tested trunk like 30 seconds ago21:45
racarron radeon21:45
racarrmy distribution21:45
racarris not entirely up to date21:45
bschaeferracarr, yeah im on ati...maybe I should update mir21:45
* bschaefer goes to rebuild trunk21:45
bschaeferall other demos work though :)21:46
robert_ancellrobotfuel, ack on the bypass testing - that seems to be the same behaviour others are getting. duflu has made some more changes so I'll update the PPA now21:50
* bschaefer notices the nice grey background of the mir server now21:58
bschaeferracarr, stack trace if you're interested: http://paste.ubuntu.com/6008119/22:00
bschaeferfull trace: http://paste.ubuntu.com/6008126/22:02
bschaeferracarr, i also just realized was getting that cursor problem as well that I commented out...22:02
bschaeferas im not even able to load the server without it complaining about the gbm not able to open ...22:03
* bschaefer goes to try that out again22:03
racarrill check it out soon22:08
racarrsecond lunch :p22:08
bschaeferyuup getting that gbm cursor error just when trying to start the mir server on normal X... http://paste.ubuntu.com/6008137/22:08
bschaeferracarr, :), well I really should ping RAOF about it22:08
bschaeferas IIRC the fix the u-s-c was to disable input but this was only for u-s-c, not the mir server...22:08
racarrNo I get RAOF22:09
racarr:p ok maybe I can figure out my dpms thing myself22:09
racarrand you can hve RAOF22:10
bschaeferracarr, haha, well yours is a bit more important :)22:10
bschaeferi think my issue is only on a few ATI cards...22:10
bschaeferracarr, i can also bug him tomorrow about it22:11
racarrok ill give you RAOF for22:11
racarr1 bitcoin22:11
racarrinsted22:11
bschaeferhaha22:11
racarr:p haha sorry im just teasing22:11
racarrdont orry about it22:11
* bschaefer doesn't have any bitcoins22:11
racarrI probably cant look hard at this right aay though, need to poke at22:11
racarrDPMS a little22:11
racarrshould be super close22:11
racarrhaha me either22:11
bschaefercool, i don't actually know what DPMS is22:12
bschaefersounds like a long word though22:12
kgunnrsalveti: so what's the current thot....should you simply revert that one commit to unblock ? or what's your recommendation ?22:13
bschaeferDisplay Power Management Signaling? Eww...22:13
rsalvetikgunn: there's not single commit to revert yet, still looking22:13
kgunnrsalveti: got it...i'll try to be patient22:13
rsalvetihaha :-)22:13
kgunnrsalveti: i get a lot of questions about it....as you can imagine :)22:15
rsalvetiyup, no worries22:15
racarrbschaefer: Its pretty ew22:26
racarrso ar I can turn the display off22:26
racarrbut then never back on :p22:26
bschaeferhaha, well who needs a working display anyway!22:26
racarrthen I tried looking in xf86-video-modesetting for examples22:27
racarrhich is where22:27
racarr21:44 < racarr>         /* bonghits in the randr 1.2 - uses dpms to disable crtc - bad buzz */22:27
racarrcomes from22:27
racarrlol22:27
bschaeferhahaha22:27
racarrRAOF: Around?22:59
RAOFracarr: Yo!23:08
racarrRAOF: Was hoping you might have some insight on my DPMS difficulties23:10
racarressentially, I am setting the DPMS property on the connector23:10
RAOFSure. Let's see if I've drunk from that particular fountain of madness.23:10
racarrwith the DPMS mode, and DPMSModeOff works kind of as you ould expect23:11
racarrthe display turns off23:11
racarrunforunately I can't verify hat happens next because23:11
racarrI can never turn the display back on23:11
RAOFHeh.23:11
racarrand if I write mir_demo_server to23:11
racarrsome output the file ends up empty23:11
racarrI guess I can use a second machine to23:11
racarrget that but23:12
racarrlol I guess I am just wondering if there is anything off the top of your head23:12
racarrthat should obviously have to be done as well, etc23:12
racarrmaybe I can use xset dpms force on23:12
racarrto recover23:12
RAOFI think possibly the answer might be "don't bother trying to use the dpms properties; just modeset a null fb"?23:13
RAOFracarr: That's what the intel driver does, at least.23:14
RAOFDPMSOff == SET_CRTC to 0; DPMSOn == set_mode_major (ie: full modeset)23:15
RAOFracarr: This is different to just removing the display from the DisplayConfig because the display is still logically connected (and so you can put windows on it, we probably still have its DisplayBuffers allocated, etc). But as far as KMS is concerned, just remove it from the config.23:18
racarrhe intel driver also sets the property23:18
racarrthe modesetting driver ignores the crt and just messes ith the property23:18
racarrcomplaining of "bonghits in the randr"23:18
racarrIll try ith the crtc...and make sure I redo things properly on ModeON23:20
RAOFAh, right. You might want to set the DPMS property in addition to doing the modeset to turn off/on the backlight.23:22
RAOF(Which we might have to do manually anyway)23:22
racarrsounds promising23:22
racarryes backlight23:22
racarris a mild concern23:22
racarrintel driver seems to do special things23:23
RAOFYeah, backlight is a swamp of madness.23:23
racarrattempts test again...*crosses fingers*23:24
RAOFBecause it's all ‘as an OEM I can add value by having a different backlight interface!’23:24
RAOF(Spoiler: no, you can't)23:24
RAOFHm. So XMir XRandR appears to work, except for the bit where Mir actually changes any modes.23:27
racarrDoo doo :(23:32
racarralso that doesnt sound great23:32
bschaeferRAOF, hey, guess what happens when I try to start the mir server on my ATI machine? http://paste.ubuntu.com/6008137/23:36
RAOFbschaefer: What ATI card do you have, again?23:36
bschaefer VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Redwood XT [Radeon HD 5670/5690/5730]23:37
bschaefer567023:37
bschaeferis the one I have23:37
bschaeferand if i comment out the creation of the cursor it works fine until I try to do software renderering (multiwin example)23:38
bschaefereverything else works if I comment that out...23:38
bschaefer(comment out the creation of the cursor)23:38
RAOFRight. This suggests that we're failing to allocate software-renderable gbm buffers.23:40
bschaeferright, which this is what I get when I run the mutli win when I turn the cursor off: http://paste.ubuntu.com/6008119/23:40
RAOFDoes the server also die?23:41
RAOFOr spew any error messages?23:41
bschaeferwell right from my perspective of it not working :)23:41
bschaeferRAOF, nope, it just fails to run the application and it dies23:41
bschaeferit spews out:23:41
bschaeferterminate called after throwing an instance of 'boost::exception_detail::clone_impl<boost::exception_detail::error_info_injector<std::runtime_error> >'23:41
bschaefer  what():  Failed to import PRIME fd for DRM buffer23:41
bschaeferalso this is me running mir natively on normal X23:43
RAOFBy "on normal X" you mean "With an X server running on another VT", right?23:45
bschaeferRAOF, hmm normal X meaning i've turned off xmir23:46
kgunnhey...anyone got phoronix installed running standalone x (not xmir)....updated like today...want to give it a shot23:46
bschaeferbut yes23:46
kgunnspecifically openarena23:47
bschaeferkgunn, i don't have phronoix installed but I can install it23:47
bschaeferand im on standalone x23:47
kgunnits pretty dead easy23:47
bschaeferyeah, it just take a while to download the games :(23:48
kgunni just ran it on a recent update and it was ~10fps23:48
bschaeferRAOF, i can try to take a better look into it since I can reproduce it well23:48
bschaeferwell at lease figure out what function is failing...23:49
bschaeferkgunn, im also on ati23:49
RAOFbschaefer: To debug you'd want to stick a breakpoint in gbm_dri_bo_create, matching width == 64 and height == 64.23:50
bschaeferRAOF, to match the size of the cursor?23:51
bschaeferRAOF, and ill give that a try after running this phoronix test23:51
RAOFbschaefer: Correct. That way you won't catch the other allocations23:51
bschaefero cool, but it seems to be the the only one, as commenting out the creation of the cursor and the server works23:51
bschaeferall egl apps work fine on it23:52
kgunnbschaefer: that's cool...its a good data point23:55
bschaeferkgunn, cool, just downloading openarea...then it should run!23:55
RAOFbschaefer: Yeah; all those other allocations will succeed, so they're uninteresting. That's why you should ‘break gbm_dri_bo_create if ((width == 64) && (height == 64))” ☺23:56
bschaeferooo i see, I was thinking all gbm buffer calls were failing, because running the example mulitwin was having problems getting a mir region23:57
bschaeferwhich seemed related...23:57
bschaeferthanks!23:57
RAOFI'd suspect it's just calls with GBM_BO_USE_WRITE set23:58
RAOFWhich the cursor will have, but also any buffer we allocate for software rendering.23:59
RAOFAnd it's easier to catch the cursor :)23:59

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