/srv/irclogs.ubuntu.com/2014/05/15/#ubuntu-mir.txt

RAOF:)00:08
bschaeferracarr__, hey, are you still around? Weird problem im seeing with mir(sdl)/finger events01:24
bschaeferracarr__, i get the first finger down, the motion events, but when i left my finger up, not finger up action?01:24
RAOFOdd.01:26
bschaeferyeah, i think ill have to dig into mir/android input stack... not sure where its going missing from01:27
bschaeferandroid doesn't handle mouse up/down very well either (up is just an empty event with no state)01:28
bschaeferish01:28
bschaeferRAOF, im assuming the finger events are working just fine on unity8, so im assuming im handling things incorrectly in SDL01:28
dufluSpeaking of which, I discovered last night that Android-input converts two-finger drags on my ThinkPad to what appears to be 3-finger events. Weird.01:29
bschaefervery strange01:29
bschaeferthe fun part, if i do a second finger down, i get a finger up event01:30
dufluEither that or there's a tool type demo-shell is supporting by accident :)01:30
RAOFduflu: Yeah, the android input stack does _weird_ things to touchpads.01:30
dufluRAOF: Also "click" is not button 1 :)01:31
RAOFbschaefer: Huh. My input acceptance tests don't check for any finger-up events. I should probably add that.01:31
RAOFduflu: It should be if you have exactly one finger down?01:31
RAOFduflu: At least, it was for me.01:31
bschaeferRAOF, well im assuming they are working, as with unity8 you can press icons :)01:31
dufluRAOF: Yeah it varies significantly between laptops :S01:31
dufluRAOF: Oh my memory failed me... it's different: https://bugs.launchpad.net/mir/+bug/119405701:32
ubot5Ubuntu bug 1194057 in Mir "mir_motion_action_move not received for thinkpad buttons (only for the clickpad)" [Low,Triaged]01:32
racarr__bschaefer: Are you looking in to silvns issue?01:36
racarr__I was looking in to his touchscreen issue and now he01:36
bschaeferracarr__, no my own now :(01:36
racarr__is on a similar issue01:36
racarr__oh01:36
bschaeferyeah01:36
racarr__he has the same issue01:36
bschaeferhe poked me about it, but i swear he had up/down fine01:36
bschaeferwith 1 finger01:36
bschaeferracarr__, also demo finger paint doesn't seem to want to work01:37
bschaefero also... im using an N701:37
bschaeferkilling unity8, and using the mir server on its own (not sure what "unsupported" with the N7)01:37
duflubschaefer: https://bugs.launchpad.net/mir/+bug/123950101:37
ubot5Ubuntu bug 1239501 in Mir "Nexus 7 does not respond to touch" [High,Expired]01:37
bschaeferwell01:37
bschaeferthat could explain a few things01:38
* bschaefer needs to get a supported touch N<number>01:38
duflubschaefer: Reopen?01:38
bschaeferduflu, well the N7 is an unsupported device01:38
bschaeferduflu, but yeah i can reproduce these issues01:38
bschaefer3 finger touch isn't moving windows01:39
duflubschaefer: Sure, but Mir is open source. All real bugs are valid just lower priority if not official :)01:39
bschaeferduflu, let me double check by getting trunk dev mir01:39
bschaeferduflu, as im using whats in main 14.04? or 14.1001:39
duflubschaefer: Your Ubuntu release presently won't matter if you're compiling Mir from source01:43
dufluLet's see how long that lasts... hopefully a while01:43
bschaeferduflu, right, but i was using main mir-demos for my testing01:43
bschaeferso i wanted to double check it was still happening on trunk mir01:43
duflubschaefer: trusty is only one point release behind utopic: https://launchpad.net/ubuntu/+source/mir01:44
bschaeferyeah im on trusty: 0.1.8+14.04.20140411-0ubuntu101:44
bschaeferfor libmirclient701:44
bschaeferduflu, that bug was closed a bit ago :), soo yeah01:45
duflubschaefer: Not closed. Only expired because it stopped happening for no apparent reason :)01:45
bschaeferduflu, yeah, i mean it stopped happing for no apparent reason a bit ago, and i would have had that fix01:45
bschaeferusing mir-demos, soo mir trunk wont do much :(01:46
duflubschaefer: There was no fix. The bug came and went even before it expired01:46
dufluSo I guess it's come again01:46
bschaeferduflu, hmm i wonder if its because everyone stopped using the N7?01:46
bschaeferyeah01:46
bschaeferduflu, o and thanks for the bug link :), i should have attempted to check bugs01:47
=== chihchun_afk is now known as chihchun
dufluRAOF: Oh the two-finger window Alt+dragging on my touchpad looks like it might be an accidental feature. And one I might make use of :)03:11
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
RAOFGrr. You can't actually change an interface at all and preserve ABI, can you?08:25
RAOFOh, well. EOD.08:25
=== dandrader is now known as dandrader|afk
duflualf_: I'm off to make dinner then back later. In the mean time, consider this problem: https://bugs.launchpad.net/mir/+bug/1317370/comments/910:02
ubot5Ubuntu bug 1317370 in Mir "[regression] [BufferQueue] BufferQueueTest.compositor_never_owns_client_buffers is very slow (up to 30 seconds) on fast machines" [High,Triaged]10:02
dufluIt's an interesting case. I hope I missed something.10:02
alan_gdednick: AIUI a helper can only have one active trust session the session mediator? So why does it need a client side handle to a trust session? Isn't the connection a sufficient identifier?10:04
dednickalan_g: when i originally wrote it, there wasn't actually that limitation, and the handle hung about. I'm not exactly sure if we want that limitation either, at the moment.10:10
dednickalan_g: but anyway, i think it adds a bit of separation of "ideas".10:11
dednickalan_g: i mean that i'm not sure we want the one-to-one limitation in the long run.10:14
alan_gdednick: Ack10:14
alan_gtvoss: opinion? ^^10:15
tvossalan_g, dednick we want one helper to be able to handle multiple trust sessions10:16
tvossfor example: multiple content picking operations10:16
dednicktvoss: at the same time?10:18
tvossdednick, yup, they are stateful10:18
dednicktvoss: ok. phase 2?10:18
alan_gtvoss: from the same connection? Or can it use a connection for each?10:19
tvossdednick, yup10:19
tvossalan_g, connection as in ordinary Mir connection?10:19
alan_gtvoss: Yes. But I'm starting to wonder if it is a new type of connection and not the same thing we have to manage surfaces.10:20
tvossalan_g, that's why we introduce trusted sessions as I understand it10:20
=== dandrader|afk is now known as dandrader
alan_gtvoss: but the proposed API implies it is owned by a connection (in the same way that surfaces are). Not that it is a different type of connection.10:22
tvossalan_g, why would it need to be owned by something different than a connection?10:22
alan_gtvoss: currently we have one type of connection. It manages surfaces. The proposed API says it manages surfaces and an optional trust session. Why can't we have one connection type for surfaces and one connection type for a trust session?10:25
tvossalan_g, why would adding another type of child (surfaces and trust sessions) require a new type of parent?10:27
tvossalan_g, also: a trust helper might want to have surfaces as well10:27
tvossalan_g, which have to be added to trust sessions10:27
alan_gWhat is the relationship between any surfaces and any trust session(s)?10:27
alan_gbut trust sessions don't know about surfaces - only processes10:28
tvosswell, they should know about surfaces ultimately10:28
tvossalan_g, a surface belongs to a connection but can be referenced by a trust session10:29
alan_gShould a trust helper surface be associated with a specific trust session or all (or some)?10:29
tvossalan_g, at most by one10:29
alan_gtvoss: at one time or over the duration of the connection?10:34
dednickalan_g: this is all "ultimately". not for this iteration.10:35
dednickjust to remind :)10:35
alan_gdednick: I know - but I'd like the conceptual model to be right before we publish APIs10:36
tvossalan_g, at any point in time, a surface can be referenced by at most one trust session10:39
alan_gBut it can be removed from one and added to a different one?10:40
tvossalan_g, yup, that's certainly possible10:40
dednickalan_g: can you take a look at: https://code.launchpad.net/~nick-dedekind/mir/test_new_fds_for_trusted_clients when you get a chance?11:29
dednickWas taking a look at the problem with child sessions not stopping. The test creates a server and client. the client creates some fds, which another client connects with.11:29
dednickThe child that connected to the fd is crashing when shutting down, and the server is not getting a disconnect for it.11:29
dednickstack for the client crash: http://pastebin.ubuntu.com/7467385/11:31
alan_gdednick: ack. Was taking a first look at ~nick-dedekind/mir/trusted_sessions/+merge/218975 before getting into the issue you mentioned yesterday11:31
dednickalan_g: ok. thanks11:32
dednicki'm taking a guess, but i think the server may be closing the fd's which were opened by the parent client.11:34
dednickwhen the parent client connection is released11:35
alan_gdednick: Hmm. The server closes FDs once it has sent them (but they're open in the other process). I don't think it tracks them after that.11:37
dednickalan_g: sent them? over to the parent client over ipc?11:37
alan_gack11:38
dednickalan_g: hm. but they won't be open in the other process by then.11:38
alan_gdednick: Oh yes they are. (The FDs not the connections)11:39
alan_gwe've been doing it for years.11:39
dednickalan_g: i think i'm getting confused. but we only start the child client process after we've got the response from the server for the new_fds_for_trusted_clients.11:41
alan_gYes, but you already have the FD in your process.11:41
* alan_g has to go make lunch - BIAB11:42
=== alan_g is now known as alan_g|lunch
=== dednick is now known as dednick|lunch
dandraderwhen a surface is resized ( mir::scene:surface::resize() ) should the client recreate his eglSurface or do anything at all?12:24
alf_dandrader: no, unless you need to draw differently somehow (otherwise everything will just be scaled)12:33
dandraderalf_, ok12:33
alf_dandrader: scaled => assuming you are drawing using relative sizes (e.g. using the -1,1 coordinate system of GL)12:34
=== greyback is now known as greyback|food
=== dandrader is now known as dandrader|lunch
=== tedg is now known as ted
=== alan_g|lunch is now known as alan_g
=== ted is now known as tedg
alan_gdednick: @https://code.launchpad.net/~nick-dedekind/mir/test_new_fds_for_trusted_clients - I don't understand what's going on with SharedRegion. Are you using shared memory to copy the FD ids from one process to another without migrating the FDs themselves?13:15
=== greyback|food is now known as greyback
dednickalan_g: ah crap. forgot that the other clients need to be created by the parent client in order to get the FDs13:17
dednicki mean the processes need to be spawed from the parent client.13:17
alan_gThat's certainly the easiest way.13:17
alan_gShould I stop looking?13:18
dednickthe shared mem was just to get the fds across process. but being used incorrectly.13:18
alan_gI mean is the problem you mentioned yesterday likely the same thing?13:18
dednickalan_g: hm. the problem yesterday was with the child processes being created by the helper, so probably not...13:19
alan_gOK. I'll start by writing a test for what should be happening.13:20
dednickalan_g: thanks. easier than me stabbing in the dark about what's supposed to happen :) I don't know much about fds apparently13:22
alan_gnp13:23
=== dandrader|lunch is now known as dandrader
dandraderalf_, is there a significant overhead in resizing a surface?13:25
alf_dandrader: we need to allocate new graphics buffers, so yes in a sense, but it all depends on your particular scenario13:27
dandraderalf_, flipping the dimensions. e.g. from 768x1280 to 1280x76813:28
alf_dandrader: it depends on the device and your requirements whether its ok, so I guess you need to try it out and see if it's acceptable13:33
alf_dandrader: do you have alternatives to flipping the dimensions (flipping rendering instead?)13:34
dandraderalf_, yes I do13:34
dandraderalf_, so overall you think it's better to flip rendering than to flip the dimentions (surface resize)?13:34
alf_dandrader: flipping rendering should be near cost free for the GPU (just a different transformation matrix)13:36
greybackalf_: so right now there's no support for properly resizing a Qt app in Mir (and platform-api & qtubuntu)?13:51
=== pete-woods is now known as pete-woods-lunch
=== alan_g is now known as alan_g|tea
=== alan_g|tea is now known as alan_g
alf_greyback: mir supports resizes14:39
greybackalf_: ok, so the qtubuntu just needs to support it14:40
greybackwhich is probably doesn't right now14:40
alf_greyback: (you can play with resizes using mir_demo_server_shell)14:40
=== pete-woods-lunch is now known as pete-woods
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== alan_g is now known as alan_g|EOD
greybackmterry: ping16:42
mterrygreyback, heyo16:42
greybackmterry: hey, can you explain me how unity-system-compostor starts. Lightdm starts it right? Is there a configuration applied (i.e. env vars set for USC)16:43
mterrygreyback, it is started by lightdm.  With some env vars it sets yeah.  And also, the configuration can change the command line used (and in fact ubuntu-touch-session does install a usc-wrapper file that sets a little more env for touch stuff)16:44
greybackmterry: are any of those env vars critical for USC to start by any chance?16:45
mterrygreyback, probs?  What's the problem?16:45
greybackmterry: http://pastebin.ubuntu.com/7468671/16:46
greybacknote it's with a custom build of mir16:46
mterrygreyback, oh yeah, you can't really start manually16:46
mterrygreyback, lightdm passes it a couple fd's for communicating with it16:46
greybackaha16:46
mterrygreyback, specified on command line16:46
greybackmterry: so "start lightdm" is the right way to start it?16:47
mterrygreyback, yeah16:47
mterrygreyback, lightdm loves its private fds16:47
greybackhow friendly16:47
greybackmterry: so I get: http://pastebin.ubuntu.com/7468684/ <- I'm missing the "ubuntu" session somehow. Where should that file be?16:49
mterrygreyback, hm, what env are you running in?  I think you're missing ubuntu-session (or ubuntu-touch-session)16:50
greybackmterry: on the phone. Some packages might have been removed (major surgery and all)16:50
mterrygreyback, yeah install ubuntu-touch-session16:50
mterrygreyback, it will change the default session to "ubuntu-touch" and install the .desktop file for it16:51
greybackack16:51
greybackmterry: there we go, that fixed it, thanks16:51
mterrynice16:51
greybackslightly worrying how easily that broke...16:52
mterrygreyback, well stop removing packages!  :)16:52
greyback:P16:54
kdub_any more reviewers on https://code.launchpad.net/~kdub/mir/post-if-optimizable/+merge/21799117:17
=== beidl_ is now known as beidl
=== tedg is now known as ted
racarr__This is where all the fun is19:49
racarr__kdub_: Ill review!19:49
racarr__just saw backlog lol19:49
racarr__p.s. looking for reviewers on cursor-spike-phase-319:49
racarr__ok I swear I have seen so many diffs that are exactly 1334 line19:50
racarr__s19:50
racarr__it seems to be a common stopping point19:50
racarr__anpok: BECAUSE ON FREENODE THE PEOPLE MAY FINALLY CONTROL THE MEANS OF PRODUCTION19:57
racarr__*waves flags*19:57
anpok:)19:58
=== beidl_ is now known as beidl
racarr__https://code.launchpad.net/~mir-team/mir/cursor-spike-bonus-phase-correct-gbm-cursor-hide/+merge/21976422:00
racarr__easy review22:00
racarr__with no dependencies :)22:00
RAOFracarr__: I hope there's going to be a cursor-spike-lightning-round at some point.23:30
racarr__RAOF: Lol23:35
racarr__ill use that for the cleanup branch at the end23:35

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