/srv/irclogs.ubuntu.com/2009/04/07/#ubuntu-kernel.txt

=== k-s is now known as k-s[AWAY]
=== __Purple__ is now known as _Purple_
=== thunderstruck is now known as gnomefreak
=== mdz_ is now known as mdz
=== thunderstruck is now known as gnomefreak
=== k-s[AWAY] is now known as k-s
=== k-s is now known as k-s[AWAY]
=== k-s[AWAY] is now known as k-s
=== k-s is now known as k-s[AWAY]
Shockhi, I have changed a config option, built source package with "debuild -S" and uploaded the package to launchpad. the build fails with "EE: Previous or current ABI file missing!". how can I fix this?13:53
rtgpgraner: CONFIG_USB_SERIAL=y bug number?14:40
=== k-s[AWAY] is now known as k-s
manjosmb_tp, the kernel prints out so much info that its hard to run the suspend resume tests with it 14:54
smb_tpmanjo, Ok, feared so much. I remove most of it and get you something lighter14:55
manjosmb_tp, also I could not figure out what it was printing coz it was printing so much stuff... 14:55
manjoso looks like the drive reports ready but still in EBUSY state14:55
smb_tpmanjo, Yeah, I turned on the verbose debugging, but saw it probably is too much14:56
manjoit happens when you suspend without power cord attach and attach the power cord after you suspend and then resume with the power cord still plugged in14:56
manjoif you give me a clue I can put some debug printfs 14:57
smb_tpHm, thats interesting14:57
manjodo you have any hints ? 14:58
manjoI think I found some points where I want to print info reading the code yesterday14:58
smb_tpI need to play around this anyways as we want/need some debugging there. But I would start around the soft/hard reset functions14:58
manjonow I have a fair understanding of libata stuff to start putting printfs14:58
manjosmb_tp, yeah that is what I had in mind14:59
smb_tpWon't prevent you from doing that then. :)14:59
manjo:)15:00
tjaaltonapw: hi, what about bug 224642, any chance to get it in before the freeze?15:04
=== nturner_ is now known as nturner
rtgtjaalton: lest you feel ignored, apw just asked me if he could push the patch for that bug.15:14
apwrtg thanks ... just got bounced from the network :(15:15
apwtjaalton, thanks for the prod.  i've pushed that change up and it should make the kernel15:17
manjoapw, smb_tp did we cancel the meeting for today?15:19
apwhrm, we cirtainly didn't actually do anything to cancel it15:19
smb_tpI thought I had written an email...15:19
manjosmb_tp, yeah see that 15:21
manjothansk15:21
tjaaltonrtg, apw: great, thanks!15:44
=== k-s is now known as k-s[AWAY]
=== k-s[AWAY] is now known as k-s
liwsysklogd looks for a symbol matching Version_[0-9]+ in System.map, but Ubuntu (intrepid, jaunty) doesn't seem to include one; Debian (lenny) does; anyone know what's up?18:59
liwthis is related to #27792418:59
loolliw: We had one in /boot/System.map-2.6.24-19-generic; perhaps something which was dropped upstream?19:03
liwI don't have a changelog.Debian that reaches that far back in time, only to 2.6.26-2.619:09
infinityliw: What kernel version is lenny shipping that has this symbol?19:10
liwinfinity, 2.6.2619:10
infinityliw: Ahh, hrm.  In the LP bug, all the reported METOOs are from 2.6.27+, so it may well be an upstream change.19:11
liwinfinity, would you be able to easily check that?19:14
infinityI have no local git love here.  Haven't worked on the Ubuntu kernel in ages. :/19:14
infinitySo, no, not any more readily than you could.19:14
infinityliw: There's a random claim from Russell King that on kernels with KALLSYMS, we should be invoking klogd with "-x" to skip the System.map lookup entirely.19:18
infinityliw: But, we had KALLSYMS=y back in hardy too, where it seemed to work fine.19:19
smb_tpliw, Hm, what do you mean exactly by Intrepid and Jaunty do not have a System.map?19:21
liwsmb_tp, they don't have a symbol that matches Version_[0-9]+ in their System.map, which is what sysklogd wants to find19:22
liwin order to figure out which kernel version the map is for, I think19:22
smb_tpliw, Ah, ok. Got it19:22
liwinfinity, indeed, in my hardy VM, with 2.6.24, System.map is correctly loaded and the symbols it there19:23
liwI haven't had any real understanding about building kernels since 0.97 or so, so I'm a bit lost here19:23
infinityliw: Well, I'd begin by a violent recursive grep of the hardy kernel source for the string "Version_", find out where the symbol comes from, then check intrepid/jaunty to see if that codepath went away, or if we're accidentally optimising it away with a certain config option.19:25
smb_tpinfinity, yeah. currently looking19:25
infinityliw: Out of curiosity, does this missing symbol (and klogd not getting to load System.map) actually lead to any real bugs, or just ugly log output? :P19:27
liwinfinity, that's a good question :)19:27
infinityliw: I mean, I'd assume it prevents klogd from doing pointer lookups to name symbols... But I have no idea why we'd care.19:27
infinityliw: Assuming that programs that actually make use of System.map (modutils, ksymoops, that sort of thing) are parsing it correctly, I'm not entirely sure that the klogd thing matters at all, and maybe we should just be invoking it with "-x" to skip the System.map read entirely.19:30
infinity       -x     Omits EIP translation and therefore doesn’t read the System.map file.19:30
smb_tpI would think that missing the right System.map has simmilar effect19:31
liwif we don't care about System.map being found by sysklogd, we don't need to do anything, really19:32
smb_tpBut since I already saw stack traces with symbol names there must be an alternate way19:32
infinitysmb_tp: Oh, I'm sure the end result is the same, just that one avoids an Sky is Falling error message. :)19:32
infinitysmb_tp: I suspect the symbol names are coming from KALLSYMS=y19:32
infinitysmb_tp: (As noted, users of KALLSYMS should be loading klogd with -x anyway)19:32
infinityliw: Optionally, if you can find a clever way to detect KALLSYMS is in effect, you could patch klogd to print a friendlier "KALLSYMS kernel loaded, skipping System.map load", instead of "OH GOD, NO SYSTEM.MAP, HALP!"19:33
liwinfinity, I think I'd rather see someone who understands what you just said do that :)19:34
smb_tpinfinity, that would be a plausible explanation. 19:34
liwthe bug was marked as medium priority by colin king, so I should perhaps ask him why19:35
infinityAnyhow, from what I can tell with some basic research and a bit of futzing in the code, I think the error message (for us, anyway) is entirely harmless.  Just a bit scary.19:35
smb_tpinfinity, Well the message is just plain "Cannot find map file" and then "Loaded x symbols from y modules". Not someting that let me panic19:35
infinitysmb_tp: "Cannot find" on a file that clearly exists is cause for panic for a lot of people. :)19:35
infinitysmb_tp: "Not loading, because of $reason" is a bit friendlier.19:36
infinity(The number of people METOOing the bug sort of backs up my assertion)19:36
liw(the real fix would might be to switch to a syslogd written by people who know C I/O)19:36
infinityliw: Our klogd works well enough.  *shrug*19:37
smb_tpinfinity, Agreed, it is slightly misleading19:37
infinitysmb_tp: No one likes the second line in their boot sequence to be "I can't find ur filez noob"19:38
liwanyway, I'm done for today, thanks for you help with this19:38
smb_tpinfinity, We could suggest "Confused by the contents of the file"19:39
infinitysmb_tp: Well, I'm curious if the Version_* symbol goes away only on KALLSYMS builds, or if it's gone completely.19:41
infinitysmb_tp: If it's gone completely, then we do have a real bug here, whether in the upstream kernel or in our klogd, where things will be suboptimal for users building their own kernels.19:41
smb_tpinfinity, Yeah I am trying to find out for curiosity. Somehow it is not quickly grepable19:41
infinitysmb_tp: If it's only gone on KALLSYMS builds, I call it a non-bug.19:41
infinity(Though the error message going away or being re-written would be swell)19:42
infinityliw: I tossed a pithy followup on the bug with very little technical explanation, because I didn't feel like typing. :P19:42
infinityliw: If you want more info there, there's plenty of backscroll here to cut and paste. :)19:43
smb_tpI would very much suspect if cking put it as medium, then its just a nuisance and probably time was lacking to bother19:44
infinitysmb_tp: On the flipside, this could be seen as an accidental optimization.  If we have symbol lookups built into the kernel, it's a terrible waste to have them also loaded into klogd's (probably non-pageable) memory.19:46
smb_tpinfinity, It makes much sense. Why have that twice. Maybe just an oversight to _not_ display the message.19:48
=== zul_ is now known as zul
smb_tpinfinity, hah, yeah...19:51
smb_tp#ifndef CONFIG_KALLSYMS19:51
smb_tp#define version(a) Version_ ## a19:51
smb_tpin init/version.c19:52
infinitysmb_tp: Perfect, that's what I was hoping for.19:52
* infinity follows up to the bug again.19:52
smb_tpcommit 197dcffc8ba0ea943fee86e28e99cd957579977219:55
smb_tpinit/version.c: define version_string only if CONFIG_KALLSYMS is not defined19:55
infinitysmb_tp: Thanks for the backtracking.  I've updated the bug to confirm that it's harmless, and to suggest making klogd's error message disappear or be made friendlier.19:56
smb_tpinfinity, Ok, cool.19:56
=== emgent_ is now known as emgent
=== bradf is now known as fdarb
rtgapw: kirkland says he is stable now without the i945 patch.23:38
apwyes he said that to me too23:38
rtgapw: are you doing a different patch for it?23:39
apwthe patch cannot be the trigger for pgraner's issue as it wasn't in .3823:39
apwand the reports of the compiz issues explicitly were seen on .3823:39
apwbug 35695123:40
ubot3Malone bug 356951 in compiz "Compiz sometimes hangs the machine" [High,Confirmed] https://launchpad.net/bugs/35695123:40
apwwe have no clue as to cause of his issue at this time23:40
rtgapw: hmm, so you're thinking that the i945 patch had nothing to do with kirkland's stability?23:42
apwsorry ... i am confusing you with use of him23:42
apwkirkland was ok on .38 and ok with 41 with the revert,. this is consistant with it being the BCHBAR patch as it was .39 material23:43
apwits not consistant with pgraner's issue as that is seen in .38 which does not have it23:44
apw(it == the patch)23:44
rickspencer3apw: could that not mean (as bryce points out to me) that pgraner suffers from yet another issue?23:44
rtgapw: so, you're OK with reverting the BCHBAR patch?23:45
apwright, that is what i was trying and failing to communicate, that pgraner has a different issue23:45
apwi think kirkland is sufficient reason to be suspicious of the patch and revert it regardless23:45
rtgapw: got it. will do.23:46
apwrtg ack23:46
brycertg, apw: if you have any other questions on that patch, jbarnes has popped in here23:50
rtgbryce, jbarnes: Its been shown to be the culprit for hard lockups on kirkland's laptop (which I think is i945).23:52
jbarnesrtg: at boot or during use?23:52
jbarness/boot/i915 load/23:52
rtgapw built a test kernel with only the BCHBAR patch reverted. it locks fairly soon after GDM starts, certainly within a few minutes.23:52
jbarnesMCHBAR I hope?23:53
apwyep23:53
rtgjbarnes: I think so, but he's busy at this conference using his laptop, so its hard to get much access to  it.23:53
apwwe are separately suffering a compix triggered machine hang on a number of systems23:54
jbarneshm23:54
apwyep saw it with my own eyes, hard locks, compiz off23:54
jbarnesthe MCHBAR patch will allow for tiled rendering on more machines...23:54
apwrevert the patch and goodness ensues23:54
jbarnesso there may be a more generic tiling related problem23:54
apwand blow up machines23:54
apw:)23:54
apwjbarnes, does sound feasable23:55
jbarnesthat could be confirmed by running with the patch but adding option "tiling" "false" to xorg.conf's intel driver section23:56

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