On 10/01/13 12:14, Jon Medhurst (Tixy) wrote:
[...]
I had to fix the line which gets the script called at boot:
-service setirqaffinity /set_irq_affinity 0xc07 +service setirqaffinity /set_irq_affinity.sh 0xc07
Thanks for fixing this. I was debating different names and looks like submitted the patch which was neither here nor there.
and testing on TC2, A9, and big.LITTLE Fast Models shows IRQ affinities getting set as we expect, i.e. to A7 cores if they exist, otherwise to all cores. I spoke to someone about the possible issues with IKS (the In Kernel Switcher for big.LITTLE) and the conclusion was that this patch would be innocuous.
Speaking to some IKS folk internally, I had reached a similar conclusion. So its good to see this being a more general view as well.
After fixing some whitespace issues and adding that '.sh' into init.arm-versatileexpress I've pushed the change into Gerrit http://review.android.git.linaro.org/#change,3240
I am not sure if review responses will make it to my mailbox but shoutout if any changes are needed.
Cheers, Punit
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.