Changes between Version 9 and Version 10 of 802.11/wlan_exp/app_notes/tutorial_token_mac/CPU_HIGH

Timestamp:

Jul 15, 2015, 11:04:59 AM (9 years ago)

Author:

chunter

Comment:

--

802.11/wlan_exp/app_notes/tutorial_token_mac/CPU_HIGH

@@ -35 +35 @@
 typedef struct{
         token_end_reservation_reason_t reason;
-        wlan_mac_low_tx_details low_tx_details;
 } ipc_token_end_reservation;
 }}}
@@ -68 +67 @@
         ipc_msg_to_low.msg_id            = IPC_MBOX_MSG_ID(IPC_MBOX_TOKEN_NEW_RESERVATION);
 
+        //The below chunk of code is just to make sure that we pad an extra u32 word if the
+        //ipc_token_new_reservation struct is not 32-bit aligned.
         if( (sizeof(u32)*(sizeof(ipc_token_new_reservation)/sizeof(u32))) ==  sizeof(ipc_token_new_reservation) ){
                 ipc_msg_to_low.num_payload_words = (sizeof(ipc_token_new_reservation)/sizeof(u32));
@@ -76 +77 @@
         ipc_msg_to_low.payload_ptr       = (u32*)(&ipc_payload);
 
-
+        //For safety, we will go ahead and disable interrupts while we are processing this function. This function
+        //iterates through the doubly-linked list of associated stations. If that list were to be modified by an interrupt
+        //context while this function is running, there may be unexpected results. By disabling interrupts, we know
+        //that we can trust the doubly-linked list for this entire function context
         curr_interrupt_state = wlan_mac_high_interrupt_stop();
-
-        // Send an IPC message down to CPU_LOW to let it know we are moving to a new
-        // reservation period.
 
         curr_station_info_entry = next_station_info_entry;
@@ -98 +99 @@
 
         } else {
+                // It's a STA's reservation
                 curr_station_info = (station_info*)(curr_station_info_entry->data);
                 if( wlan_mac_high_is_valid_association(&my_bss_info->associated_stations, curr_station_info) ){
@@ -191 +193 @@
 ----
 
-