From: Patrick Bridges <bridges@apu.cs.unm.edu>
Date: Tue, 24 May 2011 17:01:43 +0000 (-0600)
Subject: Print revision to 8254.c
X-Git-Url: http://v3vee.org/palacios/gitweb/gitweb.cgi?a=commitdiff_plain;h=b78d577da5b43dfe1da69e57e3b24faa4d9dc465;p=palacios.git

Print revision to 8254.c
---

diff --git a/palacios/src/devices/8254.c b/palacios/src/devices/8254.c
index e01eeda..fee0e1a 100644
--- a/palacios/src/devices/8254.c
+++ b/palacios/src/devices/8254.c
@@ -133,7 +133,8 @@ static int handle_crystal_tics(struct pit * pit, struct channel * ch, uint_t osc
     uint_t channel_cycles = 0;
     uint_t output_changed = 0;
   
-    // PrintDebug("8254 PIT: %d crystal tics\n", oscillations);
+    //    PrintDebug("8254 PIT (channel %d): %d crystal tics\n", 
+    //	       ch - pit->ch0, oscillations);
     if (ch->run_state == PENDING) {
 	oscillations--;
 	ch->counter = ch->reload_value;
@@ -158,6 +159,8 @@ static int handle_crystal_tics(struct pit * pit, struct channel * ch, uint_t osc
 
     if (ch->counter > oscillations) {
 	ch->counter -= oscillations;
+	//PrintDebug("8254 PIT: Counter at %u after %u oscillations.\n", 
+	//   (unsigned int)ch->counter, oscillations);
 	return output_changed;
     } else {
 	ushort_t reload_val = ch->reload_value; 
@@ -184,23 +187,33 @@ static int handle_crystal_tics(struct pit * pit, struct channel * ch, uint_t osc
 	oscillations = oscillations % reload_val;
 
 	ch->counter = reload_val - oscillations;
+	//	PrintDebug("8254 PIT: Counter reset to %u.\n", 
+	//   (unsigned int)ch->counter);
+
     }
 
-    //  PrintDebug("8254 PIT: Channel Cycles: %d\n", channel_cycles);
+    //PrintDebug("8254 PIT: Channel %ld (mode = %u) Cycles: %d\n", 
+	       //(ch - &pit->ch_0), ch->op_mode, channel_cycles);
   
-
-
     switch (ch->op_mode) {
 	case IRQ_ON_TERM_CNT:
-	    if ((channel_cycles > 0) && (ch->output_pin == 0)) {
-		ch->output_pin = 1; 
-		output_changed = 1;
+	    if (channel_cycles > 0) {
+	        if (ch->output_pin == 0) {
+		    ch->output_pin = 1; 
+		    output_changed = 1;
+	        } else {
+		   // PrintDebug("8254: Output not changed in TERM_CNT mode.\n");
+	        }
 	    }
 	    break;
 	case ONE_SHOT:
-	    if ((channel_cycles > 0) && (ch->output_pin == 0)) {
-		ch->output_pin = 1; 
-		output_changed = 1;
+	    if (channel_cycles > 0) {
+	        if ((ch->output_pin == 0)) {
+		    ch->output_pin = 1; 
+		    output_changed = 1;
+		} else {
+		    // PrintDebug("8254: Output not changed in ONE_SHOT mode.\n");
+		}
 	    }
 	    break;
 	case RATE_GEN:
@@ -216,6 +229,7 @@ static int handle_crystal_tics(struct pit * pit, struct channel * ch, uint_t osc
 		output_changed = 1;
 	    }
 
+
 	    break;
 	case SW_STROBE:
 
@@ -300,21 +314,19 @@ static void pit_update_timer(struct guest_info * info, ullong_t cpu_cycles, ullo
 	state->pit_counter = state->pit_reload - cpu_cycles;    
 
 	if (oscillations) {
-	    PrintDebug("8254 PIT: Handling %d crystal tics\n", oscillations);
-	}
+	    //    PrintDebug("8254 PIT: Handling %d crystal tics\n", oscillations);
 
-	if (handle_crystal_tics(state, &(state->ch_0), oscillations) == 1) {
-	    // raise interrupt
-	    // PrintDebug("8254 PIT: Injecting Timer interrupt to guest\n");
-	    v3_raise_irq(info->vm_info, 0);
-	}
+	    if (handle_crystal_tics(state, &(state->ch_0), oscillations) == 1) {
+	        // raise interrupt
+	        PrintDebug("8254 PIT: Injecting Timer interrupt to guest (run_state = %d)\n",
+			   state->ch_0.run_state);
+	        v3_raise_irq(info->vm_info, 0);
+	    }
 
-	//handle_crystal_tics(state, &(state->ch_1), oscillations);
-	handle_crystal_tics(state, &(state->ch_2), oscillations);
+	    //handle_crystal_tics(state, &(state->ch_1), oscillations);
+	    handle_crystal_tics(state, &(state->ch_2), oscillations);
+	}
     }
-  
-
-
  
     return;
 }
@@ -430,11 +442,21 @@ static int handle_speaker_write(uint8_t *speaker, struct channel * ch, char val)
 }
 
 static int handle_channel_cmd(struct channel * ch, struct pit_cmd_word cmd) {
-    ch->op_mode = cmd.op_mode;
-    ch->access_mode = cmd.access_mode;
 
+    if (cmd.op_mode != ch->op_mode) {
+	PrintDebug("8254 PIT: Changing channel from op mode %d to op mode %d.\n", 
+	 	   ch->op_mode, cmd.op_mode);
+    }
 
+    if (cmd.access_mode != 0) {
+      ch->op_mode = cmd.op_mode;
+    }
 
+    if (cmd.access_mode != ch->access_mode) {
+	PrintDebug("8254 PIT: Changing channel from access mode %d to access mode %d.\n", 
+		   ch->access_mode, cmd.access_mode);
+    }
+    ch->access_mode = cmd.access_mode;
 
     switch (cmd.access_mode) {
 	case LATCH_COUNT:
@@ -586,6 +608,11 @@ static int pit_write_command(struct guest_info * core, ushort_t port, void * src
 	return -1;
     }
 
+    if (cmd->op_mode == 0) {
+      V3_Print("SETTING PIT MODE TO 0!!!!!\n");
+      v3_print_guest_state(core);
+    }
+
     switch (cmd->channel) {
 	case 0:
 	    if (handle_channel_cmd(&(state->ch_0), *cmd) == -1) {