[00:32] Really, gcc? If you use C++, even if you intend to expose a purely C ABI, you are compelled to expose random std:: detritus too? [01:46] Why, hello there! [01:46] Quite clearly nobody actually suse the mir_debug_* symbols, because they're mangled. === chihchun_afk is now known as chihchun [02:08] RAOF: Umm, worthy of a bug report maybe [02:09] Or I'll just fix it. [02:09] Or both at once... [02:09] If it never worked then it's not a regression. If no one ever used it then maybe not important enough to be a bug :) === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun [07:09] Hey gdb understands STL containers now and makes them readable [07:09] that's new [07:18] Oh, the python support got fixed? [07:26] @duflu: https://bugs.launchpad.net/mir/+bug/1275398 afaik this is the issue with i915 using a slightly different init code in mesa than the newer intel. so it needs some mir love [07:26] Ubuntu bug 1275398 in Mir "i915 driver/i945 hardware: Mir GL clients are rendered as black or transparent windows when using i945 graphics" [High,Triaged] [07:26] anpok_: Please apply love where appropriate [07:26] Umm. [07:26] You know what I mean [07:26] :) [07:27] i am strugling still with qxl [07:27] it does page flip [07:27] it seems to drop the page flip event to the event queue [07:28] user space seems to get woken up but there is no page flip event === chihchun is now known as chihchun_afk [08:07] RAOF: @"we don't want to install these into the global library search path" - if not then how does client code access the symbols in e.g. libmirplatform? [08:08] Via libmirserver? [08:08] We currently dlopen them from libmirserver, right? [08:09] Then the (server) client code never needs to directly link against libmirplatform? [08:09] Nope. The point of libmirplatform being LGPL is that it doesn't require client code to link against the GPL libmirserver [08:10] BTW I like the idea of introducing .symbols [08:11] Ooooh! [08:11] we do dlopen [mesa|android]/libmirplatformgraphics.so - so it could work for those [08:11] By “client code” in this case you mean “platform drivers”? [08:12] For this example, yes [08:12] What else is going to require symbols from libmirplatform, and not also link with libmirserver? [08:13] true. But by "client code" I mean drivers, servers, client, etc [08:13] There's a separate libmircilentplatform, right? [08:14] There is [08:15] Anyway, the answer to the question “where would they get their symbols from” is “from the thing that's dlopening() them” [08:15] * alan_g is still confused why some things are .so and some are not. [08:16] I think the state we _want_ to be in is having exactly two SOs - libmirserver.so and libmirclient.so [08:17] For servers & clients maybe. But drivers introduce a third case [08:18] And I would prefer to share common code than link it into all three. [08:20] Or four if there's really a good reason for libmirclientplatform.so being different from libmirplatform.so [08:20] Drivers don't need to link against anything Mirish; they get dlopen()ed, so they always have the symbols available. [08:21] Drivers are never useful outside the context of a mirserver instance [08:21] * RAOF is somewhat foggy, so may in future disown current statements :/ [08:22] * alan_g is not going to argue now but doesn't feel quite convinced [08:23] I'm a bit annoyed that libstdc++ means that libmirclient exports >1000 symbols, rather than the ~10 or so we *actually* want to export. [08:25] RAOF: Yeah I was wondering how you were going to solve the template problem. Particularly since libmirserver needs many of the template instance symbols exported [08:26] I think it's a lot more than 10 (but a lot more less than 1000) [08:26] I think you will find it will be higher than 1000, I expect [08:26] Templates are evil things [08:28] A tip for counting unique templates: Search for "::~" to identify unique destructors [08:28] alan_g: Yeah, more like 50, true. [08:29] duflu: I gave up touching libmirserver. Practically every cpp file is a part of the ABI there. [08:30] * RAOF started trying to hide what look like purely internal classes, but it turns out that they're not purely internal. [08:30] RAOF: I think a better place to start on mirserver is search the headers. You can quickly find a bunch of classes/headers that are unreachable by the public interfaces [08:31] "unreachable" meaning we don't provide any APIs for people to access instances of said classes [08:31] RAOF: my feeling is that libmir[client]platform is already stable (if not pretty) and a better measure for driver compatibility than the volatile server ABI [08:31] duflu: That's what I did; it's not true that they're unreachable by public interfaces. [08:32] RAOF: Well, some, not all, obviously [08:32] I don't think any of them are unreachable by public interfaces, actually. [08:33] RAOF: There used to be plenty. But I haven't made a stab at it for a long time [08:33] At least by my criterion of ‘if I apply visibility=hidden to this class, does everything link?’ [08:33] Given the number of headers has increased, I doubt they're all actually needed in public now [08:34] RAOF: meaning tests that access "internal" classes? [08:34] RAOF: Obviously Mir needs those headers. I'm talking about headers no external project wants/can utilize [08:34] alan_g: No, meaning mir_demo_server. [08:34] well we can safely hide those not in include [08:34] oh [08:34] anpok_: Ba baw! Incorrect! [08:35] our examples do that? [08:35] Because we rejected the idea of separating "shared" headers from "public" [08:35] We should be able to hide those not in include [08:35] anpok_: Because we expect people to derive from DefaultServerConfiguration, so everything that DefaultServerConfiguration touches needs to be public. [08:35] And by ‘public’ I mean “linkable to”. [08:35] Ah the ServerConfiguration beast [08:36] Why? [08:36] RAOF: sure? but for those forward decls should sufficient [08:36] Not at link time. [08:36] +be [08:36] Although public projects only need access to classes used in public methods of *ServerConfiguration [08:37] I mean used in public declarations [08:37] RAOF: yes, at link time [08:37] You can't have an object with fields that have visibility less than the object. [08:37] But the fields only need the forward decls [08:38] Sounds like we may have exposed too much info in *server_configuration.h" [08:38] Certainly more than I think we do [08:39] So, empirically, if you stick a bunch of visibility=hidden attributes on classes with headers in src/server, mir_demo_server_* cannot link. [08:41] We *should* be able to do that. I don't think the root cause is DefaultServerConfiguration - but will need to investigate. [08:41] Because the class derived from DefaultServerConfiguration needed a bunch of symbols. [08:41] camako: The merge will be instantly redundant if we shuffle the development focus back to 0.5. We can now that 0.5 actually has a branch [08:41] Yeah, I'll push this branch at some point. [08:42] camako: And then there's no waiting for Jenkins ;) [08:44] Here we go... [08:45] CMakeFiles/mir_demo_server_shell.dir/__/server_configuration.cpp.o:(.data.rel.ro._ZTCN3mir8examples19ServerConfigurationE0_NS_26DefaultServerConfigurationE[_ZTVN3mir8examples19ServerConfigurationE]+0x2d0): undefined reference to `mir::DefaultServerConfiguration::the_input_registrar()' [08:45] And a thousand others. [08:45] But EOD for me here. [08:46] RAOF: goodbye. (But that is a function in DefaultServerConfiguration, not something the function "touches". [08:46] ) [08:55] duflu, I see your point. However, we have already suffered and are in the final stretch (packages built). [08:56] There is an ongoing discussion on configuring jenkins for the rtm. [08:56] Meaning two distros.. [08:56] * camako feels our problems will be addressed soon [08:57] * alan_g admires the optimism of youth [08:57] camako: Well the preferable solution would be for distro to actually use the distro configuration and build from the correct upstream branch :) https://launchpad.net/ubuntu/+source/mir [08:57] * camako hasn't considered himself as "youth" in a very long time.. But thanks.. [08:57] That was previously "nice to have" but with two distros, probably now required [08:58] duflu, yes... That's why I'm "optimistic" [08:58] dednick, I'm assuming you've seen kgunn's email? [08:59] Can we push for landing silo 18, or do you need to fix smth? [09:00] camako: just read now [09:00] camako: give me a sec [09:01] sure [09:01] o/ [09:02] duflu, thanks for updating the changelog... [09:05] camako: give me a about 15 minutes to give the silo a quick test [09:05] dednick, sounds good... Take your time... I've yet to finish testing myself.. [09:23] alf_: I'm beginning to suspect that the reason for the slowdown is the fact, while before the job of actual rendering & buffer management was split between a mir compositor and the Qt renderer, now Qt's renderer thread is rendering but also is managing buffers [09:24] greyback: I took some new traces this morning, and one thing I noticed is that at the start rendering the dash takes ~1000 calls, but after visiting "copes" and going back [09:25] greyback: ... "Scopes" scope and going back it increases to over 1500, trying to figure out what the diff is [09:25] alf_: I think that's because the contents of an off-screen scope are still being rendered [09:26] alf_: I've pushed a patch to improve things, it should have landed by now [09:26] greyback: which package? [09:26] alf_: unity8, from the silo6 [09:27] That's fun. Callgrind says we make a quarter of a million fread calls to load X cursors [09:27] alf_: Qt's scenegraph renderer does not do off-screen culling [09:27] greyback: :/ [09:27] greyback: if we make those buffers opaque? [09:27] then still no culling? [09:28] well it's a big cost for a large scenegraph, and a good developer can more easily manually manage the visibility of components [09:29] alf_: a nice way to visually see these off-screen things is to run unity8 with QSG_VISUALIZE=overdraw [09:29] anpok: sorry, I didn't follow that. Which buffers opaque? Other apps I guess you mean? [09:29] greyback: also I noticed that even in the best case when things are "smooth", the dash is consistenly above 16ms (20-30ms), and in the worst case (judder) it's about (40-50), so there is some serious work to be done there regardless of this particular issue [09:30] anpok: yeah making them opaque would help [09:30] alf_: yep [09:30] alf_: dash is doing lots of work, and lots of bad things [09:31] and as a result qt isn't able to optimise much [09:32] greyback: "actual rendering & buffer management" how do you mean that? [09:34] greyback: in theory QtCompositor should be more efficient, since it's rendering directly to the mir provided framebuffer instead of on a surface which is then composited by the default mir renderer (which I think was happening before QtComp?) [09:36] alf_: correct. I just had the idea last night that we've taken the jobs of 2 threads and combined them into 1 (removing GLRenderer though) [09:37] greyback: hmmmmm [09:37] the full screen blend blocked the caller for around 2 ms on nexus4 [09:38] anpok: the thing is, we see performance drop with zero apps open. So that is Qt rendering directly to the mir provided framebuffer [09:41] greyback: you may be right, in the sense that because we are rendering in the thread that blocks for the page flip, the effect of missing a frame boundary are more pronounced [09:43] wasnt there an option to move eglSwapBuffers to another therad in Qt? [09:44] anpok: that is enabled (the renderer runs in a separate thread per window - per display *I think* - separate to the GUI thread) [09:44] greyback: anpok: it's not that they are in the same thread, it's also that previously because we rendered to a surface first, we had a buffer to ease frame boundary misses [09:44] i thought there was another one that also moves eglswap to a third [09:46] greyback: anpok: but this is just on the top of my head, no hard facts to back it up [09:46] greyback: anpok: off the top ... [09:53] greyback: your new unity8 build is much better in terms of gl calls per frame [10:00] alf_: yeah, but you can still experience jank when switching between apps & scopes scope [10:01] so not perfect [10:09] greyback: you mean while switching between the two, or after switching to scopes and back to apps again [10:09] ? [10:11] alf_: if I just go back & forth between the 2, it seems to slow a bit [10:11] expecially with a certain flick [10:15] Good news. I shall leave you today with nothing new to propose [10:16] \\o/ [10:17] duflu: oh to continue our custom to belatedly admit we've not done anything wrt window management: I've not done anything wrt window management recently [10:17] greyback: Awesome-tastic [10:17] greyback: Me too but only because RTM trumps WM [10:18] greyback: hmm, that's to be expected I guess... during the transition we are rendering both scopes. Each scope separately fails to meet the deadline, both scopes together make a complete mess [10:18] * duflu has another idea and fails to log off... [10:18] alf_: yep [10:19] alf_: but at least a probably solution is in the pipeline: split Dash out of shell so it is a separate process [10:19] s/probably/probable/ [10:19] well "solution" [10:20] it'll still render slowly [10:20] greyback: Has something changed in this area in QtComp? Is nonQtComp somehow convincing QtSG to render only parts of the SG? [10:20] but the frame boundary missing would be more forgiving - if that theory is right (I'm convinced) [10:21] Ah. Input latency reduced ten-fold. If only we could do that on the phone. [10:21] greyback: is split-dash scheduled for rtm? [10:21] duflu: how and what's blocking this on the phone? [10:22] alf_: Framedropping. And the fact that Qt apparently lacks throttling to be able to use it safely [10:26] alf_: yes [10:28] duflu: call me ignoramus: input throttling? Is that compressing a large number of events into a single event, to reduce the amount of events a client has to process? Or something else? [10:28] greyback: Could refer to different things. Where's it written? [10:29] duflu: I dunno, I saw you write it, I wanted to know what you were referring to [10:30] greyback: Oh. I did notice with strong suspicion though that Android-input seems to group and discard input events depending on how fast you consume them. Really fast renders get more events. Slow renderers get less, but at least don't get progressively more behind [10:31] duflu: ok, that's interesting [10:32] greyback: So fingerpaint for example always keeps up with the current pointer location, but a fast renderer (frame dropping) sees many more samples... smoother curves [10:32] I know for a fact my mouse generates around 1000 events per sec. Mir is certainly not getting most of them [11:10] dednick, the silo is testing well for me. How is it on your side? [11:10] * camako saw some msgs between alan_g and tvoss [11:10] and dednick [11:18] camako: silo working for me. tvoss having some issues, but don't think it's directly related. === alan_g is now known as alan_g|lunch [12:15] alf_: I'm getting more & more convinced by your frame boundary explanation [12:17] greyback: convinced by experiments or by thinking about it? :) [12:17] alf_: I've no proper data, more just thinking about it [12:19] greyback: btw, QSG_VISUALIZE and its various options are very neat... and the dash sucks at batching (33 alpha-enabled batches, 2 opaque) :/ [12:19] alf_: I know, it's horrible [12:20] alf_: totally defeating the batching optimisation [12:20] explains why scrolling so severe on CPU and GPU [12:23] greyback: Do you think there is an easy way with the QtComp codebase to test the frame boundary theory? [12:28] alf_: possibly by comparing frame render & frame delta times between QtComp & non-QtComp [12:28] QSG_RENDER_TIMING *might* give enough info, else QSG_RENDERER_DEBUG=render [12:29] would also need to compare the same complex scene for both cases === alan_g|lunch is now known as alan_g === greyback is now known as greyback|lunch === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader [14:19] hey guys, got a gift for you - client crash on exit bug #1342694 [14:19] bug 1342694 in mir (Ubuntu) "qmlscene crashed with SIGSEGV in _M_release() on quit" [Undecided,New] https://launchpad.net/bugs/1342694 [14:19] it *could* potentially be qtubuntu, too === greyback|lunch is now known as greyback [14:37] Saviq: it's being discussed... I think alan_g just root caused it [14:37] coolz === dandrader is now known as dandrader|lunch === chihchun_afk is now known as chihchun === dandrader|lunch is now known as dandrader === alan_g is now known as alan_g|EOD === dandrader is now known as dandrader|afk [20:04] Would appreciate thoughts on https://code.launchpad.net/~mir-team/mir/cursor-spike-phase-7-name-some-cursors/+merge/225242 [20:04] once it or something similar lands can go around and add cursor support to qt, chromium, gtk, etc... === dandrader|afk is now known as dandrader [20:06] + any problem with landing https://code.launchpad.net/~mir-team/mir/improve-attribute-protocol-and-normalize-getters/+merge/225912 [20:11] opinions on https://code.launchpad.net/~mir-team/mir/touchspot-visualizer/+merge/224731 probably more important than cursor [20:15] opinions on https://code.launchpad.net/~mir-team/mir/touchspot-visualizer/+merge/224731 probably more important than cursor [20:15] whoops [21:00] how to share acceptance tests with unity-mir/qtmir/mirqt/gerry [21:00] beyond copy pasting code [21:00] ive been thinking though there are a lot of tests [21:01] racarr__: I'm open to ideas [21:01] that should be run against [21:01] the shell server configuration [21:01] racarr__: also note I've not had the time to make any of the changes you suggested yet. So don't be offended if it appears I just ignored you:) [21:01] no worries I havent done a second pass through yet. um. [21:02] I dont think any of them are in particular important to prevent landing. [21:02] i.e. I expect a read through of unity-mir [21:02] would turn up about just as much [21:02] could you make a gdoc and put your review comments in there please? Easier to deal with than an email thread IMO [21:02] I kind of tried to identify things that we may want to fix asap though. [21:02] oh sure, most of the same mistakes are probably there :) [21:02] like...I had kind of forgotten that screencast authorization was never implemented [21:02] lol [21:03] ok will [21:03] google doc it up [21:03] with QtComp, I need to disable it, as it won't work [21:03] until I combine Qt's renderer with Mir's interface properly [21:04] haha right... [21:04] we need proper permissions for display config at least [21:04] or [21:04] maybe both are just always disabled on the phone [21:04] and there is [21:05] I've no idea how shell mediates that. We've not had that discussion [21:05] developer mode :p [21:05] always disabled + developer mode to always enable [21:05] is better than [21:05] always allowed though [21:06] oh apparently I was wrong, we do send clients focus/unfocus events [21:07] :) [21:08] https://code.launchpad.net/~mir-team/mir/improve-attribute-protocol-and-normalize-getters/+merge/225912 [21:08] last chance to object pre landing [21:12] racarr__: why does a setter return the type that is set? [21:12] + MirSurfaceType set_type(MirSurfaceType t); [21:13] greyback: hmm no real reason atm...the exception throw [21:13] used to be in configure instead of set_* [21:14] I guess there is some thought that the logic is like [21:14] racarr__: it's a weird api IMO, as when I set something, I've to check if the value returned is actually the one I set. Else I cannot trust it [21:14] well [21:14] they are private [21:14] the public api is [21:14] int configure(int, int) lol [21:15] that's a crappy API and you know it [21:15] lol which part [21:15] why not just export these private methods? [21:15] ABI stability? [21:15] well as it stands the only user is frontend which is [21:15] me [21:15] happy with the ints. [21:16] ? [21:16] eh. I'm trying to be a vocal API consumer [21:16] haha. [21:16] configure(int,int) is an unpleasant way to set properties on something [21:17] aside from that, I like your changes [21:17] I think, as needed/requested, setters and getters can be added for specific properties....some of them have public getters [21:17] I guess configure/query is really the API to frontend [21:17] and the API to shell [21:17] has been ignored lol [21:19] nice :P [21:20] I hope I make sense...I may literally be the most tired I have ever been right now [21:20] the good news. No niccotine in 5 days! [21:21] and i...enjoyed the whole process this time [21:21] and am pretty confident that I am over it [21:21] the bad news. no proper sleep in 5 days! [21:23] keep it up man, you can do it [21:23] I think tonight is finally my night just due to sheer and total exhaustion. [21:23] :D Thanks [21:23] I'm not in any hurry for you to do anything (think I'm the bottleneck right now) [21:24] so if you need to snooze, go for it [21:24] I...none of it makes me want to smoke lol im just so tired atm hard to tell things make sense [21:25] haha, nah, gotta power through until the sun goes down at least lol [21:25] Think I am going to distract myself with low brain power acceptance testing. [21:25] :)