Comments for Henri.

diff --git a/src/input/input_netlist.c b/src/input/input_netlist.c
index 816830c43fb3416f4cc91d878b5e40a9a5fed5c8..b1a1c7cf2ee4392bc88acaf0a817994977c8726e 100644 (file)
--- a/src/input/input_netlist.c
+++ b/src/input/input_netlist.c
@@ -2,7 +2,7 @@
  * input_netlist.c: netlist management
  *****************************************************************************
  * Copyright (C) 1998, 1999, 2000 VideoLAN
- * $Id: input_netlist.c,v 1.23 2000/12/29 14:03:44 henri Exp $
+ * $Id: input_netlist.c,v 1.24 2000/12/30 00:38:19 massiot Exp $
  *
  * Authors: Henri Fallon <henri@videolan.org>
  *
@@ -193,9 +193,11 @@ struct iovec * input_NetlistGetiovec( void * p_method_data )
               );
  
     // je suis pas complétement sûr que je fais ce que tu voulais ici ...
+    //Hum t'as pas l'impression de faire INPUT_READ_ONCE fois la même chose ?
     
     for (i_loop=0; i_loop<INPUT_READ_ONCE; i_loop++)
     {
+        //Noooooooooooooooooooooooooooooooooooooooooooooooooooooooooon !!!
         p_netlist->pp_free_data[p_netlist->i_data_start]->p_payload_start 
             = (byte_t *)p_netlist->pp_free_data[p_netlist->i_data_start];
 
@@ -210,10 +212,23 @@ struct iovec * input_NetlistGetiovec( void * p_method_data )
     // entre deux un autre thread peut pas venir foutre le brin dans les
     // i_data_start et stop, du genre avec un newpacket ? 
     
+    //Bonne question, c'est là tout l'avantage de la FIFO : les décodeurs ne
+    //déplacent _que_ i_data_end, et l'input déplace i_data_start.
+    //Pas de concurrence possible. L'input appelle NewPacket() et les
+    //décodeurs appellent DeletePES().
+    
     // je suppose qu'on fait pas tout dans une même fonction parce que si le
     // readv a besoin de moins que INPUT_READ_ONCE on perdrait de la place, mais
+    //Oui.
     // est-ce qu'il faut pas laisser le lock à ce moment là et le rendre
     // à la fin de Mviovec ?
+    //Non. Un read() est une opération bloquante qui peut prendre un temps
+    //*énorme* sur lecture réseau notamment (pas temps CPU, temps réel
+    //d'attente). Si on lockait pendant tout ce temps les décodeurs seraient
+    //bloqués très souvent.
+    
+    //Pour le corrigé (et la version LIFO de l'algorithme) tu peux regarder
+    //input.c:input_ReadPacket() de l'input-I.
 }
 
 /*****************************************************************************
@@ -284,7 +299,10 @@ struct data_packet_s * input_NetlistNewPacket( void * p_method_data,
     p_return->p_next = NULL;
     p_return->b_discard_payload = 0;
     
+    //Nooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooon !!!
     p_return->p_payload_start = (byte_t *)p_return;
+    //Plus précisément, + i_buffer_size (il a le droit de vouloir un buffer
+    //plus petit)
     p_return->p_payload_end = p_return->p_payload_start 
                             + p_netlist->i_buffer_size;
     
@@ -352,7 +370,7 @@ void input_NetlistDeletePacket( void * p_method_data, data_packet_t * p_data )
     p_netlist->p_free_iovec[p_netlist->i_data_end].iov_base = p_data->p_buffer;
     
     /* unlock */
-    vlc_mutex_unlock (&p_netlist->lock);    
+    vlc_mutex_unlock (&p_netlist->lock);
 }
 
 /*****************************************************************************