=== Laibsch1 is now known as Laibsch === |Mike^| is now known as Mike^ [14:36] NCommander: http://pastebin.mandriva.com/17521 [14:36] dmart: ^ [14:36] lool: ? [14:36] dmart: Mandrive patch by rtp (Arnaud Patard) [14:36] Which works for him [14:37] It will work if the called function only has 4 args or less [14:37] But if there are more, the saved lr will appear in the middle of them, messing things up. [14:37] I don't know whether this framework is necessarily supposed to work for arbitrary numbers of args, but I'm assuming it is. [14:37] dmart: I guess so as well, it's meant as a generic bridge [14:38] I'm asking him to join #ubuntu-arm here [14:38] Other than the issue of where on the stack lr is stored, the patch is effectively the same as mine, so it's reassuring that it works. [14:38] dmart: I found what crashes valgrind on startup [14:38] I have disappear to a meeting right now :( but I should be back in ~1hr or so [14:38] rtp: dmart and NCommander are around and looked at your changes [14:39] rtp: dmart will be back in an hour (meeting), but says the function probably has to work for any number of args [14:39] lool: great. :) [14:39] lool: ok. [14:39] lool: I thought ARM already submitted a fixed version [14:39] NCommander: did they? [14:40] NCommander: where? [14:40] NCommander: the patch in comment #69 doesn't work on my system [14:40] lool: it was on the bug, give me a sec [14:41] rtp: hrm, worked here, but that may be a fluke. [14:41] rtp: lool: seems the patch never made it to LP, hold on, I have to go fishing for it [14:41] NCommander: fwiw, I'm not running ubuntu on my systems :) [14:42] rtp: lool: http://pastebin.ubuntu.com/395620/ [14:42] rtp: The patch wasn't distro-specific : it ought work on Mandriva as well. [14:42] rtp: of course not, but solving this bug should solve it everywhere. I know Debian, and Gentoo also were watching for a fix [14:43] armin76: Did you test the patch? [14:44] NCommander: will try this patch. Would bee nice to get it on the bug so that everyone can follow and test :) [14:44] rtp: well it hasn't even been tested yet so lets not jump the gun so quickly ;-) [14:44] rtp: I didn't even know Mandravia had an ARM port [14:44] dmart: commented on the valgrind bug; happy if you have suggestions to avoid "mvn" on Thumb 2 [14:45] NCommander: my intent was not to blame people at all. sorry [14:45] rtp: no blame taken. I should have made sure the patch made it onto LP. That was just a breakdown in communications [14:45] NCommander: our arm port is something new [14:45] NCommander: even untested, the patch belongs in the bug report [14:45] Where I've sentit now [14:45] lool: thanks [14:45] rtp: In the bug now, sorr [14:46] y [14:46] * lool & [14:46] lool: thanks :) [14:47] * NCommander hasn't run Mandriva since it was Mandrake [14:51] well, at least, you know that Mandriva was Mandrake before [15:03] NCommander: your pastebin patch is working here. [15:04] rtp: ARM will be thrilled to know that. Do you have upstream commit access to ooo-build [15:05] NCommander: no, I've no access to it. Maybe the debian ooo guy as commit access ? [15:05] s/as/has/ [15:05] rtp: I have someone I can poke to commit it :-) [15:05] :) [15:06] rtp: can you just comment that on LP? I'll nudge ccheney to replace the patch in ooo-build later today [15:07] NCommander: I've no account on LP :) [15:08] rtp: Are you sure? For a while LP was creating accounts for all sorts of open-source developer sorts (you may not know your password) [15:08] rtp: Check at https://launchpad.net/people [15:11] persia: it looks like it created something when I was "playing" with gnome-control-center. I'll prefer using my mail @work [15:11] rtp: If you can log into that account, you can feed it a different address. [15:12] ok. [15:20] persia: looks like it worked. thanks [15:20] rtp: No problem. Thanks for stopping by and helping out :) [15:21] rtp: Feel free to ask if you run into anything with which we can help. [15:31] persia: well, this bug was annoying a lot of people and needed to be fixed imho. I regret I was not able to look at it before the end of last week :( [16:12] NCommander, rtp: Hi, I can discuss the OOo fix thing now [16:16] lool: What was the background to the Valgrind issue? [16:38] dmart: rtp tested the revised fix, and it works on mandriva [16:39] rtp's own patch has the right idea, but appears incorrect in a couple of ways. [16:42] dmart: I've not played with arm asm since some years so it's possible there are bugs. I've mostly played with mips asm theses days :) [16:42] Fair enough [16:44] The issues were a) you should keep the stack 8-byte aligned when calling functions (so you need to push an even number of regs, or adjust sp)... [16:45] and b) I think the bridge is intended to work for functions with > 4 arguments. The way you push lr causes it to appear in the middle of the args, which is likely confuse functions with >4 args. My version just pushes lr last; otherwise the logic is much the same. [16:51] I see... === admin_ is now known as Meizirkki [18:38] Is there any reason for lucid alpha3 to be relatively 'slow' out of the box compared to eg. angstrom for arm7? [18:38] It feels rather sluggish, I do hope it's not march=armv-7? [18:40] it is v7 and thumb2 by default [18:41] on what HW do you see that ? beagle ? [18:42] Another OMAP3430 board (Samsung H1 smartphone) [18:43] (I borrowed the built rootfs from beagle) [18:43] well, we dont see any sluggishness on any of our supported boards (actually quite the opposite) [18:44] Odd [18:44] so it might be a kernel feature you are missing that the lucid userspace relies on [18:44] Also, is there a particular operation that is sluggish? [18:46] Well, 'general' command line use using the serial/ethernet gadget (latter using sshd). Perhaps I should try to get some figures [18:47] Have to admit I'm running root off sd, so perhaps angstrom used tmpfs for some bits or so. [18:49] persia: which patch? [18:50] armin76: The OOo patch for gcc 4.4 [18:51] persia: nobody told me anything, so no [18:51] but if it works for you there's no point me testing it :) [18:52] armin76: Not for testing: I thought you might want it for application. [18:53] If you had tested, that's a nice data point, but sharing is always good :) [18:53] nah, i just tested it because NCommander told me to [18:53] but thanks for the offer, i appreciate it :) === powderluv_ is now known as powderluv [20:11] http://blog.laptopmag.com/hands-on-marvell-armada-powered-smartbook-does-1080p-video [20:11] interesting. [20:12] Though, hinge that doesn't open to 180 degrees? bleh. [21:02] rtp: sorry, thought I had pushed the patch, but the connection had been reset; thanks for adding it [21:04] ok, seeing something unusual here during install... it appears that kernel messages are passing through to X, and I'm getting text artifacts on the screen from it [21:04] grabbing a screencapture won't see it, and moving the mouse across the affected areas redraws it correctly [21:04] rather annoying though [21:05] anyone seeing that on the current lucid images? [21:05] GrueMaster: ^ [21:06] I haven't seen it yet on Dove, but I had another issue to attend to. [22:16] weird... wine-ing "gpu caps viewer" reports a DIFFERENT version of MESA than native glxinfo reports! [22:16] 1.5 Mesa 7.7.1-DEVEL [22:16] er, wrong tab. [22:16] (a.k.a. wrong channel) === powderluv_ is now known as powderluv