Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 1 | /* -*- Mode:C++; c-file-style:"gnu"; indent-tabs-mode:nil; -*- */ |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 2 | /* |
| 3 | * Copyright (c) 2014-2019, Regents of the University of California, |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 4 | * Arizona Board of Regents, |
| 5 | * Colorado State University, |
| 6 | * University Pierre & Marie Curie, Sorbonne University, |
| 7 | * Washington University in St. Louis, |
| 8 | * Beijing Institute of Technology, |
| 9 | * The University of Memphis. |
Alexander Afanasyev | 9bcbc7c | 2014-04-06 19:37:37 -0700 | [diff] [blame] | 10 | * |
| 11 | * This file is part of NFD (Named Data Networking Forwarding Daemon). |
| 12 | * See AUTHORS.md for complete list of NFD authors and contributors. |
| 13 | * |
| 14 | * NFD is free software: you can redistribute it and/or modify it under the terms |
| 15 | * of the GNU General Public License as published by the Free Software Foundation, |
| 16 | * either version 3 of the License, or (at your option) any later version. |
| 17 | * |
| 18 | * NFD is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; |
| 19 | * without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR |
| 20 | * PURPOSE. See the GNU General Public License for more details. |
| 21 | * |
| 22 | * You should have received a copy of the GNU General Public License along with |
| 23 | * NFD, e.g., in COPYING.md file. If not, see <http://www.gnu.org/licenses/>. |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 24 | */ |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 25 | |
| 26 | #include "fw/ncc-strategy.hpp" |
Davide Pesavento | 3dade00 | 2019-03-19 11:29:56 -0600 | [diff] [blame] | 27 | |
| 28 | #include "tests/daemon/face/dummy-face.hpp" |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 29 | #include "strategy-tester.hpp" |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 30 | #include "topology-tester.hpp" |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 31 | |
| 32 | namespace nfd { |
Spyridon Mastorakis | d0381c0 | 2015-02-19 10:29:41 -0800 | [diff] [blame] | 33 | namespace fw { |
Junxiao Shi | d9ee45c | 2014-02-27 15:38:11 -0700 | [diff] [blame] | 34 | namespace tests { |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 35 | |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 36 | using NccStrategyTester = StrategyTester<NccStrategy>; |
Junxiao Shi | 890afe9 | 2016-12-15 14:34:34 +0000 | [diff] [blame] | 37 | NFD_REGISTER_STRATEGY(NccStrategyTester); |
| 38 | |
Junxiao Shi | 5e5e445 | 2015-09-24 16:56:52 -0700 | [diff] [blame] | 39 | BOOST_AUTO_TEST_SUITE(Fw) |
Davide Pesavento | cf7db2f | 2019-03-24 23:17:28 -0400 | [diff] [blame] | 40 | BOOST_FIXTURE_TEST_SUITE(TestNccStrategy, GlobalIoTimeFixture) |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 41 | |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 42 | BOOST_AUTO_TEST_CASE(FavorRespondingUpstream) |
| 43 | { |
Junxiao Shi | 18739c4 | 2016-12-22 08:03:00 +0000 | [diff] [blame] | 44 | // NccStrategy is fairly complex. |
| 45 | // The most important property is: it remembers which upstream is the fastest to return Data, |
| 46 | // and favors this upstream in subsequent Interests. |
| 47 | |
Junxiao Shi | 3cb4fc6 | 2014-12-25 22:17:39 -0700 | [diff] [blame] | 48 | LimitedIo limitedIo(this); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 49 | Forwarder forwarder; |
Junxiao Shi | 890afe9 | 2016-12-15 14:34:34 +0000 | [diff] [blame] | 50 | NccStrategyTester& strategy = choose<NccStrategyTester>(forwarder); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 51 | strategy.afterAction.connect(bind(&LimitedIo::afterOp, &limitedIo)); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 52 | |
| 53 | shared_ptr<DummyFace> face1 = make_shared<DummyFace>(); |
| 54 | shared_ptr<DummyFace> face2 = make_shared<DummyFace>(); |
| 55 | shared_ptr<DummyFace> face3 = make_shared<DummyFace>(); |
| 56 | forwarder.addFace(face1); |
| 57 | forwarder.addFace(face2); |
| 58 | forwarder.addFace(face3); |
| 59 | |
| 60 | Fib& fib = forwarder.getFib(); |
Junxiao Shi | a6de429 | 2016-07-12 02:08:10 +0000 | [diff] [blame] | 61 | fib::Entry& fibEntry = *fib.insert(Name()).first; |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 62 | fibEntry.addOrUpdateNextHop(*face1, 0, 10); |
| 63 | fibEntry.addOrUpdateNextHop(*face2, 0, 20); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 64 | |
| 65 | Pit& pit = forwarder.getPit(); |
| 66 | |
| 67 | // first Interest: strategy knows nothing and follows routing |
Junxiao Shi | f3c0781 | 2014-03-11 21:48:49 -0700 | [diff] [blame] | 68 | shared_ptr<Interest> interest1p = makeInterest("ndn:/0Jm1ajrW/%00"); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 69 | Interest& interest1 = *interest1p; |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 70 | interest1.setInterestLifetime(8_s); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 71 | shared_ptr<pit::Entry> pitEntry1 = pit.insert(interest1).first; |
| 72 | |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 73 | pitEntry1->insertOrUpdateInRecord(*face3, 0, interest1); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 74 | strategy.afterReceiveInterest(FaceEndpoint(*face3, 0), interest1, pitEntry1); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 75 | |
| 76 | // forwards to face1 because routing says it's best |
Junxiao Shi | 1553910 | 2014-10-08 22:39:51 -0700 | [diff] [blame] | 77 | // (no io run here: afterReceiveInterest has already sent the Interest) |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 78 | BOOST_REQUIRE_EQUAL(strategy.sendInterestHistory.size(), 1); |
| 79 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[0].outFaceId, face1->getId()); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 80 | |
| 81 | // forwards to face2 because face1 doesn't respond |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 82 | limitedIo.run(1, 500_ms, 10_ms); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 83 | BOOST_REQUIRE_EQUAL(strategy.sendInterestHistory.size(), 2); |
| 84 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[1].outFaceId, face2->getId()); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 85 | |
| 86 | // face2 responds |
Junxiao Shi | f3c0781 | 2014-03-11 21:48:49 -0700 | [diff] [blame] | 87 | shared_ptr<Data> data1p = makeData("ndn:/0Jm1ajrW/%00"); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 88 | Data& data1 = *data1p; |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 89 | strategy.beforeSatisfyInterest(pitEntry1, FaceEndpoint(*face2, 0), data1); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 90 | this->advanceClocks(10_ms, 500_ms); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 91 | |
| 92 | // second Interest: strategy knows face2 is best |
Junxiao Shi | f3c0781 | 2014-03-11 21:48:49 -0700 | [diff] [blame] | 93 | shared_ptr<Interest> interest2p = makeInterest("ndn:/0Jm1ajrW/%00%01"); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 94 | Interest& interest2 = *interest2p; |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 95 | interest2.setInterestLifetime(8_s); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 96 | shared_ptr<pit::Entry> pitEntry2 = pit.insert(interest2).first; |
| 97 | |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 98 | pitEntry2->insertOrUpdateInRecord(*face3, 0, interest2); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 99 | strategy.afterReceiveInterest(FaceEndpoint(*face3, 0), interest2, pitEntry2); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 100 | |
| 101 | // forwards to face2 because it responds previously |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 102 | this->advanceClocks(1_ms); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 103 | BOOST_REQUIRE_GE(strategy.sendInterestHistory.size(), 3); |
| 104 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[2].outFaceId, face2->getId()); |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 105 | } |
| 106 | |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 107 | BOOST_AUTO_TEST_CASE(Bug1853) |
| 108 | { |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 109 | Forwarder forwarder; |
Junxiao Shi | 890afe9 | 2016-12-15 14:34:34 +0000 | [diff] [blame] | 110 | NccStrategyTester& strategy = choose<NccStrategyTester>(forwarder); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 111 | |
| 112 | shared_ptr<DummyFace> face1 = make_shared<DummyFace>(); |
| 113 | shared_ptr<DummyFace> face2 = make_shared<DummyFace>(); |
| 114 | shared_ptr<DummyFace> face3 = make_shared<DummyFace>(); |
| 115 | forwarder.addFace(face1); |
| 116 | forwarder.addFace(face2); |
| 117 | forwarder.addFace(face3); |
| 118 | |
| 119 | Fib& fib = forwarder.getFib(); |
Junxiao Shi | a6de429 | 2016-07-12 02:08:10 +0000 | [diff] [blame] | 120 | fib::Entry& fibEntry = *fib.insert(Name()).first; |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 121 | fibEntry.addOrUpdateNextHop(*face1, 0, 10); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 122 | |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 123 | Pit& pit = forwarder.getPit(); |
| 124 | |
| 125 | // first Interest: strategy follows routing and forwards to face1 |
| 126 | shared_ptr<Interest> interest1 = makeInterest("ndn:/nztwIvHX/%00"); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 127 | interest1->setInterestLifetime(8_s); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 128 | shared_ptr<pit::Entry> pitEntry1 = pit.insert(*interest1).first; |
| 129 | |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 130 | pitEntry1->insertOrUpdateInRecord(*face3, 0, *interest1); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 131 | strategy.afterReceiveInterest(FaceEndpoint(*face3, 0), *interest1, pitEntry1); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 132 | |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 133 | this->advanceClocks(1_ms); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 134 | BOOST_REQUIRE_EQUAL(strategy.sendInterestHistory.size(), 1); |
| 135 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[0].outFaceId, face1->getId()); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 136 | |
| 137 | // face1 responds |
| 138 | shared_ptr<Data> data1 = makeData("ndn:/nztwIvHX/%00"); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 139 | strategy.beforeSatisfyInterest(pitEntry1, FaceEndpoint(*face1, 0), *data1); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 140 | this->advanceClocks(10_ms, 500_ms); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 141 | |
| 142 | // second Interest: bestFace is face1, nUpstreams becomes 0, |
| 143 | // therefore pitEntryInfo->maxInterval cannot be calculated from deferRange and nUpstreams |
| 144 | shared_ptr<Interest> interest2 = makeInterest("ndn:/nztwIvHX/%01"); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 145 | interest2->setInterestLifetime(8_s); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 146 | shared_ptr<pit::Entry> pitEntry2 = pit.insert(*interest2).first; |
| 147 | |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 148 | pitEntry2->insertOrUpdateInRecord(*face3, 0, *interest2); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 149 | strategy.afterReceiveInterest(FaceEndpoint(*face3, 0), *interest2, pitEntry2); |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 150 | |
| 151 | // FIB entry is changed before doPropagate executes |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 152 | fibEntry.addOrUpdateNextHop(*face2, 0, 20); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 153 | this->advanceClocks(10_ms, 1_s);// should not crash |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 154 | } |
| 155 | |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 156 | BOOST_AUTO_TEST_CASE(Bug1961) |
| 157 | { |
Junxiao Shi | 3cb4fc6 | 2014-12-25 22:17:39 -0700 | [diff] [blame] | 158 | LimitedIo limitedIo(this); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 159 | Forwarder forwarder; |
Junxiao Shi | 890afe9 | 2016-12-15 14:34:34 +0000 | [diff] [blame] | 160 | NccStrategyTester& strategy = choose<NccStrategyTester>(forwarder); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 161 | strategy.afterAction.connect(bind(&LimitedIo::afterOp, &limitedIo)); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 162 | |
| 163 | shared_ptr<DummyFace> face1 = make_shared<DummyFace>(); |
| 164 | shared_ptr<DummyFace> face2 = make_shared<DummyFace>(); |
| 165 | shared_ptr<DummyFace> face3 = make_shared<DummyFace>(); |
| 166 | forwarder.addFace(face1); |
| 167 | forwarder.addFace(face2); |
| 168 | forwarder.addFace(face3); |
| 169 | |
| 170 | Fib& fib = forwarder.getFib(); |
Junxiao Shi | a6de429 | 2016-07-12 02:08:10 +0000 | [diff] [blame] | 171 | fib::Entry& fibEntry = *fib.insert(Name()).first; |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 172 | fibEntry.addOrUpdateNextHop(*face1, 0, 10); |
| 173 | fibEntry.addOrUpdateNextHop(*face2, 0, 20); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 174 | |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 175 | Pit& pit = forwarder.getPit(); |
| 176 | |
| 177 | // first Interest: strategy forwards to face1 and face2 |
| 178 | shared_ptr<Interest> interest1 = makeInterest("ndn:/seRMz5a6/%00"); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 179 | interest1->setInterestLifetime(2_s); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 180 | shared_ptr<pit::Entry> pitEntry1 = pit.insert(*interest1).first; |
| 181 | |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 182 | pitEntry1->insertOrUpdateInRecord(*face3, 0, *interest1); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 183 | strategy.afterReceiveInterest(FaceEndpoint(*face3, 0), *interest1, pitEntry1); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 184 | limitedIo.run(2 - strategy.sendInterestHistory.size(), 2_s, 10_ms); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 185 | BOOST_REQUIRE_EQUAL(strategy.sendInterestHistory.size(), 2); |
| 186 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[0].outFaceId, face1->getId()); |
| 187 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[1].outFaceId, face2->getId()); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 188 | |
| 189 | // face1 responds |
| 190 | shared_ptr<Data> data1 = makeData("ndn:/seRMz5a6/%00"); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 191 | strategy.beforeSatisfyInterest(pitEntry1, FaceEndpoint(*face1, 0), *data1); |
Junxiao Shi | 4846f37 | 2016-04-05 13:39:30 -0700 | [diff] [blame] | 192 | pitEntry1->clearInRecords(); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 193 | this->advanceClocks(10_ms); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 194 | // face2 also responds |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 195 | strategy.beforeSatisfyInterest(pitEntry1, FaceEndpoint(*face2, 0), *data1); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 196 | this->advanceClocks(10_ms); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 197 | |
| 198 | // second Interest: bestFace should be face 1 |
| 199 | shared_ptr<Interest> interest2 = makeInterest("ndn:/seRMz5a6/%01"); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 200 | interest2->setInterestLifetime(2_s); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 201 | shared_ptr<pit::Entry> pitEntry2 = pit.insert(*interest2).first; |
| 202 | |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 203 | pitEntry2->insertOrUpdateInRecord(*face3, 0, *interest2); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 204 | strategy.afterReceiveInterest(FaceEndpoint(*face3, 0), *interest2, pitEntry2); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 205 | limitedIo.run(3 - strategy.sendInterestHistory.size(), 2_s, 10_ms); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 206 | |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 207 | BOOST_REQUIRE_GE(strategy.sendInterestHistory.size(), 3); |
| 208 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[2].outFaceId, face1->getId()); |
Junxiao Shi | 82e7f58 | 2014-09-07 15:15:40 -0700 | [diff] [blame] | 209 | } |
Junxiao Shi | cbb490a | 2014-08-13 12:24:24 -0700 | [diff] [blame] | 210 | |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 211 | BOOST_AUTO_TEST_CASE(Bug1971) |
| 212 | { |
Junxiao Shi | 3cb4fc6 | 2014-12-25 22:17:39 -0700 | [diff] [blame] | 213 | LimitedIo limitedIo(this); |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 214 | Forwarder forwarder; |
Junxiao Shi | 890afe9 | 2016-12-15 14:34:34 +0000 | [diff] [blame] | 215 | NccStrategyTester& strategy = choose<NccStrategyTester>(forwarder); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 216 | strategy.afterAction.connect(bind(&LimitedIo::afterOp, &limitedIo)); |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 217 | |
| 218 | shared_ptr<DummyFace> face1 = make_shared<DummyFace>(); |
| 219 | shared_ptr<DummyFace> face2 = make_shared<DummyFace>(); |
| 220 | forwarder.addFace(face1); |
| 221 | forwarder.addFace(face2); |
| 222 | |
| 223 | Fib& fib = forwarder.getFib(); |
Junxiao Shi | a6de429 | 2016-07-12 02:08:10 +0000 | [diff] [blame] | 224 | fib::Entry& fibEntry = *fib.insert(Name()).first; |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 225 | fibEntry.addOrUpdateNextHop(*face2, 0, 10); |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 226 | |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 227 | Pit& pit = forwarder.getPit(); |
| 228 | |
| 229 | // first Interest: strategy forwards to face2 |
| 230 | shared_ptr<Interest> interest1 = makeInterest("ndn:/M4mBXCsd"); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 231 | interest1->setInterestLifetime(2_s); |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 232 | shared_ptr<pit::Entry> pitEntry1 = pit.insert(*interest1).first; |
| 233 | |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 234 | pitEntry1->insertOrUpdateInRecord(*face1, 0, *interest1); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 235 | strategy.afterReceiveInterest(FaceEndpoint(*face1, 0), *interest1, pitEntry1); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 236 | limitedIo.run(1 - strategy.sendInterestHistory.size(), 2_s, 10_ms); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 237 | BOOST_REQUIRE_EQUAL(strategy.sendInterestHistory.size(), 1); |
| 238 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[0].outFaceId, face2->getId()); |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 239 | |
| 240 | // face2 responds |
| 241 | shared_ptr<Data> data1 = makeData("ndn:/M4mBXCsd"); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 242 | data1->setFreshnessPeriod(5_ms); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 243 | strategy.beforeSatisfyInterest(pitEntry1, FaceEndpoint(*face2, 0), *data1); |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 244 | pitEntry1->deleteOutRecord(*face2, 0); |
Junxiao Shi | 4846f37 | 2016-04-05 13:39:30 -0700 | [diff] [blame] | 245 | pitEntry1->clearInRecords(); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 246 | this->advanceClocks(10_ms); |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 247 | |
| 248 | // similar Interest: strategy should still forward it |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 249 | pitEntry1->insertOrUpdateInRecord(*face1, 0, *interest1); |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 250 | strategy.afterReceiveInterest(FaceEndpoint(*face1, 0), *interest1, pitEntry1); |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 251 | limitedIo.run(2 - strategy.sendInterestHistory.size(), 2_s, 10_ms); |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 252 | BOOST_REQUIRE_EQUAL(strategy.sendInterestHistory.size(), 2); |
| 253 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[1].outFaceId, face2->getId()); |
Junxiao Shi | 8bfd56d | 2014-10-08 10:06:00 -0700 | [diff] [blame] | 254 | } |
| 255 | |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 256 | BOOST_AUTO_TEST_CASE(Bug1998) |
| 257 | { |
| 258 | Forwarder forwarder; |
Junxiao Shi | 890afe9 | 2016-12-15 14:34:34 +0000 | [diff] [blame] | 259 | NccStrategyTester& strategy = choose<NccStrategyTester>(forwarder); |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 260 | |
| 261 | shared_ptr<DummyFace> face1 = make_shared<DummyFace>(); |
| 262 | shared_ptr<DummyFace> face2 = make_shared<DummyFace>(); |
| 263 | forwarder.addFace(face1); |
| 264 | forwarder.addFace(face2); |
| 265 | |
| 266 | Fib& fib = forwarder.getFib(); |
Junxiao Shi | a6de429 | 2016-07-12 02:08:10 +0000 | [diff] [blame] | 267 | fib::Entry& fibEntry = *fib.insert(Name()).first; |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 268 | fibEntry.addOrUpdateNextHop(*face1, 0, 10); // face1 is top-ranked nexthop |
| 269 | fibEntry.addOrUpdateNextHop(*face2, 0, 20); |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 270 | |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 271 | Pit& pit = forwarder.getPit(); |
| 272 | |
| 273 | // Interest comes from face1, which is sole downstream |
| 274 | shared_ptr<Interest> interest1 = makeInterest("ndn:/tFy5HzUzD4"); |
| 275 | shared_ptr<pit::Entry> pitEntry1 = pit.insert(*interest1).first; |
ashiqopu | d3ae85d | 2019-02-17 02:29:55 +0000 | [diff] [blame] | 276 | pitEntry1->insertOrUpdateInRecord(*face1, 0, *interest1); |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 277 | |
ashiqopu | c707948 | 2019-02-20 05:34:37 +0000 | [diff] [blame] | 278 | strategy.afterReceiveInterest(FaceEndpoint(*face1, 0), *interest1, pitEntry1); |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 279 | |
| 280 | // Interest shall go to face2, not loop back to face1 |
Junxiao Shi | a49a1ab | 2016-07-15 18:24:36 +0000 | [diff] [blame] | 281 | BOOST_REQUIRE_EQUAL(strategy.sendInterestHistory.size(), 1); |
| 282 | BOOST_CHECK_EQUAL(strategy.sendInterestHistory[0].outFaceId, face2->getId()); |
Junxiao Shi | 6316220 | 2015-01-14 22:27:33 -0700 | [diff] [blame] | 283 | } |
| 284 | |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 285 | BOOST_AUTO_TEST_CASE_EXPECTED_FAILURES(PredictionAdjustment, 1) |
| 286 | BOOST_AUTO_TEST_CASE(PredictionAdjustment) // Bug 3411 |
| 287 | { |
| 288 | /* |
| 289 | * /----------\ |
| 290 | * | consumer | |
| 291 | * \----------/ |
| 292 | * | |
| 293 | * | |
| 294 | * +---+ 5ms +---+ |
| 295 | * | A |-------| B | |
| 296 | * +---+ +---+ |
| 297 | * | | |
| 298 | * | 5ms | 15ms/5ms |
| 299 | * | | |
| 300 | * +---+ +---+ |
| 301 | * | C |-------| D | |
| 302 | * +---+ 15ms +---+ |
| 303 | * /5ms | |
| 304 | * | |
| 305 | * /----------\ |
| 306 | * | producer | |
| 307 | * \----------/ |
| 308 | */ |
| 309 | |
| 310 | TopologyTester topo; |
| 311 | TopologyNode nodeA = topo.addForwarder("A"), |
| 312 | nodeB = topo.addForwarder("B"), |
| 313 | nodeC = topo.addForwarder("C"), |
| 314 | nodeD = topo.addForwarder("D"); |
| 315 | |
| 316 | for (TopologyNode node : {nodeA, nodeB, nodeC, nodeD}) { |
| 317 | topo.setStrategy<NccStrategy>(node); |
| 318 | } |
| 319 | |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 320 | shared_ptr<TopologyLink> linkAB = topo.addLink("AB", 5_ms, {nodeA, nodeB}), |
| 321 | linkAC = topo.addLink("AC", 5_ms, {nodeA, nodeC}), |
| 322 | linkBD = topo.addLink("BD", 15_ms, {nodeB, nodeD}), |
| 323 | linkCD = topo.addLink("CD", 15_ms, {nodeC, nodeD}); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 324 | |
| 325 | topo.registerPrefix(nodeA, linkAB->getFace(nodeA), "ndn:/P"); |
| 326 | topo.registerPrefix(nodeA, linkAC->getFace(nodeA), "ndn:/P"); |
| 327 | topo.registerPrefix(nodeB, linkBD->getFace(nodeB), "ndn:/P"); |
| 328 | topo.registerPrefix(nodeC, linkCD->getFace(nodeC), "ndn:/P"); |
| 329 | |
| 330 | shared_ptr<TopologyAppLink> producer = topo.addAppFace("producer", nodeD, "ndn:/P"); |
| 331 | topo.addEchoProducer(producer->getClientFace()); |
| 332 | |
| 333 | shared_ptr<TopologyAppLink> consumer = topo.addAppFace("consumer", nodeA); |
| 334 | topo.addIntervalConsumer(consumer->getClientFace(), "ndn:/P", |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 335 | 100_ms, 300); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 336 | |
Junxiao Shi | fc02186 | 2016-08-25 21:51:18 +0000 | [diff] [blame] | 337 | auto getMeInfo = [&] () -> NccStrategy::MeasurementsEntryInfo* { |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 338 | Measurements& measurements = topo.getForwarder(nodeA).getMeasurements(); |
Junxiao Shi | 80f9fcd | 2016-07-23 02:48:36 +0000 | [diff] [blame] | 339 | measurements::Entry* me = measurements.findExactMatch("ndn:/P"); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 340 | return me == nullptr ? nullptr : me->getStrategyInfo<NccStrategy::MeasurementsEntryInfo>(); |
| 341 | }; |
| 342 | |
| 343 | // NccStrategy starts with an exploration period when the algorithm adjusts |
| 344 | // its prediction to converge near the path RTT. |
| 345 | bool isExplorationFinished = false; |
| 346 | const time::milliseconds TRUE_RTT1(40); |
| 347 | bool isLastPredictionUnder = true; |
| 348 | int nPredictionCrossings = 0; |
| 349 | for (int i = 0; i < 10000; ++i) { |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 350 | this->advanceClocks(5_ms); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 351 | auto meInfo = getMeInfo(); |
| 352 | if (meInfo == nullptr) { |
| 353 | continue; |
| 354 | } |
| 355 | |
| 356 | if ((isLastPredictionUnder && meInfo->prediction > TRUE_RTT1) || |
| 357 | (!isLastPredictionUnder && meInfo->prediction < TRUE_RTT1)) { |
| 358 | isLastPredictionUnder = !isLastPredictionUnder; |
| 359 | if (++nPredictionCrossings > 6) { |
| 360 | isExplorationFinished = true; |
| 361 | BOOST_TEST_MESSAGE("exploration finishes after " << (i * 5) << "ms"); |
| 362 | break; |
| 363 | } |
| 364 | } |
| 365 | } |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 366 | BOOST_REQUIRE_MESSAGE(isExplorationFinished, "exploration did not finish within 50s"); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 367 | |
| 368 | // NccStrategy has selected one path as the best. |
| 369 | // When we reduce the RTT of the other path, ideally it should be selected as the best face. |
| 370 | // However, this won't happen due to a weakness in NccStrategy. |
ashiqopu | 3ad49db | 2018-10-20 22:38:47 +0000 | [diff] [blame] | 371 | // See https://redmine.named-data.net/issues/3411#note-4 |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 372 | shared_ptr<Face> bestFace1 = getMeInfo()->bestFace.lock(); |
| 373 | if (bestFace1.get() == &linkAB->getFace(nodeA)) { |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 374 | linkCD->setDelay(5_ms); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 375 | } |
| 376 | else if (bestFace1.get() == &linkAC->getFace(nodeA)) { |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 377 | linkBD->setDelay(5_ms); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 378 | } |
| 379 | else { |
| 380 | BOOST_FAIL("unexpected best face"); |
| 381 | } |
| 382 | |
| 383 | bool isNewBestChosen = false; |
| 384 | for (int i = 0; i < 10000; ++i) { |
Davide Pesavento | 14e71f0 | 2019-03-28 17:35:25 -0400 | [diff] [blame] | 385 | this->advanceClocks(5_ms); |
Hila Ben Abraham | 39a79be | 2016-03-30 22:00:55 -0700 | [diff] [blame] | 386 | auto meInfo = getMeInfo(); |
| 387 | if (meInfo == nullptr) { |
| 388 | continue; |
| 389 | } |
| 390 | |
| 391 | if (meInfo->bestFace.lock() != bestFace1) { |
| 392 | isNewBestChosen = true; |
| 393 | BOOST_TEST_MESSAGE("new best face is found after " << (i * 5) << "ms"); |
| 394 | break; |
| 395 | } |
| 396 | } |
| 397 | BOOST_CHECK_MESSAGE(isNewBestChosen, "new best face is not found in 50000ms"); // expected failure |
| 398 | } |
| 399 | |
Junxiao Shi | 5e5e445 | 2015-09-24 16:56:52 -0700 | [diff] [blame] | 400 | BOOST_AUTO_TEST_SUITE_END() // TestNccStrategy |
| 401 | BOOST_AUTO_TEST_SUITE_END() // Fw |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 402 | |
Junxiao Shi | d9ee45c | 2014-02-27 15:38:11 -0700 | [diff] [blame] | 403 | } // namespace tests |
Spyridon Mastorakis | d0381c0 | 2015-02-19 10:29:41 -0800 | [diff] [blame] | 404 | } // namespace fw |
Junxiao Shi | 0b5fbbb | 2014-02-20 15:54:03 -0700 | [diff] [blame] | 405 | } // namespace nfd |