Another asinine question

Another asinine question

[Juan Carlos Castro y Castro]

If I compile and install the HG version of alsa-lib while everything 
else is 1.0.11, will Bad Things™ happen?


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid\x120709&bid&3057&dat\x121642

Re: Another asinine question

[Takashi Iwai]

At Fri, 12 May 2006 12:06:28 -0300,
Juan Carlos Castro y Castro wrote:
> 
> If I compile and install the HG version of alsa-lib while everything 
> else is 1.0.11, will Bad Things™ happen?

No, it shouldn't.  alsa-lib API is backward compatible.

But there were fixes and improvements in the driver and utils.  Better
to update all things.


Takashi


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid\x120709&bid&3057&dat\x121642

Re: Another asinine question

[Juan Carlos Castro y Castro]

Thanks for the tip (again!)

(Memo to self: always CC the particular persons you're trying to reach, 
list delays are getting out of hand)

By the way, my pcm_file extensions DO work! It's the "plug" plugin 
that's not passing the reads! Stay tuned...

Takashi Iwai wrote:

>At Fri, 12 May 2006 12:06:28 -0300,
>Juan Carlos Castro y Castro wrote:
>  
>
>>If I compile and install the HG version of alsa-lib while everything 
>>else is 1.0.11, will Bad Things™ happen?
>>    
>>
>
>No, it shouldn't.  alsa-lib API is backward compatible.
>
>But there were fixes and improvements in the driver and utils.  Better
>to update all things.
>
>
>Takashi
>
>  
>



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid\x120709&bid&3057&dat\x121642

Re: Another asinine question

[Takashi Iwai]

At Tue, 16 May 2006 11:33:22 -0300,
Juan Carlos Castro y Castro wrote:
> 
> Thanks for the tip (again!)
> 
> (Memo to self: always CC the particular persons you're trying to reach, 
> list delays are getting out of hand)
> 
> By the way, my pcm_file extensions DO work! It's the "plug" plugin 
> that's not passing the reads! Stay tuned...

Great, feel free to submit the patches to alsa-devel ML (Cc to me).


Takashi

> 
> Takashi Iwai wrote:
> 
> >At Fri, 12 May 2006 12:06:28 -0300,
> >Juan Carlos Castro y Castro wrote:
> >  
> >
> >>If I compile and install the HG version of alsa-lib while everything 
> >>else is 1.0.11, will Bad Things™ happen?
> >>    
> >>
> >
> >No, it shouldn't.  alsa-lib API is backward compatible.
> >
> >But there were fixes and improvements in the driver and utils.  Better
> >to update all things.
> >
> >
> >Takashi
> >
> >  
> >
> 


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid\x120709&bid&3057&dat\x121642

Re: Another asinine question

[Juan Carlos Castro y Castro]

[-- Attachment #1: Type: text/plain, Size: 775 bytes --]

Takashi Iwai wrote:

>Juan Carlos Castro y Castro wrote:
>  
>
>>By the way, my pcm_file extensions DO work! It's the "plug" plugin 
>>that's not passing the reads! Stay tuned...
>>    
>>
>Great, feel free to submit the patches to alsa-devel ML (Cc to me).
>
Your wish is my command, Takashi-san. ;)

Note the interleaved read has not yet been implemented. Also, I'm not 
yet dealing with blocking versus non-blocking at the moment. When 
reading from a file, read()'s just keep getting zero bytes and arecord 
doesn't mind. Some deal of sophistication will have to be added.

Another thing: you'll see I just "hung" another snd_pcm_file_t structure 
at the bottom of the original one. I'll understand if that offends 
people's coding sensibilities -- it offended mine. :-/

[-- Attachment #2: pcm_file-infile.patch --]
[-- Type: text/x-patch, Size: 5166 bytes --]

diff -r 33b6942ac42d src/pcm/pcm_file.c
--- a/src/pcm/pcm_file.c	Fri Apr 28 15:55:32 2006 +0200
+++ b/src/pcm/pcm_file.c	Tue May 16 13:21:12 2006 -0300
@@ -42,7 +42,7 @@ typedef enum _snd_pcm_file_format {
 	SND_PCM_FILE_FORMAT_RAW
 } snd_pcm_file_format_t;
 
-typedef struct {
+typedef struct snd_pcm_file_t_struct {
 	snd_pcm_generic_t gen;
 	char *fname;
 	int fd;
@@ -55,6 +55,7 @@ typedef struct {
 	char *wbuf;
 	snd_pcm_channel_area_t *wbuf_areas;
 	size_t buffer_bytes;
+	struct snd_pcm_file_t_struct *nextfile;
 } snd_pcm_file_t;
 
 #endif /* DOC_HIDDEN */
@@ -120,6 +121,14 @@ static int snd_pcm_file_close(snd_pcm_t 
 		free((void *)file->fname);
 		close(file->fd);
 	}
+	if (file->nextfile) {
+		if (file->nextfile->fname) {
+			free((void *)file->nextfile->fname);
+			close(file->nextfile->fd);
+		}
+		free(file->nextfile);
+		file->nextfile = NULL;
+	}
 	return snd_pcm_generic_close(pcm);
 }
 
@@ -222,10 +231,20 @@ static snd_pcm_sframes_t snd_pcm_file_re
 {
 	snd_pcm_file_t *file = pcm->private_data;
 	snd_pcm_channel_area_t areas[pcm->channels];
-	snd_pcm_sframes_t n = snd_pcm_readi(file->gen.slave, buffer, size);
-	if (n > 0) {
-		snd_pcm_areas_from_buf(pcm, areas, buffer);
-		snd_pcm_file_add_frames(pcm, areas, 0, n);
+	snd_pcm_sframes_t n /* , bytesn */;
+
+	if (file->nextfile) {
+		n = /* bytesn = */ read(file->nextfile->fd, buffer, size * pcm->frame_bits / 8);
+		if (n > 0)
+			n = n * 8 / pcm->frame_bits;
+		/* SNDERR("DEBUG: channels = %d, sample_bits = %d, frame_bits = %d, bytes = %d, frames = %d",
+		        pcm->channels, pcm->sample_bits, pcm->frame_bits, bytesn, n); */
+	} else {
+		n = snd_pcm_readi(file->gen.slave, buffer, size);
+		if (n > 0) {
+			snd_pcm_areas_from_buf(pcm, areas, buffer);
+			snd_pcm_file_add_frames(pcm, areas, 0, n);
+		}
 	}
 	return n;
 }
@@ -234,7 +253,14 @@ static snd_pcm_sframes_t snd_pcm_file_re
 {
 	snd_pcm_file_t *file = pcm->private_data;
 	snd_pcm_channel_area_t areas[pcm->channels];
-	snd_pcm_sframes_t n = snd_pcm_writen(file->gen.slave, bufs, size);
+	snd_pcm_sframes_t n;
+
+	if (file->nextfile) {
+		SNDERR("DEBUG: Noninterleaved read not yet implemented.\n");
+		return 0;	/* Noninterleaved read not yet implemented */
+	}
+
+	n = snd_pcm_readn(file->gen.slave, bufs, size);
 	if (n > 0) {
 		snd_pcm_areas_from_bufs(pcm, areas, bufs);
 		snd_pcm_file_add_frames(pcm, areas, 0, n);
@@ -377,11 +403,11 @@ static snd_pcm_fast_ops_t snd_pcm_file_f
  *          changed in future.
  */
 int snd_pcm_file_open(snd_pcm_t **pcmp, const char *name,
-		      const char *fname, int fd, const char *fmt, int perm,
-		      snd_pcm_t *slave, int close_slave)
+		      const char *fname, int fd, const char *ifname, int ifd, 
+		      const char *fmt, int perm, snd_pcm_t *slave, int close_slave)
 {
 	snd_pcm_t *pcm;
-	snd_pcm_file_t *file;
+	snd_pcm_file_t *file, *ifile;
 	snd_pcm_file_format_t format;
 	int err;
 	assert(pcmp);
@@ -395,7 +421,7 @@ int snd_pcm_file_open(snd_pcm_t **pcmp, 
 	if (fname) {
 		fd = open(fname, O_WRONLY|O_CREAT, perm);
 		if (fd < 0) {
-			SYSERR("open %s failed", fname);
+			SYSERR("open %s for writing failed", fname);
 			return -errno;
 		}
 	}
@@ -405,7 +431,34 @@ int snd_pcm_file_open(snd_pcm_t **pcmp, 
 			close(fd);
 		return -ENOMEM;
 	}
-	
+
+	if (ifname) {
+		ifd = open(ifname, O_RDONLY);
+		if (ifd < 0) {
+			SYSERR("open %s for reading failed", ifname);
+			if (fname)
+				close(fd);
+			return -errno;
+		}
+	}
+	if (ifd) {
+		ifile = calloc(1, sizeof(snd_pcm_file_t));
+		if (!ifile) {
+			if (ifname)
+				close(ifd);
+			if (fname)
+				close(fd);
+			return -ENOMEM;
+		}
+		if (ifname)
+			ifile->fname = strdup(ifname);
+		ifile->fd = ifd;
+		ifile->format = format;
+		ifile->gen.slave = slave;		/* Do I really have to do this? */
+		ifile->gen.close_slave = close_slave;	/* And this?                    */
+		file->nextfile = ifile;
+	}
+
 	if (fname)
 		file->fname = strdup(fname);
 	file->fd = fd;
@@ -486,9 +539,9 @@ int _snd_pcm_file_open(snd_pcm_t **pcmp,
 	int err;
 	snd_pcm_t *spcm;
 	snd_config_t *slave = NULL, *sconf;
-	const char *fname = NULL;
+	const char *fname = NULL, *ifname = NULL;
 	const char *format = NULL;
-	long fd = -1;
+	long fd = -1, ifd = -1;
 	int perm = 0600;
 	snd_config_for_each(i, next, conf) {
 		snd_config_t *n = snd_config_iterator_entry(i);
@@ -513,6 +566,17 @@ int _snd_pcm_file_open(snd_pcm_t **pcmp,
 			err = snd_config_get_string(n, &fname);
 			if (err < 0) {
 				err = snd_config_get_integer(n, &fd);
+				if (err < 0) {
+					SNDERR("Invalid type for %s", id);
+					return -EINVAL;
+				}
+			}
+			continue;
+		}
+		if (strcmp(id, "infile") == 0) {
+			err = snd_config_get_string(n, &ifname);
+			if (err < 0) {
+				err = snd_config_get_integer(n, &ifd);
 				if (err < 0) {
 					SNDERR("Invalid type for %s", id);
 					return -EINVAL;
@@ -556,7 +620,7 @@ int _snd_pcm_file_open(snd_pcm_t **pcmp,
 	snd_config_delete(sconf);
 	if (err < 0)
 		return err;
-	err = snd_pcm_file_open(pcmp, name, fname, fd, format, perm, spcm, 1);
+	err = snd_pcm_file_open(pcmp, name, fname, fd, ifname, ifd, format, perm, spcm, 1);
 	if (err < 0)
 		snd_pcm_close(spcm);
 	return err;

Re: Another asinine question

[Takashi Iwai]

At Tue, 16 May 2006 13:28:23 -0300,
Juan Carlos Castro y Castro wrote:
> 
> Takashi Iwai wrote:
> 
> >Juan Carlos Castro y Castro wrote:
> >  
> >
> >>By the way, my pcm_file extensions DO work! It's the "plug" plugin 
> >>that's not passing the reads! Stay tuned...
> >>    
> >>
> >Great, feel free to submit the patches to alsa-devel ML (Cc to me).
> >
> Your wish is my command, Takashi-san. ;)
> 
> Note the interleaved read has not yet been implemented. Also, I'm not 
> yet dealing with blocking versus non-blocking at the moment. When 
> reading from a file, read()'s just keep getting zero bytes and arecord 
> doesn't mind. Some deal of sophistication will have to be added.
 
That's OK.  We can extend the feature later.

> Another thing: you'll see I just "hung" another snd_pcm_file_t structure 
> at the bottom of the original one. I'll understand if that offends 
> people's coding sensibilities -- it offended mine. :-/

Does't only one file descriptor instead of the whole snd_pcm_file
struct suffice?  Actually, it's used only in readi().


> @@ -377,11 +403,11 @@ static snd_pcm_fast_ops_t snd_pcm_file_f
>   *          changed in future.
>   */
>  int snd_pcm_file_open(snd_pcm_t **pcmp, const char *name,
> -		      const char *fname, int fd, const char *fmt, int perm,
> -		      snd_pcm_t *slave, int close_slave)
> +		      const char *fname, int fd, const char *ifname, int ifd, 
> +		      const char *fmt, int perm, snd_pcm_t *slave, int close_slave)

Don't forget to change the doxygen comment for this function.

> @@ -405,7 +431,34 @@ int snd_pcm_file_open(snd_pcm_t **pcmp, 
>  			close(fd);
>  		return -ENOMEM;
>  	}
> -	
> +
> +	if (ifname) {
> +		ifd = open(ifname, O_RDONLY);
> +		if (ifd < 0) {
> +			SYSERR("open %s for reading failed", ifname);
> +			if (fname)
> +				close(fd);
> +			return -errno;
> +		}
> +	}
> +	if (ifd) {

Should be "if (ifd >= 0)".


Also, please make a patch against the latest HG repository.
The typo in readn() was already fixed, so your last patch conflicts
with the latest tree.


Thanks,

Takashi


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642

Re: Another asinine question

[Juan Carlos Castro y Castro]

Juan Carlos Castro y Castro wrote:

> By the way, my pcm_file extensions DO work! It's the "plug" plugin 
> that's not passing the reads! Stay tuned...

I'm pretty sure now the problem lies with the plug plugin -- I am either 
not using it right (more likely) or I need to use a different plugin 
entirely (less likely) or more hacking will be needed (unlikely).

Current status is: with my changes, if I use the file PCM as microphone 
input (and use capture parameters that match the raw input file format), 
capture works. The "plug" PCM is for format conversion, and it doesn't 
work for input, only for output. Here's my current .asoundrc:

------------------------------------------------------------
pcm.fileout {
        type file
        file /home/jcastro/alsaout.raw
        infile /home/jcastro/alsain.raw
        slave.pcm null
}

pcm_slave.sl3 {
    pcm "fileout"
    format S16_LE
    channels 1
    rate 48000
}

pcm.complex_convert {
    type plug
    slave sl3
}
------------------------------------------------------------



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642

Error compiling alsa-driver from sources

[Juan Carlos Castro y Castro]

Takashi Iwai wrote:

>Juan Carlos Castro y Castro wrote:
>  
>
>>If I compile and install the HG version of alsa-lib while everything 
>>else is 1.0.11, will Bad Things happen?
>>    
>>
>No, it shouldn't.  alsa-lib API is backward compatible.
>
>But there were fixes and improvements in the driver and utils.  Better
>to update all things.
>  
>
I see that alsa-driver requires alsa-kernel, and finds it automatically 
if present in a sibling directory. That does not happen when installing 
the stable alsa-driver tarball.

Here's the error I got:

  CC [M]  /usr/src/hg/alsa-driver/pci/pdplus/pdplus.o
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6179: error: 
`PCI_VENDOR_ID_MARIAN' undeclared here (not in a function)
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6179: error: initializer 
element is not constant
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6179: error: (near 
initialization for `pdplus_ids[0].vendor')
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6180: error: 
`PCI_DEVICE_ID_MARIAN_PRODIF_PLUS' undeclared here (not in a function)
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6180: error: initializer 
element is not constant
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6180: error: (near 
initialization for `pdplus_ids[0].device')
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6183: error: initializer 
element is not constant
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6183: error: (near 
initialization for `pdplus_ids[0]')
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6184: error: initializer 
element is not constant
/usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6184: error: (near 
initialization for `pdplus_ids[1]')
make[5]: ** [/usr/src/hg/alsa-driver/pci/pdplus/pdplus.o] Erro 1
make[4]: ** [/usr/src/hg/alsa-driver/pci/pdplus] Erro 2
make[3]: ** [/usr/src/hg/alsa-driver/pci] Erro 2
make[2]: ** [_module_/usr/src/hg/alsa-driver] Erro 2
make[1]: ** [modules] Erro 2
make[1]: Leaving directory `/usr/src/linux-2.6.16-pom'
make: ** [compile] Erro 2



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642

Re: Error compiling alsa-driver from sources

[Takashi Iwai]

At Tue, 16 May 2006 12:52:58 -0300,
Juan Carlos Castro y Castro wrote:
> 
> Takashi Iwai wrote:
> 
> >Juan Carlos Castro y Castro wrote:
> >  
> >
> >>If I compile and install the HG version of alsa-lib while everything 
> >>else is 1.0.11, will Bad Things happen?
> >>    
> >>
> >No, it shouldn't.  alsa-lib API is backward compatible.
> >
> >But there were fixes and improvements in the driver and utils.  Better
> >to update all things.
> >  
> >
> I see that alsa-driver requires alsa-kernel, and finds it automatically 
> if present in a sibling directory. That does not happen when installing 
> the stable alsa-driver tarball.
> 
> Here's the error I got:
> 
>   CC [M]  /usr/src/hg/alsa-driver/pci/pdplus/pdplus.o
> /usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6179: error: 
> `PCI_VENDOR_ID_MARIAN' undeclared here (not in a function)
> /usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6179: error: initializer 
> element is not constant
> /usr/src/hg/alsa-driver/pci/pdplus/pdplus.c:6179: error: (near 
> initialization for `pdplus_ids[0].vendor')
(snip)

It's a known issue that I've seen sometimes.
alsa-driver/include/linux/pci_ids.h is missing, which is a symlink to
../pci_ids_compat.h.  It must be created by configure script but it's
removed by some reason on hgcompile.

Create the symlink above manually or just run configure again.


Takashi


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642

Re: Error compiling alsa-driver from sources

[Juan Carlos Castro y Castro]

Takashi Iwai wrote:

>(snip)
>
>It's a known issue that I've seen sometimes.
>alsa-driver/include/linux/pci_ids.h is missing, which is a symlink to
>../pci_ids_compat.h.  It must be created by configure script but it's
>removed by some reason on hgcompile.
>
>Create the symlink above manually or just run configure again.
>  
>
./configure did it, thanks.

In the meantime I also realized alsa-utils requires gettext-devel. 
Again, unlike the stable tarball.


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642