| 00:35:49 | *** | Saving seen data "./dancer.seen" |
| 01:00 |
| 01:00:30 | foolsh | __builtin: I bit of progress, if I change -O3 to -O2 optimization level I no long get a freeze and it actually gives me an error message "Unable to allocate wave buffer for sound 22 size 28478 9095 22050 7042!" |
| 01:14:07 | __builtin | Ahh... it's probably out of memory |
| 01:14:18 | __builtin | How much ram does the zip have? |
| 01:16:52 | | Quit Acou_Bass (Ping timeout: 272 seconds) |
| 01:18:58 | | Join Acou_Bass [0] (~Acou_Bass@cpc97736-bolt17-2-0-cust152.10-3.cable.virginm.net) |
| 01:19:48 | foolsh | Don't know can't find any reliable sources online but PLUGIN_BUFFER_SIZE is 0x80000 |
| 01:20:05 | foolsh | What target do you run it on ipodv6? |
| 01:21:25 | __builtin | Yeah |
| 01:22:26 | Bilgus | I wanna say the clipzip has around 16 mb |
| 01:22:28 | __builtin | Ah, it looks like it has 8 MB (see tools/configure) |
| 01:22:41 | __builtin | Probably on the low end even for wolf |
| 01:22:48 | __builtin | No way Duke will run on it |
| 01:23:06 | Bilgus | ah yeah thats right because it stores two copies of the rom and only loads one |
| 01:23:17 | __builtin | Hmm? |
| 01:25:07 | foolsh | memory=8 # not sure |
| 01:25:44 | foolsh | hmm I'm gonna try increasing this and see what happens |
| 01:26:12 | Bilgus | foolsh do you have the multiboot bootloader?? |
| 01:26:48 | foolsh | no but point me to it and I will |
| 01:28:04 | Bilgus | http://forums.rockbox.org/index.php/topic,51844.msg240418.html\ |
| 01:28:11 | foolsh | Ah thank you |
| 01:32:29 | | Quit Huntereb (Read error: Connection reset by peer) |
| 01:32:38 | Bilgus | np I tried to get it added to the actual builds but I never heard back from any of the head mucky mucks and I've also no idea how to upload stuff to the web server for staging either |
| 01:33:43 | | Join Huntereb [0] (~Huntereb@69.161.102.241) |
| 01:34:16 | Bilgus | also it looks like jhMikeS is either gone forever or extended so I'll probably be changing the way the MB firmware works with filesystems but whats on gerrit works for the time being |
| 01:43:10 | Bilgus | __builtin >>>> hmm... I tried out my ipod6.0G and the lcd flashes are completely gone... I that from the patches fixing my reversal of lcd always on off or something else? |
| 01:46:34 | __builtin | I commented out the lcd_awake() that seemed to be causing the problem, but the flashes persisted on my 7G (slim) |
| 01:46:52 | __builtin | yet on the 6.0G (older model), the flashes were gone |
| 01:47:04 | * | __builtin suspects it's a hardware issue at the core |
| 02:00 |
| 02:25:22 | foolsh | yeah nope clipzip definitely has only 8mb. Should ifdef out those targets with limited ram |
| 02:25:34 | foolsh | shame |
| 02:35:52 | *** | Saving seen data "./dancer.seen" |
| 02:52:14 | | Quit MrZeus (Ping timeout: 264 seconds) |
| 03:00 |
| 03:30:34 | __builtin | ulmutul (logs): I think I may know the problem |
| 03:31:04 | __builtin | With my assembly, that is. There's a corner case if r3=1 when calling the function |
| 03:31:41 | __builtin | after decrementing it once at the start of the function, it will wrap around and cause big problems |
| 04:00 |
| 04:12:15 | | Join dandels [0] (~dandels@unaffiliated/dandels) |
| 04:35:55 | *** | Saving seen data "./dancer.seen" |
| 04:45:00 | __builtin | jfc, this is some coincidence I've run into here |
| 04:45:58 | __builtin | the assembly bug I've been talking about was triggered by a sound exactly 7325 samples in length |
| 04:46:43 | __builtin | what's special about 7325, you ask? It's prime modulo *every single* power of two from 4 to 4096 |
| 04:47:12 | __builtin | had the sound had just one more or less sample, it never would've triggered the bug and I never would've caught it |
| 05:00 |
| 05:14:21 | | Quit TheSeven (Ping timeout: 250 seconds) |
| 05:14:50 | | Join TheSeven [0] (~quassel@rockbox/developer/TheSeven) |
| 05:35:44 | | Quit dandels (Ping timeout: 244 seconds) |
| 06:00 |
| 06:36:00 | *** | Saving seen data "./dancer.seen" |
| 06:42:40 | | Join amdj [0] (~aaron@freenode/staff/atheme.amdj) |
| 06:45:57 | amdj | I just want to say thankyou for such a great piece of software. I have a Sansa Clip+ glued to my car dashboard and plugged into the aux port and cigarette lighter socket for audio in the car, but it was continuing to play after turning off the car, needing me to turn it off manually or let it go on for hours, losing my mental place in the playlist until the battery died. I was |
| 06:45:57 | amdj | considering removing its internal battery, but Rockbox's "Car Adapter Mode" is a much better alternative. I've just flashed it and set that up, and it works. \o/ |
| 07:00 |
| 07:07:44 | | Quit Moarc (Quit: i znowu NADMUCHAŁ BALONA) |
| 07:11:28 | | Join Moarc [0] (~chujko@a105.net128.okay.pl) |
| 07:47:57 | | Quit flabrus (Ping timeout: 245 seconds) |
| 07:48:11 | | Join flabrus [0] (~beard@flab.tech) |
| 07:48:11 | | Quit flabrus (Changing host) |
| 07:48:11 | | Join flabrus [0] (~beard@unaffiliated/flabrus) |
| 07:48:47 | | Quit bluebrother (Ping timeout: 245 seconds) |
| 07:48:56 | | Join bluebrother [0] (~dom@rockbox/developer/bluebrother) |
| 08:00 |
| 08:36:02 | *** | Saving seen data "./dancer.seen" |
| 08:41:27 | | Quit TheSeven (Ping timeout: 264 seconds) |
| 08:44:21 | | Join TheSeven [0] (~quassel@rockbox/developer/TheSeven) |
| 08:53:22 | | Quit dys (Ping timeout: 245 seconds) |
| 09:00 |
| 09:18:32 | | Join dys [0] (~dys@2003:5b:203b:100:a64c:c8ff:fef4:13a6) |
| 10:00 |
| 10:29:42 | | Quit Huntereb (Ping timeout: 244 seconds) |
| 10:36:03 | *** | Saving seen data "./dancer.seen" |
| 10:54:12 | | Join Huntereb [0] (~Huntereb@d-69-161-102-241.va.cpe.atlanticbb.net) |
| 11:00 |
| 11:04:59 | vup | trying to build rockbox manually for the clip+ results in a "undefined reference to `find_first_set_bit'" for me |
| 11:05:33 | vup | manually adding build/firmware/asm/arm/ffs.o to the linker command make is work, at |
| 11:06:17 | vup | anybody have an idea for a proper fix? |
| 11:10:54 | | Join ZincAlloy [0] (~Adium@2a02:8108:9440:dfc:2437:f781:4cd4:56c) |
| 11:34:57 | | Quit foolsh (Ping timeout: 248 seconds) |
| 11:40:39 | | Join foolsh [0] (~quassel@89.187.181.131) |
| 11:43:25 | | Join pamaury [0] (~pamaury@maths.r-prg.net.univ-paris7.fr) |
| 11:43:25 | | Quit pamaury (Changing host) |
| 11:43:25 | | Join pamaury [0] (~pamaury@rockbox/developer/pamaury) |
| 11:51:52 | | Quit ZincAlloy (Quit: Leaving.) |
| 11:57:44 | | Join vmx [0] (~vmx@p5B2A2B5D.dip0.t-ipconnect.de) |
| 12:00 |
| 12:36:06 | *** | Saving seen data "./dancer.seen" |
| 13:00 |
| 13:03:13 | | Join dandels [0] (~dandels@unaffiliated/dandels) |
| 13:26:16 | | Quit dandels (Ping timeout: 245 seconds) |
| 13:41:36 | Bilgus | vup I just build for the clip+ like 5 minutes ago I'd say its your system.. |
| 13:42:20 | Bilgus | I'd try make clean |
| 13:56:21 | | Join massiveH [0] (~massiveH@ool-18e4e27c.dyn.optonline.net) |
| 14:00 |
| 14:02:05 | vup | tried make clean already |
| 14:02:29 | vup | Bilgus: which gcc version did you use? |
| 14:09:53 | gevaerts | If it's about missing files during link, it's not going to be a difference in gcc version I think |
| 14:10:28 | vup | well the strange thing is it links in libfirmware.a which according to `nm` contains the find_first_set_bit symbol |
| 14:10:58 | vup | so not really sure where its going wrong |
| 14:12:16 | gevaerts | A link order problem, maybe? That still doesn't say why it would go wrong for you and not for other people of course |
| 14:16:52 | gevaerts | Are you using -j or anything like that? |
| 14:17:40 | gevaerts | Versions here (according to the configure output): http://paste.debian.net/1092303/ |
| 14:18:46 | foolsh | vup: do you have ccache installed, I had some strangeness recently even after running "ccache -c" I had to uninstall it to compile correctly |
| 14:19:07 | vup | nope don't have ccache installed |
| 14:19:23 | vup | i am using -j$(nproc) |
| 14:19:34 | vup | but manually running the link command also runs into this issue |
| 14:20:18 | gevaerts | What's your output for ar t ./firmware/libfirmware.a ? Mine is https://paste.debian.net/1092304/ |
| 14:21:16 | vup | also thought about order, but `-lfirmware` is specified before `-lrbcodec` which is where it complains about the missing symbol |
| 14:21:36 | vup | ok i am using gcc 9.1.0 ;) |
| 14:21:40 | vup | thats quite a bit newer |
| 14:21:58 | gevaerts | My toolchain hasn't been updated in a while :) |
| 14:22:31 | vup | well configure also tells me that 9.1.0 isn't of the recommended 4.4.4 version |
| 14:23:37 | gevaerts | That may or may not be relevant. configure recommends a specific version because we know it works, but it's been a while since rockbox development was at its peak, so people haven't tested newer compilers extensively recently |
| 14:25:04 | vup | well fixing it manually by adding build/firmware/asm/arm/ffs.o to the linker command lets in run through and seems to work fine on the actual hardware |
| 14:25:27 | vup | this is my ar -t firmware/libfirmware.a output https://paste.debian.net/1092304/ |
| 14:26:10 | gevaerts | That seems to be the wrong link. That's my list :) |
| 14:26:57 | vup | oh yeah |
| 14:26:59 | vup | https://paste.debian.net/1092305/ |
| 14:27:31 | gevaerts | Not that it matters. They're exactly the same |
| 14:27:52 | vup | yep |
| 14:29:36 | vup | ah figured it out |
| 14:29:51 | vup | using −−thumb makes this error occur |
| 14:30:06 | vup | without that it works fine |
| 14:30:53 | vup | which is kinda sad, because −−thumb brings rockbox.sansa down to 497k from 634k |
| 14:35:28 | * | gevaerts tries |
| 14:36:08 | *** | Saving seen data "./dancer.seen" |
| 14:36:11 | gevaerts | Yes, same problem here then |
| 14:39:07 | vup | interesting so nothing todo with my gcc version |
| 14:42:53 | gevaerts | It works if I put -lfirmware between -lrbcodec and -lunwarminder |
| 14:43:00 | gevaerts | It fails if it's anywhere else |
| 14:46:16 | vup | can confirm |
| 14:49:39 | gevaerts | "You are in a maze of twisty little makefiles, all alike" |
| 14:53:57 | vup | ;) |
| 14:54:14 | | Join ZincAlloy [0] (~Adium@2a02:8108:9440:dfc:79ea:9da:aac8:ee99) |
| 14:54:49 | vup | well the linker invokation is exactly the same between a −−thumb and a normal build (apart from using the ../tools/thumb-cc.py wrapper) |
| 14:55:12 | | Quit massiveH (Quit: Leaving) |
| 14:55:27 | gevaerts | Yes, I don't actually understand why that makes a difference |
| 14:56:17 | vup | yeah me neither |
| 14:57:14 | gevaerts | For clip+, a non-thumb build also works with -lfirmware between -lrbcodec and -lunwarminder, so if that also works for all other targets, changing the order might be possible |
| 14:57:28 | gevaerts | At least if someone manages to understand the makefiles enough :) |
| 15:00 |
| 15:01:53 | vup | hmm putting `-lfirmware` twice with the second one being after `-lrbcodec` also works |
| 15:02:01 | vup | that seems a little easier to add in the makefile |
| 15:02:35 | gevaerts | Getting this wrong makes the list of link erros longer than my scroll buffer :) |
| 15:06:05 | gevaerts | https://paste.debian.net/1092318/ makes it work for me for clip+ thumb and non-thumb |
| 15:06:22 | gevaerts | I'm going to try some other targets to see if it breaks anything obvious |
| 15:11:17 | vup | awesome, works for me aswell |
| 15:18:28 | fs-bluebot | Build Server message: New build round started. Revision b0de98a, 280 builds, 9 clients. |
| 15:18:34 | gevaerts | OK, let's hope nothing breaks :) |
| 15:24:07 | | Join speachy [0] (40eebded@64.238.189.237) |
| 15:27:24 | speachy | FWIW I was building my clip+ firmware with GCC 4.9 for a while. |
| 15:27:37 | speachy | generated a smaller image IIRC. |
| 15:28:01 | speachy | not that I really banged on anything but at least audio playback (with various codecs) was stable. |
| 15:30:10 | speachy | and IIRC the linker issue is caused by a change of defaults in binutils. Not really a GCC bug per se, but rather with how the overall toolchain was configured. |
| 15:33:27 | gevaerts | Maybe, but that's still an old change then |
| 15:34:09 | gevaerts | I mean, my linker is 2.20.1.20100303 |
| 15:35:10 | speachy | has to do with aggressively discarding unused symbols. |
| 15:35:14 | | Quit deevious (Quit: deevious) |
| 15:35:46 | speachy | forget the linker flag that enables/disables that behavior. |
| 15:36:45 | speachy | but I ran into this in the 2011-2012 timeframe; without that aggressive unreachable/uncalled code culling (and -Os, and LTO) the code simply didn't fit. |
| 15:37:12 | speachy | ran into a lot of LTO bugs too. :) |
| 15:37:19 | gevaerts | I'm not *against* linkers that produce efficient binaries :) |
| 15:37:40 | gevaerts | So if changing the order fixes things, I'm happy |
| 15:37:47 | speachy | I'd wager rockbox would greatly benefit from LTO |
| 15:38:04 | speachy | but we should upgrade the toolchains to at least 4.9.x first. |
| 15:40:19 | gevaerts | Maybe, although I seem to have vague memories of people playing with lto back in the day and it not making a huge difference. Could be wrong though |
| 15:41:00 | gevaerts | Mostly due to code already being fairly aggressively #ifdeffed |
| 15:41:45 | speachy | Early GCC4 is when it first became generally usable, but modern GCC+binutils is considerably better. |
| 15:44:37 | speachy | okay, I'm finally respinning the talk-in-plugins patch, rearranging some of the other stuff to be in more consistent places. |
| 15:54:53 | gevaerts | Something odd is going on with the manual build |
| 15:54:55 | speachy | heh, whoops, I just rebased prior to my gerrit push and now I'm hit by the linker thing. |
| 15:57:26 | speachy | did a clean, make dep, and now it's all peachy again |
| 16:00 |
| 16:00:12 | gevaerts | ok, looks like the build finished and nothing broke :) |
| 16:00:18 | vup | hey looks like nothung broke |
| 16:00:37 | gevaerts | Celebrations are in order! Let's get coffee :) |
| 16:10:15 | * | speachy will have a nice cup of herbal tea instead. |
| 16:10:57 | gevaerts | Coffee is a herb :) |
| 16:15:45 | speachy | hmm, are herbs technically leaves, or does any of the plant qualify? |
| 16:16:02 | speachy | coffee is strange. how can something that smells so good taste so vile? |
| 16:17:07 | gevaerts | :) |
| 16:17:08 | speachy | but in any case, I cannot tolerate caffeine, so I can't even have a cup of earl grey. "good decaffinated tea" is an oxymoron, unfortunately. |
| 16:22:05 | Bilgus | jHMikes and I played with enabling Thumb for ARM a year or so ago and didn't have those issues maybe its in the logs what we had to do to make it work |
| 16:23:34 | vup | I didn't get to test it a lot yet but with this fix it seems to work fine |
| 16:23:53 | vup | (on clip+ with thumb enabled) |
| 16:26:32 | Bilgus | ../tools/configure −−thumb 62 b |
| 16:26:39 | Bilgus | ../tools/configure −−thumb 62 n |
| 16:26:48 | Bilgus | b is for bootloader |
| 16:27:18 | Bilgus | ill try it real quick |
| 16:27:37 | gevaerts | Well, it should work *now* :) |
| 16:29:23 | | Join robertd1 [0] (~robertd@201.211.186.136) |
| 16:33:41 | Bilgus | not being that I haven't rebased.. |
| 16:36:09 | *** | Saving seen data "./dancer.seen" |
| 16:36:14 | speachy | when I get home I can dust off my failing-internal-flash clip+ and give it that a try. |
| 16:36:33 | speachy | (as opposed to my failed-internal-flash clip+) |
| 16:36:34 | Bilgus | yeah same failure weird wonder what actually changes since 2017 |
| 16:36:52 | Bilgus | oh and I guess its ../tools/configure −−thumb −−target=62 −−type=n |
| 16:37:00 | speachy | (as opposed to the mass grave of fuzes..) |
| 16:37:51 | Bilgus | I've killed 1 clip+ since ~2010ish |
| 16:38:40 | Bilgus | and it still has an external pheonix connector so I can do load measurements just the sd slot died |
| 16:38:44 | amdj | This Clip+ cost me £0 so I don't mind if it dies eventually. |
| 16:38:52 | Bilgus | I would |
| 16:39:27 | foolsh | Anyone? Is there a patch to allow rockbox to boot on usb power? I have a fuze+ I'd like to test on but it's battery is long gone |
| 16:39:47 | Bilgus | Ive been running all my other sansas off the sdcard with the multibootloader patch and I hope that makes the flash last forever |
| 16:40:20 | Bilgus | foolsh johnb had one for the clip+ and the idea should be the same |
| 16:40:32 | foolsh | thanks |
| 16:41:06 | Bilgus | I can't remember if he did it I did it or someone else did it but Ill try to find it |
| 16:42:09 | speachy | only had .rockbox on my internal flash. most likely that being baked daily in Florida sunlight took its toll.. |
| 16:42:57 | * | speachy curses at Vivado for the tenth time this morning. |
| 16:43:57 | vup | vivado as in xilinx vivado? |
| 16:44:14 | vup | if yes, i feel your pain |
| 16:48:43 | foolsh | Bilgus: Ah I found #g899 (after wrangling with gerrit's search syntax) |
| 16:48:45 | speachy | more spefically, the reference design can't be recreated due to 17 missing IP blocks and two more that are present, but require an additional license. |
| 16:49:23 | Bilgus | did better than me lol I was still looking |
| 16:49:24 | speachy | and starting from the template results in a non-booting system. |
| 16:49:51 | Bilgus | g#899 |
| 16:49:51 | speachy | the more expensive a software tool is, the worse its user experience tends to be. |
| 16:49:53 | fs-bluebot | Gerrit review #899 at http://gerrit.rockbox.org/r/899 : iPod Classic: Prevent low battery shutdown if no battery is present by Michael Sparmann |
| 16:50:18 | Bilgus | nah that isn't the one |
| 16:53:51 | vup | yeah the vivado ip stuff is great ;) i hope the open source tools will soon support xilinx fpga's |
| 16:55:26 | speachy | we're using bleeding edge stuff so I'm not going to hold my breath. meanwhile I'm tripping over bugs first publicy reported five years ago. :/ |
| 16:58:24 | Bilgus | foolsh: https://github.com/Rockbox/rockbox/blob/3e91ad5df2c32bb28236ec43a660f6287d55530d/firmware/powermgmt.c#L387 |
| 16:59:05 | Bilgus | #define NO_LOW_BATTERY_SHUTDOWN |
| 17:00 |
| 17:00:05 | speachy | so I updated g#2071, incorporating Bilgus's comments. Anyone else with comments? I've been using builds (Rocker, X3) with it (and the follow-on) changes since Feburary, and others have been using Poretsky's various builds for much longer. |
| 17:00:06 | fs-bluebot | Gerrit review #2071 at http://gerrit.rockbox.org/r/2071 : FS #7704 - Talk support for plugins by Solomon Peachy |
| 17:01:55 | Bilgus | speachy I have something to go into plugins.h as well let me know before you push it and I'll put mine in with it |
| 17:02:52 | | Quit dys (Remote host closed the connection) |
| 17:12:29 | speachy | I could go ahead and push it now. |
| 17:12:45 | speachy | your changes are new additions, or is it something I could fold into this patch? |
| 17:14:07 | Bilgus | no not related just can go with the api version bump |
| 17:14:37 | Bilgus | I'll put mine on top as a child and when I push mine it'll push yours |
| 17:18:04 | speachy | okeydokey. I have five follow-up patches staged that talk-enable more plugins, and then I can resume cherry-picking more stuff out of Poretsky's tree. |
| 17:19:00 | Bilgus | are you wanting to push it today I have about an hour I can rb wuith |
| 17:19:44 | speachy | it's been more or less ready since feburary. It can wait until you're ready. :) |
| 17:26:40 | | Quit noobineer (Ping timeout: 250 seconds) |
| 17:42:54 | | Join noobineer [0] (~noobineer@2601:401:8200:26d8:e57a:837b:402c:7a7a) |
| 17:55:08 | Bilgus | speachy I found an issue in the plugins you didn't move some of the entries in both plugins.c and plugins.h |
| 17:55:11 | Bilgus | rbversion, |
| 17:55:11 | Bilgus | root_menu_get_options, |
| 17:55:11 | Bilgus | root_menu_set_default, |
| 17:55:11 | Bilgus | root_menu_write_to_cfg, |
| 17:55:11 | Bilgus | root_menu_load_from_cfg, |
| 17:55:12 | *** | Alert Mode level 1 |
| 17:55:12 | Bilgus | settings_save, |
| 17:55:23 | Bilgus | at least these but there might be more |
| 17:56:46 | speachy | crap, okay. gimme a moment to fix |
| 18:00 |
| 18:05:13 | *** | Alert Mode OFF |
| 18:05:35 | | Quit pamaury (Quit: Konversation terminated!) |
| 18:05:37 | speachy | doing a build with -Werror this time |
| 18:09:14 | speachy | okay, it's no longer complaining. can't believe I missed that. |
| 18:09:32 | speachy | new patch set pushed to gerrit. |
| 18:10:08 | Bilgus | I only noticed it as I was moving some stuff around to make it more logical |
| 18:10:46 | Bilgus | I'm out of time for today so I'll pull the new set down this evening |
| 18:11:10 | speachy | thanks! |
| 18:11:38 | Bilgus | thank you! :p |
| 18:11:41 | | Join Rower [0] (~husvagn@m83-185-90-72.cust.tele2.se) |
| 18:36:13 | *** | Saving seen data "./dancer.seen" |
| 18:42:32 | | Part robertd1 |
| 19:00 |
| 19:00:00 | | Quit Moarc (Ping timeout: 245 seconds) |
| 19:01:34 | | Quit vmx (Remote host closed the connection) |
| 19:18:59 | | Join lebellium [0] (~lebellium@89-92-69-110.hfc.dyn.abo.bbox.fr) |
| 20:00 |
| 20:19:07 | | Join Moarc [0] (~chujko@a105.net128.okay.pl) |
| 20:36:17 | *** | Saving seen data "./dancer.seen" |
| 22:00 |
| 22:36:19 | *** | No seen item changed, no save performed. |
| 22:51:25 | | Quit Moarc (Ping timeout: 272 seconds) |
| 22:54:02 | | Quit speachy (Ping timeout: 260 seconds) |
| 22:55:37 | | Join Moarc [0] (~chujko@a105.net128.okay.pl) |
icon identifies that the person is a core developer (has commit access).