Rockbox

Tasklist

FS#9253 - logf file output patch.

Attached to Project: Rockbox
Opened by Yoshihisa Uchida (Uchida) - Friday, 08 August 2008, 13:28 GMT
Task Type Patches
Category Applications
Status Unconfirmed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

It is a patch that can automatically output the result of logf to the
file.


The setting of default doesn't output the result of the logf to the
file.
Please set and do the following when it is effective.

System > Debug > logf auto file output: select "Yes"
This task depends upon

Comment by Nicolas Pennequin (nicolas_p) - Friday, 08 August 2008, 14:36 GMT
Are you sure a whole new set of file functions is needed? Also, did ou write all of asyncfile.c?
Comment by Yoshihisa Uchida (Uchida) - Sunday, 10 August 2008, 14:46 GMT
If asyncfile.c was not used when the log is output to the file,
then it not be output to the file or only a part of the log be output
when the load of Rockbox is high (playing music or movie).

The patch file to output the log to the file without using asyncfile.c was prepared,
and confirm the output of the log is different, please.
Comment by Yoshihisa Uchida (Uchida) - Sunday, 10 August 2008, 14:59 GMT
Sorry, I don't send patch file.

fs_logf_no_asyncfile.patch
 It is a patch file to output the log to the file without using asyncfile.c.

rev_logf.txt
It is a file that upsets the top and bottom of the line to the execution result of logfdump(logf.txt).

logf.log
 It is a log file that Rockbox that applied fs_logf_no_asyncfile.patch.

When it compare rev_logf.txt with logf.log,
The 635-660th line and the 681-683th line of rev_logf.txt is
different from logf.log.

Comment by Jens Erdmann (pyro_maniac) - Tuesday, 10 March 2009, 16:49 GMT
I attached a file with a actual and minimal version but i didn't had the chance to test it enoght.
Comment by Yoshihisa Uchida (Uchida) - Wednesday, 11 March 2009, 14:22 GMT
Processing slows very much when the log is effective because logf() doesn't end if writing in the file is not completed.
For example, when the log is output when a music has been played, the slowness of processing can be experienced.
Because processing is not slowed down when the log is output, the asyncfile.c is needed.

My patch file cannot be applyed in the Rockbox of the latest version.
If the new patch file is necessary, please wait for the patch file that can be appropriated to the latest version.
Comment by Jens Erdmann (pyro_maniac) - Wednesday, 11 March 2009, 14:56 GMT
so i will wait for a new patch to come along with my port. Thanks for doing this.
Comment by Yoshihisa Uchida (Uchida) - Thursday, 12 March 2009, 12:42 GMT
I corrected my patch file to apply it to the Rockbox of the latest version.
Please report on the operation when this patch is used.
Comment by Jens Erdmann (pyro_maniac) - Thursday, 12 March 2009, 19:56 GMT
There are still several bugs in. asyncfile.c is inserted in SOURCES but there is no file. The boolean in settings.h is in the enum.
Comment by Yoshihisa Uchida (Uchida) - Saturday, 21 March 2009, 14:27 GMT
Sorry, my patch file build failure.
It updates. I think that the Rockbox firmware build success.
Comment by Jens Erdmann (pyro_maniac) - Saturday, 21 March 2009, 18:46 GMT
There are still some warnings on applying but it works for me. Thanks for help!
Comment by Yoshihisa Uchida (Uchida) - Thursday, 26 March 2009, 11:00 GMT
sync r20543.
I confirmed the build was able to be done to iPod video, iriver H120 and Sansa e200.
Comment by Jens Erdmann (pyro_maniac) - Thursday, 26 March 2009, 12:20 GMT
Patch 3 works on a h10 5g build too.

Loading...