Solved issue #14: to avoid rounding problems, ResCloudlet processes length in…

Solved issue #14: to avoid rounding problems, ResCloudlet processes length in Instructions rather than Millions of Instructions.

modules/cloudsim/src/main/java/org/cloudbus/cloudsim/CloudletSchedulerDynamicWorkload.java

@@ -76,7 +76,7 @@ public class CloudletSchedulerDynamicWorkload extends CloudletSchedulerTimeShare
 		List<ResCloudlet> cloudletsToFinish = new ArrayList<ResCloudlet>();
 
 		for (ResCloudlet rcl : getCloudletExecList()) {
-			rcl.updateCloudletFinishedSoFar((long) Math.ceil((timeSpan * getTotalCurrentAllocatedMipsForCloudlet(rcl, getPreviousTime()))));
+			rcl.updateCloudletFinishedSoFar((long) (timeSpan * getTotalCurrentAllocatedMipsForCloudlet(rcl, getPreviousTime()) * 1000000));
 
             if (rcl.getRemainingCloudletLength() == 0) { //finished: remove from the list
             	cloudletsToFinish.add(rcl);

modules/cloudsim/src/main/java/org/cloudbus/cloudsim/CloudletSchedulerSpaceShared.java

@@ -91,7 +91,7 @@ public class CloudletSchedulerSpaceShared extends CloudletScheduler {
 		capacity /= cpus; // average capacity of each cpu
 
 		for (ResCloudlet rcl : getCloudletExecList()) { // each machine in the exec list has the same amount of cpu
-			rcl.updateCloudletFinishedSoFar((long) Math.ceil((capacity * timeSpam * rcl.getPesNumber())));
+			rcl.updateCloudletFinishedSoFar((long) (capacity * timeSpam * rcl.getPesNumber() * 1000000));
 		}
 
 		if (getCloudletExecList().size() == 0 && getCloudletWaitingList().size() == 0) { // no more cloudlets in this scheduler

modules/cloudsim/src/main/java/org/cloudbus/cloudsim/CloudletSchedulerTimeShared.java

@@ -69,7 +69,7 @@ public class CloudletSchedulerTimeShared extends CloudletScheduler {
 		double timeSpam = currentTime - getPreviousTime();
 
 		for (ResCloudlet rcl : getCloudletExecList()) {
-			rcl.updateCloudletFinishedSoFar((long) Math.ceil((getCapacity(mipsShare) * timeSpam * rcl.getPesNumber())));
+			rcl.updateCloudletFinishedSoFar((long) (getCapacity(mipsShare) * timeSpam * rcl.getPesNumber() * 1000000));
 		}
 
 		if (getCloudletExecList().size() == 0) {

modules/cloudsim/src/main/java/org/cloudbus/cloudsim/ResCloudlet.java

@@ -227,7 +227,7 @@ public class ResCloudlet {
 
         // In case a Cloudlet has been executed partially by some other grid
         // hostList.
-        this.cloudletFinishedSoFar = cloudlet.getCloudletFinishedSoFar();
+        this.cloudletFinishedSoFar = cloudlet.getCloudletFinishedSoFar()*1000000;
     }
 
     /**
@@ -471,15 +471,14 @@ public class ResCloudlet {
      * @post $result >= 0
      */
     public long getRemainingCloudletLength() {
-        long length = cloudlet.getCloudletTotalLength() - cloudletFinishedSoFar;
+        long length = cloudlet.getCloudletTotalLength()*1000000 - cloudletFinishedSoFar;
 
-        // Remaining Cloudlet length can't be negative number. This can be
-        // happening when this.updateCloudletFinishedSoFar() keep calling.
+        // Remaining Cloudlet length can't be negative number.
         if (length < 0) {
-            length = 0;
+            return 0;
         }
 
-        return length;
+        return (long) Math.ceil(length/1000000);
     }
 
     /**
@@ -502,11 +501,11 @@ public class ResCloudlet {
         cloudlet.setExecParam(wallClockTime, totalCompletionTime);
 
         long finished = 0;
-        if (cloudlet.getCloudletTotalLength() < cloudletFinishedSoFar) {
+        if (cloudlet.getCloudletTotalLength()*1000000 < cloudletFinishedSoFar) {
             finished = cloudlet.getCloudletLength();
         }
         else {
-            finished = cloudletFinishedSoFar;
+            finished = cloudletFinishedSoFar/1000000;
         }
 
         cloudlet.setCloudletFinishedSoFar(finished);
@@ -515,7 +514,7 @@ public class ResCloudlet {
     /**
      * A method that updates the length of cloudlet that has been completed.
      *
-     * @param miLength cloudlet length in Million Instructions (MI)
+     * @param miLength cloudlet length in Instructions (I)
      *
      * @pre miLength >= 0.0
      * @post $none

modules/cloudsim/src/main/java/org/cloudbus/cloudsim/Vm.java

@@ -12,8 +12,6 @@ import java.util.ArrayList;
 import java.util.LinkedList;
 import java.util.List;
 
-import org.cloudbus.cloudsim.core.CloudSim;
-
 /**
  * Vm represents a VM: it runs inside a Host, sharing hostList
  * with other VMs. It processes cloudlets. This processing happens according
@@ -136,6 +134,7 @@ public class Vm {
 	 * @post $none
 	 */
 	public double updateVmProcessing(double currentTime, List<Double> mipsShare) {
+		setRecentlyCreated(false);
 		if (mipsShare != null) {
 			return getCloudletScheduler().updateVmProcessing(currentTime, mipsShare);
 		}
@@ -154,7 +153,7 @@ public class Vm {
 			//boolean mipsIsNull = true;
 			//if (CloudSim.clock() > 0) {
 				//mipsIsNull = false;
-				setRecentlyCreated(false);
+				//
 			//}
 //			for (double mips : currentRequestedMips) {
 //				if (mips > 0.0) {