Project

General

Profile

Heavy CPU load

Added by bas t about 12 years ago

Do you have any OTA EPG services enabled? Is the CPU load constantly higher or just for brief periods?

@addamsutton:
I am using the epg my cable provider sends along with the signal. That is not the origin of the problem.

The heavy cpu load is while recording a show and is constant.

I'll be glad to help debugging this, I'll need directions though.


Replies (66)

RE: Heavy CPU load - Added by Adam Sutton about 12 years ago

Which commit do you think broke it then? and have you tried hacking out the mod's to see if the problem goes away?

Adam

RE: Heavy CPU load - Added by bas t about 12 years ago

It has to be either 841ff8eb49 or 711ce27baf or both. I'm quite sure of that, since 9e53683ca0 and previous commits are not affected while a259025d31 and later are.

The commits between the two won't build and zlib was ruled out as a possible cause. So that leaves us with 841ff8eb49 and/or 711ce27baf as only possible brakers.

Sorry, I don't have a clue as to what "hacking out the mod's" even means. EDIT:

oh yeah, I do know what that means. It took me a while (I'm not a native English speaker)

Reading the code, the only thing that draw my attention was that both mmx as well as sse2 have vanished from configure.inc

RE: Heavy CPU load - Added by bas t about 12 years ago

Plus, there are way too many mod's in these commits.
I'm not capable of debugging this.

RE: Heavy CPU load - Added by Adam Sutton about 12 years ago

Neither of the listed commits do anything that significantly affects the binary output of the image.

841ff8eb49 - this is simply a revamping of the configuration and Makefile. Though only impact this could have is that optional code could now be removed, in which case I'd expect it to fail completely or the optimisation settings are wrong. The later would potentially affect performance but the settings look write to me.

Just to be sure can you do the following:
make distclean
./configure # with whatever options you need
V=1 make

and pastebin the output.

711ce27baf - this is a minor correction to existing code added in previous commits and relates to stuff that should only really be accessed on startup. Unless the decrypt code is being very inefficient and loading config constantly this will have no effect.

Adam

RE: Heavy CPU load - Added by Adam Sutton about 12 years ago

I take that back ;) I think I might have spotted the problem. Bare with me...

Adam

RE: Heavy CPU load - Added by Adam Sutton about 12 years ago

Please try this patch.

t.patch (557 Bytes) t.patch Make correction

RE: Heavy CPU load - Added by bas t about 12 years ago

Goody I will test it! :)

However, it may take a while. github seems to have problems right now.

RE: Heavy CPU load - Added by Adam Sutton about 12 years ago

Yeah github has serious problems, but shouldn't stop you testing it. You don't need access to github to download and apply the patch ;)

Adam

RE: Heavy CPU load - Added by bas t about 12 years ago

That's true, but I deleted my repo, so I'll have to wait. Later on tonight I don't have time so it could be first thing in the morning.

RE: Heavy CPU load - Added by bas t about 12 years ago

In the meantime, what about configure.inc?

Are you sure that mmx and sse2 are no longer needed in configure.inc? They must have been there for some reason, I think.

RE: Heavy CPU load - Added by Adam Sutton about 12 years ago

They are still there and the options are being correctly used. The only issue I've found is that the general CFLAGS were being lost for the FFdecsa code, with only the mmx and sse flags being used.

Adam

RE: Heavy CPU load - Added by bas t about 12 years ago

YOU HAVE HIT THE JACKPOT!!!

Congratulations, and thank you very much!

RE: Heavy CPU load - Added by bas t about 12 years ago

I mean cpuload is normal again, I was not responding to your last message.

RE: Heavy CPU load - Added by Alex . about 12 years ago

THANKS Adam!

Great, Indeed, it works!

Just applied your patch and the cpu-load is way down again for the encrypted channels !!!!
:-)

Thanks for figuring this out.

Alex.

Adam Sutton wrote:

They are still there and the options are being correctly used. The only issue I've found is that the general CFLAGS were being lost for the FFdecsa code, with only the mmx and sse flags being used.

Adam

RE: Heavy CPU load - Added by Adam Sutton about 12 years ago

Well I broke it, so I probably should've been the one to fix it ;)

Patch now applied.

RE: Heavy CPU load - Added by Alex . about 12 years ago

Adam Sutton wrote:

Well I broke it, so I probably should've been the one to fix it ;)

Lol

;-)

Great anyway that you put the effort in it !

(51-66/66)