COVESA / dlt-daemon

Diagnostic Log and Trace.
https://covesa.github.io/dlt-daemon/
Mozilla Public License 2.0
377 stars 292 forks source link

On Qnx,Setting the parameter GzipCompression=on will cause problems with the dlt-logstorage-ctrl display configuration file #645

Closed hush-soul closed 3 months ago

hush-soul commented 3 months ago

1.no problems storing uncompressed files 2.The same configuration file was successfully compressed in the Linux system

1 2

Looking forward to your reply, thank you

minminlittleshrimp commented 3 months ago

Hello @hush-soul I believe in my fix the value for parsing in ON. Kindly testing with the latest dlt version. Also, I wonder if it does not happen with ON_SPECIFIC_SIZE, we might look into the differences/mechanisms behind this sync behavior and the normal mode ON_MSG. A possible issue might come from the fsync phase. Regards

minminlittleshrimp commented 3 months ago

Besides, is this a duplicated one of https://github.com/COVESA/dlt-daemon/issues/644 ?

Update: Sorry didnt notice, this one for QNX. However we can combine them for easier syncing up

hush-soul commented 3 months ago

Hello @hush-soul I believe in my fix the value for parsing in ON. Kindly testing with the latest dlt version. Also, I wonder if it does not happen with ON_SPECIFIC_SIZE, we might look into the differences/mechanisms behind this sync behavior and the normal mode ON_MSG. A possible issue might come from the fsync phase. Regards

The same situation will occur with ON_SPECIFIC_SIZE and ON_MSG

I found some phenomena. It may be a problem with my zlib. It is still being debugged. Thanks for the reply.

hush-soul commented 3 months ago

Hello @hush-soul I believe in my fix the value for parsing in ON. Kindly testing with the latest dlt version. Also, I wonder if it does not happen with ON_SPECIFIC_SIZE, we might look into the differences/mechanisms behind this sync behavior and the normal mode ON_MSG. A possible issue might come from the fsync phase. Regards

I have solved this problem. It was my mistake. The qnx program code was not synchronized to the latest version. Thank you.

hush-soul commented 3 months ago

The new version does not have this problem

minminlittleshrimp commented 3 months ago

I see, thanks for the report. My pleasure to support you 😀