00:42:40 | | Quit massiveH (Quit: Leaving) |
00:54:53 | _bilgus_ | dconrad (logs) see thats why we can't have nice things, I really like that patch so lets hope its coincidence |
00:57:19 | _bilgus_ | its probably more likely Haas Surround has a bug that this is making come out of the wood work |
01:00 |
01:40:14 | | Quit Romster (Quit: Leaving) |
01:54:15 | _bilgus_ | well its reproducible in the simulator ASAN says AddressSanitizer: SEGV on unknown address 0x000000000000 |
01:54:43 | *** | Saving seen data "./dancer.seen" |
02:00 |
02:13:46 | _bilgus_ | changing the code back to the original before 4691 the error still occurs |
03:00 |
03:06:50 | | Join Romster [0] (~romster@158.140.215.184) |
03:11:32 | | Quit Romster (Ping timeout: 248 seconds) |
03:45:20 | bluebrother | someone familiar with ipod6g bootloader install? I've been looking at the manual and the manual install for that are ... well, outdated. |
03:45:57 | bluebrother | it still links to freemyipod, but we have our official bootloader now. Also, it instructs the user to use Rockbox Utility with the "show disabled targets" option, which is simply wrong these days −− ipod6g isn't a disabled target anymore. |
03:46:30 | bluebrother | I also expect bootloader install to work on Windows if you have mks5lboot for Windows. Though I don't have an ipod6g, so I cannot check. |
03:46:43 | bluebrother | and last, it instructs the user to install the build first. Is that really necessary? |
03:47:38 | bluebrother | (plus, using Rockbox Utility, but if you want to install manually you likely want to install from the zip anyway, and it's in the "bootloader install" subsection, so that is also messed up) |
03:48:57 | bluebrother | I'd update the docs, but should have confirmation on how it actually works :) |
03:51:24 | | Join lebellium [0] (~lebellium@2a01cb040109a6001d29f6c4d30b6294.ipv6.abo.wanadoo.fr) |
03:54:44 | *** | Saving seen data "./dancer.seen" |
04:00 |
04:32:03 | | Join Romster [0] (~romster@158.140.215.184) |
05:00 |
05:32:52 | | Quit Richard42 (Ping timeout: 268 seconds) |
05:54:28 | rb-bluebot | Build Server message: New build round started. Revision dfd9c10589, 303 builds, 7 clients. |
05:54:47 | *** | Saving seen data "./dancer.seen" |
06:00 |
06:02:09 | | Quit jackie (Quit: No Ping reply in 180 seconds.) |
06:03:17 | | Join jackie [0] (~jackie@banana-new.kilobyte22.de) |
06:28:32 | rb-bluebot | Build Server message: Build round completed after 2044 seconds. |
06:28:35 | rb-bluebot | Build Server message: Revision dfd9c10589 result: All green |
06:30:09 | rb-bluebot | Build Server message: New build round started. Revision ac9066dd44, 303 builds, 7 clients. |
06:35:39 | | Quit CH23_M (Ping timeout: 265 seconds) |
06:38:22 | | Join CH23_M [0] (~CH23@revspace/participant/ch23) |
06:55:44 | | Quit CH23_M (Read error: Connection reset by peer) |
06:56:03 | | Join CH23_M [0] (~CH23@revspace/participant/ch23) |
07:00 |
07:03:11 | rb-bluebot | Build Server message: Build round completed after 1982 seconds. |
07:03:12 | rb-bluebot | Build Server message: Revision ac9066dd44 result: All green |
07:54:50 | *** | Saving seen data "./dancer.seen" |
08:00 |
08:25:45 | _bilgus_ | besides td speed the dsp stuff doesn't check for a successful handle from core_alloc |
08:26:05 | _bilgus_ | but thats still not fixing the crash in the sim |
08:26:29 | _bilgus_ | and I've followed it to prior to asan being added |
08:26:36 | _bilgus_ | so.... |
08:36:59 | _bilgus_ | nm they check just not in the alloc proc |
08:56:48 | _bilgus_ | starting in dsp/surround.c followed it to surround_new_format> dsp_proc_activate(dsp/dsp_core.c) adding sleep(100) at the end of the function makes it not crash |
08:57:05 | _bilgus_ | oddly I don't see a rhyme or reason |
08:57:37 | _bilgus_ | race condition I suppose |
09:00 |
09:03:13 | _bilgus_ | dsp_process(dsp/dsp_core.c) looks to be our running thread |
09:54:54 | *** | No seen item changed, no save performed. |
10:00 |
10:22:59 | _bilgus_ | dconrad (logs) could you try g#4905 and see if the problem persists? |
10:23:02 | rb-bluebot | Gerrit review #4905 at https://gerrit.rockbox.org/r/c/rockbox/+/4905 : haas surround crash diag by William Wilgus |
10:35:54 | _bilgus_ | from what I know so far its calling a NULL function pointer now why is that process function pointer still NULL I don't know |
10:43:43 | | Join amachronic [0] (~amachroni@user/amachronic) |
10:48:29 | amachronic | _bilgus about that INIT_ATTR linker thing: g#4906 |
10:48:32 | rb-bluebot | Gerrit review #4906 at https://gerrit.rockbox.org/r/c/rockbox/+/4906 : configure: Detect linker's NO_CROSSREFS_TO() support by Aidan MacDonald |
10:48:46 | _bilgus_ | that patch fixes it for me and it appears the surround works too but I still don't know what is missing, doing a logf build it appears it calls the NULL function till it gets setup but why it only appears when you se through the QuickScreen Ive no idea, its almost as if it skips something critical where as running it through the settings menu it works fine, WEIRD |
10:48:52 | amachronic | sample linker script in g#4909 |
10:48:54 | rb-bluebot | Gerrit review #4909 at https://gerrit.rockbox.org/r/c/rockbox/+/4909 : x1000: Enable support for INIT_ATTR by Aidan MacDonald |
10:49:45 | _bilgus_ | amachronic, nice ill have to go through and see where else INIT_ATTR is waiting to crash us |
10:50:42 | _bilgus_ | re your playlist loader patch I think its a coincidence I'm waiting to hear back from dconrad if this patch makes his player stop crashing |
10:52:17 | _bilgus_ | ASAN really doesn't do much for tracking it down probably have to be pulled apart a bit more, either that of we just leave the empty fn there its not adding processing in the hot path that way |
10:53:50 | amachronic | huh, well i'll try an asan build now anyway |
10:54:11 | _bilgus_ | it smells like a race condition being that adding a sleep makes it work too |
10:56:19 | _bilgus_ | ASAN cathches it its just got no context I knew the pc of 0 pointed to a null deref but I just had to go through and check every pointer till I found it |
10:56:47 | _bilgus_ | the first ver of that patch has the check if you want to see where it was |
11:00 |
11:03:14 | amachronic | _bilgus_ okay I see what you mean with ASAN, when it bails like that you can use gdb to get a backtrace |
11:03:21 | amachronic | ASAN_OPTIONS="abort_on_error=1" gdb ./rockboxui |
11:03:44 | _bilgus_ | oh nice I was wondering if I could attach a debugger to it |
11:04:19 | _bilgus_ | where do I define that? in the call to configure or in the script? |
11:05:21 | amachronic | you just set the env var before you run the sim |
11:05:22 | _bilgus_ | or on the commend line? |
11:05:28 | _bilgus_ | ah ok |
11:06:28 | amachronic | well i only found out what you already did −− the process fn pointer is NULL |
11:07:51 | _bilgus_ | I have no idea why dconrad found 4691 to be the start I still found it all the way back to ASAN being added |
11:08:12 | _bilgus_ | kinda stopped me in my tracks there |
11:09:49 | amachronic | i'd like to say it's happening because dsp_proc_enable_enlink() is linking in the dsp before the fn pointer gets set by configure(DSP_PROC_INIT) |
11:10:35 | _bilgus_ | but whats different between doing it through the menu versus the QS they appear to do the same |
11:10:44 | amachronic | that I don't know :) |
11:10:51 | _bilgus_ | must be something I'm missing |
11:16:02 | | Join dconrad [0] (~dconrad@152.117.104.235) |
11:21:54 | dconrad | _builgus_ I'm getting that patch built now |
11:22:31 | dconrad | thanks for looking at it |
11:26:35 | amachronic | well I added sleep(HZ) right before ->configure(DSP_PROC_INIT) and got a crash from the menu eventually |
11:26:42 | amachronic | doesn't happen every time though |
11:29:11 | dconrad | that patch does appear to make it work |
11:29:24 | dconrad | let me try it a few more times and see if its consistent |
11:34:46 | amachronic | if I comment out the pcmbuf_set_low_latency() calls in lowlatency_callback() then the menu triggers it reliably without the sleep |
11:35:21 | amachronic | that must be the difference between the menu and quickscreen. |
11:36:10 | amachronic | i guess low latency mode makes it harder to trigger the race? |
11:37:07 | amachronic | _bilgus_ i think your patch is the best solution |
11:38:51 | dconrad | so I've tried it 4 times and didn't get the crash once |
11:39:19 | dconrad | I have noticed that dsp options seem to take a few seconds to "kick in" from the QS, is this related? |
11:40:01 | dconrad | or, could it be related |
11:43:01 | amachronic | i think that's because the QS doesn't enable low latency mode |
11:43:59 | dconrad | that would make sense, does just going into the settings enable low latency mode normally? or, certain settings I guess |
11:44:27 | amachronic | yeah only certain sound settings do it |
11:44:54 | dconrad | like, enter the haas surround setting menu and it triggers it then |
11:45:00 | amachronic | yep |
11:45:05 | dconrad | I see |
11:45:37 | dconrad | well that answers a long-standing question I've had haha |
11:50:34 | | Quit amachronic (Quit: amachronic) |
11:51:32 | _bilgus_ | thats strange I saw the low_latency call and tried adding that but still got the crash |
11:51:53 | _bilgus_ | when I was first trying to pin it down |
11:53:25 | _bilgus_ | maybe it still happens even with low_latency enabled just less frquent |
11:54:57 | *** | Saving seen data "./dancer.seen" |
12:00 |
12:34:34 | | Join dys [0] (~dys@user/dys) |
13:00 |
13:25:51 | | Quit dconrad (Remote host closed the connection) |
13:35:08 | | Join mink [0] (~mink@178.197.192.87) |
13:42:09 | | Join amiconn_ [0] (jens@p200300ea874aa500305e95fffec66ff3.dip0.t-ipconnect.de) |
13:42:09 | | Nick amiconn is now known as Guest4891 (jens@p549f0d97.dip0.t-ipconnect.de) |
13:42:09 | | Quit Guest4891 (Killed (cadmium.libera.chat (Nickname regained by services))) |
13:42:09 | | Nick amiconn_ is now known as amiconn (jens@p200300ea874aa500305e95fffec66ff3.dip0.t-ipconnect.de) |
13:42:46 | | Join pixelma_ [0] (marianne@p200300ea874aa500305e95fffec66ff3.dip0.t-ipconnect.de) |
13:42:46 | | Nick pixelma is now known as Guest4402 (marianne@p549f0d97.dip0.t-ipconnect.de) |
13:42:46 | | Nick pixelma_ is now known as pixelma (marianne@p200300ea874aa500305e95fffec66ff3.dip0.t-ipconnect.de) |
13:43:16 | | Quit Guest4402 (Ping timeout: 256 seconds) |
13:55:01 | *** | Saving seen data "./dancer.seen" |
14:00 |
14:46:37 | | Quit fourHZ (Quit: Client closed) |
14:48:01 | | Join fourHZ [0] (~fourHZ@92-52-40-121.dynamic.orange.sk) |
15:00 |
15:02:04 | | Join dconrad [0] (~dconrad@152.117.104.235) |
15:06:59 | | Quit dconrad (Ping timeout: 268 seconds) |
15:08:22 | | Join dconrad [0] (~dconrad@152.117.104.235) |
15:11:11 | dconrad | small caveat with patchset 4905 _bilgus_, it doesn't seem to save the setting if I change it from the QS and then power off the player |
15:11:36 | dconrad | though that's probably a completely separate issue |
15:14:34 | dconrad | coincidentally, after months and months of not seeing it, I got the "screen wakeup freeze" like 4 or 5 times today in quick succession... I may try some more to find a repeatable way to trigger it |
15:15:47 | | Quit fourHZ (Quit: https://www.youtube.com/watch?v=Euvo4wvuUdY) |
15:35:09 | | Quit mink (Remote host closed the connection) |
15:36:35 | _bilgus_ | dconrad, I'll look into it a bit more before I push it I still need to check a few things (in my patch comment) whats this 'screen wakeup freeze'? |
15:37:59 | _bilgus_ | I think as things change and or get faster we will probably see more of these things pop up I hate heisenbugs |
15:38:48 | dconrad | the screen wakeup freeze is something unrelated, but has been with the eros q native since the port started |
15:39:49 | dconrad | I thought it was taken care of when amachronic found the lcd instructions didn't have the end marker, and I haven't seen it in a long time, only for it to pop up again today |
15:40:19 | dconrad | basically the player is doing its thing and you wake up the screen and the whole device freezes, you have to do a force power off |
15:43:28 | dconrad | not sure if coincidence or not, but most of my library is flac, but I'm listening to some ancient mp3s today which I don't think I've listened to in a while |
15:55:03 | *** | Saving seen data "./dancer.seen" |
15:59:55 | _bilgus_ | more heisenbugs it is then :p\ |
16:00 |
16:02:50 | dconrad | yuup |
16:29:40 | dconrad | hmm, I guess I can't save the surround enable setting at all, even through the menu |
16:36:12 | dconrad | even with a blank settings file |
16:53:32 | | Join bilgus_ph [0] (~bilgus_ph@172.58.203.135) |
16:54:04 | bilgus_ph | Dconrad probably something with the recent settings rewrites I'll see what I can find tonight or tomorrow |
16:58:55 | | Quit bilgus_ph (Ping timeout: 260 seconds) |
17:00 |
17:21:26 | | Quit dconrad (Remote host closed the connection) |
17:26:08 | | Quit lebellium (Quit: Leaving) |
17:28:08 | | Quit advcomp2019_ (Quit: Leaving) |
17:33:52 | | Join advcomp2019 [0] (~advcomp20@user/advcomp2019) |
17:47:33 | | Join dconrad [0] (~dconrad@152.117.104.235) |
17:50:32 | | Quit dconrad (Remote host closed the connection) |
17:51:02 | | Join dconrad [0] (~dconrad@152.117.104.235) |
17:51:08 | | Quit dconrad (Client Quit) |
17:55:05 | *** | Saving seen data "./dancer.seen" |
18:00 |
18:56:24 | | Quit CH23_M (Read error: Connection reset by peer) |
18:56:56 | | Join CH23_M [0] (~CH23@revspace/participant/ch23) |
19:00 |
19:55:08 | *** | Saving seen data "./dancer.seen" |
21:00 |
21:55:13 | *** | No seen item changed, no save performed. |
22:00 |
22:17:14 | rb-bluebot | Build Server message: New build round started. Revision 863538c50a, 303 builds, 6 clients. |
22:54:16 | rb-bluebot | Build Server message: Build round completed after 2222 seconds. |
22:54:19 | rb-bluebot | Build Server message: Revision 863538c50a result: All green |
23:00 |
23:37:10 | | Quit m01 (Quit: Konversation terminated.) |
23:39:22 | | Join m01 [0] (~quassel@vps-b172b88b.vps.ovh.net) |
23:42:57 | | Join WilliamC [0] (~WilliamC@2603-6080-0400-0f75-6c36-1f3a-030a-493c.res6.spectrum.com) |
23:43:25 | WilliamC | I have some Vorbis files, with images, that are causing rockbox to crash. Is this a known issue? |
23:43:58 | WilliamC | I removed the images to see if this is the case, otherwise I don't know what the commonality is. |
23:45:29 | | Quit Acou_Bass (Quit: ZNC 1.8.2 - https://znc.in) |
23:48:12 | | Join Acou_Bass [0] (~eddie@cpc95736-bolt17-2-0-cust330.10-3.cable.virginm.net) |
23:55:15 | *** | Saving seen data "./dancer.seen" |