/srv/irclogs.ubuntu.com/2011/05/19/#ubuntu-x.txt

=== bryceh_ is now known as bryceh
=== bryceh is now known as bryce
feiHello,e'rbody04:06
feiI'm a kubuntu user from China04:06
feianyone here?04:07
feihi,dantti04:17
RAOFtjaalton: I don't think the nvidia/fglrx blacklists won't actually affect nouveau/ati.  They end up explicitly loading the kernel driver, which should bypass the blacklist, right?07:55
RAOFtjaalton: That said, doing that will likely race and will not necessarily do anything but make X explode as nouveau initialises before KMS is fully up, bails, and then VESA stomps all over KMS state.07:56
tjaaltonRAOF: then why do they install the blacklist if it doesn't work ?-)07:56
RAOFtjaalton: Because it prevents *autoloading* by pciid.07:56
RAOFSo without the blacklist there's a race between nvidia and nouveau kernel modules, and whichever loads first locks the other one out.07:57
tjaaltonok, so nouveau would load anyway if nvidia wouldn't autoload?07:57
RAOFNow that we no longer install an xorg.conf, I think the answer is “yes”.07:58
tjaaltonookay07:58
RAOFThis sounds like a job for The Experimental Method™!07:58
tjaalton:)07:58
RAOFAlthough I don't think I've got a system fast enough to break with the kernel/DDX kms race.07:59
RAOFBut first, to the running around!08:01
tjaaltoni need to get my head around the race thingy, last week was the first time I heard about it08:01
=== yofel_ is now known as yofel
RAOFWhich race?  The kms/DDX race?08:08
tjaaltonthat08:08
tjaaltonyou mean the xserver might start before kms is up?08:08
RAOFYes, absolutely.08:12
tjaaltonugh..08:12
RAOFI'm *sure* I've seen you comment on bugs where this has happened :)08:12
tjaaltonok, bad memory then :)08:12
RAOFParticularly - if the module is blacklisted, then it'll only start to get loaded once X has loaded the DDX.08:12
tjaaltoni do have an ssd system, though the cpu isn't on par with the disk08:12
tjaaltonnow that part i don't get, that how it'll still load the module if it's blacklisted08:15
tjaaltonbut I guess it's easy to test08:15
tjaaltonif no nvidia, X will try nouveau, and then it would forcibly load nouveau.ko?08:15
tjaaltonhmm right08:16
tjaaltonmodprobe.d blacklisting doesn't prevent 'modprobe foo' from working08:16
tjaaltonmaybe I'll delete the comment from the blueprint then08:19
tjaaltondone08:20
tjaaltonwould be great to have a diagram of all the various ways to break the system, then it would be easier to plan the failsafe stuff so that it's more robust08:21
=== ara_ is now known as ara
RAOFtjaalton: :)10:21
bryceor some sort of artificial way of reproducing those conditions so we can write a test suite13:51
cndbryce, tjaalton: do you know what the timeline is for an upload to natty-proposed for xorg-server?16:48
cndI just got another email about the coordinate transformation bug that's waiting to be pushed16:48
brycecnd, if you have a branch set up with the change I could upload it right now16:49
cndbryce, it's up16:49
cndubuntu-natty16:49
cndin the git repo16:49
tjaaltonyeah fine by me to upload it now16:50
cndcool16:50
bryceuploaded16:56
cndbryce, ta16:57
tjaaltonbryce: bad release :)17:04
brycesigh17:06
brycereuploaded17:12
brycestupid git won't let me push the branch change17:12
tjaaltonwhat does it say?17:14
bryce$ git push origin ubuntu-natty17:15
bryceTo ssh://bryce-guest@alioth.debian.org/git/pkg-xorg/xserver/xorg-server.git17:15
bryce ! [rejected]        ubuntu-natty -> ubuntu-natty (non-fast-forward)17:15
bryceerror: failed to push some refs to 'ssh://bryce-guest@alioth.debian.org/git/pkg-xorg/xserver/xorg-server.git'17:15
bryceTo prevent you from losing history, non-fast-forward updates were rejected17:15
ubot4bryce: Error: I am only a bot, please don't think I'm intelligent :)17:15
bryceMerge the remote changes (e.g. 'git pull') before pushing again.  See the17:15
bryce'Note about fast-forwards' section of 'git push --help' for details.17:15
tjaaltonhave you done rebases on your local tree?17:16
bryceno, just git commit --amend17:16
tjaaltonhttp://stackoverflow.com/questions/253055/how-do-i-push-amended-commit-to-the-remote-git-repo17:18
tjaaltonit rewrites the history, so you need to force it17:18
tjaaltonand don't amend in the future :)17:19
tjaaltonif you've already pushed17:19
tjaaltonmaybe this is why things have failed in the past?17:19
bryceit's probably the source of the problem I had yesterday.  Didn't realize --amend was actually useless post-push17:23
tjaaltonin this case forcing the update would only break my local three, but that's probably ok :)17:23
bryceno, I didn't want to do that, I just started over as a regular commit17:23
tjaaltonand force that?17:23
bryceno force needed17:24
tjaaltonah, there it is17:24
tjaaltonyeah looks good now17:25
bryceevidently I should hold off on pushing ;-)17:25
tjaaltonon the contrary, more practice :)17:26
brycehow many years have I been using git?  it's still such a time sink for me17:26
bryce1 minute to upload something, 30 min to fight git to let me commit it17:26
tjaaltondo a change, dch, debcommit -a, git push origin17:27
tjaaltonthat's it, and don't use .patch but .diff for patches :)17:27
tjaaltonso git status will complain loudly17:27
tjaaltonabout files not added to the branch17:27
brycedon't you have to do 'git push origin ubuntu'?17:28
tjaaltonwell yeah if you just want to push that branch, it'll just complain if the other branches are not uptodate17:28
bryceI dislike the idea of using .diff instead of .patch.17:28
tjaaltonwhy?17:28
tjaaltonthey are in debian/patches, so the path tells they are patches17:29
bryce'patch' is descriptive of what the file is, 'diff' just indicates how it was produced17:30
bryceseems like it would be better to just make git stop rejecting .patch files17:31
tjaaltonbesides, libxi git doesn't have the latest patch17:31
tjaaltonbryce: that change would have to be made to every .gitignore where we have an ubuntu branch, not that practical17:32
brycetjaalton, we typically add patches to only a handful, and could be done on a case-by-case basis as needed.  not that big of a deal17:34
tjaaltonmmh, i don't like case-by-case decisions, it's either-or :)17:36
bryceactually looks like we can just add debian/patches/.gitignore with "!*.patch"17:37
tjaaltonit's still ugly, for the sake of a postfix17:37
bryceaha, figured it out17:41
brycecan be set globally17:41
tjaaltonthat's good then17:55
brycehmm, can be set globally but project level .gitignore always takes precedence :-/18:26
tjaaltonbryce: ah..18:47
tjaaltoni'll upload current xorg-server git to oneiric, need to get the patch there too22:05

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