Commit 2ac915c5 authored by Christophe Massiot's avatar Christophe Massiot

Comments for Henri.

parent 12d3f35f
......@@ -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 compltement sr que je fais ce que tu voulais ici ...
//Hum t'as pas l'impression de faire INPUT_READ_ONCE fois la mme 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 dcodeurs ne
//dplacent _que_ i_data_end, et l'input dplace i_data_start.
//Pas de concurrence possible. L'input appelle NewPacket() et les
//dcodeurs appellent DeletePES().
// je suppose qu'on fait pas tout dans une mme 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 opration bloquante qui peut prendre un temps
//*norme* sur lecture rseau notamment (pas temps CPU, temps rel
//d'attente). Si on lockait pendant tout ce temps les dcodeurs seraient
//bloqus trs 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 prcisment, + 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);
}
/*****************************************************************************
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment