xbt_assert(recv_done, "Receiver killed somehow. It shouldn't");
}
-static void test_comm_dsend_and_quit()
+static void test_comm_dsend_and_quit_put_before_get()
{
XBT_INFO("%s: Launch a detached communication and end right after", __func__);
bool dsend_done = false;
simgrid::s4u::Actor::create("receiver", all_hosts[2], [&recv_done]() {
assert_exit(false, 3);
- bool got_exception = false;
simgrid::s4u::this_actor::sleep_for(2);
- try {
- void* payload = simgrid::s4u::Mailbox::by_name("mb")->get();
- xbt_free(payload);
- } catch (xbt_ex const& e) {
- got_exception = true;
- }
+ void* payload = simgrid::s4u::Mailbox::by_name("mb")->get();
+ xbt_free(payload);
recv_done = true;
- xbt_assert(not got_exception);
return;
});
xbt_assert(recv_done, "Receiver killed somehow. It shouldn't");
}
+static void test_comm_dsend_and_quit_get_before_put()
+{
+ XBT_INFO("%s: Launch a detached communication and end right after", __func__);
+ bool dsend_done = false;
+ bool recv_done = false;
+
+ simgrid::s4u::ActorPtr sender = simgrid::s4u::Actor::create("sender", all_hosts[1], [&dsend_done]() {
+ assert_exit(false, 2);
+ char* payload = xbt_strdup("toto");
+ simgrid::s4u::this_actor::sleep_for(2);
+ simgrid::s4u::Mailbox::by_name("mb")->put_init(payload, 1000)->detach();
+ dsend_done = true;
+ return;
+ });
+
+ simgrid::s4u::Actor::create("receiver", all_hosts[2], [&recv_done]() {
+ assert_exit(false, 3);
+ void* payload = simgrid::s4u::Mailbox::by_name("mb")->get();
+ xbt_free(payload);
+ recv_done = true;
+ return;
+ });
+
+ // Sleep long enough to let the test ends by itself. 3 + surf_precision should be enough.
+ simgrid::s4u::this_actor::sleep_for(4);
+ xbt_assert(dsend_done, "Sender killed somehow. It shouldn't");
+ xbt_assert(recv_done, "Receiver killed somehow. It shouldn't");
+}
+
+
static void test_comm_killsend()
{
XBT_INFO("%s: Launch a communication and kill the sender", __func__);
bool returned_from_main = false;
bool in_catch_before_on_exit = false;
bool in_catch_after_on_exit = false;
+ bool send_done = false;
simgrid::s4u::ActorPtr receiver = simgrid::s4u::Actor::create(
"receiver", all_hosts[1],
[&in_on_exit, &returned_from_main, &in_catch_before_on_exit, &in_catch_after_on_exit]() {
assert_exit(true, 1);
try {
- void *msg = simgrid::s4u::Mailbox::by_name("mb")->get();
- } catch (simgrid::HostFailureException const&) {
- in_catch_before_on_exit = not in_on_exit;
- in_catch_after_on_exit = in_on_exit;
+ simgrid::s4u::Mailbox::by_name("mb")->get();
} catch (simgrid::NetworkFailureException const&) {
+ // Shouldn't get in here
in_catch_before_on_exit = not in_on_exit;
in_catch_after_on_exit = in_on_exit;
}
returned_from_main = true;
});
- receiver->on_exit([&in_on_exit](bool failed) { in_on_exit = true; });
-
- simgrid::s4u::ActorPtr sender = simgrid::s4u::Actor::create(
- "sender", all_hosts[2],
- []() {
- try {
- int data;
- simgrid::s4u::Mailbox::by_name("mb")->put(&data, 100000);
- } catch (simgrid::HostFailureException const&) {
- } catch (simgrid::NetworkFailureException const&) {
- }
- });
+ receiver->on_exit([&in_on_exit](bool) { in_on_exit = true; });
+
+ simgrid::s4u::ActorPtr sender = simgrid::s4u::Actor::create("sender", all_hosts[2], [&send_done]() {
+ assert_exit(false, 1);
+ bool got_exception = false;
+ try {
+ int data = 42;
+ simgrid::s4u::Mailbox::by_name("mb")->put(&data, 100000);
+ } catch (simgrid::NetworkFailureException const&) {
+ got_exception = true;
+ }
+ xbt_assert(got_exception);
+ send_done = true;
+ });
simgrid::s4u::this_actor::sleep_for(1);
receiver->get_host()->turn_off();
// Note: If we don't sleep here, we don't "see" the bug
simgrid::s4u::this_actor::sleep_for(1);
+ receiver->get_host()->turn_on(); // switch host on again
xbt_assert(in_on_exit,
"Receiver's on_exit function was never called");
"Receiver mistakenly went to catch clause (after the on_exit function was called)");
xbt_assert(not returned_from_main,
"Receiver returned from main normally even though its host was killed");
+ xbt_assert(send_done, "Sender killed somehow. It shouldn't");
+}
+
+static void test_link_off()
+{
+ XBT_INFO("%s: Launch an actor that waits on a recv, turn communicating link off", __func__);
+ bool sender_got_exception = false;
+ bool receiver_got_exception = false;
+
+ simgrid::s4u::ActorPtr receiver = simgrid::s4u::Actor::create("receiver", all_hosts[1], [&receiver_got_exception]() {
+ assert_exit(false, 1);
+ try {
+ simgrid::s4u::Mailbox::by_name("mb")->get();
+ } catch (simgrid::NetworkFailureException const&) {
+ receiver_got_exception = true;
+ }
+ });
+
+ simgrid::s4u::ActorPtr sender = simgrid::s4u::Actor::create("sender", all_hosts[2], [&sender_got_exception]() {
+ assert_exit(false, 1);
+ try {
+ int data = 42;
+ simgrid::s4u::Mailbox::by_name("mb")->put(&data, 100000);
+ } catch (simgrid::NetworkFailureException const&) {
+ sender_got_exception = true;
+ }
+ });
+
+ simgrid::s4u::this_actor::sleep_for(1);
+ simgrid::s4u::Link::by_name("link1")->turn_off();
+ simgrid::s4u::Link::by_name("link1")->turn_on();
+ xbt_assert(receiver_got_exception, "Receiver should have caught a NetworkFailureException");
+ xbt_assert(sender_got_exception, "Sender should have caught a NetworkFailureException");
}
/* We need an extra actor here, so that it can sleep until the end of each test */
static void main_dispatcher()
{
- run_test("sleep", static_cast<std::function<void()>>(test_sleep));
+ run_test("sleep", test_sleep);
run_test("sleep killed at start", test_sleep_kill_begin);
run_test("sleep killed in middle", test_sleep_kill_middle);
/* We cannot kill right at the end of the action because killer actors are always rescheduled to the end of the round
* to avoid that they exit before their victim dereferences their name */
run_test("sleep restarted at start", test_sleep_restart_begin);
- run_test("sleep restarted at middle", test_sleep_restart_middle);
+ run_test("sleep restarted in middle", test_sleep_restart_middle);
// run_test("sleep restarted at end", test_sleep_restart_end);
- run_test("exec", static_cast<std::function<void()>>(test_exec));
+ run_test("exec", test_exec);
run_test("exec killed at start", test_exec_kill_begin);
run_test("exec killed in middle", test_exec_kill_middle);
run_test("exec restarted at start", test_exec_restart_begin);
- run_test("exec restarted at middle", test_exec_restart_middle);
+ run_test("exec restarted in middle", test_exec_restart_middle);
run_test("exec restarted at end", test_exec_restart_end);
run_test("comm", test_comm);
- run_test("comm dsend and quit", test_comm_dsend_and_quit);
+ run_test("comm dsend and quit (put before get)", test_comm_dsend_and_quit_put_before_get);
+ run_test("comm dsend and quit (get before put)", test_comm_dsend_and_quit_get_before_put);
run_test("comm kill sender", test_comm_killsend);
- //run_test("comm recv and kill", test_host_off_while_receive);
+ run_test("comm recv and kill", test_host_off_while_receive);
+ run_test("comm turn link off", test_link_off);
}
int main(int argc, char* argv[])