Modo compacto | Modo extendido

452 en 19 posts · 07-01-2013 20:24

Enunciado:

Escriba un programa que invoque a la función de lectura del dispositivo de audio y envíe las muestras leídas auna FIFO, luego que tome las muestras de ese FIFO y las envíe a la salida de audio correspondiente.

Bueno, yo logre leer el microfono y reproducirlo, funciono todo bien, pero cuando le agregue el mkfifo para leerlo desde ahi tuve problemas, se me cuelga el programa. Les comento con muchos asteriscos las lineas que agregue luego de hacer que el programa me funcione correctamente (solo que no guardaba lso datos en una fifo como pedian).



 



/*

 * 

 * 

 * Ejecutar:

 * 1) aoss ./tp (Si no ejecuta hacer: sudo apt-get install alsa-oss)

 * 

 */



#include <unistd.h>

#include <fcntl.h>

#include <sys/types.h>

#include <sys/ioctl.h>

#include <stdlib.h>

#include <stdio.h>

#include <string.h>

#include <linux/soundcard.h>

#include <sys/stat.h>





#define TIEMPO 10

#define VELOCIDAD 8000

#define TAMANIO 8

#define CANAL 1

#define FIFO_NAME "Mi_fifo"



void setear_parametros_audio(int);





int main(void)

{

  int fd=0, estado=0;

  char buf[TIEMPO*VELOCIDAD*TAMANIO*CANAL/8];

  

  if((fd=mkfifo(FIFO_NAME, 0666))<0)//*****************************************************DESDE ACA*******************

  {

    printf("Error creando la FIFO %s\n",FIFO_NAME);

    exit(0);

  }//*****************************************************HASTA ACA*******************

  

  fd = open(FIFO_NAME, O_RDWR);// ***************** ACA solo cambie el FIFO_NAME, en realidad iba"/dev/dsp"***

  

  if(fd < 0)

  {

    printf("Error al abrir \"/dev/dsp\", Error: %s\n", strerror (fd));

    exit(0);

  }

  

  setear_parametros_audio(fd);

  

  printf("Hable por el microfono:\n");

  

  estado = read(fd, buf, sizeof(buf));

  

  if(estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función read, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  

  printf("Se escucho:\n");

  

  estado = write(fd, buf, sizeof(buf));

  

  if (estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función write, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  close(fd);//*****************************************************ACA*******************

 return 0; 

}





void setear_parametros_audio(int fd)

{

  int argumento=TAMANIO, estado=0;

  

  estado = ioctl(fd, SOUND_PCM_WRITE_BITS, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != TAMANIO)

    fprintf (stderr,"Tamaño de muestras no soportado. Se programó %d\n",argumento);

  

  argumento = CANAL;

  estado = ioctl(fd, SOUND_PCM_WRITE_CHANNELS, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != CANAL)

    fprintf (stderr,"Canal no soportado. Se programó %d\n",argumento);

  

  argumento = VELOCIDAD;

  estado = ioctl(fd, SOUND_PCM_WRITE_RATE, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != VELOCIDAD)

    fprintf (stderr,"Velocidad no soportada. Se programó %d\n",argumento);

  

}

Compilar y ejecutar código

Son solo un par de lineas las que agregue que ahi las comente, si alguien me puede ayudar lo agradeceria mucho. Saludos!!

NOTA: Al compilar aca tira error creando la fifo, pero a mi no me lo tira en la pc. Se queda trabado en la funcion read y no sale nunca.

47 en 23 posts · 08-01-2013 22:17

Hola, creo que el problema es justamente esa línea que cambiaste:



 fd = open(FIFO_NAME, O_RDWR);

Compilar y ejecutar código

Cambiala por:



 fd = open("/dev/dsp", O_RDWR);

Compilar y ejecutar código

Se te cuelga el programa porque en la línea en la que usás read estás tratando de leer de la FIFO y no de la placa de sonido. Si mal no recuerdo, cuando leés de una FIFO, el programa se queda colgado esperando que alguien escriba. Así que para poder hacer lo que te pide el enunciado, usá un file descriptor distinto para la FIFO, y agregá lo que falta para leer y escribir de ahí.
Saludos.

452 en 19 posts · 08-01-2013 22:43

Claro, es ocmo decis, y oantes usaba eldev/dsp, pero ahora necesito guardar lo escuchado por el microfono en una FIFO (porque lo pide en enunciado del ejercicio). y cuando cambie eso fue cuando tuve problemas.

47 en 23 posts · (14-01-2013)

Lo que tenés que hacer es usar otro file descriptor para la FIFO. Primero la tenés que crear con mkfifo(), y lo que te devuelve es 0 si salió bien o -1 si hubo un error, NO te devuelve un file descriptor. Además tené en cuenta que una FIFO tiene que ser abierta simultáneamente por dos procesos, uno para lectura y otro para escritura, y la función open() te bloquea a uno hasta que el otro abre también la FIFO. Así que tendrías que hacer 2 programas, uno que lea de la placa de sonido y después guarde en la FIFO, y otro que lea de la FIFO y escriba en la placa.

El main del proceso escritor te podría quedar así (no estoy seguro si es exactamente así el uso, porque creo que sólo usé las FIFOs para preparar el final de la materia jaja):



 

int main(void)

{

  int fdaudio, fdfifo, estado;

  char buf[TIEMPO*VELOCIDAD*TAMANIO*CANAL/8];

  

  if((mkfifo(FIFO_NAME, 0666))<0)//*****************************************************DESDE ACA*******************

  {

    printf("Error creando la FIFO %s\n",FIFO_NAME);

    exit(0);

  }//*****************************************************HASTA ACA*******************

  

  fdaudio = open("/dev/dsp", O_RDWR);

  

  if(fdaudio < 0)

  {

    printf("Error al abrir \"/dev/dsp\", Error: %s\n", strerror (fdaudio));

    exit(0);

  }

  

  setear_parametros_audio(fdaudio);

  

  printf("Hable por el microfono:\n");

  

  estado = read(fd, buf, sizeof(buf));

  

  if(estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función read, Código de error: %s\n",strerror (estado));

    exit(0);

  }



  close(fdaudio);



  fdfifo = open(FIFO_NAME, O_WRONLY); // Acá va a bloquearse hasta que el proceso lector abra la FIFO



  if(fdfifo < 0)

  {

    printf("Error al abrir la FIFO, Error: %s\n", strerror (fdfifo));

    exit(0);

  }

  

  estado = write(fdfifo, buf, sizeof(buf)); // Acá se va a bloquear hasta que el otro proceso lea

  

  if (estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función write, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  close(fdfifo);//*****************************************************ACA*******************

 return 0; 

}

Compilar y ejecutar código

Después para el otro programa tendrías que hacer que abra la FIFO para lectura, que lea lo que recibe, y lo mande a la placa de sonido. El problema que veo con este código es que primero tendrías que ejecutar este programa y después el que lee, porque si los dos tratan de abrir el /dev/dsp al mismo tiempo va a tirar error.

Para ver más sobre las FIFOs fijate en la manpage:

http://linux.die.net/man/3/mkfifo

452 en 19 posts · 13-01-2013 17:03

pablo.m Gracias por la ayuda! Claro, yo me olvide que debia abrirse la fifo con otro programa porque se bloquea hasta que no es leida, ahi arregle un poco los codigos, pero aun no funciona como deberia, lo que llegue a lograr fue que (hasta donde puedo entender) tome el audio del microfono y lo grabe, pero en alguna parte de grabarlo, no se si en el programa que toma el audio del microfono o el que lo reproduce, me hace un ruido feo como de interferencia por un timepo de un segundo y se corta y terminan los 2 programas. Te pongo los 2 codigos (pongo tambien el codigo de las funciones por las dudas. Quizas vos puedas ver donde esta el error, seguro es una estupidez jaja, pasda que nunca nos explicaron el manejo de audio y lo tengo que entregar si o si porque me lo exige la catedra.

PD: El ruido lo hace ucando termina de leer y va a hacer el write en el codido escuchar.c te lo marcon con un cometnario que dice: "PROBLEMA ACA".

Este es el codigo del programa escuchar.c



 

#include "header.h"



int main(void)

{

  int fdaudio=0, fdfifo=0, estado=0;

  char buf[TIEMPO*VELOCIDAD*TAMANIO*CANAL/8];

  

  if( mkfifo(FIFO_NAME, 0666) < 0 )

  {

    printf("Error creando la FIFO %s\n",FIFO_NAME);

    exit(0);

  }

  

  fdaudio = open("/dev/dsp", O_WRONLY);

  

  if(fdaudio < 0)

  {

    printf("Error al abrir \"/dev/dsp\", Error: %s\n", strerror (fdaudio));

    exit(0);

  }

  

  setear_parametros_audio(fdaudio);

  

  printf("Hable por el microfono:\n");

  

  estado = write(fdaudio, buf, sizeof(buf));

  

  if(estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función read, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  close(fdaudio);

  

  fdfifo = open(FIFO_NAME, O_WRONLY);

  

  if( fdfifo < 0)

  {

    printf("Error al abrir la fifo, error: %s", strerror(fdfifo));

  }

  

  estado = write(fdfifo, buf, sizeof(buf)); //*********PROBLEMA ACA, o al menos luego del close(fdaudio)***********

  

  if( estado != sizeof(buf))

  {

    printf("Error en la funcion write, error: %s", strerror(estado));

  }

  

  close(fdfifo);

 return 0; 

}

Compilar y ejecutar código

Este codigo siguiente es el de reproducir.c (seguro este es el que tiene el error, lee los comentarios que les puse.



 

#include "header.h"



int main(void)

{

  int fdaudio=0, fdfifo = 0, estado=0;

  char buf[TIEMPO*VELOCIDAD*TAMANIO*CANAL/8];

  

  if(sleep(TIEMPO+1) < 0)

    printf("Problema con la funcion sleep.\n");

  

  fdaudio = open("/dev/dsp", O_RDONLY);

  

  if(fdaudio < 0)

  {

    printf("Error al abrir \"/dev/dsp\", Error: %s\n", strerror (fdaudio));

    exit(0);

  }

  

  setear_parametros_audio(fdaudio);

  fdfifo = open(FIFO_NAME, O_RDONLY);

  

  printf("Se escucho:\n");

  estado = read(fdfifo, buf, sizeof(buf));

  

  if (estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función write, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  close(fdfifo);

 return 0;

}

Compilar y ejecutar código

Aca pongo por las dudas el funciones.c y luego pongo el header.h



 

#include "header.h"



void setear_parametros_audio(int fd)

{

  int argumento=TAMANIO, estado=0;

  

  estado = ioctl(fd, SOUND_PCM_WRITE_BITS, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != TAMANIO)

    fprintf (stderr,"Tamaño de muestras no soportado. Se programó %d\n",argumento);

  

  argumento = CANAL;

  estado = ioctl(fd, SOUND_PCM_WRITE_CHANNELS, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != CANAL)

    fprintf (stderr,"Canal no soportado. Se programó %d\n",argumento);

  

  argumento = VELOCIDAD;

  estado = ioctl(fd, SOUND_PCM_WRITE_RATE, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != VELOCIDAD)

    fprintf (stderr,"Velocidad no soportada. Se programó %d\n",argumento);

  

}

Compilar y ejecutar código



 

#include <unistd.h>

#include <fcntl.h>

#include <sys/types.h>

#include <sys/ioctl.h>

#include <stdlib.h>

#include <stdio.h>

#include <string.h>

#include <linux/soundcard.h>

#include <sys/stat.h>





#define TIEMPO 10

#define VELOCIDAD 8000

#define TAMANIO 8

#define CANAL 1

#define FIFO_NAME "Mi_fifo"



void setear_parametros_audio(int);

Compilar y ejecutar código

Es el ultimo tp que me falta terminar, asi que bueno, gracias por la ayuda nuevamente

47 en 23 posts · (14-01-2013)

En el primer código, el problema que tenés es que usás write sobre fdaudio en vez de read. O sea que estás escribiendo basura en la salida de sonido (y en la FIFO), por eso escuchás cualquier cosa. Además tenés que abrir el dsp en modo O_RDONLY.

En el segundo código, te falta el write en la placa de sonido, y además te sugeriría que cambies el orden en que manejas los archivos. Acá pongo el código con esas modificaciones, fijate los comentarios.



 

#include "header.h"



int main(void)

{

  int fdaudio=0, fdfifo = 0, estado=0;

  char buf[TIEMPO*VELOCIDAD*TAMANIO*CANAL/8];

  

  // El sleep no es necesario con estas modificaciones.

  /*if(sleep(TIEMPO+1) < 0)

    printf("Problema con la funcion sleep.\n");*/

  

  fdfifo = open(FIFO_NAME, O_RDONLY); // Se va a bloquear hasta que el otro programa haya grabado todo el audio en el buffer

  

  if(fdfifo < 0)

  {

    printf("Error al abrir la FIFO. Error: %s\n", strerror(fdfifo);

    exit(0);

  }



  estado = read(fdfifo, buf, sizeof(buf)); // Se va a bloquear hasta que el otro programa escriba los datos en la FIFO.

  

  if (estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función read, Código de error: %s\n",strerror (estado));

    exit(0);

  }



  close(fdfifo); // No necesitamos más la FIFO, ya que todos los datos están en buf



  fdaudio = open("/dev/dsp", O_WRONLY); // Abrimos el dsp para escribir. A esta altura el otro programa cerró el dsp por lo que

                                                        // no debería haber ningún error.



  if(fdaudio < 0)

  {

    printf("Error al abrir \"/dev/dsp\", Error: %s\n", strerror (fdaudio));

    exit(0);

  }

  

  setear_parametros_audio(fdaudio);



  printf("Se escucho:\n");

  

  estado = write(fdaudio, buf, sizeof(buf)); // Mandamos las muestras a la placa.	   

  if(estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función write, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  close(fdaudio);



  return 0;

}

Compilar y ejecutar código

452 en 19 posts · 13-01-2013 21:29

GENIAAAL!!!! ahi me salio! mcuhisimas gracias!!! luego subo el programa terminado para aquellos que tengan una duda similar a la mia, lo que si, tengo un problema algo raro, cada vez que ejecuto el programa, se crea la fifo, luego siempre cerre con close los fdaudio y fdfifo, pero cuando vuevlo a ejecutar, en lugar de reemplazar la fifo me tira error al crearla (si la borro manualemente vuelve a funcionar y crear la fifo todo bien), alguna idea? se uqe peudo borrar la fifo desde el programa, pero no es lo que corresponde jaja, deberia reemplazarla y listo.

452 en 19 posts · 14-01-2013 11:57

Ha! ahi lo pude solucionar, n oes necesario vovler a crearla

la conexion ya esta hecha, por lo que pregunto si errno es distinto a EEXIST (archivo existente) entocnes ahi si hubo un error al crearla y cierro todo, luego suboi los codigos, slaudos y gracias!

452 en 19 posts · 15-01-2013 11:56

Bueno, como habia dicho, paso a dejarles los codigos para aquellos que los necesiten.



 

int main(void)

{

  int fdaudio=0, fdfifo = 0, estado=0;

  char buf[TIEMPO*VELOCIDAD*TAMANIO*CANAL/8];

  

  printf("Esperando finalizacion de grabado de audio....\n");

  fdfifo = open(FIFO_NAME, O_RDONLY); // Se va a bloquear hasta que el otro programa haya grabado todo el audio en el buffer

  

  if(fdfifo < 0)

  {

    printf("Error al abrir la FIFO. Error: %s\n", strerror(fdfifo));

    exit(0);

  }



  estado = read(fdfifo, buf, sizeof(buf)); // Se va a bloquear hasta que el otro programa escriba los datos en la FIFO.

  

  if (estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función read, Código de error: %s\n",strerror (estado));

    exit(0);

  }



  close(fdfifo); // No necesitamos más la FIFO, ya que todos los datos están en buf



  fdaudio = open("/dev/dsp", O_WRONLY); // Abrimos el dsp para escribir. A esta altura el otro programa cerró el dsp por lo que no debería haber ningún error.



  if(fdaudio < 0)

  {

    printf("Error al abrir \"/dev/dsp\", Error: %s\n", strerror (fdaudio));

    exit(0);

  }

  

  setear_parametros_audio(fdaudio);



  printf("Se escucho:\n");

  

  estado = write(fdaudio, buf, sizeof(buf)); // Mandamos las muestras a la placa.	   

  if(estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función write, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  close(fdaudio);



  printf("\n----FIN----\n");

  

  return 0;

}

Compilar y ejecutar código



 

int main(void)

{

  int fdaudio=0, fdfifo=0, estado=0;

  char buf[TIEMPO*VELOCIDAD*TAMANIO*CANAL/8];

  

  if( mkfifo(FIFO_NAME, 0666) < 0 )

  {

    if(errno != EEXIST)

    {

      printf("Error creando la FIFO %s error: %s\n",FIFO_NAME, strerror(errno));

      exit(0);

    }

  }

  

  fdaudio = open("/dev/dsp", O_RDONLY);

  

  if(fdaudio < 0)

  {

    printf("Error al abrir \"/dev/dsp\", Error: %s\n", strerror (fdaudio));

    exit(0);

  }

  

  setear_parametros_audio(fdaudio);

  

  printf("Hable por el microfono:\n");

  

  estado = read(fdaudio, buf, sizeof(buf));

  

  if(estado != sizeof(buf))

  {

    fprintf(stderr,"Error en función read, Código de error: %s\n",strerror (estado));

    exit(0);

  }

  close(fdaudio);

  

  printf("Esperando conexion con el programa para reproducir la fifo.\n");

  

  fdfifo = open(FIFO_NAME, O_WRONLY);

  

  if( fdfifo < 0)

  {

    printf("Error al abrir la fifo, error: %s", strerror(fdfifo));

  }

  

  estado = write(fdfifo, buf, sizeof(buf));

  

  if( estado != sizeof(buf))

  {

    printf("Error en la funcion write, error: %s\n", strerror(estado));

  }

  

  close(fdfifo);

  

  printf("\n----FIN----\n");



 return 0; 

}

Compilar y ejecutar código



 

void setear_parametros_audio(int fd)

{

  int argumento=TAMANIO, estado=0;

  

  estado = ioctl(fd, SOUND_PCM_WRITE_BITS, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != TAMANIO)

    fprintf (stderr,"Tamaño de muestras no soportado. Se programó %d\n",argumento);

  

  argumento = CANAL;

  estado = ioctl(fd, SOUND_PCM_WRITE_CHANNELS, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != CANAL)

    fprintf (stderr,"Canal no soportado. Se programó %d\n",argumento);

  

  argumento = VELOCIDAD;

  estado = ioctl(fd, SOUND_PCM_WRITE_RATE, &argumento);

  if(estado < 0)

    printf("Es probable que no se haya podido configurar todos lso parametros deseados.\n");

  if(argumento != VELOCIDAD)

    fprintf (stderr,"Velocidad no soportada. Se programó %d\n",argumento);

  

}

Compilar y ejecutar código

Bueno ahi esta. Saludos!!

384 en 135 posts · 15-01-2013 12:05

Una aclaracion boluda, el Error creando la FIFO es porque el proceso no tiene permisos sobre las colas en la maquina virtual donde se esa corriendo.
Esta bueno que avise.

47 en 23 posts · 16-01-2013 20:14

Qué bueno que te haya funcionado, lástima que ese sea uno de los temas que algunos profesores se saltean. Suerte con el final.

2.983 en 451 posts · 16-01-2013 20:20

Como voy a parir esta materia jaja.

Modo compacto \| Modo extendido [Info I] Problema con puerto de audio
Autor	Mensaje