On Mon September 10 2007 8:30 am, takaki.azuma wrote:
> Hi Kevin,
>
> The patch "[PATCH] 7/7: Non-RTAS Cell signal routing" works on our
> platform. Thank you really.
>
> Then I noticed two issues.
>
> First, when the argument 'num_signals' of celleb_activate_signals() is 0,
> an illegal value is returned.
>
> Second, I think that the argument to passthru_enable() in
> celleb_activate_signals() has a problem. The array index 'i' of the
> argument 'signals[i].cpu' doesn't indicate an effective "signals" array.
>
> I made the patch for them.

This patch looks fine. Thanks for catching those!

-- 
Kevin Corry
[EMAIL PROTECTED]
http://www.ibm.com/linux/
_______________________________________________
perfmon mailing list
[email protected]
http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/

Reply via email to