Hi,
Below is a patch to prevent a NULL function being called by the
ctrl_if_tx_tasklet. This should not really be necessary (although its a
sensible check to make), however, I have been using the control interface, with
a ctrl_if_send_message_and_get_response call and have noticed that duplicate
responses (or response notifications) are sometimes recieved by this tasklet.
Since the first response resets the callback function to NULL, this results in
the second response sending the instruction pointer to 0 within interupt context
(which makes the kernel crash horribly in a way which is almost impossible to
debug :) ).
Looking through the code, the only other use of
ctrl_if_send_message_and_get_response is in netfront, and that is in the version
of probe_interfaces which panics the kernel.
This patch stops the horrible crash, however it doesn''t solve the
problem of duplicate responses being recieved, therefore sometimes a duplicate
message will introduce some kind of race condition which prevents a following
response from being correctly handled (meaning
ctrl_if_send_message_and_get_response blocks forever waiting for a responce
which has been lost).
If anyone has any ideas why duplicates might be sent, please let me know.
Cheers,
Ross
# HG changeset patch
# User rcmcilro@localhost.localdomain
# Node ID c51d3245ce42554889cccad19c81797d4cd6c011
# Parent afcc6c319b9ceae53ee218a0f07a932feebcf340
Fix null function execution in __cntl_if_tx_tasklet
diff -r afcc6c319b9c -r c51d3245ce42
linux-2.6-xen-sparse/arch/xen/kernel/ctrl_if.c
--- a/linux-2.6-xen-sparse/arch/xen/kernel/ctrl_if.c Tue Jul 19 15:25:43
2005+++ b/linux-2.6-xen-sparse/arch/xen/kernel/ctrl_if.c Tue Jul 19 15:52:39
2005@@ -141,7 +141,7 @@
msg->type, msg->subtype);
/* Execute the callback handler, if one was specified. */
- if ( msg->id != 0xFF )
+ if ( msg->id != 0xFF &&
*ctrl_if_txmsg_id_mapping[msg->id].fn != NULL)
{
(*ctrl_if_txmsg_id_mapping[msg->id].fn)(
msg, ctrl_if_txmsg_id_mapping[msg->id].id);
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
