Ultimate Intervalometer - intermittent reboot faiure (powershot D10) - page 7 - Completed and Working Scripts

reyalp

13457

Re: Ultimate Intervalometer - intermittent reboot faiure (powershot D10)

« Reply #60 on: 24 / February / 2014, 01:23:55 »

Advertisements

Quote from: philmoz on 24 / February / 2014, 00:53:43

It happens when one task calls Close; but gets interrupted in the Close call, and another task then calls Open.

The call to Close, calls 'close' then later calls an FsNotify function to remove the closed file from the list.

When a task switch happens between these calls, the new task calls Open.
Open then calls 'open' and gets back the file handle from the previous call to 'close'.
Open then tries to call the FsNotify function to add the file handle to the list; but it is already there so it asserts.

OK, that makes sense (except the part where Canon didn't protect it with a semaphore or something...) Though it's surprising how reproducible it is with the autostart once script.

Quote

In all likelihood switching to always using 'open' in CHDK (even for writing files) would probably work and eliminate more instances of the problem - we don't use whatever the FsNotify code is doing.

It seems like this would be reasonable to try in the trunk for a while.

Logged

Don't forget what the H stands for.

Karmaschinken

Re: Ultimate Intervalometer - intermittent reboot faiure (powershot D10)

« Reply #61 on: 24 / February / 2014, 03:58:14 »

I recently finished my 50 camera rig with the possibility to download crash dumps. If I can help testing something please tell me.

Logged

waterwingz

12092

Re: Ultimate Intervalometer - intermittent reboot faiure (powershot D10)

« Reply #62 on: 24 / February / 2014, 07:37:00 »

Quote from: philmoz on 24 / February / 2014, 00:53:43

This was what was discovered for the startup crash fix, and the fix was to call 'open' for files being read at startup instead of Open. In all likelihood switching to always using 'open' in CHDK (even for writing files) would probably work and eliminate more instances of the problem - we don't use whatever the FsNotify code is doing.

Does this mean that

Code: [Select]

#define CAM_STARTUP_CRASH_FILE_OPEN_FIX 1 can be removed from platform_camera.h files?

Logged

Ported : A1200 SD940 G10 Powershot N G16

philmoz

3332

Re: Ultimate Intervalometer - intermittent reboot faiure (powershot D10)

« Reply #63 on: 24 / February / 2014, 13:39:31 »

Quote from: waterwingz on 24 / February / 2014, 07:37:00

Quote from: philmoz on 24 / February / 2014, 00:53:43
This was what was discovered for the startup crash fix, and the fix was to call 'open' for files being read at startup instead of Open. In all likelihood switching to always using 'open' in CHDK (even for writing files) would probably work and eliminate more instances of the problem - we don't use whatever the FsNotify code is doing.
Does this mean that

Code: [Select]
#define CAM_STARTUP_CRASH_FILE_OPEN_FIX 1 can be removed from platform_camera.h files?

If we replace the call to '_Open' with '_open' unconditionally in wrappers.c, then it could be removed.

Phil.

Logged

CHDK ports:
sx30is (1.00c, 1.00h, 1.00l, 1.00n & 1.00p)
g12 (1.00c, 1.00e, 1.00f & 1.00g)
sx130is (1.01d & 1.01f)
ixus310hs (1.00a & 1.01a)
sx40hs (1.00d, 1.00g & 1.00i)
g1x (1.00e, 1.00f & 1.00g)
g5x (1.00c, 1.01a, 1.01b)
g7x2 (1.01a, 1.01b, 1.10b)