- Minor modifications to step_forward()
git-svn-id: svn://svn.gna.org/svn/sgpemv2/trunk@711 3ecf2c5c-341e-0410-92b4-d18e462d057c
This commit is contained in:
parent
401c569a9f
commit
5ab575dffd
|
@ -202,7 +202,6 @@ pkginclude_HEADERS += \
|
|||
src/backend/process.hh \
|
||||
src/backend/schedulable.hh \
|
||||
src/backend/scheduler.hh \
|
||||
src/backend/slice.hh \
|
||||
src/backend/sub_request.hh \
|
||||
src/backend/thread.hh \
|
||||
src/backend/user_interrupt_exception.hh
|
||||
|
|
|
@ -116,127 +116,133 @@ Scheduler::get_policy()
|
|||
void
|
||||
Scheduler::step_forward(History& history, Policy& cpu_policy) throw(UserInterruptException)
|
||||
{
|
||||
// // This very method should be exclusive: no concurrent behaviour, from when we
|
||||
// // store a readyqueue and policy pointer for the user-policy to retrieve, to when
|
||||
// // the policy returns
|
||||
// // TODO: restrict this area to maximise parallelism
|
||||
// Glib::Mutex::Lock lock(_mutex);
|
||||
// This very method should be exclusive: no concurrent behaviour, from when we
|
||||
// store a readyqueue and policy pointer for the user-policy to retrieve, to when
|
||||
// the policy returns
|
||||
// TODO: restrict this area to maximise parallelism
|
||||
Glib::Mutex::Lock lock(_mutex);
|
||||
|
||||
// // NOTE: Be sure to read the *ORIGINAL* documentation in the design document for this method!
|
||||
// NOTE: Be sure to read the *ORIGINAL* documentation in the design document for this method!
|
||||
|
||||
// // FIXME: handle me! I'm not just a pretty boolean, I want to be *USED*! *EXPLOITED*!
|
||||
// // *RAPED*! *MAKE ME BLEED*!
|
||||
// bool simulation_ended = true; // Assume we've finished. Then prove me wrong.
|
||||
// FIXME: handle me! I'm not just a pretty boolean, I want to be *USED*! *EXPLOITED*!
|
||||
// *RAPED*! *MAKE ME BLEED*!
|
||||
bool simulation_ended = true; // Assume we've finished. Then prove me wrong.
|
||||
|
||||
// ConcreteHistory& concrete_history = (ConcreteHistory&) history;
|
||||
//
|
||||
// // Use an auto_ptr since we've some exceptions in the coming...
|
||||
// auto_ptr<ConcreteEnvironment> new_snapshot(new ConcreteEnvironment(concrete_history.get_last_environment()));
|
||||
//
|
||||
// typedef std::vector<DynamicProcess*> Processes;
|
||||
// typedef std::vector<DynamicRequest*> Requests;
|
||||
// typedef std::vector<DynamicSubRequest*> SubRequests;
|
||||
// typedef std::vector<DynamicThread*> Threads;
|
||||
//
|
||||
// Threads all_threads;
|
||||
// DynamicThread* running_thread = NULL;
|
||||
ConcreteHistory& concrete_history = (ConcreteHistory&) history;
|
||||
|
||||
// Use an auto_ptr since we've some exceptions in the coming...
|
||||
auto_ptr<ConcreteEnvironment> new_snapshot(new ConcreteEnvironment(concrete_history.get_last_environment()));
|
||||
|
||||
typedef std::vector<DynamicProcess*> Processes;
|
||||
typedef std::vector<DynamicRequest*> Requests;
|
||||
typedef std::vector<DynamicSubRequest*> SubRequests;
|
||||
typedef std::vector<DynamicThread*> Threads;
|
||||
|
||||
Threads all_threads;
|
||||
DynamicThread* running_thread = NULL;
|
||||
|
||||
// collect_threads(new_snapshot->get_processes(), all_threads);
|
||||
collect_threads(new_snapshot->get_processes(), all_threads);
|
||||
|
||||
// // designer + implementer (Matteo) comment follows:
|
||||
// designer + implementer (Matteo) comment follows:
|
||||
|
||||
// for(Threads::iterator it = all_threads.begin(); it != all_threads.end(); it++)
|
||||
// {
|
||||
// DynamicThread& current = **it;
|
||||
//
|
||||
// // 1. mark future threads as ready, if appropriate
|
||||
// if(current.get_state() == Schedulable::state_future)
|
||||
// {
|
||||
// Process& parent = current.get_process();
|
||||
// if(parent.get_elapsed_time() == current.get_arrival_time())
|
||||
// current.set_state(Schedulable::state_ready);
|
||||
// }
|
||||
//
|
||||
// // Save the current running thread for future usage, if it hasn't ended
|
||||
// // its allotted time
|
||||
// if(current.get_state() == Schedulable::state_running)
|
||||
// {
|
||||
// running_thread = ¤t; // Even if we change its state to terminated
|
||||
// // 2. mark threads that used all their allotted time as terminated
|
||||
// if(current.get_total_cpu_time() - current.get_elapsed_time() == 0)
|
||||
// current.set_state(Schedulable::state_terminated);
|
||||
// }
|
||||
for(Threads::iterator it = all_threads.begin(); it != all_threads.end(); it++)
|
||||
{
|
||||
DynamicThread& current = **it;
|
||||
|
||||
// 1. mark future threads as ready, if appropriate
|
||||
if(current.get_state() == Schedulable::state_future)
|
||||
{
|
||||
Process& parent = current.get_process();
|
||||
if(parent.get_elapsed_time() == current.get_arrival_time())
|
||||
current.set_state(Schedulable::state_ready);
|
||||
}
|
||||
|
||||
// Save the current running thread for future usage, if it hasn't ended
|
||||
// its allotted time
|
||||
if(current.get_state() == Schedulable::state_running)
|
||||
{
|
||||
// increasing the time elapsed of the running thread + process
|
||||
// should be done here as the first thing, instead than
|
||||
// directly after selecting them
|
||||
running_thread->decrease_remaining_time();
|
||||
|
||||
// // 3. check for simulation termination (we can directly use threads
|
||||
// // for this check, since processes' state is based upon threads' one)
|
||||
// if( /* we still think that */ simulation_ended &&
|
||||
// (current.get_state() & (Schedulable::state_blocked |
|
||||
// Schedulable::state_terminated)) == 0)
|
||||
// simulation_ended = false;
|
||||
// }
|
||||
running_thread = ¤t; // Even if we change its state to terminated
|
||||
// 2. mark threads that used all their allotted time as terminated
|
||||
if(current.get_total_cpu_time() - current.get_elapsed_time() == 0)
|
||||
current.set_state(Schedulable::state_terminated);
|
||||
}
|
||||
|
||||
// // What to do now if the simulation ended?
|
||||
// 3. check for simulation termination (we can directly use threads
|
||||
// for this check, since processes' state is based upon threads' one)
|
||||
if( /* we still think that */ simulation_ended &&
|
||||
(current.get_state() & (Schedulable::state_blocked |
|
||||
Schedulable::state_terminated)) == 0)
|
||||
simulation_ended = false;
|
||||
}
|
||||
|
||||
|
||||
// // FIXME: increasing the time elapsed of the running thread + process
|
||||
// // should maybe be done here as the first thing, instead than
|
||||
// // directly when selecting them
|
||||
// if(running_thread != NULL)
|
||||
// running_thread->decrease_remaining_time();
|
||||
//
|
||||
// // 4a. Requests for the running thread exhausted
|
||||
// if(running_thread != NULL) {
|
||||
// Requests& reqs = running_thread->get_dynamic_requests();
|
||||
//
|
||||
// // FIXME we lack a way to tell and/or remember for how
|
||||
// // much a subrequest has been being fulfilled
|
||||
// // THIS MEANS this part is NOT complete
|
||||
// // We should check if a request has been fulfilled
|
||||
// ---------- FIXME ----------------
|
||||
// What to do now if the simulation ended?
|
||||
|
||||
// // FIXME If a request was being fulfilled to the running thread,
|
||||
// // we should decrease the request remaining time here.
|
||||
|
||||
// // This is why we kept a ref to the old running thread,
|
||||
// // even if it was terminated
|
||||
// if(running_thread->get_state() == Schedulable::state_terminated)
|
||||
// free_all_resources_of(*running_thread); // this function isn't complete
|
||||
//
|
||||
// }
|
||||
//
|
||||
|
||||
// 4a. Requests for the running thread exhausted
|
||||
if(running_thread != NULL) {
|
||||
Requests& reqs = running_thread->get_dynamic_requests();
|
||||
if(running_thread->get_state() == Schedulable::state_terminated)
|
||||
{
|
||||
// for(Requests::iterator it = reqs.begin();
|
||||
|
||||
|
||||
// FIXME we lack a way to tell and/or remember for how
|
||||
// much a subrequest has been being fulfilled
|
||||
// THIS MEANS this part is NOT complete
|
||||
// We should check if a request has been fulfilled
|
||||
|
||||
// // /
|
||||
// // /
|
||||
// // /
|
||||
// // (I'M HERE) < * * * * * * * * * * *
|
||||
// // \
|
||||
// // \
|
||||
// // \
|
||||
// //
|
||||
// // (is it visible enough for you?)
|
||||
// FIXME If a request was being fulfilled to the running thread,
|
||||
// we should decrease the request remaining time here.
|
||||
|
||||
//
|
||||
// ReadyQueue& ready_queue = new_snapshot->get_sorted_queue();
|
||||
// prepare_ready_queue(ready_queue);
|
||||
// try
|
||||
// {
|
||||
// // ?. Use the policy to sort the queue
|
||||
// This is why we kept a ref to the old running thread,
|
||||
// even if it was terminated
|
||||
|
||||
free_all_resources_of(*running_thread); // this function isn't complete
|
||||
|
||||
}
|
||||
|
||||
|
||||
// // FIXME: how does it get the queue?
|
||||
// cpu_policy.sort_queue();
|
||||
// }
|
||||
// catch(UserInterruptException& e)
|
||||
// {
|
||||
// _policy_manager.init();
|
||||
// // ^^^^^
|
||||
// // Do we need to update something else?
|
||||
// /
|
||||
// /
|
||||
// /
|
||||
// (I'M HERE) < * * * * * * * * * * *
|
||||
// \
|
||||
// \
|
||||
// \
|
||||
//
|
||||
// (is it visible enough for you?)
|
||||
|
||||
// // Going up unwinding the stack, tell:
|
||||
// // - the user that the policy sucks
|
||||
// // - SimulationController that everything stopped
|
||||
// throw;
|
||||
// }
|
||||
//
|
||||
// // append the new snapshot...
|
||||
// // ...and remember to release the auto_ptr!
|
||||
// concrete_history.append_new_environment(new_snapshot.release());
|
||||
|
||||
ReadyQueue& ready_queue = new_snapshot->get_sorted_queue();
|
||||
prepare_ready_queue(ready_queue);
|
||||
try
|
||||
{
|
||||
// ?. Use the policy to sort the queue
|
||||
|
||||
// FIXME: how does it get the queue?
|
||||
cpu_policy.sort_queue();
|
||||
}
|
||||
catch(UserInterruptException& e)
|
||||
{
|
||||
_policy_manager.init();
|
||||
// ^^^^^
|
||||
// Do we need to update something else?
|
||||
|
||||
// Going up unwinding the stack, tell:
|
||||
// - the user that the policy sucks
|
||||
// - SimulationController that everything stopped
|
||||
throw;
|
||||
}
|
||||
|
||||
// append the new snapshot...
|
||||
// ...and remember to release the auto_ptr!
|
||||
concrete_history.append_new_environment(new_snapshot.release());
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue