]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
Merge commit '4363837772a4a6ba7842cce87f0f7410ff8d4231' into wip-pull-new-dmclock-2
authorJ. Eric Ivancich <ivancich@redhat.com>
Fri, 4 Aug 2017 16:35:35 +0000 (12:35 -0400)
committerJ. Eric Ivancich <ivancich@redhat.com>
Fri, 4 Aug 2017 16:35:35 +0000 (12:35 -0400)
Signed-off-by: J. Eric Ivancich <ivancich@redhat.com>
15 files changed:
1  2 
src/dmclock/CMakeLists.txt
src/dmclock/cmake/modules/BuildGTest.cmake
src/dmclock/sim/src/sim_client.h
src/dmclock/sim/src/sim_server.h
src/dmclock/sim/src/ssched/ssched_server.h
src/dmclock/sim/src/test_dmclock_main.cc
src/dmclock/sim/src/test_ssched_main.cc
src/dmclock/src/CMakeLists.txt
src/dmclock/src/dmclock_client.h
src/dmclock/src/dmclock_server.h
src/dmclock/src/dmclock_util.h
src/dmclock/test/CMakeLists.txt
src/dmclock/test/dmcPrCtl.h
src/dmclock/test/dmtest-config.h.in
src/dmclock/test/test_dmclock_server.cc

index 9444ea1e96afd0a27250d6d989a0b41f797cdcd2,0000000000000000000000000000000000000000..d133bc6b015067e30169952680045618dc56247b
mode 100644,000000..100644
--- /dev/null
@@@ -1,25 -1,0 +1,34 @@@
-   if(NOT(GTEST_FOUND))
-     find_package(GTest REQUIRED)
 +cmake_minimum_required(VERSION 2.8.11)
 +
 +list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_LIST_DIR}/cmake/modules")
 +
++set(CMAKE_CXX_FLAGS
++  "${CMAKE_CXX_FLAGS} -std=c++11 -Wno-write-strings -Wall -pthread")
++
++if(DO_NOT_DELAY_TAG_CALC)
++  add_definitions(-DDO_NOT_DELAY_TAG_CALC)
++endif()
++
 +if (NOT(TARGET gtest AND TARGET gtest_main))
- # add_subdirectory(support/src)
++  if (NOT GTEST_FOUND)
++    find_package(GTest QUIET)
++    if (NOT GTEST_FOUND)
++      include(BuildGTest)
++    endif()
 +  endif()
 +endif()
 +
 +if (NOT(BOOST_FOUND))
 +  find_package(Boost REQUIRED)
 +endif()
 +
 +add_subdirectory(src)
 +add_subdirectory(sim)
 +
 +enable_testing()
 +add_subdirectory(test)
 +add_subdirectory(support/test)
 +add_test(NAME dmclock-tests
 +  COMMAND $<TARGET_FILE:dmclock-tests>)
 +add_test(NAME dmclock-data-struct-tests
 +  COMMAND $<TARGET_FILE:dmclock-data-struct-tests>)
index 0000000000000000000000000000000000000000,0000000000000000000000000000000000000000..356c7d5ab31dce7f069a4d66e78bc9c9da0fbe10
new file mode 100644 (file)
--- /dev/null
--- /dev/null
@@@ -1,0 -1,0 +1,67 @@@
++macro(_build_gtest gtest_root)
++  include(ExternalProject)
++  ExternalProject_Add(googletest
++    SOURCE_DIR ${gtest_root}
++    CMAKE_ARGS -DBUILD_GMOCK=OFF -DBUILD_GTEST=ON
++    INSTALL_COMMAND ""
++    LOG_CONFIGURE ON
++    LOG_BUILD ON)
++
++  ExternalProject_Get_Property(googletest source_dir)
++  set(GTEST_INCLUDE_DIRS ${source_dir}/googletest/include)
++  set(GMOCK_INCLUDE_DIRS ${source_dir}/googlemock/include)
++
++  find_package(Threads REQUIRED)
++
++  ExternalProject_Get_Property(googletest binary_dir)
++  set(GTEST_LIBRARY_PATH ${binary_dir}/googletest/${CMAKE_FIND_LIBRARY_PREFIXES}gtest.a)
++  set(GTEST_LIBRARY gtest)
++  add_library(${GTEST_LIBRARY} STATIC IMPORTED)
++  set_target_properties(${GTEST_LIBRARY} PROPERTIES
++    INTERFACE_INCLUDE_DIRECTORIES "${GTEST_INCLUDE_DIRS}"
++    IMPORTED_LOCATION ${GTEST_LIBRARY_PATH}
++    IMPORTED_LINK_INTERFACE_LANGUAGES "CXX"
++    IMPORTED_LINK_INTERFACE_LIBRARIES ${CMAKE_THREAD_LIBS_INIT})
++  add_dependencies(${GTEST_LIBRARY} googletest)
++  set(GTEST_LIBRARIES ${GTEST_LIBRARY})
++
++  set(GTEST_MAIN_LIBRARY_PATH ${binary_dir}/googletest/${CMAKE_FIND_LIBRARY_PREFIXES}gtest_main.a)
++  set(GTEST_MAIN_LIBRARY gtest_main)
++  add_library(${GTEST_MAIN_LIBRARY} STATIC IMPORTED)
++  set_target_properties(${GTEST_MAIN_LIBRARY} PROPERTIES
++    INTERFACE_INCLUDE_DIRECTORIES "${GTEST_INCLUDE_DIRS}"
++    IMPORTED_LOCATION ${GTEST_MAIN_LIBRARY_PATH}
++    IMPORTED_LINK_INTERFACE_LIBRARIES ${CMAKE_THREAD_LIBS_INIT})
++  add_dependencies(${GTEST_MAIN_LIBRARY} googletest)
++
++  set(GMOCK_LIBRARY_PATH ${binary_dir}/googlemock/${CMAKE_FIND_LIBRARY_PREFIXES}gmock.a)
++  set(GMOCK_LIBRARY gmock)
++  add_library(${GMOCK_LIBRARY} STATIC IMPORTED)
++  set_target_properties(${GMOCK_LIBRARY} PROPERTIES
++    INTERFACE_INCLUDE_DIRECTORIES "${GMOCK_INCLUDE_DIRS}"
++    IMPORTED_LOCATION "${GMOCK_LIBRARY_PATH}"
++    IMPORTED_LINK_INTERFACE_LANGUAGES "CXX"
++    IMPORTED_LINK_INTERFACE_LIBRARIES ${CMAKE_THREAD_LIBS_INIT})
++  add_dependencies(${GMOCK_LIBRARY} googletest)
++
++  set(GMOCK_MAIN_LIBRARY_PATH ${binary_dir}/googlemock/${CMAKE_FIND_LIBRARY_PREFIXES}gmock_main.a)
++  set(GMOCK_MAIN_LIBRARY gmock_main)
++  add_library(${GMOCK_MAIN_LIBRARY} STATIC IMPORTED)
++  set_target_properties(${GMOCK_MAIN_LIBRARY} PROPERTIES
++    INTERFACE_INCLUDE_DIRECTORIES "${GMOCK_INCLUDE_DIRS}"
++    IMPORTED_LOCATION ${GMOCK_MAIN_LIBRARY_PATH}
++    IMPORTED_LINK_INTERFACE_LANGUAGES "CXX"
++    IMPORTED_LINK_INTERFACE_LIBRARIES ${CMAKE_THREAD_LIBS_INIT})
++  add_dependencies(${GMOCK_MAIN_LIBRARY} ${GTEST_LIBRARY})
++endmacro()
++
++find_path(GTEST_ROOT src/gtest.cc
++  HINTS $ENV{GTEST_ROOT}
++  PATHS /usr/src/googletest/googletest /usr/src/gtest)
++
++if(EXISTS ${GTEST_ROOT})
++  message(STATUS "Found googletest: ${GTEST_ROOT}")
++  _build_gtest(${GTEST_ROOT})
++else()
++  message(SEND_ERROR "Could NOT find googletest")
++endif()
index 6538dab2c0811a69a372fb01238330fce0a60acf,0000000000000000000000000000000000000000..fd4a81c76dac1a498ebaed534d2bc454514be059
mode 100644,000000..100644
--- /dev/null
@@@ -1,329 -1,0 +1,330 @@@
-                          const TestRequest&,
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2016 Red Hat Inc.
 + */
 +
 +
 +#pragma once
 +
 +
 +#include <atomic>
 +#include <mutex>
 +#include <condition_variable>
 +#include <thread>
 +#include <chrono>
 +#include <vector>
 +#include <deque>
 +#include <iostream>
 +
 +#include "sim_recs.h"
 +
 +
 +namespace crimson {
 +  namespace qos_simulation {
 +
 +    struct req_op_t {};
 +    struct wait_op_t {};
 +    constexpr struct req_op_t req_op {};
 +    constexpr struct wait_op_t wait_op {};
 +
 +
 +    enum class CliOp { req, wait };
 +    struct CliInst {
 +      CliOp op;
 +      union {
 +      std::chrono::milliseconds wait_time;
 +      struct {
 +        uint32_t count;
 +        std::chrono::microseconds time_bw_reqs;
 +        uint16_t max_outstanding;
 +      } req_params;
 +      } args;
 +
 +      // D is a duration type
 +      template<typename D>
 +      CliInst(wait_op_t, D duration) :
 +      op(CliOp::wait)
 +      {
 +      args.wait_time =
 +        std::chrono::duration_cast<std::chrono::milliseconds>(duration);
 +      }
 +
 +      CliInst(req_op_t,
 +            uint32_t count, double ops_per_sec, uint16_t max_outstanding) :
 +      op(CliOp::req)
 +      {
 +      args.req_params.count = count;
 +      args.req_params.max_outstanding = max_outstanding;
 +      uint32_t us = uint32_t(0.5 + 1.0 / ops_per_sec * 1000000);
 +      args.req_params.time_bw_reqs = std::chrono::microseconds(us);
 +      }
 +    };
 +
 +
 +    using ServerSelectFunc = std::function<const ServerId&(uint64_t seed)>;
 +
 +
 +    template<typename SvcTrk, typename ReqPm, typename RespPm, typename Accum>
 +    class SimulatedClient {
 +    public:
 +
 +      struct InternalStats {
 +      std::mutex mtx;
 +      std::chrono::nanoseconds track_resp_time;
 +      std::chrono::nanoseconds get_req_params_time;
 +      uint32_t track_resp_count;
 +      uint32_t get_req_params_count;
 +
 +      InternalStats() :
 +        track_resp_time(0),
 +        get_req_params_time(0),
 +        track_resp_count(0),
 +        get_req_params_count(0)
 +      {
 +        // empty
 +      }
 +      };
 +
 +      using SubmitFunc =
 +      std::function<void(const ServerId&,
-             TestRequest req(server, o, 12);
-             submit_f(server, req, id, rp);
++                         TestRequest&&,
 +                         const ClientId&,
 +                         const ReqPm&)>;
 +
 +      using ClientAccumFunc = std::function<void(Accum&,const RespPm&)>;
 +
 +      typedef std::chrono::time_point<std::chrono::steady_clock> TimePoint;
 +
 +      static TimePoint now() { return std::chrono::steady_clock::now(); }
 +
 +    protected:
 +
 +      struct RespQueueItem {
 +      TestResponse response;
 +      ServerId     server_id;
 +      RespPm       resp_params;
 +      };
 +
 +      const ClientId id;
 +      const SubmitFunc submit_f;
 +      const ServerSelectFunc server_select_f;
 +      const ClientAccumFunc accum_f;
 +
 +      std::vector<CliInst> instructions;
 +
 +      SvcTrk service_tracker;
 +
 +      // TODO: use lock rather than atomic???
 +      std::atomic_ulong        outstanding_ops;
 +      std::atomic_bool         requests_complete;
 +
 +      std::deque<RespQueueItem> resp_queue;
 +
 +      std::mutex               mtx_req;
 +      std::condition_variable  cv_req;
 +
 +      std::mutex               mtx_resp;
 +      std::condition_variable  cv_resp;
 +
 +      using RespGuard = std::lock_guard<decltype(mtx_resp)>;
 +      using Lock = std::unique_lock<std::mutex>;
 +
 +      // data collection
 +
 +      std::vector<TimePoint>   op_times;
 +      Accum                    accumulator;
 +      InternalStats            internal_stats;
 +
 +      std::thread              thd_req;
 +      std::thread              thd_resp;
 +
 +    public:
 +
 +      SimulatedClient(ClientId _id,
 +                    const SubmitFunc& _submit_f,
 +                    const ServerSelectFunc& _server_select_f,
 +                    const ClientAccumFunc& _accum_f,
 +                    const std::vector<CliInst>& _instrs) :
 +      id(_id),
 +      submit_f(_submit_f),
 +      server_select_f(_server_select_f),
 +      accum_f(_accum_f),
 +      instructions(_instrs),
 +      service_tracker(),
 +      outstanding_ops(0),
 +      requests_complete(false)
 +      {
 +      size_t op_count = 0;
 +      for (auto i : instructions) {
 +        if (CliOp::req == i.op) {
 +          op_count += i.args.req_params.count;
 +        }
 +      }
 +      op_times.reserve(op_count);
 +
 +      thd_resp = std::thread(&SimulatedClient::run_resp, this);
 +      thd_req = std::thread(&SimulatedClient::run_req, this);
 +      }
 +
 +
 +      SimulatedClient(ClientId _id,
 +                    const SubmitFunc& _submit_f,
 +                    const ServerSelectFunc& _server_select_f,
 +                    const ClientAccumFunc& _accum_f,
 +                    uint16_t _ops_to_run,
 +                    double _iops_goal,
 +                    uint16_t _outstanding_ops_allowed) :
 +      SimulatedClient(_id,
 +                      _submit_f, _server_select_f, _accum_f,
 +                      {{req_op, _ops_to_run, _iops_goal, _outstanding_ops_allowed}})
 +      {
 +      // empty
 +      }
 +
 +
 +      SimulatedClient(const SimulatedClient&) = delete;
 +      SimulatedClient(SimulatedClient&&) = delete;
 +      SimulatedClient& operator=(const SimulatedClient&) = delete;
 +      SimulatedClient& operator=(SimulatedClient&&) = delete;
 +
 +      virtual ~SimulatedClient() {
 +      wait_until_done();
 +      }
 +
 +      void receive_response(const TestResponse& resp,
 +                          const ServerId& server_id,
 +                          const RespPm& resp_params) {
 +      RespGuard g(mtx_resp);
 +      resp_queue.push_back(RespQueueItem{resp, server_id, resp_params});
 +      cv_resp.notify_one();
 +      }
 +
 +      const std::vector<TimePoint>& get_op_times() const { return op_times; }
 +
 +      void wait_until_done() {
 +      if (thd_req.joinable()) thd_req.join();
 +      if (thd_resp.joinable()) thd_resp.join();
 +      }
 +
 +      const Accum& get_accumulator() const { return accumulator; }
 +
 +      const InternalStats& get_internal_stats() const { return internal_stats; }
 +
 +    protected:
 +
 +      void run_req() {
 +      size_t ops_count = 0;
 +      for (auto i : instructions) {
 +        if (CliOp::wait == i.op) {
 +          std::this_thread::sleep_for(i.args.wait_time);
 +        } else if (CliOp::req == i.op) {
 +          Lock l(mtx_req);
 +          for (uint64_t o = 0; o < i.args.req_params.count; ++o) {
 +            while (outstanding_ops >= i.args.req_params.max_outstanding) {
 +              cv_req.wait(l);
 +            }
 +
 +            l.unlock();
 +            auto now = std::chrono::steady_clock::now();
 +            const ServerId& server = server_select_f(o);
 +
 +            ReqPm rp =
 +              time_stats_w_return<decltype(internal_stats.get_req_params_time),
 +                                  ReqPm>(internal_stats.mtx,
 +                                         internal_stats.get_req_params_time,
 +                                         [&]() -> ReqPm {
 +                                           return service_tracker.get_req_params(server);
 +                                         });
 +            count_stats(internal_stats.mtx,
 +                        internal_stats.get_req_params_count);
 +
++            submit_f(server,
++                     TestRequest{server, static_cast<uint32_t>(o), 12},
++                     id, rp);
 +            ++outstanding_ops;
 +            l.lock(); // lock for return to top of loop
 +
 +            auto delay_time = now + i.args.req_params.time_bw_reqs;
 +            while (std::chrono::steady_clock::now() < delay_time) {
 +              cv_req.wait_until(l, delay_time);
 +            } // while
 +          } // for
 +          ops_count += i.args.req_params.count;
 +        } else {
 +          assert(false);
 +        }
 +      } // for loop
 +
 +      requests_complete = true;
 +
 +      // all requests made, thread ends
 +      }
 +
 +
 +      void run_resp() {
 +      std::chrono::milliseconds delay(1000);
 +      int op = 0;
 +
 +      Lock l(mtx_resp);
 +
 +      // since the following code would otherwise be repeated (except for
 +      // the call to notify_one) in the two loops below; let's avoid
 +      // repetition and define it once.
 +      const auto proc_resp = [this, &op, &l](const bool notify_req_cv) {
 +        if (!resp_queue.empty()) {
 +          RespQueueItem item = resp_queue.front();
 +          resp_queue.pop_front();
 +
 +          l.unlock();
 +
 +          // data collection
 +
 +          op_times.push_back(now());
 +          accum_f(accumulator, item.resp_params);
 +
 +          // processing
 +
 +#if 0 // not needed
 +          TestResponse& resp = item.response;
 +#endif
 +
 +          time_stats(internal_stats.mtx,
 +                     internal_stats.track_resp_time,
 +                     [&](){
 +                       service_tracker.track_resp(item.server_id, item.resp_params);
 +                     });
 +          count_stats(internal_stats.mtx,
 +                      internal_stats.track_resp_count);
 +
 +          --outstanding_ops;
 +          if (notify_req_cv) {
 +            cv_req.notify_one();
 +          }
 +
 +          l.lock();
 +        }
 +      };
 +
 +      while(!requests_complete.load()) {
 +        while(resp_queue.empty() && !requests_complete.load()) {
 +          cv_resp.wait_for(l, delay);
 +        }
 +        proc_resp(true);
 +      }
 +
 +      while(outstanding_ops.load() > 0) {
 +        while(resp_queue.empty() && outstanding_ops.load() > 0) {
 +          cv_resp.wait_for(l, delay);
 +        }
 +        proc_resp(false); // don't call notify_one as all requests are complete
 +      }
 +
 +      // all responses received, thread ends
 +      }
 +    }; // class SimulatedClient
 +
 +
 +  }; // namespace qos_simulation
 +}; // namespace crimson
index e318d6e90ac328e6810aa4b7a1fc4f628003ee37,0000000000000000000000000000000000000000..42b5269d780323085c3d807ce1a5c84b0c7cc8ab
mode 100644,000000..100644
--- /dev/null
@@@ -1,227 -1,0 +1,227 @@@
-       void post(const TestRequest& request,
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2016 Red Hat Inc.
 + */
 +
 +
 +#pragma once
 +
 +
 +#include <thread>
 +#include <mutex>
 +#include <condition_variable>
 +#include <chrono>
 +#include <deque>
 +
 +#include "sim_recs.h"
 +
 +
 +namespace crimson {
 +  namespace qos_simulation {
 +
 +    template<typename Q, typename ReqPm, typename RespPm, typename Accum>
 +    class SimulatedServer {
 +
 +      struct QueueItem {
 +      ClientId                     client;
 +      std::unique_ptr<TestRequest> request;
 +      RespPm                       additional;
 +
 +      QueueItem(const ClientId&                _client,
 +                std::unique_ptr<TestRequest>&& _request,
 +                const RespPm&                  _additional) :
 +        client(_client),
 +        request(std::move(_request)),
 +        additional(_additional)
 +      {
 +        // empty
 +      }
 +      }; // QueueItem
 +
 +    public:
 +
 +      struct InternalStats {
 +      std::mutex mtx;
 +      std::chrono::nanoseconds add_request_time;
 +      std::chrono::nanoseconds request_complete_time;
 +      uint32_t add_request_count;
 +      uint32_t request_complete_count;
 +
 +      InternalStats() :
 +        add_request_time(0),
 +        request_complete_time(0),
 +        add_request_count(0),
 +        request_complete_count(0)
 +      {
 +        // empty
 +      }
 +      };
 +
 +      using ClientRespFunc = std::function<void(ClientId,
 +                                              const TestResponse&,
 +                                              const ServerId&,
 +                                              const RespPm&)>;
 +
 +      using ServerAccumFunc = std::function<void(Accum& accumulator,
 +                                               const RespPm& additional)>;
 +
 +    protected:
 +
 +      const ServerId                 id;
 +      Q*                             priority_queue;
 +      ClientRespFunc                 client_resp_f;
 +      int                            iops;
 +      size_t                         thread_pool_size;
 +
 +      bool                           finishing;
 +      std::chrono::microseconds      op_time;
 +
 +      std::mutex                     inner_queue_mtx;
 +      std::condition_variable        inner_queue_cv;
 +      std::deque<QueueItem>          inner_queue;
 +
 +      std::thread*                   threads;
 +
 +      using InnerQGuard = std::lock_guard<decltype(inner_queue_mtx)>;
 +      using Lock = std::unique_lock<std::mutex>;
 +
 +      // data collection
 +
 +      ServerAccumFunc accum_f;
 +      Accum accumulator;
 +
 +      InternalStats internal_stats;
 +
 +    public:
 +
 +      using CanHandleRequestFunc = std::function<bool(void)>;
 +      using HandleRequestFunc =
 +      std::function<void(const ClientId&,std::unique_ptr<TestRequest>,const RespPm&)>;
 +      using CreateQueueF = std::function<Q*(CanHandleRequestFunc,HandleRequestFunc)>;
 +                                      
 +
 +      SimulatedServer(ServerId _id,
 +                    int _iops,
 +                    size_t _thread_pool_size,
 +                    const ClientRespFunc& _client_resp_f,
 +                    const ServerAccumFunc& _accum_f,
 +                    CreateQueueF _create_queue_f) :
 +      id(_id),
 +      priority_queue(_create_queue_f(std::bind(&SimulatedServer::has_avail_thread,
 +                                               this),
 +                                     std::bind(&SimulatedServer::inner_post,
 +                                               this,
 +                                               std::placeholders::_1,
 +                                               std::placeholders::_2,
 +                                               std::placeholders::_3))),
 +      client_resp_f(_client_resp_f),
 +      iops(_iops),
 +      thread_pool_size(_thread_pool_size),
 +      finishing(false),
 +      accum_f(_accum_f)
 +      {
 +      op_time =
 +        std::chrono::microseconds((int) (0.5 +
 +                                         thread_pool_size * 1000000.0 / iops));
 +      std::chrono::milliseconds delay(1000);
 +      threads = new std::thread[thread_pool_size];
 +      for (size_t i = 0; i < thread_pool_size; ++i) {
 +        threads[i] = std::thread(&SimulatedServer::run, this, delay);
 +      }
 +      }
 +
 +      virtual ~SimulatedServer() {
 +      Lock l(inner_queue_mtx);
 +      finishing = true;
 +      inner_queue_cv.notify_all();
 +      l.unlock();
 +
 +      for (size_t i = 0; i < thread_pool_size; ++i) {
 +        threads[i].join();
 +      }
 +
 +      delete[] threads;
 +
 +      delete priority_queue;
 +      }
 +
-                    priority_queue->add_request(request, client_id, req_params);
++      void post(TestRequest&& request,
 +              const ClientId& client_id,
 +              const ReqPm& req_params)
 +      {
 +      time_stats(internal_stats.mtx,
 +                 internal_stats.add_request_time,
 +                 [&](){
-           TestResponse resp(req->epoch);
++                   priority_queue->add_request(std::move(request),
++                                               client_id, req_params);
 +                 });
 +      count_stats(internal_stats.mtx,
 +                  internal_stats.add_request_count);
 +      }
 +
 +      bool has_avail_thread() {
 +      InnerQGuard g(inner_queue_mtx);
 +      return inner_queue.size() <= thread_pool_size;
 +      }
 +
 +      const Accum& get_accumulator() const { return accumulator; }
 +      const Q& get_priority_queue() const { return *priority_queue; }
 +      const InternalStats& get_internal_stats() const { return internal_stats; }
 +
 +    protected:
 +
 +      void inner_post(const ClientId& client,
 +                    std::unique_ptr<TestRequest> request,
 +                    const RespPm& additional) {
 +      Lock l(inner_queue_mtx);
 +      assert(!finishing);
 +      accum_f(accumulator, additional);
 +      inner_queue.emplace_back(QueueItem(client,
 +                                         std::move(request),
 +                                         additional));
 +      inner_queue_cv.notify_one();
 +      }
 +
 +      void run(std::chrono::milliseconds check_period) {
 +      Lock l(inner_queue_mtx);
 +      while(true) {
 +        while(inner_queue.empty() && !finishing) {
 +          inner_queue_cv.wait_for(l, check_period);
 +        }
 +        if (!inner_queue.empty()) {
 +          auto& front = inner_queue.front();
 +          auto client = front.client;
 +          auto req = std::move(front.request);
 +          auto additional = front.additional;
 +          inner_queue.pop_front();
 +
 +          l.unlock();
 +
 +          // simulation operation by sleeping; then call function to
 +          // notify server of completion
 +          std::this_thread::sleep_for(op_time);
 +
-           client_resp_f(client, resp, id, additional);
 +          // TODO: rather than assuming this constructor exists, perhaps
 +          // pass in a function that does this mapping?
++          client_resp_f(client, TestResponse{req->epoch}, id, additional);
 +
 +          time_stats(internal_stats.mtx,
 +                     internal_stats.request_complete_time,
 +                     [&](){
 +                       priority_queue->request_completed();
 +                     });
 +          count_stats(internal_stats.mtx,
 +                      internal_stats.request_complete_count);
 +
 +          l.lock(); // in prep for next iteration of loop
 +        } else {
 +          break;
 +        }
 +      }
 +      }
 +    }; // class SimulatedServer
 +
 +  }; // namespace qos_simulation
 +}; // namespace crimson
index ee4c1e6e3efff40df6d0f81aba10c50e90d24ff7,0000000000000000000000000000000000000000..610c2ef665c432ad1b2a9a9d6be4c7ebc0337a21
mode 100644,000000..100644
--- /dev/null
@@@ -1,182 -1,0 +1,183 @@@
-       void add_request(const R& request,
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2016 Red Hat Inc.
 + */
 +
 +#pragma once
 +
 +#include <memory>
 +#include <mutex>
 +#include <deque>
 +
 +#include "boost/variant.hpp"
 +
 +#include "ssched_recs.h"
 +
 +#ifdef PROFILE
 +#include "profile.h"
 +#endif
 +
 +namespace crimson {
 +
 +  namespace simple_scheduler {
 +
 +    template<typename C, typename R, typename Time>
 +    class SimpleQueue {
 +
 +    public:
 +
 +      using RequestRef = std::unique_ptr<R>;
 +
 +      // a function to see whether the server can handle another request
 +      using CanHandleRequestFunc = std::function<bool(void)>;
 +
 +      // a function to submit a request to the server; the second
 +      // parameter is a callback when it's completed
 +      using HandleRequestFunc =
 +      std::function<void(const C&,RequestRef,NullData)>;
 +
 +      struct PullReq {
 +      enum class Type { returning, none };
 +
 +      struct Retn {
 +        C           client;
 +        RequestRef  request;
 +      };
 +
 +      Type                 type;
 +      boost::variant<Retn> data;
 +      };
 +
 +    protected:
 +
 +      enum class Mechanism { push, pull };
 +
 +      struct QRequest {
 +      C          client;
 +      RequestRef request;
 +      };
 +
 +      bool finishing = false;
 +      Mechanism mechanism;
 +
 +      CanHandleRequestFunc can_handle_f;
 +      HandleRequestFunc handle_f;
 +
 +      mutable std::mutex queue_mtx;
 +      using DataGuard = std::lock_guard<decltype(queue_mtx)>;
 +
 +      std::deque<QRequest> queue;
 +
 +#ifdef PROFILE
 +    public:
 +      ProfileTimer<std::chrono::nanoseconds> pull_request_timer;
 +      ProfileTimer<std::chrono::nanoseconds> add_request_timer;
 +      ProfileTimer<std::chrono::nanoseconds> request_complete_timer;
 +    protected:
 +#endif
 +
 +    public:
 +
 +      // push full constructor
 +      SimpleQueue(CanHandleRequestFunc _can_handle_f,
 +                HandleRequestFunc _handle_f) :
 +      mechanism(Mechanism::push),
 +      can_handle_f(_can_handle_f),
 +      handle_f(_handle_f)
 +      {
 +      // empty
 +      }
 +
 +      SimpleQueue() :
 +      mechanism(Mechanism::pull)
 +      {
 +      // empty
 +      }
 +
 +      ~SimpleQueue() {
 +      finishing = true;
 +      }
 +
-       add_request(RequestRef(new R(request)), client_id, req_params);
++      void add_request(R&& request,
 +                     const C& client_id,
 +                     const ReqParams& req_params) {
++      add_request(RequestRef(new R(std::move(request))),
++                  client_id, req_params);
 +      }
 +
 +      void add_request(RequestRef&& request,
 +                     const C& client_id,
 +                     const ReqParams& req_params) {
 +      DataGuard g(queue_mtx);
 +
 +#ifdef PROFILE
 +      add_request_timer.start();
 +#endif
 +      queue.emplace_back(QRequest{client_id, std::move(request)});
 +
 +      if (Mechanism::push == mechanism) {
 +        schedule_request();
 +      }
 +
 +#ifdef PROFILE
 +      add_request_timer.stop();
 +#endif
 +      } // add_request
 +
 +      void request_completed() {
 +      assert(Mechanism::push == mechanism);
 +      DataGuard g(queue_mtx);
 +
 +#ifdef PROFILE
 +      request_complete_timer.start();
 +#endif
 +      schedule_request();
 +
 +#ifdef PROFILE
 +      request_complete_timer.stop();
 +#endif
 +      } // request_completed
 +
 +      PullReq pull_request() {
 +      assert(Mechanism::pull == mechanism);
 +      PullReq result;
 +      DataGuard g(queue_mtx);
 +
 +#ifdef PROFILE
 +      pull_request_timer.start();
 +#endif
 +
 +      if (queue.empty()) {
 +        result.type = PullReq::Type::none;
 +      } else {
 +        auto front = queue.front();
 +        result.type = PullReq::Type::returning;
 +        result.data =
 +          typename PullReq::Retn{front.client, std::move(front.request)};
 +        queue.pop();
 +      }
 +
 +#ifdef PROFILE
 +      pull_request_timer.stop();
 +#endif
 +
 +      return result;
 +      }
 +
 +    protected:
 +
 +      // queue_mtx should be held when called; should only be called
 +      // when mechanism is push
 +      void schedule_request() {
 +      if (!queue.empty() && can_handle_f()) {
 +        auto& front = queue.front();
 +        static NullData null_data;
 +        handle_f(front.client, std::move(front.request), null_data);
 +        queue.pop_front();
 +      }
 +      }
 +    };
 +  };
 +};
index 668869b8d72fb040d3b74997db89a071e0f9a72a,0000000000000000000000000000000000000000..57b733e860cede4065a36d2ad2034450c35f8399
mode 100644,000000..100644
--- /dev/null
@@@ -1,324 -1,0 +1,324 @@@
-                       const sim::TestRequest& request,
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2016 Red Hat Inc.
 + */
 +
 +
 +#include "test_dmclock.h"
 +#include "config.h"
 +
 +#ifdef PROFILE
 +#include "profile.h"
 +#endif
 +
 +
 +namespace dmc = crimson::dmclock;
 +namespace test = crimson::test_dmc;
 +namespace sim = crimson::qos_simulation;
 +
 +using namespace std::placeholders;
 +
 +
 +namespace crimson {
 +    namespace test_dmc {
 +        void server_data(std::ostream& out,
 +                         test::MySim* sim,
 +                         test::MySim::ServerFilter server_disp_filter,
 +                         int head_w, int data_w, int data_prec);
 +
 +        void client_data(std::ostream& out,
 +                         test::MySim* sim,
 +                         test::MySim::ClientFilter client_disp_filter,
 +                         int head_w, int data_w, int data_prec);
 +    }
 +}
 +
 +
 +int main(int argc, char* argv[]) {
 +    std::vector<const char*> args;
 +    for (int i = 1; i < argc; ++i) {
 +      args.push_back(argv[i]);
 +    }
 +
 +    std::string conf_file_list;
 +    sim::ceph_argparse_early_args(args, &conf_file_list);
 +
 +    sim::sim_config_t g_conf;
 +    std::vector<sim::cli_group_t> &cli_group = g_conf.cli_group;
 +    std::vector<sim::srv_group_t> &srv_group = g_conf.srv_group;
 +
 +    if (!conf_file_list.empty()) {
 +      int ret;
 +      ret = sim::parse_config_file(conf_file_list, g_conf);
 +      if (ret) {
 +      // error
 +      _exit(1);
 +      }
 +    } else {
 +      // default simulation parameter
 +      g_conf.client_groups = 2;
 +
 +      sim::srv_group_t st;
 +      srv_group.push_back(st);
 +
 +      sim::cli_group_t ct1(99, 0);
 +      cli_group.push_back(ct1);
 +
 +      sim::cli_group_t ct2(1, 10);
 +      cli_group.push_back(ct2);
 +    }
 +
 +    const uint server_groups = g_conf.server_groups;
 +    const uint client_groups = g_conf.client_groups;
 +    const bool server_random_selection = g_conf.server_random_selection;
 +    const bool server_soft_limit = g_conf.server_soft_limit;
 +    uint server_total_count = 0;
 +    uint client_total_count = 0;
 +
 +    for (uint i = 0; i < client_groups; ++i) {
 +      client_total_count += cli_group[i].client_count;
 +    }
 +
 +    for (uint i = 0; i < server_groups; ++i) {
 +      server_total_count += srv_group[i].server_count;
 +    }
 +
 +    std::vector<test::dmc::ClientInfo> client_info;
 +    for (uint i = 0; i < client_groups; ++i) {
 +      client_info.push_back(test::dmc::ClientInfo 
 +                        { cli_group[i].client_reservation,
 +                          cli_group[i].client_weight,
 +                          cli_group[i].client_limit } );
 +    }
 +
 +    auto ret_client_group_f = [&](const ClientId& c) -> uint {
 +      uint group_max = 0;
 +      uint i = 0;
 +      for (; i < client_groups; ++i) {
 +      group_max += cli_group[i].client_count;
 +      if (c < group_max) {
 +        break;
 +      }
 +      }
 +      return i;
 +    };
 +
 +    auto ret_server_group_f = [&](const ServerId& s) -> uint {
 +      uint group_max = 0;
 +      uint i = 0;
 +      for (; i < server_groups; ++i) {
 +      group_max += srv_group[i].server_count;
 +      if (s < group_max) {
 +        break;
 +      }
 +      }
 +      return i;
 +    };
 +
 +    auto client_info_f = [=](const ClientId& c) -> test::dmc::ClientInfo {
 +      return client_info[ret_client_group_f(c)];
 +    };
 +
 +    auto client_disp_filter = [=] (const ClientId& i) -> bool {
 +        return i < 3 || i >= (client_total_count - 3);
 +    };
 +
 +    auto server_disp_filter = [=] (const ServerId& i) -> bool {
 +        return i < 3 || i >= (server_total_count - 3);
 +    };
 +
 +
 +    test::MySim *simulation;
 +  
 +
 +    // lambda to post a request to the identified server; called by client
 +    test::SubmitFunc server_post_f =
 +        [&simulation](const ServerId& server,
-         s.post(request, client_id, req_params);
++                      sim::TestRequest&& request,
 +                      const ClientId& client_id,
 +                      const test::dmc::ReqParams& req_params) {
 +        test::DmcServer& s = simulation->get_server(server);
++        s.post(std::move(request), client_id, req_params);
 +    };
 +
 +    std::vector<std::vector<sim::CliInst>> cli_inst;
 +    for (uint i = 0; i < client_groups; ++i) {
 +      if (cli_group[i].client_wait == std::chrono::seconds(0)) {
 +      cli_inst.push_back(
 +          { { sim::req_op, 
 +              (uint32_t)cli_group[i].client_total_ops,
 +              (double)cli_group[i].client_iops_goal, 
 +              (uint16_t)cli_group[i].client_outstanding_ops } } );
 +      } else {
 +      cli_inst.push_back(
 +          { { sim::wait_op, cli_group[i].client_wait },
 +            { sim::req_op, 
 +              (uint32_t)cli_group[i].client_total_ops,
 +              (double)cli_group[i].client_iops_goal, 
 +              (uint16_t)cli_group[i].client_outstanding_ops } } );
 +      }
 +    }
 +
 +    simulation = new test::MySim();
 +
 +    test::DmcServer::ClientRespFunc client_response_f =
 +        [&simulation](ClientId client_id,
 +                      const sim::TestResponse& resp,
 +                      const ServerId& server_id,
 +                      const dmc::PhaseType& phase) {
 +        simulation->get_client(client_id).receive_response(resp,
 +                                                           server_id,
 +                                                           phase);
 +    };
 +
 +    test::CreateQueueF create_queue_f =
 +        [&](test::DmcQueue::CanHandleRequestFunc can_f,
 +            test::DmcQueue::HandleRequestFunc handle_f) -> test::DmcQueue* {
 +        return new test::DmcQueue(client_info_f, can_f, handle_f, server_soft_limit);
 +    };
 +
 + 
 +    auto create_server_f = [&](ServerId id) -> test::DmcServer* {
 +      uint i = ret_server_group_f(id);
 +      return new test::DmcServer(id,
 +                                 srv_group[i].server_iops,
 +                               srv_group[i].server_threads,
 +                               client_response_f,
 +                               test::dmc_server_accumulate_f,
 +                               create_queue_f);
 +    };
 +
 +    auto create_client_f = [&](ClientId id) -> test::DmcClient* {
 +      uint i = ret_client_group_f(id);
 +      test::MySim::ClientBasedServerSelectFunc server_select_f;
 +      uint client_server_select_range = cli_group[i].client_server_select_range;
 +      if (!server_random_selection) {
 +      server_select_f = simulation->make_server_select_alt_range(client_server_select_range);
 +      } else {
 +      server_select_f = simulation->make_server_select_ran_range(client_server_select_range);
 +      }
 +      return new test::DmcClient(id,
 +                               server_post_f,
 +                               std::bind(server_select_f, _1, id),
 +                               test::dmc_client_accumulate_f,
 +                               cli_inst[i]);
 +    };
 +
 +#if 1
 +    std::cout << "[global]" << std::endl << g_conf << std::endl;
 +    for (uint i = 0; i < client_groups; ++i) {
 +      std::cout << std::endl << "[client." << i << "]" << std::endl;
 +      std::cout << cli_group[i] << std::endl;
 +    }
 +    for (uint i = 0; i < server_groups; ++i) {
 +      std::cout << std::endl << "[server." << i << "]" << std::endl;
 +      std::cout << srv_group[i] << std::endl;
 +    }
 +    std::cout << std::endl;
 +#endif
 +
 +    simulation->add_servers(server_total_count, create_server_f);
 +    simulation->add_clients(client_total_count, create_client_f);
 +
 +    simulation->run();
 +    simulation->display_stats(std::cout,
 +                              &test::server_data, &test::client_data,
 +                              server_disp_filter, client_disp_filter);
 +
 +    delete simulation;
 +} // main
 +
 +
 +void test::client_data(std::ostream& out,
 +               test::MySim* sim,
 +               test::MySim::ClientFilter client_disp_filter,
 +               int head_w, int data_w, int data_prec) {
 +    // report how many ops were done by reservation and proportion for
 +    // each client
 +
 +    int total_r = 0;
 +    out << std::setw(head_w) << "res_ops:";
 +    for (uint i = 0; i < sim->get_client_count(); ++i) {
 +        const auto& client = sim->get_client(i);
 +        auto r = client.get_accumulator().reservation_count;
 +        total_r += r;
 +        if (!client_disp_filter(i)) continue;
 +        out << " " << std::setw(data_w) << r;
 +    }
 +    out << " " << std::setw(data_w) << std::setprecision(data_prec) <<
 +        std::fixed << total_r << std::endl;
 +
 +    int total_p = 0;
 +    out << std::setw(head_w) << "prop_ops:";
 +    for (uint i = 0; i < sim->get_client_count(); ++i) {
 +        const auto& client = sim->get_client(i);
 +        auto p = client.get_accumulator().proportion_count;
 +        total_p += p;
 +        if (!client_disp_filter(i)) continue;
 +        out << " " << std::setw(data_w) << p;
 +    }
 +    out << " " << std::setw(data_w) << std::setprecision(data_prec) <<
 +        std::fixed << total_p << std::endl;
 +}
 +
 +
 +void test::server_data(std::ostream& out,
 +               test::MySim* sim,
 +               test::MySim::ServerFilter server_disp_filter,
 +               int head_w, int data_w, int data_prec) {
 +    out << std::setw(head_w) << "res_ops:";
 +    int total_r = 0;
 +    for (uint i = 0; i < sim->get_server_count(); ++i) {
 +        const auto& server = sim->get_server(i);
 +        auto rc = server.get_accumulator().reservation_count;
 +        total_r += rc;
 +        if (!server_disp_filter(i)) continue;
 +        out << " " << std::setw(data_w) << rc;
 +    }
 +    out << " " << std::setw(data_w) << std::setprecision(data_prec) <<
 +        std::fixed << total_r << std::endl;
 +
 +    out << std::setw(head_w) << "prop_ops:";
 +    int total_p = 0;
 +    for (uint i = 0; i < sim->get_server_count(); ++i) {
 +        const auto& server = sim->get_server(i);
 +        auto pc = server.get_accumulator().proportion_count;
 +        total_p += pc;
 +        if (!server_disp_filter(i)) continue;
 +        out << " " << std::setw(data_w) << pc;
 +    }
 +    out << " " << std::setw(data_w) << std::setprecision(data_prec) <<
 +        std::fixed << total_p << std::endl;
 +
 +    const auto& q = sim->get_server(0).get_priority_queue();
 +    out << std::endl <<
 +      " k-way heap: " << q.get_heap_branching_factor() << std::endl
 +      << std::endl;
 +
 +#ifdef PROFILE
 +    crimson::ProfileCombiner<std::chrono::nanoseconds> art_combiner;
 +    crimson::ProfileCombiner<std::chrono::nanoseconds> rct_combiner;
 +    for (uint i = 0; i < sim->get_server_count(); ++i) {
 +      const auto& q = sim->get_server(i).get_priority_queue();
 +      const auto& art = q.add_request_timer;
 +      art_combiner.combine(art);
 +      const auto& rct = q.request_complete_timer;
 +      rct_combiner.combine(rct);
 +    }
 +    out << "Server add_request_timer: count:" << art_combiner.get_count() <<
 +      ", mean:" << art_combiner.get_mean() <<
 +      ", std_dev:" << art_combiner.get_std_dev() <<
 +      ", low:" << art_combiner.get_low() <<
 +      ", high:" << art_combiner.get_high() << std::endl;
 +    out << "Server request_complete_timer: count:" << rct_combiner.get_count() <<
 +      ", mean:" << rct_combiner.get_mean() <<
 +      ", std_dev:" << rct_combiner.get_std_dev() <<
 +      ", low:" << rct_combiner.get_low() <<
 +      ", high:" << rct_combiner.get_high() << std::endl;
 +    out << "Server combined mean: " <<
 +      (art_combiner.get_mean() + rct_combiner.get_mean()) <<
 +      std::endl;
 +#endif
 +}
index 6df20dc5f896eac656e594c8f8b5b69a38e29424,0000000000000000000000000000000000000000..14ff7e9b47040eef906489d7352746506759a4a3
mode 100644,000000..100644
--- /dev/null
@@@ -1,187 -1,0 +1,187 @@@
-                 const sim::TestRequest& request,
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2016 Red Hat Inc.
 + */
 +
 +
 +#include "test_ssched.h"
 +
 +
 +#ifdef PROFILE
 +#include "profile.h"
 +#endif
 +
 +
 +namespace test = crimson::test_simple_scheduler;
 +namespace ssched = crimson::simple_scheduler;
 +namespace sim = crimson::qos_simulation;
 +
 +using namespace std::placeholders;
 +
 +
 +namespace crimson {
 +  namespace test_simple_scheduler {
 +    void client_data(std::ostream& out,
 +                   test::MySim* sim,
 +                   test::MySim::ClientFilter client_disp_filter,
 +                   int head_w, int data_w, int data_prec);
 +
 +    void server_data(std::ostream& out,
 +                   test::MySim* sim,
 +                   test::MySim::ServerFilter server_disp_filter,
 +                   int head_w, int data_w, int data_prec);
 +  } // namespace test_simple
 +} // namespace crimson
 +    
 +
 +int main(int argc, char* argv[]) {
 +  // server params
 +
 +  const uint server_count = 100;
 +  const uint server_iops = 40;
 +  const uint server_threads = 1;
 +
 +  // client params
 +
 +  const uint client_total_ops = 1000;
 +  const uint client_count = 100;
 +  const uint client_server_select_range = 10;
 +  const uint client_wait_count = 1;
 +  const uint client_iops_goal = 50;
 +  const uint client_outstanding_ops = 100;
 +  const std::chrono::seconds client_wait(10);
 +
 +  auto client_disp_filter = [=] (const ClientId& i) -> bool {
 +    return i < 3 || i >= (client_count - 3);
 +  };
 +
 +  auto server_disp_filter = [=] (const ServerId& i) -> bool {
 +    return i < 3 || i >= (server_count - 3);
 +  };
 +
 +
 +  test::MySim *simulation;
 +
 +  // lambda to post a request to the identified server; called by client
 +  test::SubmitFunc server_post_f =
 +    [&simulation](const ServerId& server_id,
-     server.post(request, client_id, req_params);
++                sim::TestRequest&& request,
 +                const ClientId& client_id,
 +                const ssched::ReqParams& req_params) {
 +    auto& server = simulation->get_server(server_id);
++    server.post(std::move(request), client_id, req_params);
 +  };
 +
 +  static std::vector<sim::CliInst> no_wait =
 +    { { sim::req_op, client_total_ops, client_iops_goal, client_outstanding_ops } };
 +  static std::vector<sim::CliInst> wait =
 +    { { sim::wait_op, client_wait },
 +      { sim::req_op, client_total_ops, client_iops_goal, client_outstanding_ops } };
 +
 +  simulation = new test::MySim();
 +
 +#if 1
 +  test::MySim::ClientBasedServerSelectFunc server_select_f =
 +    simulation->make_server_select_alt_range(client_server_select_range);
 +#elif 0
 +  test::MySim::ClientBasedServerSelectFunc server_select_f =
 +    std::bind(&test::MySim::server_select_random, simulation, _1, _2);
 +#else
 +  test::MySim::ClientBasedServerSelectFunc server_select_f =
 +    std::bind(&test::MySim::server_select_0, simulation, _1, _2);
 +#endif
 +
 +  test::SimpleServer::ClientRespFunc client_response_f =
 +    [&simulation](ClientId client_id,
 +                const sim::TestResponse& resp,
 +                const ServerId& server_id,
 +                const ssched::NullData& resp_params) {
 +    simulation->get_client(client_id).receive_response(resp,
 +                                                     server_id,
 +                                                     resp_params);
 +  };
 +
 +  test::CreateQueueF create_queue_f =
 +    [&](test::SimpleQueue::CanHandleRequestFunc can_f,
 +      test::SimpleQueue::HandleRequestFunc handle_f) -> test::SimpleQueue* {
 +    return new test::SimpleQueue(can_f, handle_f);
 +  };
 +
 +  auto create_server_f = [&](ServerId id) -> test::SimpleServer* {
 +    return new test::SimpleServer(id,
 +                                server_iops, server_threads,
 +                                client_response_f,
 +                                test::simple_server_accumulate_f,
 +                                create_queue_f);
 +  };
 +
 +  auto create_client_f = [&](ClientId id) -> test::SimpleClient* {
 +    return new test::SimpleClient(id,
 +                                server_post_f,
 +                                std::bind(server_select_f, _1, id),
 +                                test::simple_client_accumulate_f,
 +                                id < (client_count - client_wait_count)
 +                                ? no_wait : wait);
 +  };
 +
 +  simulation->add_servers(server_count, create_server_f);
 +  simulation->add_clients(client_count, create_client_f);
 +
 +  simulation->run();
 +  simulation->display_stats(std::cout,
 +                          &test::server_data, &test::client_data,
 +                          server_disp_filter, client_disp_filter);
 +} // main
 +
 +
 +void test::client_data(std::ostream& out,
 +                     test::MySim* sim,
 +                     test::MySim::ClientFilter client_disp_filter,
 +                     int head_w, int data_w, int data_prec) {
 +  // empty
 +}
 +
 +
 +void test::server_data(std::ostream& out,
 +                     test::MySim* sim,
 +                     test::MySim::ServerFilter server_disp_filter,
 +                     int head_w, int data_w, int data_prec) {
 +  out << std::setw(head_w) << "requests:";
 +  int total_req = 0;
 +  for (uint i = 0; i < sim->get_server_count(); ++i) {
 +    const auto& server = sim->get_server(i);
 +    auto req_count = server.get_accumulator().request_count;
 +    total_req += req_count;
 +    if (!server_disp_filter(i)) continue;
 +    out << std::setw(data_w) << req_count;
 +  }
 +  out << std::setw(data_w) << std::setprecision(data_prec) <<
 +    std::fixed << total_req << std::endl;
 +
 +#ifdef PROFILE
 +    crimson::ProfileCombiner<std::chrono::nanoseconds> art_combiner;
 +    crimson::ProfileCombiner<std::chrono::nanoseconds> rct_combiner;
 +    for (uint i = 0; i < sim->get_server_count(); ++i) {
 +      const auto& q = sim->get_server(i).get_priority_queue();
 +      const auto& art = q.add_request_timer;
 +      art_combiner.combine(art);
 +      const auto& rct = q.request_complete_timer;
 +      rct_combiner.combine(rct);
 +    }
 +    out << "Server add_request_timer: count:" << art_combiner.get_count() <<
 +      ", mean:" << art_combiner.get_mean() <<
 +      ", std_dev:" << art_combiner.get_std_dev() <<
 +      ", low:" << art_combiner.get_low() <<
 +      ", high:" << art_combiner.get_high() << std::endl;
 +    out << "Server request_complete_timer: count:" << rct_combiner.get_count() <<
 +      ", mean:" << rct_combiner.get_mean() <<
 +      ", std_dev:" << rct_combiner.get_std_dev() <<
 +      ", low:" << rct_combiner.get_low() <<
 +      ", high:" << rct_combiner.get_high() << std::endl;
 +    out << "Server combined mean: " <<
 +      (art_combiner.get_mean() + rct_combiner.get_mean()) <<
 +      std::endl;
 +#endif
 +}
index 7178f266e2a41d13276ad1fbe3a92bc49135cd37,0000000000000000000000000000000000000000..65d09f6a46915d05405a4f540dbc1261f059fea1
mode 100644,000000..100644
--- /dev/null
@@@ -1,24 -1,0 +1,10 @@@
- set(local_flags "-std=c++11 -Wno-write-strings -Wall -pthread")
- if(DO_NOT_DELAY_TAG_CALC)
-   set(local_flags "${local_flags} -DDO_NOT_DELAY_TAG_CALC")
- endif()
 +include_directories(${GTEST_INCLUDE_DIRS})
 +include_directories(${Boost_INCLUDE_DIRS})
 +include_directories(../support/src)
 +
- set_source_files_properties(${dmc_srcs}
-   PROPERTIES
-   COMPILE_FLAGS "${local_flags}"
-   )
- if ("${CMAKE_CXX_COMPILER_ID}" MATCHES "Clang")
-   set(warnings_off " -Wno-unused-variable -Wno-unused-function")
- elseif ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU")
-   set(warnings_off " -Wno-unused-but-set-variable -Wno-unused-function")
- endif()
++set(CMAKE_CXX_FLAGS
++  "${CMAKE_CXX_FLAGS} -std=c++11 -Wno-write-strings -Wall -pthread")
 +
 +set(dmc_srcs dmclock_util.cc ../support/src/run_every.cc)
 +
 +add_library(dmclock STATIC ${dmc_srcs})
index b44e1211b53f667daf23a8ed651480f4574a931d,0000000000000000000000000000000000000000..92f4cf83bb1abbdad5c792b5377258f9227dd653
mode 100644,000000..100644
--- /dev/null
@@@ -1,194 -1,0 +1,193 @@@
- #include "gtest/gtest_prod.h"
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2017 Red Hat Inc.
 + */
 +
 +
 +#pragma once
 +
 +#include <map>
 +#include <deque>
 +#include <chrono>
 +#include <thread>
 +#include <mutex>
 +#include <condition_variable>
 +
 +#include "run_every.h"
 +#include "dmclock_util.h"
 +#include "dmclock_recs.h"
 +
-       FRIEND_TEST(dmclock_client, server_erase);
 +
 +namespace crimson {
 +  namespace dmclock {
 +    struct ServerInfo {
 +      Counter   delta_prev_req;
 +      Counter   rho_prev_req;
 +      uint32_t  my_delta;
 +      uint32_t  my_rho;
 +
 +      ServerInfo(Counter _delta_prev_req,
 +               Counter _rho_prev_req) :
 +      delta_prev_req(_delta_prev_req),
 +      rho_prev_req(_rho_prev_req),
 +      my_delta(0),
 +      my_rho(0)
 +      {
 +      // empty
 +      }
 +
 +      inline void req_update(Counter delta, Counter rho) {
 +      delta_prev_req = delta;
 +      rho_prev_req = rho;
 +      my_delta = 0;
 +      my_rho = 0;
 +      }
 +
 +      inline void resp_update(PhaseType phase) {
 +      ++my_delta;
 +      if (phase == PhaseType::reservation) ++my_rho;
 +      }
 +    };
 +
 +
 +    // S is server identifier type
 +    template<typename S>
 +    class ServiceTracker {
++      // we don't want to include gtest.h just for FRIEND_TEST
++      friend class dmclock_client_server_erase_Test;
 +
 +      using TimePoint = decltype(std::chrono::steady_clock::now());
 +      using Duration = std::chrono::milliseconds;
 +      using MarkPoint = std::pair<TimePoint,Counter>;
 +
 +      Counter                 delta_counter; // # reqs completed
 +      Counter                 rho_counter;   // # reqs completed via reservation
 +      std::map<S,ServerInfo>  server_map;
 +      mutable std::mutex      data_mtx;      // protects Counters and map
 +
 +      using DataGuard = std::lock_guard<decltype(data_mtx)>;
 +
 +      // clean config
 +
 +      std::deque<MarkPoint>     clean_mark_points;
 +      Duration                  clean_age;     // age at which ServerInfo cleaned
 +
 +      // NB: All threads declared at end, so they're destructed firs!
 +
 +      std::unique_ptr<RunEvery> cleaning_job;
 +
 +
 +    public:
 +
 +      // we have to start the counters at 1, as 0 is used in the
 +      // cleaning process
 +      template<typename Rep, typename Per>
 +      ServiceTracker(std::chrono::duration<Rep,Per> _clean_every,
 +                   std::chrono::duration<Rep,Per> _clean_age) :
 +      delta_counter(1),
 +      rho_counter(1),
 +      clean_age(std::chrono::duration_cast<Duration>(_clean_age))
 +      {
 +      cleaning_job =
 +        std::unique_ptr<RunEvery>(
 +          new RunEvery(_clean_every,
 +                       std::bind(&ServiceTracker::do_clean, this)));
 +      }
 +
 +
 +      // the reason we're overloading the constructor rather than
 +      // using default values for the arguments is so that callers
 +      // have to either use all defaults or specify all timings; with
 +      // default arguments they could specify some without others
 +      ServiceTracker() :
 +      ServiceTracker(std::chrono::minutes(5), std::chrono::minutes(10))
 +      {
 +      // empty
 +      }
 +
 +
 +      /*
 +       * Incorporates the RespParams received into the various counter.
 +       */
 +      void track_resp(const S& server_id, const PhaseType& phase) {
 +      DataGuard g(data_mtx);
 +
 +      auto it = server_map.find(server_id);
 +      if (server_map.end() == it) {
 +        // this code can only run if a request did not precede the
 +        // response or if the record was cleaned up b/w when
 +        // the request was made and now
 +        ServerInfo si(delta_counter, rho_counter);
 +        si.resp_update(phase);
 +        server_map.emplace(server_id, si);
 +      } else {
 +        it->second.resp_update(phase);
 +      }
 +
 +      ++delta_counter;
 +      if (PhaseType::reservation == phase) {
 +        ++rho_counter;
 +      }
 +      }
 +
 +
 +      /*
 +       * Returns the ReqParams for the given server.
 +       */
 +      ReqParams get_req_params(const S& server) {
 +      DataGuard g(data_mtx);
 +      auto it = server_map.find(server);
 +      if (server_map.end() == it) {
 +        server_map.emplace(server, ServerInfo(delta_counter, rho_counter));
 +        return ReqParams(1, 1);
 +      } else {
 +        Counter delta =
 +          1 + delta_counter - it->second.delta_prev_req - it->second.my_delta;
 +        Counter rho =
 +          1 + rho_counter - it->second.rho_prev_req - it->second.my_rho;
 +
 +        it->second.req_update(delta_counter, rho_counter);
 +
 +        return ReqParams(uint32_t(delta), uint32_t(rho));
 +      }
 +      }
 +
 +    private:
 +
 +      /*
 +       * This is being called regularly by RunEvery. Every time it's
 +       * called it notes the time and delta counter (mark point) in a
 +       * deque. It also looks at the deque to find the most recent
 +       * mark point that is older than clean_age. It then walks the
 +       * map and delete all server entries that were last used before
 +       * that mark point.
 +       */
 +      void do_clean() {
 +      TimePoint now = std::chrono::steady_clock::now();
 +      DataGuard g(data_mtx);
 +      clean_mark_points.emplace_back(MarkPoint(now, delta_counter));
 +
 +      Counter earliest = 0;
 +      auto point = clean_mark_points.front();
 +      while (point.first <= now - clean_age) {
 +        earliest = point.second;
 +        clean_mark_points.pop_front();
 +        point = clean_mark_points.front();
 +      }
 +
 +      if (earliest > 0) {
 +        for (auto i = server_map.begin();
 +             i != server_map.end();
 +             /* empty */) {
 +          auto i2 = i++;
 +          if (i2->second.delta_prev_req <= earliest) {
 +            server_map.erase(i2);
 +          }
 +        }
 +      }
 +      } // do_clean
 +    }; // class ServiceTracker
 +  }
 +}
index 8aaad3977264847c0c3351d76eff05da7d629171,0000000000000000000000000000000000000000..2c9940dc6c17f8af7ff6c9b990951a7672ddb547
mode 100644,000000..100644
--- /dev/null
@@@ -1,1602 -1,0 +1,1601 @@@
- #include "gtest/gtest_prod.h"
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2017 Red Hat Inc.
 + */
 +
 +
 +#pragma once
 +
 +/* COMPILATION OPTIONS
 + *
 + * By default we include an optimization over the originally published
 + * dmclock algorithm using not the values of rho and delta that were
 + * sent in with a request but instead the most recent rho and delta
 + * values from the requests's client. To restore the algorithm's
 + * original behavior, define DO_NOT_DELAY_TAG_CALC (i.e., compiler
 + * argument -DDO_NOT_DELAY_TAG_CALC).
 + *
 + * The prop_heap does not seem to be necessary. The only thing it
 + * would help with is quickly finding the mininum proportion/prioity
 + * when an idle client became active. To have the code maintain the
 + * proportional heap, define USE_PROP_HEAP (i.e., compiler argument
 + * -DUSE_PROP_HEAP).
 + */
 +
 +#include <assert.h>
 +
 +#include <cmath>
 +#include <memory>
 +#include <map>
 +#include <deque>
 +#include <queue>
 +#include <atomic>
 +#include <mutex>
 +#include <condition_variable>
 +#include <thread>
 +#include <iostream>
 +#include <sstream>
 +#include <limits>
 +
 +#include <boost/variant.hpp>
 +
 +#include "indirect_intrusive_heap.h"
 +#include "run_every.h"
 +#include "dmclock_util.h"
 +#include "dmclock_recs.h"
 +
 +#ifdef PROFILE
 +#include "profile.h"
 +#endif
 +
-       FRIEND_TEST(dmclock_server, client_idle_erase);
 +
 +namespace crimson {
 +
 +  namespace dmclock {
 +
 +    namespace c = crimson;
 +
 +    constexpr double max_tag = std::numeric_limits<double>::is_iec559 ?
 +      std::numeric_limits<double>::infinity() :
 +      std::numeric_limits<double>::max();
 +    constexpr double min_tag = std::numeric_limits<double>::is_iec559 ?
 +      -std::numeric_limits<double>::infinity() :
 +      std::numeric_limits<double>::lowest();
 +    constexpr uint tag_modulo = 1000000;
 +
 +    struct ClientInfo {
 +      const double reservation;  // minimum
 +      const double weight;       // proportional
 +      const double limit;        // maximum
 +
 +      // multiplicative inverses of above, which we use in calculations
 +      // and don't want to recalculate repeatedly
 +      const double reservation_inv;
 +      const double weight_inv;
 +      const double limit_inv;
 +
 +      // order parameters -- min, "normal", max
 +      ClientInfo(double _reservation, double _weight, double _limit) :
 +      reservation(_reservation),
 +      weight(_weight),
 +      limit(_limit),
 +      reservation_inv(0.0 == reservation ? 0.0 : 1.0 / reservation),
 +      weight_inv(     0.0 == weight      ? 0.0 : 1.0 / weight),
 +      limit_inv(      0.0 == limit       ? 0.0 : 1.0 / limit)
 +      {
 +      // empty
 +      }
 +
 +
 +      friend std::ostream& operator<<(std::ostream& out,
 +                                    const ClientInfo& client) {
 +      out <<
 +        "{ ClientInfo:: r:" << client.reservation <<
 +        " w:" << std::fixed << client.weight <<
 +        " l:" << std::fixed << client.limit <<
 +        " 1/r:" << std::fixed << client.reservation_inv <<
 +        " 1/w:" << std::fixed << client.weight_inv <<
 +        " 1/l:" << std::fixed << client.limit_inv <<
 +        " }";
 +      return out;
 +      }
 +    }; // class ClientInfo
 +
 +
 +    struct RequestTag {
 +      double reservation;
 +      double proportion;
 +      double limit;
 +      bool   ready; // true when within limit
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +      Time   arrival;
 +#endif
 +
 +      RequestTag(const RequestTag& prev_tag,
 +               const ClientInfo& client,
 +               const uint32_t delta,
 +               const uint32_t rho,
 +               const Time time,
 +               const double cost = 0.0) :
 +      reservation(cost + tag_calc(time,
 +                                  prev_tag.reservation,
 +                                  client.reservation_inv,
 +                                  rho,
 +                                  true)),
 +      proportion(tag_calc(time,
 +                          prev_tag.proportion,
 +                          client.weight_inv,
 +                          delta,
 +                          true)),
 +      limit(tag_calc(time,
 +                     prev_tag.limit,
 +                     client.limit_inv,
 +                     delta,
 +                     false)),
 +      ready(false)
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +      , arrival(time)
 +#endif
 +      {
 +      assert(reservation < max_tag || proportion < max_tag);
 +      }
 +
 +      RequestTag(const RequestTag& prev_tag,
 +               const ClientInfo& client,
 +               const ReqParams req_params,
 +               const Time time,
 +               const double cost = 0.0) :
 +      RequestTag(prev_tag, client, req_params.delta, req_params.rho, time, cost)
 +      { /* empty */ }
 +
 +      RequestTag(double _res, double _prop, double _lim, const Time _arrival) :
 +      reservation(_res),
 +      proportion(_prop),
 +      limit(_lim),
 +      ready(false)
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +      , arrival(_arrival)
 +#endif
 +      {
 +      assert(reservation < max_tag || proportion < max_tag);
 +      }
 +
 +      RequestTag(const RequestTag& other) :
 +      reservation(other.reservation),
 +      proportion(other.proportion),
 +      limit(other.limit),
 +      ready(other.ready)
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +      , arrival(other.arrival)
 +#endif
 +      {
 +      // empty
 +      }
 +
 +      static std::string format_tag_change(double before, double after) {
 +      if (before == after) {
 +        return std::string("same");
 +      } else {
 +        std::stringstream ss;
 +        ss << format_tag(before) << "=>" << format_tag(after);
 +        return ss.str();
 +      }
 +      }
 +
 +      static std::string format_tag(double value) {
 +      if (max_tag == value) {
 +        return std::string("max");
 +      } else if (min_tag == value) {
 +        return std::string("min");
 +      } else {
 +        return format_time(value, tag_modulo);
 +      }
 +      }
 +
 +    private:
 +
 +      static double tag_calc(const Time time,
 +                           double prev,
 +                           double increment,
 +                           uint32_t dist_req_val,
 +                           bool extreme_is_high) {
 +      if (0.0 == increment) {
 +        return extreme_is_high ? max_tag : min_tag;
 +      } else {
 +        if (0 != dist_req_val) {
 +          increment *= dist_req_val;
 +        }
 +        return std::max(time, prev + increment);
 +      }
 +      }
 +
 +      friend std::ostream& operator<<(std::ostream& out,
 +                                    const RequestTag& tag) {
 +      out <<
 +        "{ RequestTag:: ready:" << (tag.ready ? "true" : "false") <<
 +        " r:" << format_tag(tag.reservation) <<
 +        " p:" << format_tag(tag.proportion) <<
 +        " l:" << format_tag(tag.limit) <<
 +#if 0 // try to resolve this to make sure Time is operator<<'able.
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +        " arrival:" << tag.arrival <<
 +#endif
 +#endif
 +        " }";
 +      return out;
 +      }
 +    }; // class RequestTag
 +
 +
 +    // C is client identifier type, R is request type, B is heap
 +    // branching factor
 +    template<typename C, typename R, uint B>
 +    class PriorityQueueBase {
-       bool remove_by_req_filter_fw(std::function<bool(const R&)> filter_accum) {
++      // we don't want to include gtest.h just for FRIEND_TEST
++      friend class dmclock_server_client_idle_erase_Test;
 +
 +    public:
 +
 +      using RequestRef = std::unique_ptr<R>;
 +
 +    protected:
 +
 +      using TimePoint = decltype(std::chrono::steady_clock::now());
 +      using Duration = std::chrono::milliseconds;
 +      using MarkPoint = std::pair<TimePoint,Counter>;
 +
 +      enum class ReadyOption {ignore, lowers, raises};
 +
 +      // forward decl for friend decls
 +      template<double RequestTag::*, ReadyOption, bool>
 +      struct ClientCompare;
 +
 +      class ClientReq {
 +      friend PriorityQueueBase;
 +
 +      RequestTag tag;
 +      C          client_id;
 +      RequestRef request;
 +
 +      public:
 +
 +      ClientReq(const RequestTag& _tag,
 +                const C&          _client_id,
 +                RequestRef&&      _request) :
 +        tag(_tag),
 +        client_id(_client_id),
 +        request(std::move(_request))
 +      {
 +        // empty
 +      }
 +
 +      friend std::ostream& operator<<(std::ostream& out, const ClientReq& c) {
 +        out << "{ ClientReq:: tag:" << c.tag << " client:" <<
 +          c.client_id << " }";
 +        return out;
 +      }
 +      }; // class ClientReq
 +
 +    public:
 +
 +      // NOTE: ClientRec is in the "public" section for compatibility
 +      // with g++ 4.8.4, which complains if it's not. By g++ 6.3.1
 +      // ClientRec could be "protected" with no issue. [See comments
 +      // associated with function submit_top_request.]
 +      class ClientRec {
 +      friend PriorityQueueBase<C,R,B>;
 +
 +      C                     client;
 +      RequestTag            prev_tag;
 +      std::deque<ClientReq> requests;
 +
 +      // amount added from the proportion tag as a result of
 +      // an idle client becoming unidle
 +      double                prop_delta = 0.0;
 +
 +      c::IndIntruHeapData   reserv_heap_data;
 +      c::IndIntruHeapData   lim_heap_data;
 +      c::IndIntruHeapData   ready_heap_data;
 +#if USE_PROP_HEAP
 +      c::IndIntruHeapData   prop_heap_data;
 +#endif
 +
 +      public:
 +
 +      ClientInfo            info;
 +      bool                  idle;
 +      Counter               last_tick;
 +      uint32_t              cur_rho;
 +      uint32_t              cur_delta;
 +
 +      ClientRec(C _client,
 +                const ClientInfo& _info,
 +                Counter current_tick) :
 +        client(_client),
 +        prev_tag(0.0, 0.0, 0.0, TimeZero),
 +        info(_info),
 +        idle(true),
 +        last_tick(current_tick),
 +        cur_rho(1),
 +        cur_delta(1)
 +      {
 +        // empty
 +      }
 +
 +      inline const RequestTag& get_req_tag() const {
 +        return prev_tag;
 +      }
 +
 +      static inline void assign_unpinned_tag(double& lhs, const double rhs) {
 +        if (rhs != max_tag && rhs != min_tag) {
 +          lhs = rhs;
 +        }
 +      }
 +
 +      inline void update_req_tag(const RequestTag& _prev,
 +                                 const Counter& _tick) {
 +        assign_unpinned_tag(prev_tag.reservation, _prev.reservation);
 +        assign_unpinned_tag(prev_tag.limit, _prev.limit);
 +        assign_unpinned_tag(prev_tag.proportion, _prev.proportion);
 +        last_tick = _tick;
 +      }
 +
 +      inline void add_request(const RequestTag& tag,
 +                              const C&          client_id,
 +                              RequestRef&&      request) {
 +        requests.emplace_back(ClientReq(tag, client_id, std::move(request)));
 +      }
 +
 +      inline const ClientReq& next_request() const {
 +        return requests.front();
 +      }
 +
 +      inline ClientReq& next_request() {
 +        return requests.front();
 +      }
 +
 +      inline void pop_request() {
 +        requests.pop_front();
 +      }
 +
 +      inline bool has_request() const {
 +        return !requests.empty();
 +      }
 +
 +      inline size_t request_count() const {
 +        return requests.size();
 +      }
 +
 +      // NB: because a deque is the underlying structure, this
 +      // operation might be expensive
-           if (filter_accum(*i->request)) {
++      bool remove_by_req_filter_fw(std::function<bool(R&&)> filter_accum) {
 +        bool any_removed = false;
 +        for (auto i = requests.begin();
 +             i != requests.end();
 +             /* no inc */) {
-       bool remove_by_req_filter_bw(std::function<bool(const R&)> filter_accum) {
++          if (filter_accum(std::move(*i->request))) {
 +            any_removed = true;
 +            i = requests.erase(i);
 +          } else {
 +            ++i;
 +          }
 +        }
 +        return any_removed;
 +      }
 +
 +      // NB: because a deque is the underlying structure, this
 +      // operation might be expensive
-           if (filter_accum(*i->request)) {
++      bool remove_by_req_filter_bw(std::function<bool(R&&)> filter_accum) {
 +        bool any_removed = false;
 +        for (auto i = requests.rbegin();
 +             i != requests.rend();
 +             /* no inc */) {
-       remove_by_req_filter(std::function<bool(const R&)> filter_accum,
++          if (filter_accum(std::move(*i->request))) {
 +            any_removed = true;
 +            i = decltype(i){ requests.erase(std::next(i).base()) };
 +          } else {
 +            ++i;
 +          }
 +        }
 +        return any_removed;
 +      }
 +
 +      inline bool
-       bool remove_by_req_filter(std::function<bool(const R&)> filter_accum,
++      remove_by_req_filter(std::function<bool(R&&)> filter_accum,
 +                           bool visit_backwards) {
 +        if (visit_backwards) {
 +          return remove_by_req_filter_bw(filter_accum);
 +        } else {
 +          return remove_by_req_filter_fw(filter_accum);
 +        }
 +      }
 +
 +      friend std::ostream&
 +      operator<<(std::ostream& out,
 +                 const typename PriorityQueueBase<C,R,B>::ClientRec& e) {
 +        out << "{ ClientRec::" <<
 +          " client:" << e.client <<
 +          " prev_tag:" << e.prev_tag <<
 +          " req_count:" << e.requests.size() <<
 +          " top_req:";
 +        if (e.has_request()) {
 +          out << e.next_request();
 +        } else {
 +          out << "none";
 +        }
 +        out << " }";
 +
 +        return out;
 +      }
 +      }; // class ClientRec
 +
 +      using ClientRecRef = std::shared_ptr<ClientRec>;
 +
 +      // when we try to get the next request, we'll be in one of three
 +      // situations -- we'll have one to return, have one that can
 +      // fire in the future, or not have any
 +      enum class NextReqType { returning, future, none };
 +
 +      // specifies which queue next request will get popped from
 +      enum class HeapId { reservation, ready };
 +
 +      // this is returned from next_req to tell the caller the situation
 +      struct NextReq {
 +      NextReqType type;
 +      union {
 +        HeapId    heap_id;
 +        Time      when_ready;
 +      };
 +      };
 +
 +
 +      // a function that can be called to look up client information
 +      using ClientInfoFunc = std::function<ClientInfo(const C&)>;
 +
 +
 +      bool empty() const {
 +      DataGuard g(data_mtx);
 +      return (resv_heap.empty() || ! resv_heap.top().has_request());
 +      }
 +
 +
 +      size_t client_count() const {
 +      DataGuard g(data_mtx);
 +      return resv_heap.size();
 +      }
 +
 +
 +      size_t request_count() const {
 +      DataGuard g(data_mtx);
 +      size_t total = 0;
 +      for (auto i = resv_heap.cbegin(); i != resv_heap.cend(); ++i) {
 +        total += i->request_count();
 +      }
 +      return total;
 +      }
 +
 +
-       static void request_sink(const R& req) {
++      bool remove_by_req_filter(std::function<bool(R&&)> filter_accum,
 +                              bool visit_backwards = false) {
 +      bool any_removed = false;
 +      DataGuard g(data_mtx);
 +      for (auto i : client_map) {
 +        bool modified =
 +          i.second->remove_by_req_filter(filter_accum, visit_backwards);
 +        if (modified) {
 +          resv_heap.adjust(*i.second);
 +          limit_heap.adjust(*i.second);
 +          ready_heap.adjust(*i.second);
 +#if USE_PROP_HEAP
 +          prop_heap.adjust(*i.second);
 +#endif
 +          any_removed = true;
 +        }
 +      }
 +      return any_removed;
 +      }
 +
 +
 +      // use as a default value when no accumulator is provide
-                           std::function<void (const R&)> accum = request_sink) {
++      static void request_sink(R&& req) {
 +      // do nothing
 +      }
 +
 +
 +      void remove_by_client(const C& client,
 +                          bool reverse = false,
-           accum(*j->request);
++                          std::function<void (R&&)> accum = request_sink) {
 +      DataGuard g(data_mtx);
 +
 +      auto i = client_map.find(client);
 +
 +      if (i == client_map.end()) return;
 +
 +      if (reverse) {
 +        for (auto j = i->second->requests.rbegin();
 +             j != i->second->requests.rend();
 +             ++j) {
-           accum(*j->request);
++          accum(std::move(*j->request));
 +        }
 +      } else {
 +        for (auto j = i->second->requests.begin();
 +             j != i->second->requests.end();
 +             ++j) {
-       inline void add_request(const R& request,
++          accum(std::move(*j->request));
 +        }
 +      }
 +
 +      i->second->requests.clear();
 +
 +      resv_heap.adjust(*i->second);
 +      limit_heap.adjust(*i->second);
 +      ready_heap.adjust(*i->second);
 +#if USE_PROP_HEAP
 +      prop_heap.adjust(*i->second);
 +#endif
 +      }
 +
 +
 +      uint get_heap_branching_factor() const {
 +      return B;
 +      }
 +
 +
 +      friend std::ostream& operator<<(std::ostream& out,
 +                                    const PriorityQueueBase& q) {
 +      std::lock_guard<decltype(q.data_mtx)> guard(q.data_mtx);
 +
 +      out << "{ PriorityQueue::";
 +      for (const auto& c : q.client_map) {
 +        out << "  { client:" << c.first << ", record:" << *c.second <<
 +          " }";
 +      }
 +      if (!q.resv_heap.empty()) {
 +        const auto& resv = q.resv_heap.top();
 +        out << " { reservation_top:" << resv << " }";
 +        const auto& ready = q.ready_heap.top();
 +        out << " { ready_top:" << ready << " }";
 +        const auto& limit = q.limit_heap.top();
 +        out << " { limit_top:" << limit << " }";
 +      } else {
 +        out << " HEAPS-EMPTY";
 +      }
 +      out << " }";
 +
 +      return out;
 +      }
 +
 +      // for debugging
 +      void display_queues(std::ostream& out,
 +                        bool show_res = true,
 +                        bool show_lim = true,
 +                        bool show_ready = true,
 +                        bool show_prop = true) const {
 +      auto filter = [](const ClientRec& e)->bool { return true; };
 +      DataGuard g(data_mtx);
 +      if (show_res) {
 +        resv_heap.display_sorted(out << "RESER:", filter);
 +      }
 +      if (show_lim) {
 +        limit_heap.display_sorted(out << "LIMIT:", filter);
 +      }
 +      if (show_ready) {
 +        ready_heap.display_sorted(out << "READY:", filter);
 +      }
 +#if USE_PROP_HEAP
 +      if (show_prop) {
 +        prop_heap.display_sorted(out << "PROPO:", filter);
 +      }
 +#endif
 +      } // display_queues
 +
 +
 +    protected:
 +
 +      // The ClientCompare functor is essentially doing a precedes?
 +      // operator, returning true if and only if the first parameter
 +      // must precede the second parameter. If the second must precede
 +      // the first, or if they are equivalent, false should be
 +      // returned. The reason for this behavior is that it will be
 +      // called to test if two items are out of order and if true is
 +      // returned it will reverse the items. Therefore false is the
 +      // default return when it doesn't matter to prevent unnecessary
 +      // re-ordering.
 +      //
 +      // The template is supporting variations in sorting based on the
 +      // heap in question and allowing these variations to be handled
 +      // at compile-time.
 +      //
 +      // tag_field determines which tag is being used for comparison
 +      //
 +      // ready_opt determines how the ready flag influences the sort
 +      //
 +      // use_prop_delta determines whether the proportional delta is
 +      // added in for comparison
 +      template<double RequestTag::*tag_field,
 +             ReadyOption ready_opt,
 +             bool use_prop_delta>
 +      struct ClientCompare {
 +      bool operator()(const ClientRec& n1, const ClientRec& n2) const {
 +        if (n1.has_request()) {
 +          if (n2.has_request()) {
 +            const auto& t1 = n1.next_request().tag;
 +            const auto& t2 = n2.next_request().tag;
 +            if (ReadyOption::ignore == ready_opt || t1.ready == t2.ready) {
 +              // if we don't care about ready or the ready values are the same
 +              if (use_prop_delta) {
 +                return (t1.*tag_field + n1.prop_delta) <
 +                  (t2.*tag_field + n2.prop_delta);
 +              } else {
 +                return t1.*tag_field < t2.*tag_field;
 +              }
 +            } else if (ReadyOption::raises == ready_opt) {
 +              // use_ready == true && the ready fields are different
 +              return t1.ready;
 +            } else {
 +              return t2.ready;
 +            }
 +          } else {
 +            // n1 has request but n2 does not
 +            return true;
 +          }
 +        } else if (n2.has_request()) {
 +          // n2 has request but n1 does not
 +          return false;
 +        } else {
 +          // both have none; keep stable w false
 +          return false;
 +        }
 +      }
 +      };
 +
 +      ClientInfoFunc       client_info_f;
 +
 +      mutable std::mutex data_mtx;
 +      using DataGuard = std::lock_guard<decltype(data_mtx)>;
 +
 +      // stable mapping between client ids and client queues
 +      std::map<C,ClientRecRef> client_map;
 +
 +      c::IndIntruHeap<ClientRecRef,
 +                    ClientRec,
 +                    &ClientRec::reserv_heap_data,
 +                    ClientCompare<&RequestTag::reservation,
 +                                  ReadyOption::ignore,
 +                                  false>,
 +                    B> resv_heap;
 +#if USE_PROP_HEAP
 +      c::IndIntruHeap<ClientRecRef,
 +                    ClientRec,
 +                    &ClientRec::prop_heap_data,
 +                    ClientCompare<&RequestTag::proportion,
 +                                  ReadyOption::ignore,
 +                                  true>,
 +                    B> prop_heap;
 +#endif
 +      c::IndIntruHeap<ClientRecRef,
 +                    ClientRec,
 +                    &ClientRec::lim_heap_data,
 +                    ClientCompare<&RequestTag::limit,
 +                                  ReadyOption::lowers,
 +                                  false>,
 +                    B> limit_heap;
 +      c::IndIntruHeap<ClientRecRef,
 +                    ClientRec,
 +                    &ClientRec::ready_heap_data,
 +                    ClientCompare<&RequestTag::proportion,
 +                                  ReadyOption::raises,
 +                                  true>,
 +                    B> ready_heap;
 +
 +      // if all reservations are met and all other requestes are under
 +      // limit, this will allow the request next in terms of
 +      // proportion to still get issued
 +      bool             allow_limit_break;
 +
 +      std::atomic_bool finishing;
 +
 +      // every request creates a tick
 +      Counter tick = 0;
 +
 +      // performance data collection
 +      size_t reserv_sched_count = 0;
 +      size_t prop_sched_count = 0;
 +      size_t limit_break_sched_count = 0;
 +
 +      Duration                  idle_age;
 +      Duration                  erase_age;
 +      Duration                  check_time;
 +      std::deque<MarkPoint>     clean_mark_points;
 +
 +      // NB: All threads declared at end, so they're destructed first!
 +
 +      std::unique_ptr<RunEvery> cleaning_job;
 +
 +
 +      // COMMON constructor that others feed into; we can accept three
 +      // different variations of durations
 +      template<typename Rep, typename Per>
 +      PriorityQueueBase(ClientInfoFunc _client_info_f,
 +                      std::chrono::duration<Rep,Per> _idle_age,
 +                      std::chrono::duration<Rep,Per> _erase_age,
 +                      std::chrono::duration<Rep,Per> _check_time,
 +                      bool _allow_limit_break) :
 +      client_info_f(_client_info_f),
 +      allow_limit_break(_allow_limit_break),
 +      finishing(false),
 +      idle_age(std::chrono::duration_cast<Duration>(_idle_age)),
 +      erase_age(std::chrono::duration_cast<Duration>(_erase_age)),
 +      check_time(std::chrono::duration_cast<Duration>(_check_time))
 +      {
 +      assert(_erase_age >= _idle_age);
 +      assert(_check_time < _idle_age);
 +      cleaning_job =
 +        std::unique_ptr<RunEvery>(
 +          new RunEvery(check_time,
 +                       std::bind(&PriorityQueueBase::do_clean, this)));
 +      }
 +
 +
 +      ~PriorityQueueBase() {
 +      finishing = true;
 +      }
 +
 +
 +      // data_mtx must be held by caller
 +      void do_add_request(RequestRef&& request,
 +                        const C& client_id,
 +                        const ReqParams& req_params,
 +                        const Time time,
 +                        const double cost = 0.0) {
 +      ++tick;
 +
 +      // this pointer will help us create a reference to a shared
 +      // pointer, no matter which of two codepaths we take
 +      ClientRec* temp_client;
 +
 +      auto client_it = client_map.find(client_id);
 +      if (client_map.end() != client_it) {
 +        temp_client = &(*client_it->second); // address of obj of shared_ptr
 +      } else {
 +        ClientInfo info = client_info_f(client_id);
 +        ClientRecRef client_rec =
 +          std::make_shared<ClientRec>(client_id, info, tick);
 +        resv_heap.push(client_rec);
 +#if USE_PROP_HEAP
 +        prop_heap.push(client_rec);
 +#endif
 +        limit_heap.push(client_rec);
 +        ready_heap.push(client_rec);
 +        client_map[client_id] = client_rec;
 +        temp_client = &(*client_rec); // address of obj of shared_ptr
 +      }
 +
 +      // for convenience, we'll create a reference to the shared pointer
 +      ClientRec& client = *temp_client;
 +
 +      if (client.idle) {
 +        // We need to do an adjustment so that idle clients compete
 +        // fairly on proportional tags since those tags may have
 +        // drifted from real-time. Either use the lowest existing
 +        // proportion tag -- O(1) -- or the client with the lowest
 +        // previous proportion tag -- O(n) where n = # clients.
 +        //
 +        // So we don't have to maintain a propotional queue that
 +        // keeps the minimum on proportional tag alone (we're
 +        // instead using a ready queue), we'll have to check each
 +        // client.
 +        //
 +        // The alternative would be to maintain a proportional queue
 +        // (define USE_PROP_TAG) and do an O(1) operation here.
 +
 +        // Was unable to confirm whether equality testing on
 +        // std::numeric_limits<double>::max() is guaranteed, so
 +        // we'll use a compile-time calculated trigger that is one
 +        // third the max, which should be much larger than any
 +        // expected organic value.
 +        constexpr double lowest_prop_tag_trigger =
 +          std::numeric_limits<double>::max() / 3.0;
 +
 +        double lowest_prop_tag = std::numeric_limits<double>::max();
 +        for (auto const &c : client_map) {
 +          // don't use ourselves (or anything else that might be
 +          // listed as idle) since we're now in the map
 +          if (!c.second->idle) {
 +            double p;
 +            // use either lowest proportion tag or previous proportion tag
 +            if (c.second->has_request()) {
 +              p = c.second->next_request().tag.proportion +
 +                c.second->prop_delta;
 +            } else {
 +              p = c.second->get_req_tag().proportion + c.second->prop_delta;
 +            }
 +
 +            if (p < lowest_prop_tag) {
 +              lowest_prop_tag = p;
 +            }
 +          }
 +        }
 +
 +        // if this conditional does not fire, it
 +        if (lowest_prop_tag < lowest_prop_tag_trigger) {
 +          client.prop_delta = lowest_prop_tag - time;
 +        }
 +        client.idle = false;
 +      } // if this client was idle
 +
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +      RequestTag tag(0, 0, 0, time);
 +
 +      if (!client.has_request()) {
 +        tag = RequestTag(client.get_req_tag(),
 +                         client.info,
 +                         req_params,
 +                         time,
 +                         cost);
 +
 +        // copy tag to previous tag for client
 +        client.update_req_tag(tag, tick);
 +      }
 +#else
 +      RequestTag tag(client.get_req_tag(), client.info, req_params, time, cost);
 +      // copy tag to previous tag for client
 +      client.update_req_tag(tag, tick);
 +#endif
 +
 +      client.add_request(tag, client.client, std::move(request));
 +      if (1 == client.requests.size()) {
 +        // NB: can the following 4 calls to adjust be changed
 +        // promote? Can adding a request ever demote a client in the
 +        // heaps?
 +        resv_heap.adjust(client);
 +        limit_heap.adjust(client);
 +        ready_heap.adjust(client);
 +#if USE_PROP_HEAP
 +        prop_heap.adjust(client);
 +#endif
 +      }
 +
 +      client.cur_rho = req_params.rho;
 +      client.cur_delta = req_params.delta;
 +
 +      resv_heap.adjust(client);
 +      limit_heap.adjust(client);
 +      ready_heap.adjust(client);
 +#if USE_PROP_HEAP
 +      prop_heap.adjust(client);
 +#endif
 +      } // add_request
 +
 +
 +      // data_mtx should be held when called; top of heap should have
 +      // a ready request
 +      template<typename C1, IndIntruHeapData ClientRec::*C2, typename C3>
 +      void pop_process_request(IndIntruHeap<C1, ClientRec, C2, C3, B>& heap,
 +                             std::function<void(const C& client,
 +                                                RequestRef& request)> process) {
 +      // gain access to data
 +      ClientRec& top = heap.top();
 +
 +      RequestRef request = std::move(top.next_request().request);
++#ifndef DO_NOT_DELAY_TAG_CALC
 +      RequestTag tag = top.next_request().tag;
++#endif
 +
 +      // pop request and adjust heaps
 +      top.pop_request();
 +
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +      if (top.has_request()) {
 +        ClientReq& next_first = top.next_request();
 +        next_first.tag = RequestTag(tag, top.info,
 +                                    top.cur_delta, top.cur_rho,
 +                                    next_first.tag.arrival);
 +
 +        // copy tag to previous tag for client
 +        top.update_req_tag(next_first.tag, tick);
 +      }
 +#endif
 +
 +      resv_heap.demote(top);
 +      limit_heap.adjust(top);
 +#if USE_PROP_HEAP
 +      prop_heap.demote(top);
 +#endif
 +      ready_heap.demote(top);
 +
 +      // process
 +      process(top.client, request);
 +      } // pop_process_request
 +
 +
 +      // data_mtx should be held when called
 +      void reduce_reservation_tags(ClientRec& client) {
 +      for (auto& r : client.requests) {
 +        r.tag.reservation -= client.info.reservation_inv;
 +
 +#ifndef DO_NOT_DELAY_TAG_CALC
 +        // reduce only for front tag. because next tags' value are invalid
 +        break;
 +#endif
 +      }
 +      // don't forget to update previous tag
 +      client.prev_tag.reservation -= client.info.reservation_inv;
 +      resv_heap.promote(client);
 +      }
 +
 +
 +      // data_mtx should be held when called
 +      void reduce_reservation_tags(const C& client_id) {
 +      auto client_it = client_map.find(client_id);
 +
 +      // means the client was cleaned from map; should never happen
 +      // as long as cleaning times are long enough
 +      assert(client_map.end() != client_it);
 +      reduce_reservation_tags(*client_it->second);
 +      }
 +
 +
 +      // data_mtx should be held when called
 +      NextReq do_next_request(Time now) {
 +      NextReq result;
 +
 +      // if reservation queue is empty, all are empty (i.e., no active clients)
 +      if(resv_heap.empty()) {
 +        result.type = NextReqType::none;
 +        return result;
 +      }
 +
 +      // try constraint (reservation) based scheduling
 +
 +      auto& reserv = resv_heap.top();
 +      if (reserv.has_request() &&
 +          reserv.next_request().tag.reservation <= now) {
 +        result.type = NextReqType::returning;
 +        result.heap_id = HeapId::reservation;
 +        return result;
 +      }
 +
 +      // no existing reservations before now, so try weight-based
 +      // scheduling
 +
 +      // all items that are within limit are eligible based on
 +      // priority
 +      auto limits = &limit_heap.top();
 +      while (limits->has_request() &&
 +             !limits->next_request().tag.ready &&
 +             limits->next_request().tag.limit <= now) {
 +        limits->next_request().tag.ready = true;
 +        ready_heap.promote(*limits);
 +        limit_heap.demote(*limits);
 +
 +        limits = &limit_heap.top();
 +      }
 +
 +      auto& readys = ready_heap.top();
 +      if (readys.has_request() &&
 +          readys.next_request().tag.ready &&
 +          readys.next_request().tag.proportion < max_tag) {
 +        result.type = NextReqType::returning;
 +        result.heap_id = HeapId::ready;
 +        return result;
 +      }
 +
 +      // if nothing is schedulable by reservation or
 +      // proportion/weight, and if we allow limit break, try to
 +      // schedule something with the lowest proportion tag or
 +      // alternatively lowest reservation tag.
 +      if (allow_limit_break) {
 +        if (readys.has_request() &&
 +            readys.next_request().tag.proportion < max_tag) {
 +          result.type = NextReqType::returning;
 +          result.heap_id = HeapId::ready;
 +          return result;
 +        } else if (reserv.has_request() &&
 +                   reserv.next_request().tag.reservation < max_tag) {
 +          result.type = NextReqType::returning;
 +          result.heap_id = HeapId::reservation;
 +          return result;
 +        }
 +      }
 +
 +      // nothing scheduled; make sure we re-run when next
 +      // reservation item or next limited item comes up
 +
 +      Time next_call = TimeMax;
 +      if (resv_heap.top().has_request()) {
 +        next_call =
 +          min_not_0_time(next_call,
 +                         resv_heap.top().next_request().tag.reservation);
 +      }
 +      if (limit_heap.top().has_request()) {
 +        const auto& next = limit_heap.top().next_request();
 +        assert(!next.tag.ready || max_tag == next.tag.proportion);
 +        next_call = min_not_0_time(next_call, next.tag.limit);
 +      }
 +      if (next_call < TimeMax) {
 +        result.type = NextReqType::future;
 +        result.when_ready = next_call;
 +        return result;
 +      } else {
 +        result.type = NextReqType::none;
 +        return result;
 +      }
 +      } // do_next_request
 +
 +
 +      // if possible is not zero and less than current then return it;
 +      // otherwise return current; the idea is we're trying to find
 +      // the minimal time but ignoring zero
 +      static inline const Time& min_not_0_time(const Time& current,
 +                                             const Time& possible) {
 +      return TimeZero == possible ? current : std::min(current, possible);
 +      }
 +
 +
 +      /*
 +       * This is being called regularly by RunEvery. Every time it's
 +       * called it notes the time and delta counter (mark point) in a
 +       * deque. It also looks at the deque to find the most recent
 +       * mark point that is older than clean_age. It then walks the
 +       * map and delete all server entries that were last used before
 +       * that mark point.
 +       */
 +      void do_clean() {
 +      TimePoint now = std::chrono::steady_clock::now();
 +      DataGuard g(data_mtx);
 +      clean_mark_points.emplace_back(MarkPoint(now, tick));
 +
 +      // first erase the super-old client records
 +
 +      Counter erase_point = 0;
 +      auto point = clean_mark_points.front();
 +      while (point.first <= now - erase_age) {
 +        erase_point = point.second;
 +        clean_mark_points.pop_front();
 +        point = clean_mark_points.front();
 +      }
 +
 +      Counter idle_point = 0;
 +      for (auto i : clean_mark_points) {
 +        if (i.first <= now - idle_age) {
 +          idle_point = i.second;
 +        } else {
 +          break;
 +        }
 +      }
 +
 +      if (erase_point > 0 || idle_point > 0) {
 +        for (auto i = client_map.begin(); i != client_map.end(); /* empty */) {
 +          auto i2 = i++;
 +          if (erase_point && i2->second->last_tick <= erase_point) {
 +            delete_from_heaps(i2->second);
 +            client_map.erase(i2);
 +          } else if (idle_point && i2->second->last_tick <= idle_point) {
 +            i2->second->idle = true;
 +          }
 +        } // for
 +      } // if
 +      } // do_clean
 +
 +
 +      // data_mtx must be held by caller
 +      template<IndIntruHeapData ClientRec::*C1,typename C2>
 +      void delete_from_heap(ClientRecRef& client,
 +                          c::IndIntruHeap<ClientRecRef,ClientRec,C1,C2,B>& heap) {
 +      auto i = heap.rfind(client);
 +      heap.remove(i);
 +      }
 +
 +
 +      // data_mtx must be held by caller
 +      void delete_from_heaps(ClientRecRef& client) {
 +      delete_from_heap(client, resv_heap);
 +#if USE_PROP_HEAP
 +      delete_from_heap(client, prop_heap);
 +#endif
 +      delete_from_heap(client, limit_heap);
 +      delete_from_heap(client, ready_heap);
 +      }
 +    }; // class PriorityQueueBase
 +
 +
 +    template<typename C, typename R, uint B=2>
 +    class PullPriorityQueue : public PriorityQueueBase<C,R,B> {
 +      using super = PriorityQueueBase<C,R,B>;
 +
 +    public:
 +
 +      // When a request is pulled, this is the return type.
 +      struct PullReq {
 +      struct Retn {
 +        C                           client;
 +        typename super::RequestRef  request;
 +        PhaseType                   phase;
 +      };
 +
 +      typename super::NextReqType   type;
 +      boost::variant<Retn,Time>     data;
 +
 +      bool is_none() const { return type == super::NextReqType::none; }
 +
 +      bool is_retn() const { return type == super::NextReqType::returning; }
 +      Retn& get_retn() {
 +        return boost::get<Retn>(data);
 +      }
 +
 +      bool is_future() const { return type == super::NextReqType::future; }
 +      Time getTime() const { return boost::get<Time>(data); }
 +      };
 +
 +
 +#ifdef PROFILE
 +      ProfileTimer<std::chrono::nanoseconds> pull_request_timer;
 +      ProfileTimer<std::chrono::nanoseconds> add_request_timer;
 +#endif
 +
 +      template<typename Rep, typename Per>
 +      PullPriorityQueue(typename super::ClientInfoFunc _client_info_f,
 +                      std::chrono::duration<Rep,Per> _idle_age,
 +                      std::chrono::duration<Rep,Per> _erase_age,
 +                      std::chrono::duration<Rep,Per> _check_time,
 +                      bool _allow_limit_break = false) :
 +      super(_client_info_f,
 +            _idle_age, _erase_age, _check_time,
 +            _allow_limit_break)
 +      {
 +      // empty
 +      }
 +
 +
 +      // pull convenience constructor
 +      PullPriorityQueue(typename super::ClientInfoFunc _client_info_f,
 +                      bool _allow_limit_break = false) :
 +      PullPriorityQueue(_client_info_f,
 +                        std::chrono::minutes(10),
 +                        std::chrono::minutes(15),
 +                        std::chrono::minutes(6),
 +                        _allow_limit_break)
 +      {
 +      // empty
 +      }
 +
 +
-       add_request(typename super::RequestRef(new R(request)),
++      inline void add_request(R&& request,
 +                            const C& client_id,
 +                            const ReqParams& req_params,
 +                            double addl_cost = 0.0) {
-       inline void add_request(const R& request,
++      add_request(typename super::RequestRef(new R(std::move(request))),
 +                  client_id,
 +                  req_params,
 +                  get_time(),
 +                  addl_cost);
 +      }
 +
 +
-       add_request(typename super::RequestRef(new R(request)),
++      inline void add_request(R&& request,
 +                            const C& client_id,
 +                            double addl_cost = 0.0) {
 +      static const ReqParams null_req_params;
-       inline void add_request_time(const R& request,
++      add_request(typename super::RequestRef(new R(std::move(request))),
 +                  client_id,
 +                  null_req_params,
 +                  get_time(),
 +                  addl_cost);
 +      }
 +
 +
 +
-       add_request(typename super::RequestRef(new R(request)),
++      inline void add_request_time(R&& request,
 +                                 const C& client_id,
 +                                 const ReqParams& req_params,
 +                                 const Time time,
 +                                 double addl_cost = 0.0) {
-         break;
++      add_request(typename super::RequestRef(new R(std::move(request))),
 +                  client_id,
 +                  req_params,
 +                  time,
 +                  addl_cost);
 +      }
 +
 +
 +      inline void add_request(typename super::RequestRef&& request,
 +                            const C& client_id,
 +                            const ReqParams& req_params,
 +                            double addl_cost = 0.0) {
 +      add_request(request, req_params, client_id, get_time(), addl_cost);
 +      }
 +
 +
 +      inline void add_request(typename super::RequestRef&& request,
 +                            const C& client_id,
 +                            double addl_cost = 0.0) {
 +      static const ReqParams null_req_params;
 +      add_request(request, null_req_params, client_id, get_time(), addl_cost);
 +      }
 +
 +
 +      // this does the work; the versions above provide alternate interfaces
 +      void add_request(typename super::RequestRef&& request,
 +                     const C&                     client_id,
 +                     const ReqParams&             req_params,
 +                     const Time                   time,
 +                     double                       addl_cost = 0.0) {
 +      typename super::DataGuard g(this->data_mtx);
 +#ifdef PROFILE
 +      add_request_timer.start();
 +#endif
 +      super::do_add_request(std::move(request),
 +                            client_id,
 +                            req_params,
 +                            time,
 +                            addl_cost);
 +      // no call to schedule_request for pull version
 +#ifdef PROFILE
 +      add_request_timer.stop();
 +#endif
 +      }
 +
 +
 +      inline PullReq pull_request() {
 +      return pull_request(get_time());
 +      }
 +
 +
 +      PullReq pull_request(Time now) {
 +      PullReq result;
 +      typename super::DataGuard g(this->data_mtx);
 +#ifdef PROFILE
 +      pull_request_timer.start();
 +#endif
 +
 +      typename super::NextReq next = super::do_next_request(now);
 +      result.type = next.type;
 +      switch(next.type) {
 +      case super::NextReqType::none:
 +        return result;
-         break;
 +      case super::NextReqType::future:
 +        result.data = next.when_ready;
 +        return result;
-       inline void add_request(const R& request,
 +      case super::NextReqType::returning:
 +        // to avoid nesting, break out and let code below handle this case
 +        break;
 +      default:
 +        assert(false);
 +      }
 +
 +      // we'll only get here if we're returning an entry
 +
 +      auto process_f =
 +        [&] (PullReq& pull_result, PhaseType phase) ->
 +        std::function<void(const C&,
 +                           typename super::RequestRef&)> {
 +        return [&pull_result, phase](const C& client,
 +                                     typename super::RequestRef& request) {
 +          pull_result.data =
 +          typename PullReq::Retn{client, std::move(request), phase};
 +        };
 +      };
 +
 +      switch(next.heap_id) {
 +      case super::HeapId::reservation:
 +        super::pop_process_request(this->resv_heap,
 +                                   process_f(result, PhaseType::reservation));
 +        ++this->reserv_sched_count;
 +        break;
 +      case super::HeapId::ready:
 +        super::pop_process_request(this->ready_heap,
 +                                   process_f(result, PhaseType::priority));
 +        { // need to use retn temporarily
 +          auto& retn = boost::get<typename PullReq::Retn>(result.data);
 +          super::reduce_reservation_tags(retn.client);
 +        }
 +        ++this->prop_sched_count;
 +        break;
 +      default:
 +        assert(false);
 +      }
 +
 +#ifdef PROFILE
 +      pull_request_timer.stop();
 +#endif
 +      return result;
 +      } // pull_request
 +
 +
 +    protected:
 +
 +
 +      // data_mtx should be held when called; unfortunately this
 +      // function has to be repeated in both push & pull
 +      // specializations
 +      typename super::NextReq next_request() {
 +      return next_request(get_time());
 +      }
 +    }; // class PullPriorityQueue
 +
 +
 +    // PUSH version
 +    template<typename C, typename R, uint B=2>
 +    class PushPriorityQueue : public PriorityQueueBase<C,R,B> {
 +
 +    protected:
 +
 +      using super = PriorityQueueBase<C,R,B>;
 +
 +    public:
 +
 +      // a function to see whether the server can handle another request
 +      using CanHandleRequestFunc = std::function<bool(void)>;
 +
 +      // a function to submit a request to the server; the second
 +      // parameter is a callback when it's completed
 +      using HandleRequestFunc =
 +      std::function<void(const C&,typename super::RequestRef,PhaseType)>;
 +
 +    protected:
 +
 +      CanHandleRequestFunc can_handle_f;
 +      HandleRequestFunc    handle_f;
 +      // for handling timed scheduling
 +      std::mutex  sched_ahead_mtx;
 +      std::condition_variable sched_ahead_cv;
 +      Time sched_ahead_when = TimeZero;
 +
 +#ifdef PROFILE
 +    public:
 +      ProfileTimer<std::chrono::nanoseconds> add_request_timer;
 +      ProfileTimer<std::chrono::nanoseconds> request_complete_timer;
 +    protected:
 +#endif
 +
 +      // NB: threads declared last, so constructed last and destructed first
 +
 +      std::thread sched_ahead_thd;
 +
 +    public:
 +
 +      // push full constructor
 +      template<typename Rep, typename Per>
 +      PushPriorityQueue(typename super::ClientInfoFunc _client_info_f,
 +                      CanHandleRequestFunc _can_handle_f,
 +                      HandleRequestFunc _handle_f,
 +                      std::chrono::duration<Rep,Per> _idle_age,
 +                      std::chrono::duration<Rep,Per> _erase_age,
 +                      std::chrono::duration<Rep,Per> _check_time,
 +                      bool _allow_limit_break = false) :
 +      super(_client_info_f,
 +            _idle_age, _erase_age, _check_time,
 +            _allow_limit_break)
 +      {
 +      can_handle_f = _can_handle_f;
 +      handle_f = _handle_f;
 +      sched_ahead_thd = std::thread(&PushPriorityQueue::run_sched_ahead, this);
 +      }
 +
 +
 +      // push convenience constructor
 +      PushPriorityQueue(typename super::ClientInfoFunc _client_info_f,
 +                      CanHandleRequestFunc _can_handle_f,
 +                      HandleRequestFunc _handle_f,
 +                      bool _allow_limit_break = false) :
 +      PushPriorityQueue(_client_info_f,
 +                        _can_handle_f,
 +                        _handle_f,
 +                        std::chrono::minutes(10),
 +                        std::chrono::minutes(15),
 +                        std::chrono::minutes(6),
 +                        _allow_limit_break)
 +      {
 +      // empty
 +      }
 +
 +
 +      ~PushPriorityQueue() {
 +      this->finishing = true;
 +      sched_ahead_cv.notify_one();
 +      sched_ahead_thd.join();
 +      }
 +
 +    public:
 +
-       add_request(typename super::RequestRef(new R(request)),
++      inline void add_request(R&& request,
 +                            const C& client_id,
 +                            const ReqParams& req_params,
 +                            double addl_cost = 0.0) {
++      add_request(typename super::RequestRef(new R(std::move(request))),
 +                  client_id,
 +                  req_params,
 +                  get_time(),
 +                  addl_cost);
 +      }
 +
 +
 +      inline void add_request(typename super::RequestRef&& request,
 +                            const C& client_id,
 +                            const ReqParams& req_params,
 +                            double addl_cost = 0.0) {
 +      add_request(request, req_params, client_id, get_time(), addl_cost);
 +      }
 +
 +
 +      inline void add_request_time(const R& request,
 +                                 const C& client_id,
 +                                 const ReqParams& req_params,
 +                                 const Time time,
 +                                 double addl_cost = 0.0) {
 +      add_request(typename super::RequestRef(new R(request)),
 +                  client_id,
 +                  req_params,
 +                  time,
 +                  addl_cost);
 +      }
 +
 +
 +      void add_request(typename super::RequestRef&& request,
 +                     const C& client_id,
 +                     const ReqParams& req_params,
 +                     const Time time,
 +                     double addl_cost = 0.0) {
 +      typename super::DataGuard g(this->data_mtx);
 +#ifdef PROFILE
 +      add_request_timer.start();
 +#endif
 +      super::do_add_request(std::move(request),
 +                            client_id,
 +                            req_params,
 +                            time,
 +                            addl_cost);
 +      schedule_request();
 +#ifdef PROFILE
 +      add_request_timer.stop();
 +#endif
 +      }
 +
 +
 +      void request_completed() {
 +      typename super::DataGuard g(this->data_mtx);
 +#ifdef PROFILE
 +      request_complete_timer.start();
 +#endif
 +      schedule_request();
 +#ifdef PROFILE
 +      request_complete_timer.stop();
 +#endif
 +      }
 +
 +    protected:
 +
 +      // data_mtx should be held when called; furthermore, the heap
 +      // should not be empty and the top element of the heap should
 +      // not be already handled
 +      //
 +      // NOTE: the use of "super::ClientRec" in either the template
 +      // construct or as a parameter to submit_top_request generated
 +      // a compiler error in g++ 4.8.4, when ClientRec was
 +      // "protected" rather than "public". By g++ 6.3.1 this was not
 +      // an issue. But for backwards compatibility
 +      // PriorityQueueBase::ClientRec is public.
 +      template<typename C1,
 +             IndIntruHeapData super::ClientRec::*C2,
 +             typename C3,
 +             uint B4>
 +      C submit_top_request(IndIntruHeap<C1,typename super::ClientRec,C2,C3,B4>& heap,
 +                         PhaseType phase) {
 +      C client_result;
 +      super::pop_process_request(heap,
 +                                 [this, phase, &client_result]
 +                                 (const C& client,
 +                                  typename super::RequestRef& request) {
 +                                   client_result = client;
 +                                   handle_f(client, std::move(request), phase);
 +                                 });
 +      return client_result;
 +      }
 +
 +
 +      // data_mtx should be held when called
 +      void submit_request(typename super::HeapId heap_id) {
 +      C client;
 +      switch(heap_id) {
 +      case super::HeapId::reservation:
 +        // don't need to note client
 +        (void) submit_top_request(this->resv_heap, PhaseType::reservation);
 +        // unlike the other two cases, we do not reduce reservation
 +        // tags here
 +        ++this->reserv_sched_count;
 +        break;
 +      case super::HeapId::ready:
 +        client = submit_top_request(this->ready_heap, PhaseType::priority);
 +        super::reduce_reservation_tags(client);
 +        ++this->prop_sched_count;
 +        break;
 +      default:
 +        assert(false);
 +      }
 +      } // submit_request
 +
 +
 +      // data_mtx should be held when called; unfortunately this
 +      // function has to be repeated in both push & pull
 +      // specializations
 +      typename super::NextReq next_request() {
 +      return next_request(get_time());
 +      }
 +
 +
 +      // data_mtx should be held when called; overrides member
 +      // function in base class to add check for whether a request can
 +      // be pushed to the server
 +      typename super::NextReq next_request(Time now) {
 +      if (!can_handle_f()) {
 +        typename super::NextReq result;
 +        result.type = super::NextReqType::none;
 +        return result;
 +      } else {
 +        return super::do_next_request(now);
 +      }
 +      } // next_request
 +
 +
 +      // data_mtx should be held when called
 +      void schedule_request() {
 +      typename super::NextReq next_req = next_request();
 +      switch (next_req.type) {
 +      case super::NextReqType::none:
 +        return;
 +      case super::NextReqType::future:
 +        sched_at(next_req.when_ready);
 +        break;
 +      case super::NextReqType::returning:
 +        submit_request(next_req.heap_id);
 +        break;
 +      default:
 +        assert(false);
 +      }
 +      }
 +
 +
 +      // this is the thread that handles running schedule_request at
 +      // future times when nothing can be scheduled immediately
 +      void run_sched_ahead() {
 +      std::unique_lock<std::mutex> l(sched_ahead_mtx);
 +
 +      while (!this->finishing) {
 +        if (TimeZero == sched_ahead_when) {
 +          sched_ahead_cv.wait(l);
 +        } else {
 +          Time now;
 +          while (!this->finishing && (now = get_time()) < sched_ahead_when) {
 +            long microseconds_l = long(1 + 1000000 * (sched_ahead_when - now));
 +            auto microseconds = std::chrono::microseconds(microseconds_l);
 +            sched_ahead_cv.wait_for(l, microseconds);
 +          }
 +          sched_ahead_when = TimeZero;
 +          if (this->finishing) return;
 +
 +          l.unlock();
 +          if (!this->finishing) {
 +            typename super::DataGuard g(this->data_mtx);
 +            schedule_request();
 +          }
 +          l.lock();
 +        }
 +      }
 +      }
 +
 +
 +      void sched_at(Time when) {
 +      std::lock_guard<std::mutex> l(sched_ahead_mtx);
 +      if (this->finishing) return;
 +      if (TimeZero == sched_ahead_when || when < sched_ahead_when) {
 +        sched_ahead_when = when;
 +        sched_ahead_cv.notify_one();
 +      }
 +      }
 +    }; // class PushPriorityQueue
 +
 +  } // namespace dmclock
 +} // namespace crimson
index d12c6f9eb63c9a61b9ae6446386c3d27b82137d4,0000000000000000000000000000000000000000..c46d09f39319a53e96f936abd25bb11e241c8110
mode 100644,000000..100644
--- /dev/null
@@@ -1,45 -1,0 +1,53 @@@
-       (void) result;
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +
 +/*
 + * Copyright (C) 2017 Red Hat Inc.
 + */
 +
 +
 +#pragma once
 +
 +
 +#include <unistd.h>
 +#include <assert.h>
 +#include <sys/time.h>
 +
 +#include <limits>
 +#include <cmath>
 +#include <chrono>
 +
 +
 +namespace crimson {
 +  namespace dmclock {
 +    // we're using double to represent time, but we could change it by
 +    // changing the following declarations (and by making sure a min
 +    // function existed)
 +    using Time = double;
 +    static const Time TimeZero = 0.0;
 +    static const Time TimeMax = std::numeric_limits<Time>::max();
 +    static const double NaN = nan("");
 +
 +
 +    inline Time get_time() {
++#if defined(__linux__)
++      struct timespec now;
++      auto result = clock_gettime(CLOCK_REALTIME, &now);
++      (void) result; // reference result in case assert is compiled out
++      assert(0 == result);
++      return now.tv_sec + (now.tv_nsec / 1.0e9);
++#else
 +      struct timeval now;
 +      auto result = gettimeofday(&now, NULL);
-       return now.tv_sec + (now.tv_usec / 1000000.0);
++      (void) result; // reference result in case assert is compiled out
 +      assert(0 == result);
++      return now.tv_sec + (now.tv_usec / 1.0e6);
++#endif
 +    }
 +
 +    std::string format_time(const Time& time, uint modulo = 1000);
 +
 +    void debugger();
 +
 +  } // namespace dmclock
 +} // namespace crimson
index 24da2e7320a5d02343355ee442b825a424bdbf17,0000000000000000000000000000000000000000..aff35d5d77581ccfd918ead823efd75beaa7fa4c
mode 100644,000000..100644
--- /dev/null
@@@ -1,31 -1,0 +1,37 @@@
++INCLUDE (CheckIncludeFiles)
++CHECK_INCLUDE_FILES("sys/prctl.h" HAVE_SYS_PRCTL_H)
++CONFIGURE_FILE(dmtest-config.h.in dmtest-config.h)
++
++include_directories(${CMAKE_CURRENT_BINARY_DIR})
 +include_directories(../src)
 +include_directories(../support/src)
 +include_directories(../sim/src)
 +
 +set(support_srcs ../sim/src/test_dmclock.cc)
 +set(test_srcs
 +  test_test_client.cc
 +  test_dmclock_server.cc
 +  test_dmclock_client.cc
 +  )
 +
 +set_source_files_properties(${core_srcs} ${test_srcs}
 +  PROPERTIES
 +  COMPILE_FLAGS "${local_flags}"
 +  )
 +
 +add_executable(dmclock-tests ${test_srcs} ${support_srcs})
++target_include_directories(dmclock-tests PRIVATE "${GTEST_INCLUDE_DIRS}")
 +
 +if (TARGET gtest AND TARGET gtest_main)
 +  add_dependencies(dmclock-tests gtest gtest_main)
 +  target_link_libraries(dmclock-tests
 +    LINK_PRIVATE $<TARGET_FILE:dmclock>
 +    pthread
 +    $<TARGET_FILE:gtest>
 +    $<TARGET_FILE:gtest_main>)
 +else()
 +  target_link_libraries(dmclock-tests
 +    LINK_PRIVATE $<TARGET_FILE:dmclock> pthread ${GTEST_LIBRARIES} ${GTEST_MAIN_LIBRARIES})
 +endif()
 +  
 +add_dependencies(dmclock-tests dmclock)
index 0000000000000000000000000000000000000000,0000000000000000000000000000000000000000..39f98293dfda274672df9674b4c06eb357e431eb
new file mode 100644 (file)
--- /dev/null
--- /dev/null
@@@ -1,0 -1,0 +1,42 @@@
++// essentially the same as ceph's PrCtl.h, copied into the dmclock library
++
++#include <dmtest-config.h>
++#ifdef HAVE_SYS_PRCTL_H
++#include <iostream>
++#include <sys/prctl.h>
++#include <errno.h>
++
++struct PrCtl {
++  int saved_state = -1;
++  int set_dumpable(int new_state) {
++    int r = prctl(PR_SET_DUMPABLE, new_state);
++    if (r) {
++      r = -errno;
++      std::cerr << "warning: unable to " << (new_state ? "set" : "unset")
++                << " dumpable flag: " << strerror(r)
++                << std::endl;
++    }
++    return r;
++  }
++  PrCtl(int new_state = 0) {
++    int r = prctl(PR_GET_DUMPABLE);
++    if (r == -1) {
++      r = errno;
++      std::cerr << "warning: unable to get dumpable flag: " << strerror(r)
++                << std::endl;
++    } else if (r != new_state) {
++      if (!set_dumpable(new_state)) {
++        saved_state = r;
++      }
++    }
++  }
++  ~PrCtl() {
++    if (saved_state < 0) {
++      return;
++    }
++    set_dumpable(saved_state);
++  }
++};
++#else
++struct PrCtl {};
++#endif
index 0000000000000000000000000000000000000000,0000000000000000000000000000000000000000..ecd2044a89a581d725ee94fc53ec78cb87e63f97
new file mode 100644 (file)
--- /dev/null
--- /dev/null
@@@ -1,0 -1,0 +1,2 @@@
++/* Define to 1 if you have the <sys/prctl.h> header file. */
++#cmakedefine HAVE_SYS_PRCTL_H 1
index 4555e377323e9ff87af6341efb47740ea60207cc,0000000000000000000000000000000000000000..95def410fb69f44ded3f78b6d8962bdf170506b9
mode 100644,000000..100644
--- /dev/null
@@@ -1,826 -1,0 +1,825 @@@
-       Request req;
 +// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 +// vim: ts=8 sw=2 smarttab
 +/*
 + * Copyright (C) 2016 Red Hat Inc.
 + */
 +
 +
 +#include <memory>
 +#include <chrono>
 +#include <iostream>
 +#include <list>
 +#include <vector>
 +
 +
 +#include "dmclock_server.h"
 +#include "dmclock_util.h"
 +#include "gtest/gtest.h"
 +
++// process control to prevent core dumps during gtest death tests
++#include "dmcPrCtl.h"
++
 +
 +namespace dmc = crimson::dmclock;
 +
 +
 +// we need a request object; an empty one will do
 +struct Request {
 +};
 +
 +
 +namespace crimson {
 +  namespace dmclock {
 +
 +    /*
 +     * Allows us to test the code provided with the mutex provided locked.
 +     */
 +    static void test_locked(std::mutex& mtx, std::function<void()> code) {
 +      std::unique_lock<std::mutex> l(mtx);
 +      code();
 +    }
 +
 +
 +    TEST(dmclock_server, bad_tag_deathtest) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      ClientId client1 = 17;
 +      ClientId client2 = 18;
 +
 +      double reservation = 0.0;
 +      double weight = 0.0;
 +
 +      dmc::ClientInfo ci1(reservation, weight, 0.0);
 +      dmc::ClientInfo ci2(reservation, weight, 1.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      if (client1 == c) return ci1;
 +      else if (client2 == c) return ci2;
 +      else {
 +        ADD_FAILURE() << "got request from neither of two clients";
 +        return ci1; // must return
 +      }
 +      };
 +
 +      QueueRef pq(new Queue(client_info_f, false));
-       EXPECT_DEATH_IF_SUPPORTED(pq->add_request(req, client1, req_params),
 +      ReqParams req_params(1,1);
 +
-       EXPECT_DEATH_IF_SUPPORTED(pq->add_request(req, client2, req_params),
++      // Disable coredumps
++      PrCtl unset_dumpable;
++
++      EXPECT_DEATH_IF_SUPPORTED(pq->add_request(Request{}, client1, req_params),
 +                              "Assertion.*reservation.*max_tag.*"
 +                              "proportion.*max_tag") <<
 +      "we should fail if a client tries to generate a reservation tag "
 +      "where reservation and proportion are both 0";
 +
 +
-       Request req;
++      EXPECT_DEATH_IF_SUPPORTED(pq->add_request(Request{}, client2, req_params),
 +                              "Assertion.*reservation.*max_tag.*"
 +                              "proportion.*max_tag") <<
 +      "we should fail if a client tries to generate a reservation tag "
 +      "where reservation and proportion are both 0";
 +    }
 +
 +
 +    TEST(dmclock_server, client_idle_erase) {
 +      using ClientId = int;
 +      using Queue = dmc::PushPriorityQueue<ClientId,Request>;
 +      int client = 17;
 +      double reservation = 100.0;
 +
 +      dmc::ClientInfo ci(reservation, 1.0, 0.0);
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo { return ci; };
 +      auto server_ready_f = [] () -> bool { return true; };
 +      auto submit_req_f = [] (const ClientId& c,
 +                            std::unique_ptr<Request> req,
 +                            dmc::PhaseType phase) {
 +      // empty; do nothing
 +      };
 +
 +      Queue pq(client_info_f,
 +             server_ready_f,
 +             submit_req_f,
 +             std::chrono::seconds(3),
 +             std::chrono::seconds(5),
 +             std::chrono::seconds(2),
 +             false);
 +
 +      auto lock_pq = [&](std::function<void()> code) {
 +      test_locked(pq.data_mtx, code);
 +      };
 +
 +
 +      /* The timeline should be as follows:
 +       *
 +       *     0 seconds : request created
 +       *
 +       *     1 seconds : map is size 1, idle is false
 +       *
 +       * 2 seconds : clean notes first mark; +2 is base for further calcs
 +       *
 +       * 4 seconds : clean does nothing except makes another mark
 +       *
 +       *   5 seconds : when we're secheduled to idle (+2 + 3)
 +       *
 +       * 6 seconds : clean idles client
 +       *
 +       *   7 seconds : when we're secheduled to erase (+2 + 5)
 +       *
 +       *     7 seconds : verified client is idle
 +       *
 +       * 8 seconds : clean erases client info
 +       *
 +       *     9 seconds : verified client is erased
 +       */
 +
 +      lock_pq([&] () {
 +        EXPECT_EQ(0u, pq.client_map.size()) <<
 +          "client map initially has size 0";
 +      });
 +
 +      Request req;
 +      dmc::ReqParams req_params(1, 1);
 +      pq.add_request_time(req, client, req_params, dmc::get_time());
 +
 +      std::this_thread::sleep_for(std::chrono::seconds(1));
 +
 +      lock_pq([&] () {
 +        EXPECT_EQ(1u, pq.client_map.size()) <<
 +          "client map has 1 after 1 client";
 +        EXPECT_FALSE(pq.client_map.at(client)->idle) <<
 +          "initially client map entry shows not idle.";
 +      });
 +
 +      std::this_thread::sleep_for(std::chrono::seconds(6));
 +
 +      lock_pq([&] () {
 +        EXPECT_TRUE(pq.client_map.at(client)->idle) <<
 +          "after idle age client map entry shows idle.";
 +      });
 +
 +      std::this_thread::sleep_for(std::chrono::seconds(2));
 +
 +      lock_pq([&] () {
 +        EXPECT_EQ(0u, pq.client_map.size()) <<
 +          "client map loses its entry after erase age";
 +      });
 +    } // TEST
 +
 +
 +#if 0
 +    TEST(dmclock_server, reservation_timing) {
 +      using ClientId = int;
 +      // NB? PUSH OR PULL
 +      using Queue = std::unique_ptr<dmc::PriorityQueue<ClientId,Request>>;
 +      using std::chrono::steady_clock;
 +
 +      int client = 17;
 +
 +      std::vector<dmc::Time> times;
 +      std::mutex times_mtx;
 +      using Guard = std::lock_guard<decltype(times_mtx)>;
 +
 +      // reservation every second
 +      dmc::ClientInfo ci(1.0, 0.0, 0.0);
 +      Queue pq;
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo { return ci; };
 +      auto server_ready_f = [] () -> bool { return true; };
 +      auto submit_req_f = [&] (const ClientId& c,
 +                             std::unique_ptr<Request> req,
 +                             dmc::PhaseType phase) {
 +      {
 +        Guard g(times_mtx);
 +        times.emplace_back(dmc::get_time());
 +      }
 +      std::thread complete([&](){ pq->request_completed(); });
 +      complete.detach();
 +      };
 +
 +      // NB? PUSH OR PULL
 +      pq = Queue(new dmc::PriorityQueue<ClientId,Request>(client_info_f,
 +                                                        server_ready_f,
 +                                                        submit_req_f,
 +                                                        false));
 +
 +      Request req;
 +      ReqParams<ClientId> req_params(client, 1, 1);
 +
 +      for (int i = 0; i < 5; ++i) {
 +      pq->add_request_time(req, req_params, dmc::get_time());
 +      }
 +
 +      {
 +      Guard g(times_mtx);
 +      std::this_thread::sleep_for(std::chrono::milliseconds(5500));
 +      EXPECT_EQ(5, times.size()) <<
 +        "after 5.5 seconds, we should have 5 requests times at 1 second apart";
 +      }
 +    } // TEST
 +#endif
 +
 +
 +    TEST(dmclock_server, remove_by_req_filter) {
 +      struct MyReq {
 +      int id;
 +
 +      MyReq(int _id) :
 +        id(_id)
 +      {
 +        // empty
 +      }
 +      }; // MyReq
 +
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,MyReq>;
 +
 +      ClientId client1 = 17;
 +      ClientId client2 = 98;
 +
 +      dmc::ClientInfo info1(0.0, 1.0, 0.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info1;
 +      };
 +
 +      Queue pq(client_info_f, true);
 +
 +      EXPECT_EQ(0u, pq.client_count());
 +      EXPECT_EQ(0u, pq.request_count());
 +
 +      ReqParams req_params(1,1);
 +
 +      pq.add_request(MyReq(1), client1, req_params);
 +      pq.add_request(MyReq(11), client1, req_params);
 +      pq.add_request(MyReq(2), client2, req_params);
 +      pq.add_request(MyReq(0), client2, req_params);
 +      pq.add_request(MyReq(13), client2, req_params);
 +      pq.add_request(MyReq(2), client2, req_params);
 +      pq.add_request(MyReq(13), client2, req_params);
 +      pq.add_request(MyReq(98), client2, req_params);
 +      pq.add_request(MyReq(44), client1, req_params);
 +
 +      EXPECT_EQ(2u, pq.client_count());
 +      EXPECT_EQ(9u, pq.request_count());
 +
 +      pq.remove_by_req_filter([](const MyReq& r) -> bool {return 1 == r.id % 2;});
 +
 +      EXPECT_EQ(5u, pq.request_count());
 +
 +      std::list<MyReq> capture;
 +      pq.remove_by_req_filter(
 +      [&capture] (const MyReq& r) -> bool {
 +        if (0 == r.id % 2) {
 +          capture.push_front(r);
 +          return true;
 +        } else {
 +          return false;
 +        }
 +      },
 +      true);
 +
 +      EXPECT_EQ(0u, pq.request_count());
 +      EXPECT_EQ(5u, capture.size());
 +      int total = 0;
 +      for (auto i : capture) {
 +      total += i.id;
 +      }
 +      EXPECT_EQ(146, total) << " sum of captured items should be 146";
 +    } // TEST
 +
 +
 +    TEST(dmclock_server, remove_by_req_filter_ordering_forwards_visit) {
 +      struct MyReq {
 +      int id;
 +
 +      MyReq(int _id) :
 +        id(_id)
 +      {
 +        // empty
 +      }
 +      }; // MyReq
 +
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,MyReq>;
 +
 +      ClientId client1 = 17;
 +
 +      dmc::ClientInfo info1(0.0, 1.0, 0.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info1;
 +      };
 +
 +      Queue pq(client_info_f, true);
 +
 +      EXPECT_EQ(0u, pq.client_count());
 +      EXPECT_EQ(0u, pq.request_count());
 +
 +      ReqParams req_params(1,1);
 +
 +      pq.add_request(MyReq(1), client1, req_params);
 +      pq.add_request(MyReq(2), client1, req_params);
 +      pq.add_request(MyReq(3), client1, req_params);
 +      pq.add_request(MyReq(4), client1, req_params);
 +      pq.add_request(MyReq(5), client1, req_params);
 +      pq.add_request(MyReq(6), client1, req_params);
 +
 +      EXPECT_EQ(1u, pq.client_count());
 +      EXPECT_EQ(6u, pq.request_count());
 +
 +      // remove odd ids in forward order and append to end
 +
 +      std::vector<MyReq> capture;
 +      pq.remove_by_req_filter(
 +      [&capture] (const MyReq& r) -> bool {
 +        if (1 == r.id % 2) {
 +          capture.push_back(r);
 +          return true;
 +        } else {
 +          return false;
 +        }
 +      },
 +      false);
 +
 +      EXPECT_EQ(3u, pq.request_count());
 +      EXPECT_EQ(3u, capture.size());
 +      EXPECT_EQ(1, capture[0].id) << "items should come out in forward order";
 +      EXPECT_EQ(3, capture[1].id) << "items should come out in forward order";
 +      EXPECT_EQ(5, capture[2].id) << "items should come out in forward order";
 +
 +      // remove even ids in reverse order but insert at front so comes
 +      // out forwards
 +
 +      std::vector<MyReq> capture2;
 +      pq.remove_by_req_filter(
 +      [&capture2] (const MyReq& r) -> bool {
 +        if (0 == r.id % 2) {
 +          capture2.insert(capture2.begin(), r);
 +          return true;
 +        } else {
 +          return false;
 +        }
 +      },
 +      false);
 +
 +      EXPECT_EQ(0u, pq.request_count());
 +      EXPECT_EQ(3u, capture2.size());
 +      EXPECT_EQ(6, capture2[0].id) << "items should come out in reverse order";
 +      EXPECT_EQ(4, capture2[1].id) << "items should come out in reverse order";
 +      EXPECT_EQ(2, capture2[2].id) << "items should come out in reverse order";
 +    } // TEST
 +
 +
 +    TEST(dmclock_server, remove_by_req_filter_ordering_backwards_visit) {
 +      struct MyReq {
 +      int id;
 +
 +      MyReq(int _id) :
 +        id(_id)
 +      {
 +        // empty
 +      }
 +      }; // MyReq
 +
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,MyReq>;
 +
 +      ClientId client1 = 17;
 +
 +      dmc::ClientInfo info1(0.0, 1.0, 0.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info1;
 +      };
 +
 +      Queue pq(client_info_f, true);
 +
 +      EXPECT_EQ(0u, pq.client_count());
 +      EXPECT_EQ(0u, pq.request_count());
 +
 +      ReqParams req_params(1,1);
 +
 +      pq.add_request(MyReq(1), client1, req_params);
 +      pq.add_request(MyReq(2), client1, req_params);
 +      pq.add_request(MyReq(3), client1, req_params);
 +      pq.add_request(MyReq(4), client1, req_params);
 +      pq.add_request(MyReq(5), client1, req_params);
 +      pq.add_request(MyReq(6), client1, req_params);
 +
 +      EXPECT_EQ(1u, pq.client_count());
 +      EXPECT_EQ(6u, pq.request_count());
 +
 +      // now remove odd ids in forward order
 +
 +      std::vector<MyReq> capture;
 +      pq.remove_by_req_filter(
 +      [&capture] (const MyReq& r) -> bool {
 +        if (1 == r.id % 2) {
 +          capture.insert(capture.begin(), r);
 +          return true;
 +        } else {
 +          return false;
 +        }
 +      },
 +      true);
 +
 +      EXPECT_EQ(3u, pq.request_count());
 +      EXPECT_EQ(3u, capture.size());
 +      EXPECT_EQ(1, capture[0].id) << "items should come out in forward order";
 +      EXPECT_EQ(3, capture[1].id) << "items should come out in forward order";
 +      EXPECT_EQ(5, capture[2].id) << "items should come out in forward order";
 +
 +      // now remove even ids in reverse order
 +
 +      std::vector<MyReq> capture2;
 +      pq.remove_by_req_filter(
 +      [&capture2] (const MyReq& r) -> bool {
 +        if (0 == r.id % 2) {
 +          capture2.push_back(r);
 +          return true;
 +        } else {
 +          return false;
 +        }
 +      },
 +      true);
 +
 +      EXPECT_EQ(0u, pq.request_count());
 +      EXPECT_EQ(3u, capture2.size());
 +      EXPECT_EQ(6, capture2[0].id) << "items should come out in reverse order";
 +      EXPECT_EQ(4, capture2[1].id) << "items should come out in reverse order";
 +      EXPECT_EQ(2, capture2[2].id) << "items should come out in reverse order";
 +    } // TEST
 +
 +
 +    TEST(dmclock_server, remove_by_client) {
 +      struct MyReq {
 +      int id;
 +
 +      MyReq(int _id) :
 +        id(_id)
 +      {
 +        // empty
 +      }
 +      }; // MyReq
 +
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,MyReq>;
 +
 +      ClientId client1 = 17;
 +      ClientId client2 = 98;
 +
 +      dmc::ClientInfo info1(0.0, 1.0, 0.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info1;
 +      };
 +
 +      Queue pq(client_info_f, true);
 +
 +      EXPECT_EQ(0u, pq.client_count());
 +      EXPECT_EQ(0u, pq.request_count());
 +
 +      ReqParams req_params(1,1);
 +
 +      pq.add_request(MyReq(1), client1, req_params);
 +      pq.add_request(MyReq(11), client1, req_params);
 +      pq.add_request(MyReq(2), client2, req_params);
 +      pq.add_request(MyReq(0), client2, req_params);
 +      pq.add_request(MyReq(13), client2, req_params);
 +      pq.add_request(MyReq(2), client2, req_params);
 +      pq.add_request(MyReq(13), client2, req_params);
 +      pq.add_request(MyReq(98), client2, req_params);
 +      pq.add_request(MyReq(44), client1, req_params);
 +
 +      EXPECT_EQ(2u, pq.client_count());
 +      EXPECT_EQ(9u, pq.request_count());
 +
 +      std::list<MyReq> removed;
 +
 +      pq.remove_by_client(client1,
 +                        true,
 +                        [&removed] (const MyReq& r) {
 +                          removed.push_front(r);
 +                        });
 +
 +      EXPECT_EQ(3u, removed.size());
 +      EXPECT_EQ(1, removed.front().id);
 +      removed.pop_front();
 +      EXPECT_EQ(11, removed.front().id);
 +      removed.pop_front();
 +      EXPECT_EQ(44, removed.front().id);
 +      removed.pop_front();
 +
 +      EXPECT_EQ(6u, pq.request_count());
 +
 +      Queue::PullReq pr = pq.pull_request();
 +      EXPECT_TRUE(pr.is_retn());
 +      EXPECT_EQ(2, pr.get_retn().request->id);
 +
 +      pr = pq.pull_request();
 +      EXPECT_TRUE(pr.is_retn());
 +      EXPECT_EQ(0, pr.get_retn().request->id);
 +
 +      pq.remove_by_client(client2);
 +      EXPECT_EQ(0u, pq.request_count()) <<
 +      "after second client removed, none left";
 +    } // TEST
 +
 +
 +    TEST(dmclock_server_pull, pull_weight) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      ClientId client1 = 17;
 +      ClientId client2 = 98;
 +
 +      dmc::ClientInfo info1(0.0, 1.0, 0.0);
 +      dmc::ClientInfo info2(0.0, 2.0, 0.0);
 +
 +      QueueRef pq;
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      if (client1 == c) return info1;
 +      else if (client2 == c) return info2;
 +      else {
 +        ADD_FAILURE() << "client info looked up for non-existant client";
 +        return info1;
 +      }
 +      };
 +
 +      pq = QueueRef(new Queue(client_info_f, false));
 +
-       pq->add_request(req, client1, req_params);
-       pq->add_request(req, client2, req_params);
 +      ReqParams req_params(1,1);
 +
 +      auto now = dmc::get_time();
 +
 +      for (int i = 0; i < 5; ++i) {
-       Request req;
++      pq->add_request(Request{}, client1, req_params);
++      pq->add_request(Request{}, client2, req_params);
 +      now += 0.0001;
 +      }
 +
 +      int c1_count = 0;
 +      int c2_count = 0;
 +      for (int i = 0; i < 6; ++i) {
 +      Queue::PullReq pr = pq->pull_request();
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +      auto& retn = boost::get<Queue::PullReq::Retn>(pr.data);
 +
 +      if (client1 == retn.client) ++c1_count;
 +      else if (client2 == retn.client) ++c2_count;
 +      else ADD_FAILURE() << "got request from neither of two clients";
 +
 +      EXPECT_EQ(PhaseType::priority, retn.phase);
 +      }
 +
 +      EXPECT_EQ(2, c1_count) <<
 +      "one-third of request should have come from first client";
 +      EXPECT_EQ(4, c2_count) <<
 +      "two-thirds of request should have come from second client";
 +    }
 +
 +
 +    TEST(dmclock_server_pull, pull_reservation) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      ClientId client1 = 52;
 +      ClientId client2 = 8;
 +
 +      dmc::ClientInfo info1(2.0, 0.0, 0.0);
 +      dmc::ClientInfo info2(1.0, 0.0, 0.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      if (client1 == c) return info1;
 +      else if (client2 == c) return info2;
 +      else {
 +        ADD_FAILURE() << "client info looked up for non-existant client";
 +        return info1;
 +      }
 +      };
 +
 +      QueueRef pq(new Queue(client_info_f, false));
 +
-       pq->add_request_time(req, client1, req_params, old_time);
-       pq->add_request_time(req, client2, req_params, old_time);
 +      ReqParams req_params(1,1);
 +
 +      // make sure all times are well before now
 +      auto old_time = dmc::get_time() - 100.0;
 +
 +      for (int i = 0; i < 5; ++i) {
-       Request req;
++      pq->add_request_time(Request{}, client1, req_params, old_time);
++      pq->add_request_time(Request{}, client2, req_params, old_time);
 +      old_time += 0.001;
 +      }
 +
 +      int c1_count = 0;
 +      int c2_count = 0;
 +
 +      for (int i = 0; i < 6; ++i) {
 +      Queue::PullReq pr = pq->pull_request();
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +      auto& retn = boost::get<Queue::PullReq::Retn>(pr.data);
 +
 +      if (client1 == retn.client) ++c1_count;
 +      else if (client2 == retn.client) ++c2_count;
 +      else ADD_FAILURE() << "got request from neither of two clients";
 +
 +      EXPECT_EQ(PhaseType::reservation, retn.phase);
 +      }
 +
 +      EXPECT_EQ(4, c1_count) <<
 +      "two-thirds of request should have come from first client";
 +      EXPECT_EQ(2, c2_count) <<
 +      "one-third of request should have come from second client";
 +    } // dmclock_server_pull.pull_reservation
 +
 +
 +    // This test shows what happens when a request can be ready (under
 +    // limit) but not schedulable since proportion tag is 0. We expect
 +    // to get some future and none responses.
 +    TEST(dmclock_server_pull, ready_and_under_limit) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      ClientId client1 = 52;
 +      ClientId client2 = 8;
 +
 +      dmc::ClientInfo info1(1.0, 0.0, 0.0);
 +      dmc::ClientInfo info2(1.0, 0.0, 0.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      if (client1 == c) return info1;
 +      else if (client2 == c) return info2;
 +      else {
 +        ADD_FAILURE() << "client info looked up for non-existant client";
 +        return info1;
 +      }
 +      };
 +
 +      QueueRef pq(new Queue(client_info_f, false));
 +
-       pq->add_request_time(req, client1, req_params, start_time);
-       pq->add_request_time(req, client2, req_params, start_time);
 +      ReqParams req_params(1,1);
 +
 +      // make sure all times are well before now
 +      auto start_time = dmc::get_time() - 100.0;
 +
 +      // add six requests; for same client reservations spaced one apart
 +      for (int i = 0; i < 3; ++i) {
-       Request req;
++      pq->add_request_time(Request{}, client1, req_params, start_time);
++      pq->add_request_time(Request{}, client2, req_params, start_time);
 +      }
 +
 +      Queue::PullReq pr = pq->pull_request(start_time + 0.5);
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      pr = pq->pull_request(start_time + 0.5);
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      pr = pq->pull_request(start_time + 0.5);
 +      EXPECT_EQ(Queue::NextReqType::future, pr.type) <<
 +      "too soon for next reservation";
 +
 +      pr = pq->pull_request(start_time + 1.5);
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      pr = pq->pull_request(start_time + 1.5);
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      pr = pq->pull_request(start_time + 1.5);
 +      EXPECT_EQ(Queue::NextReqType::future, pr.type) <<
 +      "too soon for next reservation";
 +
 +      pr = pq->pull_request(start_time + 2.5);
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      pr = pq->pull_request(start_time + 2.5);
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      pr = pq->pull_request(start_time + 2.5);
 +      EXPECT_EQ(Queue::NextReqType::none, pr.type) << "no more requests left";
 +    }
 +
 +
 +    TEST(dmclock_server_pull, pull_none) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      dmc::ClientInfo info(1.0, 1.0, 1.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info;
 +      };
 +
 +      QueueRef pq(new Queue(client_info_f, false));
 +
 +      // Request req;
 +      ReqParams req_params(1,1);
 +
 +      auto now = dmc::get_time();
 +
 +      Queue::PullReq pr = pq->pull_request(now + 100);
 +
 +      EXPECT_EQ(Queue::NextReqType::none, pr.type);
 +    }
 +
 +
 +    TEST(dmclock_server_pull, pull_future) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      ClientId client1 = 52;
 +      // ClientId client2 = 8;
 +
 +      dmc::ClientInfo info(1.0, 0.0, 1.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info;
 +      };
 +
 +      QueueRef pq(new Queue(client_info_f, false));
 +
-       pq->add_request_time(req, client1, req_params, now + 100);
 +      ReqParams req_params(1,1);
 +
 +      // make sure all times are well before now
 +      auto now = dmc::get_time();
 +
-       Request req;
++      pq->add_request_time(Request{}, client1, req_params, now + 100);
 +      Queue::PullReq pr = pq->pull_request(now);
 +
 +      EXPECT_EQ(Queue::NextReqType::future, pr.type);
 +
 +      Time when = boost::get<Time>(pr.data);
 +      EXPECT_EQ(now + 100, when);
 +    }
 +
 +
 +    TEST(dmclock_server_pull, pull_future_limit_break_weight) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      ClientId client1 = 52;
 +      // ClientId client2 = 8;
 +
 +      dmc::ClientInfo info(0.0, 1.0, 1.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info;
 +      };
 +
 +      QueueRef pq(new Queue(client_info_f, true));
 +
-       pq->add_request_time(req, client1, req_params, now + 100);
 +      ReqParams req_params(1,1);
 +
 +      // make sure all times are well before now
 +      auto now = dmc::get_time();
 +
-       Request req;
++      pq->add_request_time(Request{}, client1, req_params, now + 100);
 +      Queue::PullReq pr = pq->pull_request(now);
 +
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      auto& retn = boost::get<Queue::PullReq::Retn>(pr.data);
 +      EXPECT_EQ(client1, retn.client);
 +    }
 +
 +
 +    TEST(dmclock_server_pull, pull_future_limit_break_reservation) {
 +      using ClientId = int;
 +      using Queue = dmc::PullPriorityQueue<ClientId,Request>;
 +      using QueueRef = std::unique_ptr<Queue>;
 +
 +      ClientId client1 = 52;
 +      // ClientId client2 = 8;
 +
 +      dmc::ClientInfo info(1.0, 0.0, 1.0);
 +
 +      auto client_info_f = [&] (ClientId c) -> dmc::ClientInfo {
 +      return info;
 +      };
 +
 +      QueueRef pq(new Queue(client_info_f, true));
 +
-       pq->add_request_time(req, client1, req_params, now + 100);
 +      ReqParams req_params(1,1);
 +
 +      // make sure all times are well before now
 +      auto now = dmc::get_time();
 +
++      pq->add_request_time(Request{}, client1, req_params, now + 100);
 +      Queue::PullReq pr = pq->pull_request(now);
 +
 +      EXPECT_EQ(Queue::NextReqType::returning, pr.type);
 +
 +      auto& retn = boost::get<Queue::PullReq::Retn>(pr.data);
 +      EXPECT_EQ(client1, retn.client);
 +    }
 +  } // namespace dmclock
 +} // namespace crimson