Skip to content

Conversation

@aaronriekenberg
Copy link

On ubuntu 16.10 with the provided mbpfan.service file, mbpfan runs with the -fv options by default.

This causes output to go to /var/log/syslog, but I noticed the logs there were delayed due to buffering of the standard out i/o stream. This caused a bunch of logs to appear in /var/log/syslog at random times with wrong timestamps.

Example:

Dec 28 15:20:39 oldmacbook mbpfan[18215]: Old Temp 43: New Temp: 43, Fan Speed: 2000
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Old Temp 43: New Temp: 43, Fan Speed: 2000
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Old Temp 43: New Temp: 43, Fan Speed: 2000
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Old Temp 43: New Temp: 42, Fan Speed: 2000
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Old Temp 42: New Temp: 43, Fan Speed: 2000
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Old Temp 43: New Temp: 42, Fan Speed: 2000
Dec 28 15:20:39 oldmacbook mbpfan[18215]: Sleeping for 7 seconds

Dec 28 15:27:53 oldmacbook mbpfan[18215]: Old Temp 42: New Temp: 42, Fan Speed: 2000
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Old Temp 42: New Temp: 42, Fan Speed: 2000
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Old Temp 42: New Temp: 43, Fan Speed: 2000
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Old Temp 43: New Temp: 43, Fan Speed: 2000
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Old Temp 43: New Temp: 42, Fan Speed: 2000
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Old Temp 42: New Temp: 42, Fan Speed: 2000
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Sleeping for 7 seconds
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Old Temp 42: New Temp: 42, Fan Speed: 2000
Dec 28 15:27:53 oldmacbook mbpfan[18215]: Sleeping for 7 seconds

Adding fflush(stdout) just before sleeping fixes this:

Dec 28 16:06:17 oldmacbook mbpfan[20594]: Old Temp 54: New Temp: 53, Fan Speed: 2000
Dec 28 16:06:17 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:06:24 oldmacbook mbpfan[20594]: Old Temp 53: New Temp: 53, Fan Speed: 2000
Dec 28 16:06:24 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:06:31 oldmacbook mbpfan[20594]: Old Temp 53: New Temp: 53, Fan Speed: 2000
Dec 28 16:06:31 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:06:38 oldmacbook mbpfan[20594]: Old Temp 53: New Temp: 53, Fan Speed: 2000
Dec 28 16:06:38 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:06:45 oldmacbook mbpfan[20594]: Old Temp 53: New Temp: 56, Fan Speed: 2000
Dec 28 16:06:45 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:06:52 oldmacbook mbpfan[20594]: Old Temp 56: New Temp: 52, Fan Speed: 2000
Dec 28 16:06:52 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:06:59 oldmacbook mbpfan[20594]: Old Temp 52: New Temp: 52, Fan Speed: 2000
Dec 28 16:06:59 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:07:06 oldmacbook mbpfan[20594]: Old Temp 52: New Temp: 52, Fan Speed: 2000
Dec 28 16:07:06 oldmacbook mbpfan[20594]: Sleeping for 7 seconds
Dec 28 16:07:13 oldmacbook mbpfan[20594]: Old Temp 52: New Temp: 52, Fan Speed: 2000
Dec 28 16:07:13 oldmacbook mbpfan[20594]: Sleeping for 7 seconds

@dgraziotin dgraziotin merged commit 11ec211 into linux-on-mac:master Apr 2, 2017
@dgraziotin
Copy link
Collaborator

thank you very much, and..err..sorry for the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants