/srv/irclogs.ubuntu.com/2013/09/23/#ubuntu-mir.txt

=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== davmor2_ is now known as davmor2
=== greyback is now known as greyback|food
=== chihchun is now known as chihchun_afk
racarrMorning!12:26
racarrgreyback|food: I found the crash!12:26
greyback|foodracarr: yeah!12:27
greyback|foodhave a branch I can try?12:27
racarrgreyback|food: Yeah, of unity-mir actually, the final solution isn't totally evident12:27
racarrjuts a sec12:27
greyback|foodracarr: oh, what have I done wrong?12:28
racarrgreyback|food: Eh, no not really your fault which is why I say the final solution12:28
racarrisn't evident12:28
greyback|foodok12:28
racarrgreyback|food: http://pastebin.ubuntu.com/6145326/12:28
racarrbasically snapshot crashes if default_surface is null12:29
greyback|foodoh interesting12:29
racarri.e. the application is open but not surface yet12:29
racarrso it could throw an exception12:29
racarror perhaps return an empty image or12:29
racarrI dunno12:29
greyback|foodwell I should not be asking for a snapshot if the surface hasn't been created yet12:30
racarrthe good news is it seems really hard to crash with this applied12:30
racarrhavent seen any12:30
greyback|foodcool12:30
=== chihchun_afk is now known as chihchun
kgunnracarr: nice12:33
greyback|foodracarr: wanna propose that change as a MR, and I can review?12:33
kgunnalf_: can you propose an mr to bump the so name of the mir server?12:34
racarrgreyback|food: Ok12:35
racarrMorning kgunn :)12:35
kgunnmornin12:35
kgunngonna grab toast and coffee....12:35
racarr:)12:35
racarrOh also im not really up early :p (re: other channel)12:36
racarrim in Pennsylvania12:36
racarrgreyback|food: https://code.launchpad.net/~robertcarr/unity-mir/check-default-surface-before-snapshot/+merge/18701212:37
greyback|foodracarr: thanks12:38
=== greyback|food is now known as greyback
racarrgreyback: What's remaining to flip mir in the image.12:45
racarrDPMS landed btw and alexandros is working on galaxy nexus support via sysfs stuff12:45
racarrIs it just flicker?12:45
greybackracarr: yes, just flicker12:46
racarrgreat12:48
racarrgreyback: There is a lot of reason to believe that has to do with our HWC module and that upgrading the appropriate bionic side bits12:48
racarrwill fix it :)12:48
racarrkdub is on it last I heard12:48
greybackcool12:49
greybackwould be great if we can flip soon12:49
=== mzanetti is now known as mzanetti|food
racarrGood afternoon alan :)13:10
=== mzanetti|food is now known as mzanetti
kgunnalf_: scratch my last ping14:08
racarrMy focus-tell-dont-ask branch is generating the strangest test failures. ApplicationMEdiatorReport session_create_surface and surface_create_buffer failing with broken pipe14:10
racarrlooks like it's exposing something weird...14:10
=== chihchun is now known as chihchun_afk
kgunndidrocks: for libmirclient ffe https://bugs.launchpad.net/ubuntu/+source/mir/+bug/122921214:23
ubot5Ubuntu bug 1229212 in mir (Ubuntu) "[FFe] mir api bump post sprint for libmirclient" [Undecided,New]14:23
didrockskgunn: you should add that it doesn't have any impact on the default ubuntu experience and the risk of regression is low14:23
didrockskgunn: also, as per wiki page, you need to subscribe the release team14:23
kgunndidrocks: ack14:25
didrockskgunn: then, let's join #ubuntu-release and get people looking at it ;)14:25
didrocks(please ping me once there)14:25
smartboyhwNo symbols? Ow........14:26
=== dandrader is now known as dandrader|afk
racarr...my laptop power cord is failing me so it seems like I am goign to dissapear for a while soon14:50
racarrwhile I reinstall on an old macbook I have in the closet14:50
racarrits depressing to watch your laptop battery go down and know there is nothing you can do about it14:54
racarrgreyback_: kgunn: I guess we need resizing for october? (phone rotation)14:56
racarrso thinking of switching full steam to that?14:56
kgunnracarr: greyback_ actually...cleaning up (controlling) the api for server should come first14:57
kgunnat least something better than we have atm14:58
kgunni thnk resize is actually something we can use post V1....but i'll defer to greyback_14:58
=== dandrader|afk is now known as dandrader
racarrkgunn: It's needed for rotation15:00
racarrbecause of the panel15:00
racarror maybe we could use a hack in qtubuntu with artificially inflated surface sizes to work around it in the interim15:00
kgunngreyback_ racarr ....panel didn't rotate before (not arguing its not needed...), so was assuming parity ok15:01
kgunnlemme check15:01
greyback_kgunn: panel does not rotate right now.15:02
racarrkgunn: Well, parity is ok for landing right, but not ok for october final?15:04
kgunnracarr: i believe its ok for oct...but lemme check15:05
racarrOk15:06
racarrI can start working on stabilizzing the server APIs too15:06
racarrI have some ideas for reducing the width of the interface between unity-mir and mir15:06
racarror rather, the number of interfaces used15:06
racarrbut I'm not sure  I have a full picture solution15:07
racarrok laptop is about to kill itself. if all goes well should be back in an hour or two. Can reach me on15:07
racarrgoogle talk via phone in the mean time15:08
kdubhello folks15:12
=== chihchun_afk is now known as chihchun
kgunngreyback_: can you change the build dep for unity-mir15:39
kgunnricmm: can you change it for platform-api ?15:39
greyback_kgunn: can do.15:39
kgunnthanks...didrocks is gonna try to land mir15:40
kgunnlatest15:40
didrocksdon't forget u-s-c15:40
kgunndidrocks: sorry if i am slow...but why are we making this change ?15:42
kgunnhttps://code.launchpad.net/~didrocks/mir/remove-hack/+merge/18702915:42
didrockskgunn: because we don't force reverse depdencies anymore to use the exact mir version they were built against15:43
kgunndidrocks: ah...backward compat15:44
didrocksyeah ;)15:44
greyback_kgunn: it's just a client api bump?15:48
kgunngreyback_: ricmm both client & server15:49
kdubricmm, rsalveti how easy (or difficult) would it be to try a cm-10.2 based phablet build?15:49
=== dandrader is now known as dandrader|lunch
kgunnalf_ racarr kdub read your mail....let me know if you think we should just ressurect the integration branch ?16:22
kgunnis alan_g having connectivity issues ?16:22
ricmmkgunn: didrocks what exact versions should we start depping on in unity-mir and platform-api?16:23
ricmmfor the new process, wahts the first dep to dep on16:23
didrocksricmm: 0.0.11 seems to be the version they will bump Mir to16:23
mterrykgunn, hi16:24
kgunndidrocks: i thot you said since it bumped once16:24
kgunnthat we shouldn't bump again16:24
kgunnor do you want us to ?16:24
didrockskgunn: there are 2 bumps16:25
didrocksthe packaging ABI name bump -> not needed to change (in debian/control)16:25
didrocksthe upstream version bump (the version you are releasing), you need to bump it to 0.0.11 in debian/changelog16:26
ricmmgreyback_: ^ we need to dep client/server on .11 then16:29
ricmmso that we benefit of a rebuild16:29
kgunnkdub: moving here...do you think its worthwhile to chase a workaround of forced second layer for a bit ?16:59
kgunni'm not sure the feasibility of cm10.2 attempt16:59
kgunndidrocks: you gonna be on for a few more ?16:59
didrockskgunn: ~30 minutes, I doubt Mir will have the time to finish building by then (even if we start now)17:00
didrockskgunn: just push everything, I'll staged to proposed an have RAOF tomorrow morning doing the final xmir uploads17:00
didrockssounds right?17:00
kdubkgunn, unsure. i don't understand how that forces anything to be right though17:00
kdubso it might just be shuffling the problem around17:00
RAOFdidrocks: I'm at XDC at the moment; I can do uploads, but not necessarily when you want them :)17:03
didrocksRAOF: can you just email me a link with the 3 xmir packages source I need to upload then?17:04
didrocksI'll do that for you in that case17:04
RAOFdidrocks: Sure17:04
didrocksRAOF: thanks! I'll upload that my tomorrow morning then17:04
didrockskgunn: ^17:04
kgunncool17:04
RAOFdidrocks: Oh, it's already in the experimental-prevalidation PPA. Are you not going to be copying that over?17:04
greyback_didrocks: kgunn: unity-mir version bump branch https://code.launchpad.net/~gerboland/unity-mir/bump-mirclient-to-3/+merge/18707117:06
=== chihchun is now known as chihchun_afk
didrocksRAOF: we can't copy from that, just send me the link of the name of the source then, I'll just resign and uploda then17:06
didrocksRAOF: I prefer to have an email with the name to ensure we don't forget anything ;)17:06
RAOFdidrocks: Ok. I'll do that over lunch.17:06
didrocksgreyback_: approved (not top approved yet, see the comment)17:07
didrocksRAOF: thanks!17:07
greyback_didrocks: ack17:07
kgunndidrocks: greyback_ ricmm mterry ... please do so https://code.launchpad.net/~kgunn72/mir/changlog-update-0.0.11-for-mirserver/+merge/18707017:08
didrockskgunn: commented17:11
didrockshttps://code.launchpad.net/~kgunn72/mir/changlog-update-0.0.11-for-mirserver/+merge/187070/comments/42668617:11
kgunndidrocks: ack all that...but for server, i thot ffe was for touch ? hence blanket...only client has ffe right ?17:16
mterryah, that'll teach me17:16
didrockskgunn: oh, it's just to list it, so that we don't get any issue with the release team :)17:17
kgunndidrocks: ok...will include17:17
didrocksthx17:17
kgunndidrocks: but going fwd...if we break server again....then what ?17:17
didrockskgunn: no need to list a bug17:18
didrockskgunn: but otherwise, same procedure17:18
kgunncool17:18
didrocksmterry: I feel sad that you approved :p17:18
didrocksmterry: I can do now 5 NEW without looking at the copyright file, relying on you catching up during the MIR ;)17:19
didrocksthat will be your punishment!17:19
mterrydidrocks, Now I feel sad too17:19
didrocksahah17:19
kgunndidrocks: ok...think its right now...17:24
kgunngreyback_: thanks for the list of headers17:25
didrockskgunn: the syntax is (LP: #1229212), not (LP#1229212)17:25
ubot5Launchpad bug 1229212 in mir (Ubuntu) "[FFe] mir api bump post sprint for libmirclient" [Undecided,In progress] https://launchpad.net/bugs/122921217:25
kgunndidrocks: ok...one more time17:25
didrockskgunn: otherwise, almost good (well, you have -0ubuntu1, but daily release will strip and fix that anyway)17:25
kgunndidrocks: ok...just the : and that's it?17:28
didrockskgunn: : + space17:28
kgunndidrocks: ok...for real for real...it should be done :)17:31
* didrocks checking :)17:31
didrockskgunn: globally approved ;)17:32
kgunnracarr: you still around or flying ?17:34
kgunngreyback_: so weird...i must have had a bug in my bash...those were there....just moved them, it now seems to work17:44
* kgunn so lucky17:44
didrockskgunn: I didn't point them out on purpose, not that important (you will remain with "kg" as the signer)17:46
=== dandrader|lunch is now known as dandrader
kgunngreyback_: ok...lucky me has updated again17:48
greyback_kgunn: did you push?17:49
kgunngreyback_: yes but just...17:49
greyback_kgunn: ok, approving here17:50
kgunnkdub: back to the forcing 2nd overlay...i thot you were thinking it'd force diff code paths? or...maybe that  wasn't your idea & someone else was puting it fwd ?17:53
kgunnkdub: looking at the code, only thing i saw was an extra invalidate17:53
kdubkgunn, i've patched around the obvious problem with the code path, which didn't make much difference17:54
kgunnkdub: so, atm we're back to needing to try CM 10.217:54
kdubkgunn, well, at the moment, i'm trying to compile the kernel so I can dig right down to the hardware and figure out what's going on17:55
kgunnkdub: need help? or are you able to build & boot ?17:55
kdubthe kernel guys were helpful in pointing me to how to build, now its just a matter of me getting my chroot in order17:56
rf9020ps aux | grep unity-system-compositor19:14
rf9020patricia  2833  0.0  0.0   4464   808 pts/0    S+   21:12   0:00 grep --color=auto unity-system-compositor  what is it ?19:14
ogra_the process entry for the grep you are running19:15
ogra_ps aux | grep unity-system-compositor | grep -v grep19:15
ogra_that suppresses finding yourself while searching for something19:16
rf9020my problem on xorg kernel 3.11.0.8 reso 800x600 with nvidia 319.32 monitor 24" , mir ok if driver no proprietaire , , if driver Nouveaux screen blank !!19:18
rf9020how to have 1920x120019:18
=== mterry__ is now known as mterry
=== dandrader is now known as dandrader|afk
racarrReturn of working computer!19:37
racarrkgunn: I am still around. I am flying on wednesday19:37
racarrI was briefly out due to a failure of my laptop power cable19:37
rf9020what video card supports 100% mir19:39
racarris amd64 ci still19:44
racarrproblematic19:44
racarrit seems to be :(19:44
RAOFrf9020: Your choice of non-ancient intel, nouveau, radeon.19:46
=== dandrader|afk is now known as dandrader
rf9020?? radeon ok and nvidia ??19:48
racarrnvidia is ok as long as the nouveau supports the card19:48
rf9020i have  8500 gt nvidia19:49
racarrhttp://nouveau.freedesktop.org/wiki/FeatureMatrix/19:49
rf9020nouveau with 8500 gt screen blank with cross19:51
rf9020no click right , no terminal , but ctrl+alt f1 ok19:53
RAOFOooh, sweet! There's a new new DRI2Authenticate mechanism that'll let us async that bastartd.19:53
kgunnracarr: hey, was just checking on what you decided to attack - my preference is api :)...basically, pat sent a mail saying all ui changes should be deferred anyway...20:22
kgunnalmost viewing it as risk now20:22
racarrkgunn: API it is :)20:25
racarrthat and trying to get anything to pass jenkins20:26
kgunnracarr: no doubt on making jenkins happy....jenkins is so fickle20:27
racarr[  FAILED  ] DisplayConfigurationTest.display_change_request_for_unauthorized_client_fails (103032 ms)20:32
racarrfhqwfqwhfqwf20:33
ricmmkdub: kgunn can we talk here21:06
ricmmthe other channel is noisy21:07
ricmmkdub: can you explain more about what was failing?21:07
ricmmwe know SF does the 2 layer rendering with the same kernel you are trying21:07
ricmmand same HWC21:07
kdubricmm, but i'm not convinced they don't have rendering glitches either21:08
ricmmjust use the phone with surface flinger then21:08
kdubricmm, i haven't gotten the "hwc's force-2 layers" to work21:08
ricmmyou wont see the issues21:08
ricmmthey might still have other issues, but its not the same issue for sure21:09
ricmmvsync fencing is happening the right way21:09
kdubi won't see the flashing in the same way, i do see problems though21:09
ricmmand im not talking android, im talking the default image with SF + unity821:09
ricmmwhat problems?21:09
ricmmwell I mean but the difference is majestic21:09
ricmmits clearly obvious that something is failing with the Mir one21:09
ricmmthats triggering this21:09
ricmmthe 2-layer rendering being the most obvious cause for the missed fencing on vsync21:10
kdubbut the problems i see with surfaceflinger are similar to vsync timing issues21:10
ricmmporting to 10.2 is not really an option in our current timeframe21:10
kdubright21:10
ricmmwhat I want is exactly what surface flinger does to circumvent the problem, in our code21:10
ricmmat least as a PoC or a hack until we can move forward with 10.221:11
ricmmas it wont happen before 13.1021:11
kdubright, i'm trying to patch what we have for that21:11
kduband, i haven't been able to prove that surfaceflinger is doing the right thing either21:11
kdubso i'm currently in the process of trying to patch things21:11
kdubpreferably hwc21:11
ricmmI understand, what I'm saying is that even if SF is also having vsync issues... they are not as apparent to the naked eye as the Mir issue21:11
ricmmso you cant put them on the same plane21:12
ricmmas the SF flinger one can be considered "flawless" as we've been shipping it for a while, even if there are underlying hwc issues21:12
ricmmthey clearly dont generate the same outcome as the current tearing with Mir21:12
ricmmso, I'd say please keep trying to patch/hack to do what SF is doing re: 2lyr rendering21:12
ricmmuntil we can trigger the right fencing21:13
ricmmbecause it feels like you are right on the money with the missed syncs analysis, we just havent been able to get the hack to work to prove it21:13
kdubi can still pursue, i'm just worried that chasing the hack won't be sufficient21:14
kgunnkdub: maybe we can enlist duflu here21:15
ricmmkdub: unless another better option comes along...21:16
ricmmand they pretty much can only come from you as you are the one who knows the issue at hand21:16
ricmm10.2 is not a realistic option for shipping21:16
kdubsure, understand that21:18
kgunnricmm: sorry...got distracted, i still think if we could gen up a CM10.2 not for releasing but just to test...it would be very very useful21:28
kgunnricmm: just confirming that we could get some love from rsalveti to give it a shot at least21:28
kdubi'll try to stabilize the 2 layer hwc hack i have right now21:29
=== jhodapp is now known as jhodapp|afk
ricmmkgunn: it might not even be possible to get that love21:34
ricmmas both salveti and I are buried with stuff21:35
kgunnricmm: is flicker not important ? :)21:35
ricmmyes, but I prefer if it can be sorted internally in Mir21:35
ricmmas, if SF can do it, Mir can do it21:35
ricmmthats our motto after all ;)21:35
kgunnricmm: yeah...but i thot we were cool to get some help fom rsalveti ? ...no ?21:36
ollikgunn, isn't that an asac q?21:36
kgunnricmm: i don't even know how much work i'm asking for....can you give a swag ?21:37
kgunnolli: yeah..i suppose so21:38
kgunnricmm: it doesn't have to be rsalveti if someone else knows how21:38
ricmmguys, if the hack works for SF it will work for us21:39
ricmmits just not complete yet21:39
ricmmlets not sign off on a huge task as testing 10.2 can be21:39
kdubthe way sf and mir composition is triggered is a bit different... i guess right now, the first focus is testing the hack and seeing if it does help the flicker21:40
ricmmkdub: lets exhaust that route as I asked on friday first21:41
ricmmand then we'll see what we can do if all that fails21:41
ricmmrsalveti isnt back until tomorrow and he is the one who can tell you how much work it will be to test 10.221:41
kgunnricmm: thanks...21:41
ricmmbut I can assure you than more things will be broken, and that we dont have the manpower to get it in place for 13.1021:41
ricmmso it is not really a viable solution21:41
ricmmtherefore whatever cycles we spend there, will be wasted time21:41
kdubyeah, it seems like it might be a lot of code to move21:42
kgunnbut what if that is the right answer?21:42
ChickenCutlasskgunn, getting 10.2 up and running is really a big task.21:42
ricmmit is not, and that is proven21:42
kgunnrock/hardplace21:42
ricmmhow? because I dont see the same degree of flickering in SF21:42
ricmmso, that is not really the only solution21:42
ricmmwe need to contain the fix in the least amount of code21:43
ricmmand that is either Mir or hwcomposer itself21:43
ricmmbut not moving the whole base to 10.2, that will break some serious havoc :)21:43
kgunnor FB21:43
ricmmor fb yes21:44
kgunni'm not asking for CM10.2 to be released21:44
kgunnand i got the message21:44
kgunnits too big for you to attempt21:44
kgunnso new requirement is "flicker less than or equal to SF"21:44
kgunn..not no flicker21:45
ricmmthe req is feature/performance parity with SF21:45
ricmmif we get that, we ship Mir21:45
ricmmand as not shipping Mir isnt an option, we will get that for sure ;)21:45
kgunnricmm: i used to do phablet-dev-bootstrap but...did it and couldn't build...did it change to phablet-devenv-bootstrap21:57
ChickenCutlasskgunn, phablet-dev is still the way21:59
kgunnChickenCutlass: great...its just me22:00
kgunnracarr: any thots on the amd ci test ?22:08
kgunnracarr: i think francis is going to try to build/ci on a dedicated/non-shared host to see if that helps22:18
ice9what is the benefit of Mir over Wayland?22:36

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