/srv/irclogs.ubuntu.com/2010/09/24/#ubuntu-kernel.txt

=== jjohansen is now known as jj-afk
=== bjf is now known as bjf[afk]
=== bjf is now known as bjf[afk]
=== BenC___ is now known as BenC
jj-afkbjf[afk]: around?04:36
bjf[afk]yup04:36
jj-afkI have a ppa abi question04:36
bjf[afk]ok04:37
jj-afkI have a ppa that is failing on the abi check04:37
jj-afkhowever it worked the first build and I haven't bumped abi since04:37
bjf[afk]wow, that's weird04:38
jj-afkyeah, I am sure I have done something wrong but, the whole abi bump, upload part is uhmm poorly documented04:38
jj-afkdo you have some docs you follow when working through the bump?04:39
jj-afkI noticed certain steps in the maintanence guide don't seem to work anymore04:40
bjf[afk]trying to find my notes04:40
bjf[afk]mostly we use the maint-starnewrelease script, it advances the build number however i'm not sure how it knows to bump the abi04:42
jj-afkit doesn't04:42
bjf[afk]one thing you can do is "git mv <old-abi-directory> <new-abi-directory>"04:42
jj-afkwon't that cause a failure if the abi has change though04:43
bjf[afk]let me send you my notes and see if they make any sense to you, they don't to me right now04:44
jj-afkokay, and I will give moving the abi file a try04:44
bjf[afk]i've only had to go through this once04:46
bjf[afk]email sent04:46
jj-afkah, okay.  I've only bumbled through it on my ppa once and fluked out on getting it to work, and now it won't :)04:46
jj-afkthanks04:46
bjf[afk]now that my notes don't make sense to me i'm going to have to go through them again tomorrow and fix them up04:47
=== amitk-afk is now known as amitk
=== amitk is now known as amitk-afk
ograapw, ouch ...11:59
ograapw, we have another package description pointing wrongly to versatile ...11:59
ograapw, omap4 meta seems to still have it12:01
=== amitk-afk is now known as amitk
=== ivoks-afk is now known as ivoks
dokoapw: who can tell me about binary demotions of kernel packages? see http://people.canonical.com/~ubuntu-archive/component-mismatches.txt15:07
dokoor ogasawara ? ^^^15:07
tseliotsmb: do you know if all of the releases of 2.6.36 have the security fix with arch_compat_alloc_user_space?15:08
smbtseliot, Some of the early -rc's may not but the change is in upstream now15:10
smbtseliot, commit was between -rc4 and -rc515:11
tseliotsmb: ok, as I'm facing a weird issue with fglrx (from maverick) and (mainline) 2.6.36 where the workaround doesn't work (i.e. the compilation flag seems to be ignored. Therefore I was wondering if I should just replace compat_alloc_user_space with arch_compat_alloc_user_space when dealing with 2.6.36 kernels (I can do this with DKMS)15:12
tseliotsmb: make.sh.log: http://pastebin.ubuntu.com/499734/15:13
smbtseliot, That seems to use a slightly different approach. Is the code in fglrx expecting COMPAT_ALLOC_USER_SPACE being a macro?15:15
tseliotsmb: make.sh sets COMPAT_ALLOC_USER_SPACE to either of the compat_alloc_ functions and then passes it as a flag (MODFLAGS). Then the C file calls COMPAT_ALLOC_USER_SPACE(size)15:18
smbtseliot, I would be a bit careful with the wholesale replacement. You never know against which 2.6.36 you may be compiling against and -rc4 and earlier are different. 15:18
tseliotyes, that's why I asked15:18
smbThe error sounds like something generally goes wrong with the definition15:19
smband COMPAT_ALLOC_USER_SPACE is not replaced at all15:19
tseliotsmb: make.sh http://pastebin.ubuntu.com/499736/15:20
* tseliot nods15:21
tseliotsmb: the weird thing is that the same fix works with 2.6.3515:22
smbtseliot, This is quite odd. I'd probably try to modify the Makefile in the subdir to pass V=1, to see what gets passed in15:23
tseliotok, thanks15:24
smbIt feels like the flag might not get repeatedly get set in there15:24
tseliotsmb: I definitely don't see it there: http://pastebin.ubuntu.com/499750/15:33
smbNo if it had been correctly passed on it would be somewhere around PAGE_ATTR_FIX15:35
smbHave you checked the module Makefile? At least thea PAGE_ATTR_FIX was set again there for the kernel compile15:35
tseliotsmb: wait a second, PAGE_ATTR_FIX is being passed while COMPAT_ALLOC_USER_SPACE is not: http://pastebin.ubuntu.com/499752/15:38
tseliotbut why does this work with 2.6.35???15:38
smbtseliot, i bet if you add something there it works. I am as puzzled as you may be.15:39
smbIt should not work there either15:39
tseliotsmb: yes, that's what I'm trying now15:39
ogasawaradoko: I'll send email to the team asking about the binary demotions of kernel packages.  most of the guys are in taipei at the moment.15:44
dokoogasawara: thanks!15:44
tseliotsmb: something's definitely wrong. I doesn't fail now but COMPAT_ALLOC_USER_SPACE is set to nothing http://pastebin.ubuntu.com/499762/15:50
tseliotsmb: and I added a -DCOMPAT_ALLOC_USER_SPACE=$(COMPAT_ALLOC_USER_SPACE) in the makefile15:51
tseliotO_o15:51
smbI am not sure, but make.sh currently adds it to MODULE_FLAGS which I am not sure about how this gets passed on15:52
smbI think in the code i saw it was passed on differently15:53
tseliotyes, I really should ask AMD how they pass MODULE_FLAGS15:54
=== ivoks is now known as ivoks-afk
smbtseliot, Maybe its something that used to be in older kernels... let me check something15:55
tseliotoh15:56
tseliotthat would explain it15:56
smbtseliot, Seems AMD got hit by progress again "kbuild: allow assignment to {A,C,LD}FLAGS_MODULE on the command line" in 2.6.36-rc1 replaced MODFLAGS with something new16:01
smb"Note2: MODFLAGS was not documented and is dropped16:02
smb    without any notice. I do not expect much/any breakage16:02
smb    from this."16:02
tseliotsmb: d'oh16:02
tseliotsmb: do you have a link to the commit?16:03
smbtseliot, 6588169d516560f68672e2928680b71c647b7806 (a second I try to give you a link)16:04
smbhttp://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=6588169d516560f68672e2928680b71c647b780616:05
tseliotthanks16:05
tseliotI guess a new workaround will be required for 2.6.36. The one we adopted in lucid and in other releases should work16:06
smbYeah, generally EXTRA_CFLAGS is probably longer supported. Even if not as specific as CFLAGS_MODULE now. But its an out of tree module anyway16:08
tseliotsmb: ok, so I guess that any other flag that relied on MODFLAGS will have to be passed in a different way (not only the one we need)16:13
smbright16:13
scottie#freeipmi16:13
tseliotsmb: thanks for your help16:19
smbtseliot, np. At least the riddle is solved and we can sleep over the weekend. :)16:19
tseliotsmb: hehe16:20
tseliotsmb: replacing MODFLAGS with CFLAGS_MODULE did it. Now I can sleep at night :D16:31
smb:D16:31
=== ivoks-afk is now known as ivoks
mpoirierogasawara: mumble ?17:01
ogasawarampoirier: sure, just a sec17:02
david506Once an LTS is released, will changes by made to the kernel that would require me to recompile drivers ( modules ) that were compiled against an earlier kernel in the same LTS version ?17:16
smbdavid506, yes, there can be changes that require to re-compile (everytime the abi number changes)17:19
=== jj-afk is now known as jjohansen
david506Why would the ABI number change ?17:20
david506ok, so if the ABI number changes, can I recompile that same source with no modifications ?17:21
ogasawarajjohansen: just curious if you have the AppArmor patches pushed to a branch that I can look at.  I just need to write up a proposal for the 0-day kernel upload and wanted to get the specific details for your patches.17:21
jjohansenogasawara: not pushed to a branch but I can send you them17:22
jjohansenboth are 1 line17:22
jjohansenonly one is needed 0 day17:22
ogasawarajjohansen: ok cool, just send me the one we need for 0 day17:22
jjohansenokay17:22
smbdavid506, because some functions or structure have changed. Usually yes. There can be rare changes that make the compile fail. But this should be really rare. The only reason it happened now once was for non-gpl modules17:23
ogasawarajjohansen: you can even just pastebin it if you want17:23
smbdavid506, So I would rather say, there should be no need to change the source of the module in 99.9% of the cases17:24
david506but a severe security issue might warrant that .1% ?17:25
erUSULFWIW this was david506 original question in #ubuntu 18:11 < david506> I have a hardware provider who might agree to run a repository for their specialized software or have  their software included in the LTS versions, but I need to know what changes normally happen in the  kernel in LTS versions.17:25
erUSULdavid506: hope you do not mind17:25
smbdavid506, correct17:25
david506:)17:26
david506I was trying to make the question more clear :d17:26
david506If you could give me links to any relevant web pages to my questions, I would be most happy to read them17:26
smbI would not know of a page where this written down in detail. Not that you probably could. It also depends on what you modules rely on. If it is only using more general kernel interfaces change will less likely happen than if someone tries to hook to a very specific module17:30
=== bjf[afk] is now known as bjf
bjfjjohansen, when you have some time, i have a question (no hurry)17:33
jjohansenbjf: shoot17:34
bjfjjohansen, i'm rebasing Lucid ec2 and ran into a conflict17:35
jjohansenbjf: big surprise17:36
bjfjjohansen, actually, it went pretty smooth, just a couple of minor conflicts and only one that i have a question about17:36
jjohansenokay17:37
bjfjjohansen, sorry wife interrupt17:38
jjohansennp17:39
jjohansenI have server team interrupts17:40
bjfjjohansen, can you look at the diff of kernel/irq/manage.c between the master branch and the ec2 branch?17:41
bjfjjohansen, to resolve the conflict, i went with the ec2 version17:41
jjohansenbjf sure where is it17:41
bjfjjohansen, my public ubuntu-lucid has it17:42
bjfjjohansen, actually, let me just pastebin it17:42
bjfjjohansen, http://pastebin.ubuntu.com/499829/17:43
bjfjjohansen, i went with this patch17:44
jjohansenbjf what did the upstream patch look like17:46
* jjohansen is stunned the conflict was this small17:46
jjohansenbjf: so I redid the abi check following your guide, it worked fine when I built on tangerine but failed again when I pushed to the ppa17:49
jjohansenogasawara: how much do you know of kernel ppa builds, and abi checks?17:49
ogasawarajjohansen: I know a bit about the abi checks, the kernel ppa builds though might suffer from bug 64565317:50
ubot2Launchpad bug 645653 in linux (Ubuntu) "kernel is unbuildably large (affects: 1) (heat: 8)" [High,Confirmed] https://launchpad.net/bugs/64565317:50
jjohansenhrmm, I suspect it is different as the abi check comes after the build and the failure log clearly shows its failing in the abi check17:51
ogasawarajjohansen: have a pointer to the build log17:51
ogasawara?17:51
jjohansenthis is on a lucid kernel btw17:51
jjohansenhttp://launchpadlibrarian.net/56380500/buildlog_ubuntu-lucid-amd64.linux_2.6.32-27.47~debugmpath_FAILEDTOBUILD.txt.gz17:52
ogasawarajjohansen: this looks a little weird... you're buildling version 2.6.32-27.47~debugmpath but it's looking for ABI 2.6.32-26.45?  I'd think it'd look for 2.6.32-26.4617:57
ogasawarajjohansen: does debian.master/abi/2.6.32-26.45/amd64/server actually exist in your tree?17:57
ogasawarajjohansen: and did it get committed?17:58
jjohansenogasawara: no, it got moved to 2.6.32-27.4717:58
jjohansenogasawara: basically I skipped up a couple abi drivers for testing the mptsas and skipped .26 entirely18:00
ogasawarajjohansen: ah, so you really need the previous ABI directory to exist for the current build to check against18:00
smb-26 ... -27 how far in the future is htis?18:00
jjohansensmb: its not, its something that will die18:01
jjohansensmb: its for testing only18:01
ogasawarajjohansen: but you mentioned it built ok on tangerine?18:01
ogasawarajjohansen: which is odd18:01
jjohansenyes, that is what puzzels me18:01
smbjjohansen, Not sure you are aware but as long as you would make your tree so the previous version remains what it was you only need to bump the abi once18:01
jjohansensmb: right, I recognize that it only needs to be bumped once18:02
smbjjohansen, Did you clean completely between the builds?18:02
ogasawarajjohansen: mind if I look at your tree on tangerine?18:02
smbon tangerine18:02
jjohansenbut I noticed a stable update bump the abi during testing and figured I would get ahead of that one a wee bit18:03
jjohansenogasawara: hrmm, you could but its changed since then18:03
jjohansenbasically, it worked and I pushed and moved on to the next thing I wanted to test18:04
jjohansensmb: I did an fdr clean18:04
jjohansenI followed the abi doc as close as I could18:04
jjohansenI had a couple of odd things happen like18:04
smbjjohansen, So if you look at you tree and do a fdr printenv, it would tell you what it considers as the previous release. Then the tree needs an abi dir for that18:04
jjohansenfakeroot debian/rules insertchanges doesn't insert changes for me18:05
smbHm, an fdr clean would remove any generated abi i believe18:05
ogasawarajjohansen: ah, so on tangerine ubuntu-lucid/debian.master/abi/2.6.32-26.45 exists (but is not committed)18:06
jjohansensmb: okay, so that may be it, if it is looking for a previous abi and I skipped 2 would it be smart enough to figure that out18:06
smbinsertchanges relies on the previous version being commited with a message that is formatted "UBUNTU: Ubuntu-..."18:06
ogasawarajjohansen: explains why it passes on tangerine18:06
jjohansenogasawara: really?  weird okay that explains it then18:06
jjohansenalright I take another stab at this.  /me wonders why that one exists18:07
ogasawarajjohansen: well according to git status it's not committed in that tree on tangerine18:07
smbjjohansen, It gets created on compile. Though I had been thinking clean would remove it18:07
ogasawarajjohansen: not sure if you used that tree or another to build the package to upload18:07
smbbut maybe that is not ture18:07
jjohansenogasawara: yeah no, it wouldn't be.  I git am'd the patches from my local tree18:07
jjohansenso, it doesn't exist locally, on tangerine I built the latest pull, then branched to an earlier state and applied my patches as I didn't want any extra changes in while testing.18:09
* jjohansen doesn't get why an fdr clean didn't clean up abi cruft is all18:09
jjohansenthanks guys, at least it makes sense now18:09
ogasawarajjohansen: yah, I usually just to a git clean -f -d since fdr clean doesn't remove everything (and actually generates some files)18:10
jjohansenyeah18:10
smbor even git clean -d -f -x18:11
ogasawarasmb: heh, I like that you pass your flags alphabetically18:13
smbogasawara, And I did not even try to. :)18:13
ogasawarahaha18:14
smbWell, before I order all letters I type alphabetically... Its Friday and past 7pm, I think I be gone18:15
ogasawarasmb: yes, it's well past beer:30 for you18:19
bjfsmb, which patches did you want on ti-omap?18:21
sconklin so, kamal taught something great yesterday - meld understands git repos. If you didn't know this, check it out. "meld ./" works in a git repo and shows your changed files.18:29
jjohansensweet18:33
achiangwhat is meld?19:19
kamalsweet GUI diff utility...   apt-get install meld   (you'll like it :-)19:19
bjfachiang, it's like taking a flame-thrower to your code :-)19:20
=== bjf is now known as bjf[afk]
achiangkamal: are you saying that the output of diff -Nurp is insufficient? ;)19:21
achiangj/k, the tool looks pretty cool19:21
kamalachiang: far be it from me to ever cast aspersions upon the glory of diff -- just another tool^Wflamethrower for the toolbox :-)19:23
achiangfwiw, when i need to see visual differences, i use gvimdiff19:24
achiangbut maybe i'll try meld next time19:24
* ogasawara lunch20:10
* jjohansen lunch21:16
th1hi22:03
th1I'm building a custom kernel with patches to support pv-hvm for better performance under XenClient. how do I go about creating a new flavour like there is the -pae one currently?22:04
th1I followed the instructions on https://help.ubuntu.com/community/Kernel/Compile for how to get the sources and patched them from there22:04
th1sorry after reading http://blog.avirtualhome.com/2010/05/05/how-to-compile-a-ubuntu-lucid-kernel/ I'm unstuck..22:10
=== MTeck is now known as MTecknology
=== yofel_ is now known as yofel
=== ivoks is now known as ivoks-afk

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