[00:27] robert_ancell: I think it just exposes less implementation detail [00:27] I mean it's just a lock in implementation right, but the interface to the clients [00:28] is "perform these operations atomically" [00:29] and yes, what did my emacs do? [00:58] racarr, + // from a second thread before input focus is set.xs [00:58] think you missed hitting ctrl there [01:27] True! [01:29] thanks :) fixed in r858 [02:59] Woo... another morning of just fixing conflicts [03:39] robert_ancell: ping [03:40] duflu, yep [03:40] robert_ancell: You mentioned that shortened versions was a possibility: major.minor only [03:40] I think I agree with that... [03:40] So long as no one complains that we change major a lot [03:40] duflu, for the project? [03:41] Yes [03:41] duflu, yeah, I like it too. The second number is only incremented if we branch, so for most purposes we only have one number [03:42] robert_ancell: Or rather the second number represents fixes that don't affect ABI [03:42] ... ? [03:43] duflu, then you can't do branches. i.e. if we release Mir 94 in saucy, then we need to fix it, we need version 94.1 because in t-series we'll be on (say) 113 [03:43] So the version become . [03:46] robert_ancell: I don't understand the problem. Once saucy is released as N.M, it will get updates N.(M+1). T-series is the only distro that will get (N+1) [03:48] Just like we do for Unity releases... [03:48] duflu, so what do we do if we release 1.2 in saucy and 1.3 in t-series then need to make a patch to saucy? [03:49] we can't go 1.3 because that would collide with a newer version. Then we'd have to do 1.2.1 [03:50] robert_ancell: There's no problem if each new distro gets a major rev [03:50] Numbers are cheap [03:50] yes they are [03:51] but there's no need to unnecessarily break ABI every single release which is what this scheme would do [03:51] ... so it would be 1.2 in saucy (up to 1.X in updates), and 2.Y in T [03:52] And _eventually_ when the ABI stabilizes, we could stay on the same major and branch to 3 components [03:53] All the while ABI == major == a single integer [03:53] there's no need to combine the project version and the three (for the moment) ABIs together into one number [03:53] it just creates unnecessary work [03:55] robert_ancell: That's kind of true. But it does create clarity.... Alternatively, see my latest MP comments [04:03] duflu, I have to go but I updated both MPs [05:08] good morning :) [05:08] RAOF, ping [05:33] Morning tvoss_ [06:18] mlankhorst, RAOF in case you are around: I noticed that SNA is not enabled by default in the XMir drivers. Is that by accident or on purpose? [06:25] tvoss_: hey, ar you going to push today the disable armhf android tests? [06:25] are* [06:26] didrocks, sure, I assumed you had done it already, happy to do it, though [06:26] alf_, mind if I set https://bugs.launchpad.net/mir/+bug/1102760 to in-progress? [06:26] Launchpad bug 1102760 in Mir "Multi-monitor support incomplete - can't show different images on each screen" [Critical,Triaged] [06:26] tvoss_: ah, sorry, I'm more trying to decipher the race right now [06:26] tvoss_: sure [06:26] tvoss_: .1 doesn't seem enough though, just had a case where it's not enough [06:26] alf_, the correct answer is nope :) [06:27] * didrocks will try .5 and give several try to it === 65MAANWI2 is now known as tvoss [06:30] tvoss: if you want, I can disable all tests, but I thought you wanted to be more fine grain and I don't know ctest [06:30] didrocks, let me see what I can do :) [07:15] tvoss: Um, that would be by accident. [07:15] * RAOF checks the packaging branches. [07:16] RAOF, I have a custom /etc/X11/xorg.conf to enable sna [07:22] didrocks, https://code.launchpad.net/~thomas-voss/mir/disable_acceptance_integration_tests_on_armhf/+merge/175746 [07:25] RAOF, alan_g|EOD you might want to have a look, too: https://code.launchpad.net/~thomas-voss/mir/disable_acceptance_integration_tests_on_armhf/+merge/175746 [07:29] tvoss: approved [07:29] thanks! [07:29] tvoss: I may need to extend the sleep to .5s FYI [07:29] just ensuring that all the runs pass [07:29] didrocks, not sure if that is enough, we might need to disable unit tests too [07:29] (by priority to dailies to finish tests) [07:29] so let's see what the builders say [07:30] tvoss: ok, I'll rerun a mir build once merged, ok? [07:30] didrocks, ack [07:30] Hm. Why is it defaulting to UXA? The packaging branch clearly defaults to sna. [07:37] RAOF, can you reproduce defaulting to UXA locally? [07:58] RAOF: on ati, we have another issue in fact making unity-system-compositor segfaulting [07:58] RAOF: libEGL warning: Could not open driver /usr/lib/i386-linux-gnu/egl/egl_gallium.so (libOpenVG.so.1: cannot open shared object [07:58] file: No such file or directory) [07:58] didrocks: Oooh, really? [07:58] That's odd. [07:58] I clearly had libOpenVG installed. [07:58] RAOF: so, we should install libopenvg1-mesa I think [07:58] Yeah. [07:59] RAOF: but maybe the deps don't take it [07:59] let me see [07:59] Actually, that shouldn't be fatal; I think it should go on and load egl_dri2.so [08:01] RAOF: so libegl1-mesa:i386 9.2~git20130628.g6b676e6-0ubuntu0+mir4-jenkins86saucy0 [08:01] but libopenvg1-mesa:i386 9.1.4-0ubuntu3 [08:01] RAOF: weird that shlibdeps doesn't force the latest version [08:01] RAOF: we need libopenvg1-mesa from the ppa, right? [08:02] We shouldn't, no. [08:02] libopenvg1-mesa is installed, then? [08:03] RAOF: right, but not the version from the ppa [08:04] (you didn't list it in the things to upgrade) [08:04] can the ABI has changed? [08:04] Yeah, I didn't expect it to be necessary; libopenvg should have a fixed ABI. [08:04] RAOF: as soon as the slot is free, I'll retry forcing the upgrade to latest [08:05] I was thinking it was the race again, but longer timeout needed on ati [08:05] but not the case :) [08:05] didrocks: Yeah, thanks. I'm surprised. Looks like this might be a libegl1-mesa-drivers bug (lack of appropriate depends: on libopenvg1-mesa) [08:05] RAOF: sounds likely the cause, will keep you posted (I think the slot will be free in 10 minutes [08:05] ) [08:07] Test-building patch series to ensure they build at each commit is annoying. [08:12] RAOF: ok, now we get: [08:12] ERROR: /build/buildd/mir-0.0.7+13.10.20130719ubuntu.unity.next/src/server/graphics/gbm/gbm_cursor.cpp(46): Throw in function mir::graphics::gbm::GBMCursor::GBMBOWrapper::GBMBOWrapper(mir::graphics::gbm::GBMPlatform&) [08:12] Dynamic exception type: boost::exception_detail::clone_impl > [08:12] std::exception::what: failed to create gbm buffer [08:20] didrocks: Hm. What platform is that? [08:20] didrocks: Obviously it's ATI, but what chipset? [08:20] RAOF: 05:00.0 VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Cedar [Radeon HD 5000/6000/7350/8350 Series] === alan_g|EOD is now known as alan_g [08:20] RAOF: do you want a ssh access to the machine? [08:21] didrocks: Sure, why not. [08:21] RAOF: prefered ssh public key? ;) [08:21] From launchpad? [08:21] oui have 2! [08:22] seems you are someone who can't decide! ;-) [08:22] chris@Ed [08:22] ok, grabbing that one :) [08:58] duflu: alan_g: @remove-graphics-viewable-area, my concern about using *Region is that "Region" has a particular meaning in graphics (an arbitrary collection of pixels) and we are not providing any related operations [08:58] alf_: Maybe so, but I feel it's closer. So another word... ? [08:58] * duflu reserves the right to not think with so little time left in the week [08:59] alf_: duflu: with members "clip_to_screen" and "make_fullscreen" I have a feeling that "screens" might be part of the role/name [09:00] alan_g: perhaps we should not have *Boundaries/Regions, but completely different names for the too interfaces [09:00] s/too/two/g [09:00] ScreenLayout? [09:00] alf_: I think Region is most accurate. Qualify it as FooRegion if you like. The word "Boundaries" makes me imagine a set of lines in space, where the line equations are most important, rather than their contents [09:01] duflu: that was the intention, the boundaries are important, not the contents for these interfaces [09:01] Desktop? [09:02] A "Desktop" is the set of all displays rectangles... [09:03] My "desktop" is the thing that holds up tea, monitors, paper and pens... [09:04] desktop->clean(later) [09:04] ;) [09:06] People generally understand "is it on the desktop", and the concept extends to multiple monitors per desktop [09:07] If you want to steal X/Compiz terminology then it's a "Screen", but that's confusing terminology when "Screen" encompasses multiple monitors [09:08] Although, arguably, the desktop could be transformed to take up less than the monitor :P [09:09] Should we avoid using "screen" then and use, say, "output" [09:09] alan_g: duflu: I think that the Input(Coordinate?)Boundaries interface makes sense from the input subsystem's (the consumer's) perspective. [09:10] alan_g: duflu: I am not completely happy with SurfaceBoundaries, give the actual operations that it has, as Alan pointed out. [09:11] I would still hope for a name that describes the area/region, rather than just its edges [09:11] Area? [09:13] DisplayMap ... clip_to_display() [09:15] Map has become an overloaded word, methinks [09:19] woot, not a single gpu hang with sna after bios upgrade on x220 while running a complete phoronix test suite [09:20] tvoss_: There's a new BIOS? Or you were out of date? [09:20] duflu, bios is from May 2013 [09:21] tvoss_: All other possible variables eliminated? :) [09:21] duflu, yes, I'm staring at this for quite some time now ;) [09:23] tvoss_: Oh, I seem to have installed that BIOS already. No wonder mine has worked... (?) [09:23] duflu, I was actually down to inspecting individual registers of the gpu [09:23] Eek [09:25] duflu, but: intel_gpu_top is really an awesome tool === tvoss_ is now known as tvoss|benchmark_ [09:34] alan_g: duflu: OK, so how about, InputRegion::bounding_rectangle(), InputRegion::confine(Point) and DisplayOutputLayout::clip_to_output() DisplayOutputLayout::make_fullscreen()/resize_to_output() [09:34] alf_: Sounds reasonable [09:35] Though I'm not sure what a "Layout" is. Didn't get that far [09:35] If you make the change soon, I will abstain on it [09:35] "size_to_output()"? [09:36] DisplayOutputLayout seems a bit wordy [09:36] alan_g: DisplayLayout? [09:37] DisplayArea? [09:37] Oh. Yes it was so wordy I didn't understand "DisplayOutputLayout". I did understand "DisplayLayout" (or would "OutputLayout") [09:38] DisplayLayout would be OK [09:38] OutputRegion? [09:39] If it works for input, then why not for output? [09:42] alan_g: I don't think that DisplayArea/OutputRegion communicate that there are multiple outputs. InputRegion doesn't care about that (yet). Which brings up another point, that these interfaces may need to change names as the needs of shell/input change. This is just a first cut. [09:42] DisplayLayout would be OK [09:46] alan_g: duflu: ok, so it's InputRegion and DisplayLayout (although I still think *Boundaries makes sense, since all these operations are about clipping/checking against/sizing to some boundary) [09:46] alf_: OK. Please resubmit so I don't have to do any more reviews today :) [09:47] ... so that it's at least not blocked on me [09:47] alf_: Something can have boundaries without being named for them [09:47] duflu: Feel free to pre-abstain :) [09:50] alan_g: Sure, but as I mentioned above, in this case the boundaries are more important than that something in terms of what the consumer needs. Anyway, I think both approaches are reasonable, so I am not fussed :) [09:52] tvoss|benchmark_: that wasn't enough: https://launchpadlibrarian.net/145330493/buildlog_ubuntu-saucy-armhf.mir_0.0.7%2B13.10.20130719.1ubuntu.unity.next-0ubuntu1_FAILEDTOBUILD.txt.gz [09:52] tvoss|benchmark_: at least, it doesn't hang, but unit-tests are segfaulting [09:57] tvoss|benchmark_: duflu: alf_: any of you wanting to ack https://code.launchpad.net/~didrocks/mir/disable-unit-armh-tests/+merge/175786? [09:58] as per discussion, we disable the tests right now, then before a week pass by, tvoss|benchmark_ promised to work on them again (with some platform-api trick) [09:58] didrocks: Nothing to see yet :) [09:58] duflu: click on the commit :p [09:58] Also.. don't we usually fix failing tests? [09:59] duflu: well, it's been 3 weeks they are failing and preventing us for pushing mir [09:59] didrocks, +1 [09:59] duflu: so no, you don't usually fix failing tests ;) [09:59] thanks tvoss|benchmark_! I'll rerun a build again once merged [10:00] duflu: so the new deal is "no mir by default until this is fixed" [10:00] btw, let me add that on the spreadsheet === tvoss|benchmark_ is now known as tvoss_ [10:01] tvoss_: mind approving? (not tvoss|benchmark*ed* now? ;)) [10:03] didrocks: I will let the western hemisphere decide. I need to start winding up... [10:03] duflu: enjoy your week-end! [10:03] didrocks, want a happrovie? [10:03] tvoss_: I would love one! :) [10:04] didrocks, there you are [10:05] thanks! [10:08] * duflu also means Greece and Germany even though they're not in the western hemisphere [10:09] Bah, details [10:14] didrocks, https://bugs.launchpad.net/platform-api/+bug/1203003 [10:14] Launchpad bug 1203003 in platform-api "The platform api should offer an init method that resolves all required symbols and indicates whether the platform is available or not" [Critical,New] [10:14] tvoss_: sounds like the right strategy, thanks! [10:16] didrocks, https://bugs.launchpad.net/mir/+bug/1203004 [10:16] Launchpad bug 1203004 in Mir "Tests disabled on armhf" [Critical,New] === om26er is now known as om26er|away === om26er|away is now known as om26er [11:46] tvoss_: mir built on armhf [11:46] didrocks, \o/ [11:46] time for unity-system-compositor ;) [11:46] didrocks, \o/ [11:46] heh ;) === alan_g is now known as alan_g|lunch === alan_g|lunch is now known as alan_g [13:24] kgunn: we have a saucy version of the ihv tests now http://certification-static.canonical.com/checkbox-ihv/ [13:28] robotfuel: hey thanks [13:29] kgunn: I'll run that on some of the systems today in the lexington lab [13:30] robotfuel: good stuff & checkbox is single sign on right? so anyone could run their machine? [13:32] kgunn: I think the machine has to be added to hexr first to show up on c3, but anyone can run checkbox and view the results locally. it also produces an xls spreadsheet of the results. [13:33] kgunn: with the ihv tests there is no need for a network connection [13:35] kgunn: to run it you unpack the tarball cd in to the checkbox-ihv directory then run start_testing [13:35] kgunn: it installs all the required packages and runs checkbox's graphics tests === mpt_ is now known as mpt === alan_g is now known as alan_g|tea === alan_g|tea is now known as alan_g === om26er is now known as om26er|afk === bschaefer_ is now known as bschaefer === om26er|afk is now known as om26er [19:03] * kdub is tempted to just keep overriding MirEvent for sending messages to client [19:03] but thats probably already overridden too much [19:16] kdub: I think so! I wish we didn't use it at all [19:16] for sending messages to the client [19:16] kdub: The thing is, most messages to the client [19:16] will eventually need some sort of subscription mask [19:17] well maybe not :p [19:17] but I think a lot do, like, the client can "subscribe" to display changes [19:17] and this way, I think rather than use the mechanism we do now of the MirEvent, which is sort of half out of band [19:18] to our protobuf IRC, even though it is [19:18] on the same socket [19:18] we can model it as a call [19:18] with multiple responses [19:19] and then on the client side, I think using [19:19] more fine grained delegates, i.e. handle_input(MirEvent), handle_display_changed(information), handle_focus_lost [19:19] etc. is just good loose coupling [19:20] even though we don't have many events to the client so far [19:20] it already causes noise in some of the tests [19:20] where for example, a test which wants to match client input events [19:20] i.e. "this client gets an A key" [19:21] has to either mute, or expect, irrelevant events like [19:21] got Focus, lost focus (because it's a 3 client test) [19:21] or maybe later like, display changed! [19:22] and then, what is qtubuntu going to do? [19:22] qtubuntu_handle_input(Event) switch(event->type) case mouse_event: qtubuntu_handle_mouse_event, case key_event: bla bla. case surface_configuration_event: switch event->surface->attrib... [19:23] blablabla [19:23] so I think we should just have [19:23] seperate delegates [19:33] kdub: i think i know why someone did it...but isn't there another way we could flip buffers on our mir gl rendering ? [19:33] other than glreadpixels [19:33] glreadpixels not good for deferred arch's....it makes you wait for the entire pipe to finish rendering [19:34] when, you could run away and do more stuff [19:34] also...there is some overlap in processing on the gpu that can be gained (not just the exploiting the cpu side of things) [19:35] on a deferred mode arch....you can processing geometry & stuff...of the next frame, while your rendering out the previous frame [19:35] thots ? [19:35] tvoss_ was seeing some perf hits on galaxynexus with unity8....and i was poking around and noticed that we're doing glreadpixels [19:36] [21:33] kdub: i think i know why someone did it...but isn't there another way we could flip buffers on our mir gl rendering ? [19:36] err.. [19:36] sorry, slippage of the mouse [19:37] (kid on my arm, ..) [19:39] kdub: tvoss_ http://forum.imgtec.com/discussion/comment/7821#Comment_7821 [19:42] Where are we doing [19:42] glReadPixels! [19:42] besides the snapshot functionality. [19:44] racarr: hmmm...now you're making me wonder if i know what i'm talking about :) [19:44] maybe that's where it is [19:45] gl_pixel_buffer.cpp [19:46] kgunn: I think it's only for snapshots [19:46] 90% confidence index ;) [19:46] racarr: thanks...i'll poke a bit more to ensure [19:47] I heard about some galaxy nexus performance problems though [19:47] so I guess something is going on [19:47] racarr: it does make sense tho now that you say it....there's a bunch of copying etc going on [19:48] racarr: yeah...i don't think anyone has tried demo shell on nexus4 (if you were feeling jumpy :) [19:48] I did last week when it didn't really do anything [19:48] and it was fine [19:48] racarr: and yeah...i know way too many little bastard issues on omap4460 [19:48] I will try again soon ;) [19:48] to really worry too much [19:48] kind of fell out of sync with gerry this week because I missed thursday morning and was distracted by the stress-test wednesday morning [19:49] racarr: did you launch an app ? and if so, any perf delta appearant ? [19:49] racarr: he mainly bug fixed and put quality goodness into app management [19:49] so the ui perf prob wouldn't have changed much [19:50] It wasn't apparent on nexus 4 or desktop [19:50] but I also didn't really push it XD [19:50] at the time it was like "ok lets figure out why nothing shows up on screen..." [19:50] *hours of furious recompiling* [19:50] greyback: ^ [19:50] "Yay it shows up on screen! *flicks aimlessly for a minute* [19:50] :p [19:50] wrt nexus 4 [19:52] greyback: in case you read scrollback...would be interesting to know...how often you're doing snapshot for apps [19:53] that could be a very bad thing (glreadpixels + a copy) [20:11] kgunn, there is another way [20:11] (sorry for the delay, was eating lunch) [20:12] when Mir is shiped with the next release. How is it going to be with ati proprietary drivers ? [20:12] kgunn, nexus4 has similar problems with long glReadPixels performance [20:26] "There are only two hard things in Computer Science: cache invalidation and naming things." [20:26] haha, I feel like this speaks to our [20:27] scenegraph/synchronization data store topic *waves hands* [20:29] hah [20:31] kdub: so that glreadpixels is your trigger for flipping ? or rather is it just for snapshot support (in which case this is ok)....and i tend to agree with racarr that's what this is [20:32] enlighten if i'm wrong [20:32] kgunn, i'm unsure what the question means [20:33] kdub: sorry...i am refering to the use of glreadpixels in the code [20:33] i am thinking its only relevant to when a shell client wants a snaphot [20:33] (e.g. it shouldn;t be called every frame) [20:33] right, of course [20:33] the normal render pass does not hit that function [20:34] MichaelP: in 13.10 we would be delighted to see proprietary driver support appear from the various vendors [20:34] but that's not ours to promise [20:36] kgunn: yeah i know.. distro's like arch you either have to downgrade xorg or use ati beta drivers... [20:37] now whats mir supose to do that xorg don't ? [20:38] MichaelP: be younger :) [20:38] seriously though....that's one part [20:39] being a bit more focused w/o legacy of decades of special interest add-ons will help maintainability [20:39] Sometimes younger is dummer !!lol [20:39] and this is all a journey [20:39] if you ask what xmir will do...nothing [20:39] its about a conservative step of introducing mir into distro, yet with real road miles on it [20:40] its easy to keep designing, tweaking, etc...but maturity cannot be delivered like a feature [20:41] One thing that mir is supposed to do that X doesn't is make sense from the ground up :p [20:41] like Orsen Wells said of Paul Masson wine (and if you don't know what that is...then you're the younger of us ;) [20:41] and shine some sense up the whole stack, as opposed to [20:42] pushing nonsense and hacks up the whole stack :p [20:42] from what i was reading on google about mir.. was sound like with in the next few releases of other distro's everyone eles wants to switch to wayland [20:43] MichaelP: can't learn everything from reading [20:43] Im downloading 13.10 right now to check it out [20:44] MichaelP: awesome! [20:44] kubuntu said kwin doesn't support mir [20:44] MichaelP: i heard that too... [20:45] the beauty of open source... [20:45] MichaelP: another thing i think mir is bringing beyond xorg [20:45] But yet i watched a video on google.. with unity.. gnome shell.. kde xcfe useing mir [20:46] is that its been built around having an answer on mobile & desktop... [20:46] xmir :) [20:46] using android drivers and then of course mesa [20:47] another big decision is server side alloc'd buffers [20:48] I have trouble with the free ati drivers... i use HP laptop... then to watch music video's i plug into my hdmi tv.. So if im in like say gnome.. i have trouble sizing the screen.. on the hdmi [21:13] racarr, why do input and events come back to the clients with the same callback? [21:14] is it driven by convenience or necessity? :) [21:49] kdub: That's what I was talking about earlier [21:49] I think it [21:49] 's wrong [21:52] kdub: If you look 2.5 hours up in scrollback [21:52] I wrote a bunch on it [22:14] racarr, ah [23:55] * racarr sets clock ahead 5 minutes [23:55] * racarr beer top flies with a *clang*