FS#7528 - Superdom should yield during AI calculations
Attached to Project:
Rockbox
Opened by Nikkhil (AceNik) - Friday, 03 August 2007, 14:59 GMT
Last edited by Jonas Häggqvist (rasher) - Thursday, 11 December 2008, 02:02 GMT
Opened by Nikkhil (AceNik) - Friday, 03 August 2007, 14:59 GMT
Last edited by Jonas Häggqvist (rasher) - Thursday, 11 December 2008, 02:02 GMT
|
DetailsHey guys the new superdom plugin is very nice, but , when i try playing the game with increasing the computers farms & factories to above normal level like suppose 2,2 & also have music playback in the background the H10[20GB] seems to have lot of hang ups, i dont know if this is true for other players if yes, please feel free to add in your comments
btw this is regarding rockbox build no. 14156 [admin edit: Title updated to better reflect the problem] |
This task depends upon
Closed by Jonas Häggqvist (rasher)
Thursday, 11 December 2008, 02:02 GMT
Reason for closing: Fixed
Additional comments about closing: Patch that should fix this issue committed in r19389.
Thursday, 11 December 2008, 02:02 GMT
Reason for closing: Fixed
Additional comments about closing: Patch that should fix this issue committed in r19389.

- Field changed: Summary (Frequent Hang-Ups During Superdom → Superdom should yield during AI calculations)
- Field changed: Details
Looking at the code, Superdom doesn't yield a single time. This should probably be fixed and should hopefully let music playback continue.

Patch that adds yielding. I can't be 100% sure it works as my H10 is a 6GB, so it's screen is too small to test. I tried it on my Gigabeat though and hasn't caused any freezes.

I haven't got a DAP to test this patch on, but it takes Johnathon's idea and tweaks it a bit. If this fixes everything up then I'll commit, but I have no way to know until I can get some feedback.

Hopefully a more accurate patch. I do need feedback on these though, as I have no way of telling if these work or not.

Will, can this be commited?

Let me test it on my gigabeat, with and without patches and I'll see how it goes.

Played two whole games with music running in the background, no noticeable freezes without patch - perhaps someone with a lower powered target than the Gigabeat S should test this.

Well, I don't think the yields will introduce many problems but at least fix some so I think this should be committed rather than to wait for testers...

Commit this. I agree with Daniel, we can't be sure this entirely fixes the problem, but it will likely help and is harmless.