/srv/irclogs.ubuntu.com/2010/06/25/#ubuntu-x.txt

Sarvattwhoops RAOF your edgers expired, renewed ya :)00:53
RAOFSarvatt: Thanks.  I was going to ping you about that :)00:55
rippsWooo!!! Internet is back.01:13
tseliotRAOF, Sarvatt: I'm almost done packaging the new nvidia driver. Did you put the libraries back where they used to be in Mesa?13:27
bjsnidertseliot, did you integrate the new nvidia_supported script?13:32
tseliotbjsnider: yep13:32
bjsnidercan you post the relevant section of the rules file somewhere?13:33
tseliotbjsnider: I'll commit my changes soon in this branch: http://github.com/tseliot/nvidia-graphics-drivers13:34
bjsniderk13:34
tseliotbjsnider: ok, pushed (note: this is not final)13:57
bjsnidertseliot, where is the code that creates the /dev/nvidiactrl file?14:02
tseliotbjsnider: ???14:02
tseliotbjsnider: you will only find packaging scripts there14:03
bjsniderwell, i was looking for the correct settings for that file, i assume it is owned by root:video and that the user must be part of the video group14:06
tseliotI wouldn't know14:07
bjsnideri have to get hold of someone who has the blob installed right now14:08
bjsniderunfortunately it got screwed up thanks to ext4 when i lost power14:08
bjsnideri always lose data with ext4 if the system shuts down unexpectedly14:09
tseliot:-/14:21
iontseliot: Re: the commit message, it’s in fact the README that is missing IDs (and has been for all releases of the driver). The README is only used by the new script to find the correct symbol from the blob. The resulting list is based on the blob, not the README.14:55
iontseliot: The script traverses through the .rodata symbols until it finds one that correlates to the README list closely enough. Then that list is converted to the modaliases format.14:57
iontseliot: The list from various versions of the blob has consistently had zero ID deletions (compared to README) and almost 100 % additions compared to the number of IDs in the README.14:59
iontseliot: So, currently the thresholds for picking the symbol are: deletions = 0, additions ≤ 1.5·n where n is the number of IDs in README.15:00
tseliotion: ah, it seemed weird to me that they removed only some ids. I'll read your changes again15:00
iontseliot: Sarvatt said the IDs README is missing are things like AGP-PCIe bridge chips etc.15:01
tseliotion: aah, that's where I heard that something was removed. I must have mixed things up15:02
* tseliot changes the changelog15:04
iontseliot: For previous releases of the driver, the blob had two consecutive symbols duplicating the ID list, and that was the heuristic used to find the right symbol. They removed the duplicate and the script stopped working. So now the new heuristic for finding the right symbol is the comparison to the list from README.15:06
iontseliot: If you haven’t already done it, i could as well do the changelog change.15:13
tseliotion: I did this but I haven't pushed it yet: http://pastebin.ubuntu.com/455000/15:15
tseliotion: if you want to add a detailed explanation to the changelog, you're welcome to do so15:15
tseliotBTW why 1.5·n?15:16
ionIt’s pretty much an arbitrary number for sanity checking. The number of additions has correlated with the number of total IDs in README, and since it has consistently been *almost* 1.0·n, i added a safety buffer of about 50 % in case the number of additions exceeds the number of IDs in README in a future release.15:20
tseliotok, I suspected that but I wanted to be sure15:20
tseliotion: where's your branch on git-hub?15:21
iontseliot: Just pushed to http://github.com/ion1/nvidia-graphics-drivers / git://github.com/ion1/nvidia-graphics-drivers.git15:24
tseliotion: ok, I'll merge from it15:26
tseliotmerged and pushed. Thanks15:33
tseliotthe driver works well now15:47
tseliotI guess nvidia-173 and nvidia-96 are not installable in maverick, are they?15:51
tseliotbjsnider: ^15:51
ionHave you considered having just a single source package for all versions of the nvidia driver, btw? Wouldn’t that reduce duplication? I made sure the new nvidia_supported script works for all versions, for instance. It could just as well exist in a single source package.15:54
bjsniderion, i added y our script to the older distros and it generated the expected modaliases file15:55
tseliotion: that would make the source package too big with all the installers15:56
tseliotalso bug tracking would be harder15:58
ionGood point15:58
=== virtuald_ is now known as virtuald

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