View Issue Details

IDProjectCategoryView StatusLast Update
0016920MediaMonkey 5Generalpublic2020-10-21 01:38
ReporterLudek Assigned To 
PriorityimmediateSeveritycrashReproducibilityhave not tried
Status feedbackResolutionreopened 
Product Version5.0 
Target Version5.0Fixed in Version5.0 
Summary0016920: Various crashes because insufficient system disc space
DescriptionBased on crash logs (e.g. 058F0000) if the system disc gets out of space, then:
- MediaMonkey.ini fails to write with EWriteError: Stream write error
- writing to SQLite DB freezes in ntdll.dll -> ZwWriteFile
- playback freezes while decoding because of freeze in ntdll -> ZwReadFile

We can hardly predict all various crashes caused by insufficient disc space.

Solution would be to show warning on MM5 start when there is insufficient space available in the MM data directories.
Something like:

Directory C:\User\Ludek\AppData\Roaming\MediaMonkey5\ gets out of space.
Please free the C:\ disc space and try again.
[OK]

As the required quota we could use at least something like 200 MB ?


TagsNo tags attached.
Fixed in build2267

Activities

Ludek

2020-09-08 11:21

developer   ~0059573

Assigned to Rusty for wording of the message.

peke

2020-09-08 16:49

developer   ~0059575

Last edited: 2020-09-08 16:52

View 2 revisions

I would suggest at least size of MM5.DB.

Also it could be useful to check DB backups as last time I checked "Roaming\MediaMonkey 5\" was 17GB and that we push #16176 fix to 5.0 (especially as Ludek updated ZIP libraries due the other bug fixes. Not sure if encoding is available or just decoder) or possibly add maintenance to delete old DB files.

Ludek

2020-09-10 12:36

developer   ~0059577

Last edited: 2020-09-10 12:38

View 2 revisions

Good point with the DB backups, but size of MM5.DB isn't the only aspect, e.g. caching of looked up artwork (and artist images) also can take a lot of disc space.
Caching of artist images is the default (while DB backups are optional AFAIK)

I used this message in 2267 : https://www.dropbox.com/s/x0f0bvviyodjgjg/screenshot%202020-09-10%2014.33.24.png?dl=0

So the new strings to translate are:
'Directory %s gets out of space.'
'Please free the disc space for MediaMonkey to function properly.'

I used quota 500 MB for the warning to show.

Ludek

2020-09-10 12:45

developer   ~0059578

Resolved for testing in 2267, then to be assigned to Rusty for the wording review.

Ludek

2020-09-17 10:06

developer   ~0059594

New wording applied in 2267:

'Drive %s is running out of space.'
'Please free up space for MediaMonkey to function properly.'

lowlander

2020-09-18 01:30

developer   ~0059598

Could the message include that MediaMonkey recommends to have at least x MB of free space on the drive?

Ludek

2020-09-21 10:07

developer   ~0059626

Last edited: 2020-09-21 10:13

View 2 revisions

There isn't any value to recommend, it all depends on many unpredictable factors, e.g.
- how many artist images is going to be downloaded/cached
- how many streaming content is cached and where (Options > Player > Streaming > Maximum cache size)
- how many auto-converted files is cached when sharing media (Options > Media Sharing > Maximum cache size)
- how often user uses the backup DB feature (Manage Database > Backup database now...) and how big is his DB
etc. etc.

The point of this issue was just to inform the user that the system disk is running out of space (to prevent from submitting various crash logs as a consequence of the insufficient disk space).

peke

2020-10-21 01:38

developer   ~0059922

Verified 2270 in simulation of low disk space.

As wording is corrected and there is not much We can do to estimate amount of space needed fr MM operation, but we now present user with warning MSG I would consider it closed?