/srv/irclogs.ubuntu.com/2012/09/25/#ubuntu-x.txt

=== stgraber_ is now known as stgraber
brycehanyone know, are we getting ARM video drivers in the archive for Q?00:49
bjsniderbryceh, i have .51 ready to go01:00
bjsnideri thought that was the right one01:00
bjsniderit says it's a long-lived release01:01
bjsnideri've been using my download bandwidth for something else all day but that will be done in a few minutes and i was going to send it in then01:01
brycehbjsnider, awesome thanks01:09
brycehdidn't realize there was a .51 already; good to know01:09
bjsnideri noticed that ricotz had sent it into xorg-edgers01:15
bjsniderhe must pay somebody inside nvidia for the info on when new blobs are released01:15
RAOFtjaalton: I'd love to try the auto-power-off patch :)01:15
fubadahi, is the ppa add procedure under 12.04 still "apt-add-repository ppa:ubuntu-x-swat/x-updates'?03:52
tjaaltonbryceh: ok thanks, i'll discuss it with airlied04:12
tjaaltonRAOF: fof the kernel? i could build a paxkage with it/them04:13
tjaaltonpackage too04:13
RAOFtjaalton: Oh, they're kernel patches? That seems odd.04:13
RAOFThe kernel already exports all the necessary juju for userspace to do that.04:13
tjaaltonhmm04:14
tjaaltonok, don't know which patch you're referring to then  :)04:14
RAOFtjaalton: The one you pinged me about earlier?04:14
tjaaltoni did?-)04:15
RAOF21:31 <tjaalton> RAOF: I'm adding the patch for dynamic gpu poweroff too, would be fun to test how that works04:15
tjaaltonwth was that about. probably meant the kernel patches there04:16
RAOFHeh04:16
tjaaltonI was meant to build a kernel for that, but didn't yet04:17
RAOFAgain, I don't know why that's a kernel patch; X should have all the information necessary to parse /sys/kernel/debug/vgaswitcheroo/switch and turn off any GPU its not using.04:17
tjaaltonthere are a couple of threads on dri-devel about this04:17
tjaalton[RFC] drm dynamic power off support04:18
tjaaltonwell, this is _dynamic_!04:18
ajmitchso with the q-lts-backports PPA, if I install the renamed packages & then try to upgrade to quantal, will things break? I didn't see any conflicts/replaces or breaks from the quantal X to the renamed packages04:18
tjaalton:)04:18
RAOFOh, hey. Evolution's crashed again.04:19
RAOFMaybe the OOM killer hit it again.04:19
RAOFAh, ok. I see those. Interesting!04:21
RAOFThat'll obviously be *more* interesting when powering on the radeon doesn't crash X, of course.04:21
tjaaltonheh04:21
tjaaltonbtw, push -ati git ;)04:21
fubadado you guys know if I need to use jockey-text to switch to nvidia_current from x-swat04:22
fubadai dont have x11, using ubuntu-server04:22
tjaaltonif you already have nvidia enabled then no04:24
fubadaby defaault it uses the free drive nouveau04:25
RAOFtjaalton: Ta for the prod there.04:25
tjaaltonfubada: yes, so why do you need the blob if it's a server?04:26
fubadaim going to install xbmc as it is also my htpc04:27
fubadait doesnt need any dm04:27
fubadabut it will be running xbmc04:27
tjaaltonok, try jockey-text then04:27
fubadaok04:28
bjsniderfubada, why use the blob on an x11-free server?04:29
tjaaltonreplied already ;)04:30
bjsniderstill not sure it makes much sense04:31
bjsnideri wouldn't put anything by nvidia in a server04:32
fubadaits not a real server, its just an ion2 box with 4tb of swraid04:33
fubadahooked up to my tv04:33
fubadaa bunch of private torrents04:33
fubadaand running xbmc04:33
fubadai just dont want the overhead of gdm, etc04:33
fubadafor music/movies04:34
bjsnideroh, i kind of jumped to conclusions when you said server04:34
bjsniderjust rtorrent i guess04:34
fubadatransmission with a webui04:34
bjsnider...04:35
=== Amaranthus is now known as Amaranth
tjaaltonbryceh: btw, the patches are referenced in the changelog, just not with the name06:27
brycehtjaalton, right 228* and 229* are not referenced in the changelog.06:30
tjaaltonyeah, it's kinda silly to have numbers anyway06:30
tjaaltonsince there are huge gaps that mean nothing :)06:31
brycehtjaalton, the patch names without the numbers aren't mentioned either :-P06:32
tjaaltonright, but the context should be clear :)06:32
brycehtjaalton, not to the emacs search function :-)06:33
tjaaltonheh06:33
tjaaltonright, names should be there06:33
tjaaltoni just followed mlankhorst's lead! :)06:34
bryceh+1 to retaining patch names in changelog comments.  and +1 to context (moar context!)06:35
tjaaltonthinking about adding new categories to patches/series, so it's more clear what are from upstream, what's never meant there etc06:36
tjaaltonand drop the numbers06:36
tjaaltonfrom our patches06:36
brycehinteresting thought.  I'm in favor of file naming consistency, but there's no technical reason why it needs to be a linear set of numbers06:38
brycehhaving patches grouped together that are interdependent is a good idea, so they can be popped and pushed as a unit06:39
tjaaltonright06:39
tjaaltonand no need to review them again and again after rebasing to a new version06:40
* bryceh nods06:40
tjaaltonI might do it after this fix, it's a non-functional change but no need to rush it for the beta06:40
brycehthen the upstream cherrypicks could all be grouped at the top so they get applied first.  Would cut down on porting work.06:41
tjaaltonyeah06:41
tjaaltonalthough it could mean more churn in the distro patches :)06:42
brycehnon-functional but if you change the ordering (as you have to), you might have to rework a few patches06:42
brycehyeah, a bit of churn but that'd happen anyway when we go to the next upstream version06:42
tjaaltonright, but when backporting patches for a stable release..06:43
tjaaltonthough there should be no conflicts either way06:43
mlankhorstmaybe do it for xorg 1.14?06:43
brycehI suppose by the time we get to a perfect patch organizational system we'll be switching to wayland or whatnot06:43
tjaaltonguess so :)06:43
mlankhorstunless you can show there is no diff before and after changing the patch order06:43
brycehor maybe trim the numbers and do the easy recategorization now, and leave any tough patch reworks to 1.14?  Timo has a lot of patch-fu I bet he can pull it off.06:47
tjaaltonI'll try it out, it's easy to see if it fails in horrible ways or not. don't think it should :)06:48
tjaaltonI'll test this patch first tho..06:50
mlankhorstfamous last words06:51
mlankhorstbut.. back to fencing I suppose..06:52
tjaaltondidn't crash, ship it06:56
brycehhttp://www.bryceharrington.org/Arsenal/ubuntu-x-swat/Reports/totals-quantal-workqueue.svg07:00
brycehfirst time we've had more -nouveau bugs than -intel07:01
tjaaltonyeah it's in a bad shape07:01
tjaaltoncrashes in libexa etc07:01
mlankhorstI know, bit annoying too when upstream (darktama) doesn't respond :(07:02
brycehyeah I can't get piglit to run on it without the gpu locking up07:02
brycehand that's with 804.  with 802 in precise the kernel OOPSes07:02
mlankhorst:>07:04
mlankhorstthe annoying part is the kernel rework is not complete yet so the maintainer is focusing on that..07:04
brycehmlankhorst, in hindsight do you think we should have stuck with an older version before the rework started?07:05
tjaaltonolder version of?07:05
mlankhorstnot going to matter, both were broken07:05
tjaaltonthe rework is in the kernel07:05
brycehold version of whatever07:07
mlankhorstno it just needs some bugtracking love, we couldn't use the old ddx because of the exa changes07:07
tjaaltonnot really possible07:07
brycehso, nothing really we could have done?  just chalk it up to poor maintainership.  Bummer.07:08
tjaaltonhmm, wonder if patch 188 is breaking stuff now..07:08
tjaaltonguess we can drop it with the hotplug stuff now in07:09
tjaaltonand ask sabdfl to test :)07:10
brycehtjaalton, regardless, it's not relevant if there is better hybrid support now07:10
tjaaltonyeah I'm pretty sure the current code is good enough07:10
tjaaltonand handles this case07:11
brycehit was a pretty obvious bug when it happened07:11
tjaaltonin the future, it would be great if the distro patches had a patch header :)07:12
tjaaltongit log works most of the time07:12
tjaaltonbut it would be even easier to just look at the header07:12
brycehthis patch was from karmic, we were pretty fast and loose back then07:13
tjaaltonhehe07:13
brycehI've been trying to do the full git format patch stuff these days, when it's something I authored.  At least I always include a preface description.07:14
mlankhorstI think some headers might have been lost by my xorg codestyle rebase/update script07:14
brycehdoh07:14
mlankhorstpersonally, I prefer patches just have something in git log about them07:14
mlankhorstthat's what the metadata is for..07:14
tjaaltonha, indeed..07:15
* tjaalton spanks mlankhorst with a giant herring07:15
tjaaltonhttp://dep.debian.net/deps/dep3/07:16
tjaaltonalso, git log ceases to work when they are renamed07:16
mlankhorsttalk about overengineering..07:16
tjaaltonwell, it's usually enough to just have a from tag and a buglink07:17
mlankhorstNot really? git log will keep working..07:17
mlankhorstUnless you do a rewrite as well07:17
brycehgit log works ok for _us_07:17
tjaaltonok, not sure about that07:17
tjaaltonyeah that's a good point07:17
mlankhorstgit log --follow07:18
brycehbut for all our gitless co-workers, or users who only know about apt-get source, having a description in the patch itself can be a big boon07:18
tjaaltonlike ubuntu-release/sru folks07:18
mlankhorstseems to pick up on my debian/patches/series rename :-)07:18
mlankhorstbut nowadays I tend to edit patches manually when updating..07:19
tjaaltonquilt refresh shouldn't touch the headers07:20
tjaaltonor dquilt if you have the alias07:20
mlankhorstnah I just set quilt to always assume I'm using debian07:21
brycehhey, how do you check that a package is in NEW?  Is there a report for that?07:23
bryceh(trying to find nvidia-graphics-drivers-experimental-304 for precise-proposed)07:23
tjaaltonhttps://launchpad.net/ubuntu/precise/+queue07:23
brycehaha bingo, thanks.07:24
tjaaltonno drama after reorganizing the patches07:29
brycehnice07:35
brycehmlankhorst, I agree dep3 as a whole is overkill, but the concept is sound.  Basically the idea is to explain where the patch originated and what it does.  As long as that's clear, that's good, even if it's not dep3 compliant.07:38
tjaaltonand usually it's enough to list from:, subject:, short description or a buglink, that's it07:38
mlankhorstso pretty much what we already do :p07:39
tjaaltonyeah, when they are not stripped off on rebase :)07:40
mlankhorstI don't think there will be more 'change the entire X server coding style' patches, so it should be safe..07:41
tjaaltonhttp://pastebin.com/BrDFnAt307:47
tjaaltoncomments?07:47
tjaaltonsome offsets when patching, nothing more severe07:48
tjaaltonnot that many pure distro patches07:48
mlankhorstcan you check if the source code before and after is exactly the same?07:48
tjaaltonshould be easy07:48
bryceh"waiting for review" -> "waiting for review by upstream" ?  Maybe including a ML url or bug # would facilitate following up?07:51
tjaaltontook a git diff from both, diff'ing the diffs show no diff :)07:51
tjaaltonyeah07:51
bryceh"from upstream, drop when rebasing"...  anything break if these are placed first instead of last?07:52
tjaaltonwell, I think it's easier to rebase a single backported commit than possible several earlier ones07:53
tjaaltonusually they don't need any changes07:53
tjaaltonthan just a refresh maybe07:53
brycehhmm07:54
brycehwell I'll remain open minded07:55
mlankhorstI just don't want to break things so as long as there is no diff I'm fine with it07:56
tjaaltonis it more confusing to leave the numbers or not?07:56
mlankhorstrenumber after you have the final sequence07:57
tjaaltonno numbers anymore :)07:57
brycehtjaalton, I think you're probably right to just drop the numbers08:07
brycehI'm partial to having the numbers myself, but objectively I think the purpose they serve is largely lost with xserver08:07
tjaaltonbryceh: yeah, it didn't really work out too well trying to use the numbers for different categories08:16
tjaaltonthat was my idea too :)08:16
tjaaltonthis should scale better08:16
mlankhorstwe should do a better job at dropping unneeded crap in xorg-server though :)09:10
tjaaltonsure09:10
tjaaltonthis should make it easier09:10
ricotzhi, why is there a need to call the package "nvidia-graphics-drivers-experimental-304" instead of just "nvidia-graphics-drivers-304"10:17
ricotzand uploading 304.51 is probably the better choice here10:19
tjaaltontseliot knows10:35
tseliotricotz: because we're going to use that package for experimental drivers and yes, I know a newer release has been available since yesterday10:42
tjaaltonand the new one seems somewhat broken11:42
tjaaltonbug 105605211:43
ubottuLaunchpad bug 1056052 in xorg (Ubuntu) "Launcher not showing in auto-hide mode" [Undecided,New] https://launchpad.net/bugs/105605211:44
=== mdeslaur_ is now known as mdeslaur
ricotztseliot, i see, but including "experimental" and "304" doesnt make sense imo?11:56
=== lilstevie_ is now known as lilstevie
tseliotricotz: the technical board decided that for a reason12:44
tseliotricotz: http://ubuntu.5.n6.nabble.com/Minutes-from-the-Technical-Board-meeting-2012-09-17-td4992536.html12:45
ricotztseliot, alright, that makes sense12:49
brycehricotz, don't get hung up about it being 304, that's just being used to seed things with since we don't have a beta driver yet16:43
brycehricotz, in practice we'll only be shipping pre-release drivers in -experimental16:43
ricotzbryceh, yeah, that is why i was wondering. having "nvidia-graphics-drivers-experimental" and later "nvidia-graphics-drivers-304/ -306" seemed more logical, but it is fine since it was discussed16:47
=== pzanoni_ is now known as pzanoni
=== yofel_ is now known as yofel
czajkowskielo anyone able to help me, not sure if's a bug or something has happened, I am running 12.10, I close the lid of machine and i come back to it and go to use it again, the screen is black but lit and I can see the cursor but no login prompt19:41
czajkowskipossibly this thinks laney https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-intel/+bug/966744  but it's marked as not affecting 12.1019:43
ubottuLaunchpad bug 966744 in xserver-xorg-video-intel (Ubuntu Quantal) "[i965] Resume from suspend leaves me with black screen or a screen of the desktop before it suspended. Compiz hung in intel_update_renderbuffers() from intel_prepare_render() from brw_draw_prims()" [Critical,Confirmed]19:43
stgraberI'm seeing something similar here though not limited to closing lid. For the past few days (3-4 days I think), whenever X shuts down the screen it seems to get stuck. When I get back to the machine, the screen turns back on but all I get is a blank screen with the mouse cursor19:44
stgraberfor now my workaround is to disable the screensaver completely so the screen never turns off, but that's no really ideal ;)19:45
stgraber(intel hd4000 on an ivy bridge system)19:45
stgraberI confirmed that it's not gnome-screensaver (killed from another tty), it could be compiz (killing it gets me the wallpaper but nothing else) and I'm not seeing anything in dmesg so if it's kernel, it's not obvious19:46
stgraberoh, actually, I just noticed this in my X logfile:19:46
stgraber[  7657.397] (WW) intel(0): flip queue failed: Invalid argument19:46
stgraber[  7657.397] (WW) intel(0): Page flip failed: Invalid argument19:46
czajkowskitjaalton: you about ?19:46
toabctlczajkowski, stgraber i have the same issue here (i386, 00:02.1 Display controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics Controller (rev 07))19:50
tjaaltonczajkowski: yes it's the same bug19:56
czajkowskitjaalton: yes but it's now saying it doesnt affect Q from your latest comment on the bug 19:57
tjaaltonno, i removed compiz from there19:57
czajkowskiah ok19:59
czajkowskiwel that bug is a bit of a pita right now :/19:59
czajkowskialthough may disable my screensaver thanks to stgraber 19:59
tjaaltonand i have three intel laptops but can't reproduce it like others..20:00
czajkowskitjaalton: :/20:01
czajkowskitjaalton: you in the bluefin?20:01
tjaaltonno, just FI :)20:02
czajkowskiFI?20:03
tjaaltonfinland20:04
czajkowskiah right 20:18
czajkowskiwell I cant send you my laptop 20:18
czajkowskiI need it for work20:19
czajkowskiif I can do any debugging and let you know stuff please just ask 20:19
tjaaltonczajkowski: https://bugs.launchpad.net/ubuntu/precise/+source/xserver-xorg-video-intel/+bug/966744/comments/22620:22
ubottuLaunchpad bug 966744 in xserver-xorg-video-intel (Ubuntu Quantal) "[i965] Resume from suspend leaves me with black screen or a screen of the desktop before it suspended. Compiz hung in intel_update_renderbuffers() from intel_prepare_render() from brw_draw_prims()" [Critical,Confirmed]20:22
tjaaltonthere are a couple of traces attached, but I'm not convinced they're ok..20:23
tjaaltonso if it hangs on the first lid close, it should be easy to get a good one20:24
tjaaltonalso, boot with 'drm.debug=0xe' and attach dmesg output from the hang20:30
bjsnidertjaalton, you're located in finland?23:38
brycehbjsnider, he is23:45
brycehtjaalton, btw curious about your plan of attack for that bug?  Would it be worthwhile to post your directions on what you would do if you had the HW in front of you, so someone else that does have it could try those steps?23:46

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