<br><font size=2 face="sans-serif">Gerhard,</font>
<br>
<br><font size=2 face="sans-serif">I think you meant to actually say &quot;update&quot;
instead of &quot;query&quot;. Is this true?</font>
<br><font size=2 face="sans-serif"><br>
&quot;</font><tt><font size=2>The functionality of spe_callback_handler_update()
can be achieved by<br>
using spe_callback_handler_register() in update mode.&quot;</font></tt>
<br>
<br><font size=2 face="sans-serif">Sincerely,</font>
<br><font size=2 face="sans-serif">Brian Watt</font>
<br>
<br><tt><font size=2>cbe-oss-dev-bounces+bwatt=us.ibm.com@ozlabs.org wrote
on 07/09/2007 09:16:08 AM:<br>
<br>
&gt; <br>
&gt; This patch removes a function which has never been documented and
<br>
&gt; should therefore be not in the API layer.<br>
&gt; The functionality of spe_callback_handler_query() can be achieved
by<br>
&gt; using spe_callback_handler_register() in update mode.<br>
&gt; <br>
&gt; Signed-off-by: Gerhard Stenzel &lt;stenzel at de.ibm.com&gt;<br>
&gt; <br>
&gt; ===================================================================<br>
&gt; Index: libspe2/libspe2.c<br>
&gt; ===================================================================<br>
&gt; --- libspe2/libspe2.c &nbsp; (revision 71)<br>
&gt; +++ libspe2/libspe2.c &nbsp; (working copy)<br>
&gt; @@ -464,14 +464,6 @@ void * spe_callback_handler_query(unsign<br>
&gt; &nbsp;}<br>
&gt; &nbsp;<br>
&gt; &nbsp;/*<br>
&gt; - * spe_callback_handler_update<br>
&gt; - */<br>
&gt; -int spe_callback_handler_update(void *handler, unsigned int callnum)
<br>
&gt; -{<br>
&gt; - &nbsp; return _base_spe_callback_handler_update(handler, callnum);<br>
&gt; -}<br>
&gt; -<br>
&gt; -/*<br>
&gt; &nbsp; * spe_info_get<br>
&gt; &nbsp; */<br>
&gt; &nbsp;int spe_cpu_info_get(int info_requested, int cpu_node) <br>
&gt; Index: libspe2/spebase/spebase.h<br>
&gt; ===================================================================<br>
&gt; --- libspe2/spebase/spebase.h &nbsp; (revision 71)<br>
&gt; +++ libspe2/spebase/spebase.h &nbsp; (working copy)<br>
&gt; @@ -556,12 +556,6 @@ extern int _base_spe_callback_handler_de<br>
&gt; &nbsp; */<br>
&gt; &nbsp;extern void * _base_spe_callback_handler_query(unsigned int
callnum );<br>
&gt; &nbsp;<br>
&gt; -/*<br>
&gt; - * spe_callback_handler_update<br>
&gt; - */<br>
&gt; -extern int _base_spe_callback_handler_update(void *handler, <br>
&gt; unsigned int callnum);<br>
&gt; -<br>
&gt; -<br>
&gt; &nbsp;/**<br>
&gt; &nbsp; * _base_spe_stop_reason_get<br>
&gt; &nbsp; * <br>
&gt; Index: libspe2/spebase/lib_builtin.c<br>
&gt; ===================================================================<br>
&gt; --- libspe2/spebase/lib_builtin.c &nbsp; (revision 71)<br>
&gt; +++ libspe2/spebase/lib_builtin.c &nbsp; (working copy)<br>
&gt; @@ -120,22 +120,6 @@ void * _base_spe_callback_handler_query(<br>
&gt; &nbsp; &nbsp; return handlers[callnum];<br>
&gt; &nbsp;}<br>
&gt; &nbsp;<br>
&gt; -int _base_spe_callback_handler_update(void *handler, unsigned int
callnum)<br>
&gt; -{<br>
&gt; - &nbsp; errno = 0;<br>
&gt; - &nbsp; if (callnum &gt; MAX_CALLNUM) {<br>
&gt; - &nbsp; &nbsp; &nbsp;errno = EINVAL;<br>
&gt; - &nbsp; &nbsp; &nbsp;return -1;<br>
&gt; - &nbsp; }<br>
&gt; - &nbsp; if (handlers[callnum] == NULL) {<br>
&gt; - &nbsp; &nbsp; &nbsp;errno = ESRCH;<br>
&gt; - &nbsp; &nbsp; &nbsp;return -1;<br>
&gt; - &nbsp; }<br>
&gt; - &nbsp; <br>
&gt; - &nbsp; handlers[callnum] = handler;<br>
&gt; - &nbsp; return 0;<br>
&gt; -}<br>
&gt; -<br>
&gt; &nbsp;int handle_library_callback(struct spe_context *spe, int callnum,<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp;unsigned int npc)<br>
&gt; &nbsp;{<br>
&gt; Index: libspe2/libspe2.h<br>
&gt; ===================================================================<br>
&gt; --- libspe2/libspe2.h &nbsp; (revision 71)<br>
&gt; +++ libspe2/libspe2.h &nbsp; (working copy)<br>
&gt; @@ -207,11 +207,6 @@ int spe_callback_handler_deregister (uns<br>
&gt; &nbsp;void * spe_callback_handler_query(unsigned int callnum);<br>
&gt; &nbsp;<br>
&gt; &nbsp;/*<br>
&gt; - * spe_callback_handler_update<br>
&gt; - */<br>
&gt; -int spe_callback_handler_update(void *handler, unsigned int callnum);<br>
&gt; -<br>
&gt; -/*<br>
&gt; &nbsp; * spe_info_get<br>
&gt; &nbsp; */<br>
&gt; &nbsp;int spe_cpu_info_get(int info_requested, int cpu_node); <br>
&gt; <br>
&gt; ===================================================================<br>
&gt; <br>
&gt; Best regards, <br>
&gt; <br>
&gt; Gerhard Stenzel, Linux on Cell Development, LTC<br>
&gt; -------------------------------------------------------------------------------------<br>
&gt; IBM Deutschland Entwicklung GmbH<br>
&gt; Vorsitzender des Aufsichtsrats: Martin Jetter | Geschaeftsfuehrung:
<br>
&gt; Herbert Kircher<br>
&gt; Sitz der Gesellschaft: Boeblingen | Registergericht: Amtsgericht <br>
&gt; Stuttgart, HRB 243294<br>
&gt; _______________________________________________<br>
&gt; cbe-oss-dev mailing list<br>
&gt; cbe-oss-dev@ozlabs.org<br>
&gt; https://ozlabs.org/mailman/listinfo/cbe-oss-dev<br>
</font></tt>