srphowdy. Is there any way to compare kernel config params (i.e. the CONFIG_ bits in config-<kernel-version>) between two different versions?00:59
srpother than writing a program to do so or comparing by hand, I meant01:00
=== Mikee_C_afk is now known as Mikee_C
apwsrp (N,BFTL), not really, mostly by writing comparitors07:58
leighmanhey, trying to load a backported drm module on 14.04 using dkms but it says it is tainted10:02
leighmanis this to do with signing? can I enable signing somehow?10:02
leighmantaint code is 409810:02
leighmandrm and i915 module10:02
apwleighman, i think  it tells you why when  it taints, in dmesg10:17
apwthough it is very likley them not being signed is what it says10:17
apwyou cannot sign external modules as the key used to sign internal modules has been destroyed deliberatly10:18
apwwhy do you care that it is tainted10:18
leighmanapw: it disables trace events10:18
leighmandue to http://lwn.net/Articles/588799/ I think?10:19
leighman/proc/.../tainted gives 409810:20
leighmanwhich seems to be TAINT_OOT_MODULE and TAINT_FORCED_MODULE10:20
leighmanTAINT_FORCED_MODULE causing trace events to be disabled as in the lwn article10:20
leighmanso I think I might be out of luck?10:20
leighmanbut eg. vboxdrv does not appear in modinfo to be signed but is not listed as tainted10:22
apwno idea how vbox would avoid that, unless it specifically messes with those flags somehow10:23
apwleighman, it sounds like you have a valid usecase for the patch mentioned, no idea how ugly it would be to port10:23
apwyou might want to start a bug on this against "linux" so we can consider fixing that10:24
apwas we have a lot of OOT modules which tracing sounds useful in10:24
apwactually the patch looks pretty self contained, so it might be applicable10:24
apwleighman, ^10:24
leighmandoes look like a nice simple patch actually *he says*10:26
leighmanokay, will file a bug against linux and subscribe you?10:26
leighmanapw: ^10:28
apwlet me know the bug # here as i won't find it otherwise10:30
sorensmb: Thanks for looking into https://bugs.launchpad.net/ubuntu/+source/linux/+bug/135816210:31
ubot5Ubuntu bug 1358162 in linux (Ubuntu Trusty) "skb_get_rxhash prematurely renamed to skb_get_hash" [Undecided,Fix committed]10:31
sorensmb: I have a small request, though.10:31
sorensmb: Here's that patch I applied to handle it: https://github.com/JioCloud/contrail-vrouter/commit/c31ab9b1d05acb6814a3ee8ddda16882cdcc067210:32
sorensmb: After your patch gets applied, skb_get_rxhash will be #defined twice.10:33
apwthen i suggest you make yours say10:33
apw#ifndef foo10:33
sorensmb: What would you suggest? A more specific KERNEL_VERSION (does that even change between kernel revisions in Ubuntu)?10:33
apw#define foo10:33
sorenapw: That's what I was meaning to do, but I wasn't sure if that'd be sufficient (since it depends on import order).10:34
apwin the version check?10:34
apwsoren, though this new version will be a new ABI i am sure10:34
apw(our new version)10:34
sorenapw: How does that help me?10:35
apwyou know you only need the compat for that one abi10:35
sorenapw: Anyway, what I was wondering was if you could wrap your #define in #ifdef as well... Or am I completely off here?10:35
sorenapw: ...so that it won't depend on a specific order of #include's.10:36
leighmanapw: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/135967010:36
ubot5Ubuntu bug 1359670 in linux (Ubuntu) "Unsigned oot modules are wrongly tainted and trace events disabled" [Undecided,New]10:36
* apw suspects if anything we should #undef it10:36
sorenapw: I dunno. It might not be a problem at all. I just saw the change and wasn't sure if #ifndef'ing on my side would be sufficient.10:36
apwsoren, might not, i think you are lucky in that it has not been uploaded yet.  I suspect for us we should #undef before defining it to be sure we are getting what we expect.10:43
apwsmb, ^10:43
sorenapw: I was exactly hoping to get this addressed before the upload.10:44
sorenapw: Thanks for considering it.10:44
apwleighman, i assume this is 14.04 which is giving you trouble10:46
leighmanapw: yup10:46
apwleighman, ok i've pushed some test kernels to your bug, if you could do the do on those.11:07
leighmanapw: is there a linux-tools-3.13.0-35  - I get unsatisfiable dependency on linux-tools-3.13.0-35-generic11:35
leighmanapw: anyway taint bitmask looks correct and no messages in dmesg about disabling tracepoints11:36
apwleighman, that is good enough for me, i'll push that out for review11:36
leighmanapw: awesome, thanks11:38
leighmanapw: when would it land if so?11:38
apwif accepted, in the next SRU round, of the order of 3 weeks11:40
leighmanapw: cool,thanks11:41
=== JanC_ is now known as JanC
rtgstgraber, https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1344049/comments/317:14
ubot5Ubuntu bug 1344049 in linux (Ubuntu Trusty) "Please cherry-pick tc rule fix for userns" [Undecided,Fix committed]17:14
=== Mikee_C is now known as Mikee_C_afk

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