/srv/irclogs.ubuntu.com/2012/04/04/#ubuntu-x.txt

brycehI did a diff between the upstream 1.11 git tree's include/ dir and ours (before applying debian/patches):  http://paste.ubuntu.com/913875/00:00
brycehdidn't spot any commonalities with your valgrind output.  mostly it's cnd's touch bits00:01
cndRAOF, valgrind ioctl warnings are either: correct, or they need wrappers to tell valgrind how they really work00:02
cndRAOF, I just added wrappers for the uinput ioctls to the precise valgrind00:02
cndyou might be interested in doing the same for drm?00:03
RAOFcnd: Apparently upstream already has; in the libdrm sources - we just need to build-depend on valgrind to get those intrinsics added.00:03
cndRAOF, that seems odd00:03
cndthe fixes should be in the valgrind sources00:04
cndI suspect libdrm is either really fixing the issues, or just papering over them00:04
RAOFAh, no; what it's doing is using some intrinsics to annotate stuff for valgrind - VALGRIND_MAKE_MEM_DEFINED(bo_gem->gtt_virtual, bo->size) and such.00:06
RAOFI think they've *also* fixed an issue or two.00:06
cndRAOF, ahh, that looks like it makes some sense for drm00:07
cnddrm is funny in how it uses memory :)00:08
cndRAOF, btw, the ioctl warnings at device init time are red herrings again, I'm fairly certain00:09
RAOFHm.00:09
cndthe evdev ioctls probably need wrappers too00:09
cndjust like the uinput ones00:10
cndvalgrind assumes that all ioctls take an argument, and the argument is a pointer to valid userspace memory00:10
RAOFThere don't seem to be any suspicious writes, which is annoying.00:10
cndyou have to manually tell it if a specific ioctl either: doesn't pass an argument, or passes an argument by value00:11
RAOFBut, given I can reproduce this easily, I guess it's time to go poking around in the core.00:12
RAOFTo the bat-gdb!00:12
cndRAOF, what trackpad are you thrashing on?00:12
RAOFA synaptics one.00:12
cndRAOF, thrashing with one finger?00:12
RAOFTwo finger scrolling was the last thing I reproduced it with.00:12
cndRAOF, does xinput list a touch class for your touchpad?00:13
cndhooray! a one-liner in the server fixes compiz touchscreen interactions00:14
RAOFcnd: Hm, how do I get xinput to do that?00:15
cndRAOF, xinput list <device name|id>00:15
RAOFHm.00:15
RAOFEBADATOM00:15
RAOFFor some reason that fails.00:15
cndI've started seeing that too...00:16
cndit's next on my list of bugs :)00:16
cndI was hoping it was just me00:16
RAOFIt reports 8 classes though, from which I think you can work out if there'll be a pair of touch classes or not.00:17
cndRAOF, install utouch-frame-tools00:17
cndthen run utouch-frame-test-x11 <window>00:17
cndwhere window is some window other than the root window00:17
cndit will spit out all devices that have touch classes00:17
RAOFTwo touches00:19
RAOFhttp://paste2.org/p/196582600:19
cndok00:21
cndcool00:21
cndso what you are seeing could be multitouch related00:21
cndonce I fix this bug, I'll thrash on my magic trackpad to see if I can cause any misbehavior00:21
cndRAOF, is this on a netbook or something else that is less performant?00:22
RAOFNope, it's on my shiny sandybridge i500:22
RAOFHowever, it's on my shiny sandybridge i5 while running under valgrind, so you may assume it is significantly non-performant :)00:22
RAOFAnd, as you might note in the valgrind backtrace, the initial SIGSEGV comes from the ErrorF in mieq saying “[mi] EQ overflowing”00:27
cndRAOF, oh, you're running under valgrind00:31
cndI'm guessing that you are hitting the queue overflow because it's so slow00:31
cndis it crashing?00:31
RAOFYes.00:32
RAOFYeah, that's exactly what's happening.00:32
RAOFIt's queue overflowing.00:32
cndRAOF, oh, I bet you're hitting the same bug I was seeing yesterday :)00:32
RAOFBut! It's SEGVing when trying to print the queue overflow.00:32
cndif you hit ErrorF within signal context under valgrind00:32
cndthen it will segfault00:32
cndI think sprintf is technically signal unsafe00:32
RAOFBut only under valgrind?00:32
cndit seems to work when not under valgrind00:33
cndthough that may mean that valgrind always crashes, while in real world it only crashes 1/1000 times00:33
RAOFRight.00:33
RAOFIt is indeed not listed as signal safe00:33
RAOFThis lacks awesome.00:34
cndyeah...00:34
cndI had to stop to look at other things00:34
cndit should probably be brought up on xorg-devel at the very least though00:34
RAOFYes.00:34
cndRAOF, the sprintf is only used to print the timestamp in the X log00:35
cndso we may be able to get away without it, with some manual number printing00:35
cndwhen I commented the sprintf out, I didn't get the crash anymore00:35
RAOFAha.00:36
RAOFHm.00:38
RAOFPractically nothing done in ErrorF is signal-safe.00:38
RAOFfwrite: no, memcpy: no, fflush: no.00:39
RAOFOooh, and if you're unlucky it'll try to realloc00:39
RAOFvsnprintf: no.00:42
cndfun00:42
cndRAOF, maybe there should be a separate code path for ErrorF if in signal context?00:43
RAOFQuite possibly.00:43
cnddon't attempt to do anything other than just write directly to the log file00:43
cndactually, maybe a separate function altogether00:43
RAOFYes.00:43
cndand then in ErrorF, merely print "you're dumb" if in signal context00:43
cnd:)00:43
cndbecause ErrorF attempts to print a vararg printf style message00:44
cndwhich I think may be impossible without reimplementing vsprintf in signal safe code :)00:44
RAOFProbably, yes.00:46
cndRAOF, bryceh, Sarvatt: I have a one-liner fix for compiz touchscreen interactions for xorg-server00:46
cndany issues if I upload now?00:46
RAOFNone from me.00:46
cndfixes bug 97298500:47
ubottuLaunchpad bug 972985 in xorg-server (Ubuntu) "XQueryPointer does not return button 1 set when touch on touchscreen is active" [High,In progress] https://launchpad.net/bugs/97298500:47
RAOFAlthough what would be much more awesome is to just have a gorram input *thread* and stop futzing around in the SIGIO context.00:47
cndheh00:47
RAOFWe call *so much* code in that context.00:47
RAOFIt's scary.00:47
cndyeah00:47
cndit seemed like we were so close too...00:48
brycehcnd, pastebin the patch?00:48
cndbryceh, 505_query_pointer_touchscreen.patch00:48
cndoops00:48
cndhttp://paste.ubuntu.com/913912/00:48
cndI guess it's technically a two-liner :)00:49
brycehcnd, ok; looking at event_get_corestate this effectively just ORs in mouse->touch->state if defined00:53
cndbryceh, yep00:53
brycehheh, you know what, this might fix an issue that I *just* saw about 3 minutes ago00:54
brycehon my fujitsu touch screen, it responds to touch drag events, but if I just tap with my finger, nothing happens00:55
cndbryceh, really?00:55
brycehcnd, does that sound like this?00:55
cndbryceh, what are you trying to tap?00:55
cndare you trying to raise a window?00:55
brycehyeah, or click buttons on the launcher00:56
cndbryceh, btw, I still have a bug that is locking up the desktop after some touch interactions00:56
cndI was hoping this would fix it for good, but apparently not00:56
brycehdrags work00:56
brycehwell sort of work.  the initial touch doesn't move the cursor, so you end up dragging offset00:57
cndhmm00:57
cndbryceh, oh, we just pushed up a fix in utouch-grail00:58
cndtouchscreens are quite broken without version 3.0.500:58
brycehok.  I just updated a couple hours ago00:58
cndwithout 3.0.5, grail was holding on to touches until they physically ended00:58
brycehok, I've got 3.0.4 installed, I'll try that00:59
cndthat sounds like the issue you are seeing00:59
brycehfwiw, stylus and eraser work fine in gimp00:59
brycehok00:59
cndgood :)00:59
cndso I see that my pointer lockup is a core passive button grab01:00
cndI think there are issues in the compiz stack...01:00
RAOFReally?  Heaven forfend!01:00
RAOF:)01:00
cndhaving peeked at their code today, I'm very afraid01:00
DarxusSame problems with my Radeon with the upstream kernel.  01:21
RAOFDarxus: Sarvatt was saying something about lack of radeon-kms in the -edgers packages earlier, but it's dropped off my backscroll.01:25
Sarvattthat was in xorg-edgers01:27
Sarvatt(which its not recommended to even use)01:27
Sarvattuntil mesa 8.1 builds in there, its still going through automake changes breaking it every few days and i cant keep up so its on 8.0 branch01:28
Sarvattwhich is the same as in precise01:28
Sarvattwayland egl builds have been broken since the beginning of march for out of tree builds aka what the debian packages do01:29
Sarvattevery automake commit breaks something new because they always include $(top_srcdir) instead of $(top_builddir)01:33
Sarvattlike http://cgit.freedesktop.org/mesa/mesa/commit/?id=ca760181b4420696c7e86aa2951d7203522ad1e8 doing -I$(top_srcdir)/src/mapi ,but that ones fixed already01:36
SarvattDarxus: i'm going to push libdrm 2.4.33 and newer x-x-v-ati to x-updates tomorrow for precise, might as well be using that and not have screwed up input like you will in xorg-edgers if you're using that :)01:41
DarxusSarvatt: I'm not using xorg-edgers.01:42
Sarvattthen again i dont know what problems you really have with radeon, might be assuming too much that you're using edgers because of RAOF :)01:42
Darxushttps://bugs.launchpad.net/ubuntu/+source/linux/+bug/97120401:42
ubottuLaunchpad bug 971204 in linux (Ubuntu) "graphics fails with setgfxpayload=keep, AMD Radeon" [Medium,Incomplete]01:42
RAOFDarxus: Sorry, I was confusing you with someone else.01:44
DarxusNo problem.01:44
Sarvatthmm how does grub-gfxpayload-lists have nothing but vmware added in it in precise, i know tjaalton added a bunch of systems to it01:46
Sarvattall the systems added in the natty lists that were SRUed aren't there, hmm01:49
Sarvatttjaalton: any idea whats going on there? surely those should be forward ported to later releases01:49
* Sarvatt hopes nothing failed certification in 11.10 because it was dropped01:50
brycehcnd, btw even after updating to 3.0.5, the fujitsu touchscreen still misbehavin03:07
cndbryceh, describe what it's doing03:07
cndI still see some issues here03:07
brycehthe pointer is at position x,y03:08
brycehyou touch at position x+100,y+10003:08
brycehthe pointer remains at x,y03:08
brycehyou drag your finger from x+100,y+100 to x+150,y+15003:09
brycehthe pointer moves from x,y to x+50,y+50 03:09
cndwhoa03:09
cndthat's weird03:09
cndI've not seen that03:09
brycehactually it's a bit worse03:10
cndbryceh, it sounds like X thinks your touchscreen is a touchpad03:10
cndbryceh, can you pastebin your x log?03:10
brycehthere is some scaling going on, so moving by +50/+50 moves pointer say +100/+8003:10
brycehpastebin.ubuntu.com/91402403:11
cndbryceh, it's using the wacom driver03:12
cndI think it thinks your touchscreen is a tablet03:12
cndso its behaving like it's a trackpad for pointing03:12
brycehah, hum03:12
cndbryceh, I don't know if it's multitouch, but the only way to get the full touch experience would be to use the evdev driver03:13
brycehit is multitouch, but it is a serial interface03:13
cndbryceh, I think that means you have to use set_serial first03:14
cndbdmurray was trying to get it to work03:14
cndI don't know if he made it work out of the box in ubuntu though03:14
brycehset_serial via xinput?03:15
cndno, it's a separate utility03:15
cndbryceh, http://www.murraytwins.com/blog/?p=10303:15
brycehalright, well I'll mess with it more some day.03:15
cndlooks like I meant inputattach03:16
Sarvattsucks most all touchscreens are wacom these days :(03:24
cndor ntrig..03:28
Sarvattcnd: xt3 and m6600 were the only ntrigs that ever shipped with ubuntu, everyone else switched to wacom in the past few years :(04:21
Sarvatt(since i started tracking it in 2010)04:22
Sarvattx220 tablet, all lenovo W series touchscreens switched to wacom04:24
Sarvattreviews of xt3 were saying it was a negative it still used "ntrig crap"04:24
cndSarvatt, the dell xps 15 and 17 are still available with it04:30
cndbut yeah, I don't blame them04:30
cndI don't know why we can't get an atmel maxtouch touchscreen in an x86 laptop04:31
tjaaltonSarvatt: they worked with 3.0, so the quirks were only needed for 11.0404:47
mornauhi, i'm trying to get a copy of the linux 3.2.0-21.34 kernel build by/for xorg-edgers: https://launchpad.net/~xorg-edgers/+archive/ppa/+build/337966711:41
mornaubut i can't seem to find it in the repository.11:42
mornauis it still awaiting an upload?11:42
ricotzmornau, it should be available, try run "apt-get update" again12:22
mornauricotz: i did this right before posting but there was no updated package available, yet. it worked this time, though. 12:25
mornaubtw i'm gettnig this warning when installing the package: W: Possible missing firmware /lib/firmware/rtl_nic/rtl8168f-1.fw for module r816912:25
mornaui also got it for the current image (the last 3.2 kernel from the edgers repository), though, and it least for me networking isdoing fine.12:26
ricotzright, linux-firmware isnt updated to include the new firmware binaries12:27
ricotzSarvatt, do you think it is safe and worth to put an update linux-firmware package in edgers too?12:28
=== Darxus_ is now known as Darxus
mornauricotz, Sarvatt: i'm on your linux-image 3.2.0-20.32 now, but DHCP issues remain: "dhclient: Error creating socket to list interfaces; Permission denied." this then results in: "NetworkManager[1520]: <info> (eth0): device state change: ip-config -> failed (reason 'ip-config-unavailable') [70 120 5]". also the libvirt-bin process keep getting terminated by apparmor and keeps respawning.14:10
mornautype=1400 audit(1333548406.883:1402): apparmor="DENIED" operation="create" parent=1 profile="/usr/sbin/libvirtd" pid=31996 comm="libvirtd" family="inet" sock_type="dgram" protocol=014:11
mornauboth doesn't happen with the oneiric linux image. i'm happy o file proper bug reports if that's of help. if so, please explain what to file those against.14:12
ricotzmornau, please try 3.2.0-21.34 first, note this is a plain rebuild of the current official precise kernel14:15
ricotzmornau, so you have a r8169 ethernet device and not having the updated firmware files seems to cause problems?14:15
mornauricotz: sorry i pasted the wrong version there, i'm actually on 3.2.0-21.34 and the above report refers to this version (but also to 3.2.0-21.34)14:16
mornau... but also to 3.2.0-20.3214:17
ricotzmornau, could you try https://launchpad.net/ubuntu/+source/linux-firmware/1.78/+build/3380028/+files/linux-firmware_1.78_all.deb14:17
mornaui do have a r8169 ethernet device. i would be surprised if this caused dhcp to fail but static ip assignment to work14:19
mornaui'm currently using static ip assignment14:19
mornau(i also have multiple NICs)14:19
ricotzmornau, the newer kernel seems to rely on the newer firmware and this might fix this problem14:20
mornauokay, i'll try with the firmware, thanks14:20
Sarvattricotz: no problems from me putting firmware in there14:25
Sarvattthats really strange he's hitting that though, maybe libvirt's apparmor profile in oneiric doesn't work wth precise's kernel apparmor?14:26
* Sarvatt has no clue about apparmor really14:26
ricotzSarvatt, ok, -21.34 contains a lot appamor changes/fixes14:27
ricotzSarvatt, i will copy the firmware package to the ppa14:31
ricotzSarvatt, but this is probably a problem for lts-backports of the kernel too14:32
ricotzapw, hi ^14:32
mornauricotz: i installed the firmware package you pointed me to, ran "sudo update-initramfs -u -k all", rebooted, to no effect.14:41
mornau# dpkg -l linux-firmware\* | grep ^ii ii  linux-firmware                                1.78                                                                         Firmware for Linux kernel drivers14:42
mornauhmm silly qwebirc, it eats line wraps. but i guess you know what this output should loook like.14:43
mornaui also reinstalled the kernel image and there were no more warnings about possibly missing firmware.14:47
apwricotz, you saying there is a new dependancy between the kernel and firmware in precise?14:49
apwricotz, there won't be a precise lts backport to lucid iirc14:50
ricotzmornau, i see, i am not that familiar with libvirt14:50
ricotzapw, ok, but i saw a similar issue with the 3.0 backport14:51
ricotzapw, .. complaining about missing nouveau firmware files14:51
apwricotz, stupid backports, make life so difficult14:52
ricotzapw, so this also effects the oneiric ltsp backport14:52
ricotz*lts14:52
apwricotz, could you file a bug against the kernel with the details14:52
ricotzapw, yeah, i know, another problem is also not having an updated linux-libc14:52
mornauricotz: it's an issue with its apparmor profile, i guess. i don't need libvirt really so i just purged it. dhclient not working is a bit of a show stopper though. (so unless you want me to look into this more) i guess i'll just downgrade to the oneiric images until precise is released. thanks so far.14:53
ricotzmornau, yeah, might be the best to use the oneiric kernel then, sorry for the trouble14:55
* Sarvatt would just purge apparmor instead :P14:58
Sarvattlooking at the apparmor bugs it looks like mismatched apparmor kernel and userspace breaks dhclient all the time though :(14:59
Sarvattsame bugs filed in maverick and natty14:59
ricotzso much for abi stability ;)15:00
mornau:))15:00
* ricotz is surprisingly happy with 3.4rc1 so far15:02
Sarvattand karmic and jaunty, thats a deep rabbit hole of a problem :)15:02
mlankhorststill on 3.2 here :)15:07
apwricotz, linux-libc-dev being 'downrev' is deliberate as in theory at least the interfaces are backwards compatible15:10
apwtherefore you want to build things for the oldest interface on the machine15:10
ricotzapw, hmm, i see, putting the kernel package in xedgers is mostly for this reason to have an updated linux-libc-dev15:14
apwricotz, well in there you may be using newer things which need the newer interfaces for instance15:14
ricotzsince it is expecting people using xedgers would use a newer kernel too15:14
apwricotz, its all a bit of a mess.  actually we should not really offer a backport kernel, but a backport ppa for each release to contain this crap15:15
ricotzyeah, while having libdrm git and kernel git would fit more15:15
ricotzapw, having a ppa might lead to more inconsistencies while x-backports are starting next cycle15:17
apwricotz, well i think i mean there needs to be one PPA for each source series destination series, with kernel, mesa, X etc in for that combination15:17
ricotzah, i see15:18
apwthere is no other safe way to do it imo, anyhow ... fun15:18
ricotzi think this should also get stricter tied together15:19
ricotzso wanting a newer x should suggest you a newer kernel15:19
ricotz(or even force you)15:20
Sarvattricotz: like having a new pocket for each backport series, and having the kernel be called linux and all of the x packages having the same names too? :P15:20
ricotzSarvatt, yeah, this would be way easier to be maintained15:21
Sarvattwe're gonna get so much wrong namespacing all of x/mesa/libdrm the way lts backport kernels do15:21
apwSarvatt, you describe Nirvana15:21
=== lool- is now known as lool
cndbryceh, RAOF: just got a good valgrind trace on X :)19:05
cndoh wait, it might be in the area where I was patching stuff in19:05
cndfor debugging :(19:05
cndhmm, I think I found issues though19:11
cndI don't know if people would see them outside of touchscreen cases though19:12
brycehcnd, awesome19:29
cndyeah, the way we look for pointer listeners for direct touch devices is all wrong right now19:31
cndthe code is looking at the wrong event masks19:31
cndit's looking at the core event mask, but interpreting it as a pointer to an XI2 event mask19:31
cndso it's all wrong19:31
brycehcnd, is that something due to the mixed server stack, or a legit upstream bug?19:50
cndlegit upstream19:50
cndit just hasn't been tested properly, basically19:51
brycehok19:52
tjaaltonbryceh: the 'touchscreen is reset to relative mode' is likely a bug in g-s-d or so, since it's happening after login20:50
tjaaltonbug 949097 was filed earlier20:50
ubottuLaunchpad bug 949097 in xf86-input-wacom (Ubuntu) "Serial Wacom Tablet touch not in absolute mode when logged in" [Undecided,Incomplete] https://launchpad.net/bugs/94909720:50
brycehfriggin gnome20:51
tjaalton:)20:52
brycehso yeah, then 973133 would be a dupe of that.  was just looking at that a bit ago20:52
brycehis there a bug against g-s-d yet?20:52
tjaaltonno, probably would be right to add it to the earlier bug, until it's clear the bug is there20:53
tjaaltonsince the login screen should be running g-s-d as well..20:53
tjaaltonhmm20:55
tjaaltonwould be easy to test20:55
tjaaltonby disabling the wacom plugin20:55
Sarvattits not running g-s-d plugins though20:55
tjaaltonright20:55
tjaaltonrealized that20:55
tjaaltonactually, the bug might be that the touchscreen isn't actually reported as one by the kernel20:56
tjaaltonbut a tablet20:56
tjaaltonwhere relative mode makes sense20:56
tjaaltonhmm no20:56
tjaaltonit should be in relative mode in the login screen as well then20:57
tjaaltonbryceh: so, flip the wacom plugin off, org.gnome.settings-daemon.plugins.gsdwacom:active, and login again20:59
brycehalrighty, is there a utility for turning it off, or a config file?21:01
tjaaltondconf-editor, or gsettings21:02
tjaaltongsettings set org.gnome.settings-daemon.plugins.gsdwacom active false21:04
tjaaltonthen 'get ... active' to check that it worked21:04
brycehyep, that did the trick21:07
brycehworks properly on both login screen and logged in21:07
tjaaltonwhee21:07
brycehstylus and eraser also working21:07
tjaaltonyeah those should've worked before too21:08
brycehthey did, was just checking that they still did ;-)21:08
tjaaltonhehe, right21:08
cndbryceh, have you seen any bugs where people are attempting to perform a two touch tap on a trackpad and it is generating a middle button click instead of right?21:14
cndolli ries, my manager, is seeing it21:14
cndfirst, the defaults in X are wrong due to a distro patch from 2009 :)21:14
cndbut then gsd should be overwriting the config21:14
cndwhich is why no one sees the issue21:15
brycehcnd, not that I remember offhand, but I can take a quick scan21:50
cndbryceh, this is the bug olli filed: https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/97378421:50
ubottuLaunchpad bug 973784 in gnome-settings-daemon (Ubuntu) "configuration not set properly" [Undecided,New]21:50
cndit seems like something someone might toss in the X package bugs mistakenly21:50
brycehbug 972125 maybe21:53
ubottuLaunchpad bug 972125 in xserver-xorg-input-synaptics (Ubuntu) "Right and middle mouse button do not work on clickpad" [Undecided,New] https://launchpad.net/bugs/97212521:53
brycehthere's lots of reports of touchpads that just stop working entirely after thus and such21:53
brycehbug 97272721:54
ubottuLaunchpad bug 972727 in xinput (Ubuntu) "Synaptics Clickpad functionalities incomplete: Right button area, two-tap disabling" [Undecided,New] https://launchpad.net/bugs/97272721:54
=== yofel_ is now known as yofel
cndok, I got another patch for the xserver today :)22:56
* cnd knocks down more bugs22:56
bryceh:-)23:17

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