Bug#429326: [alsa-devel] [Pkg-alsa-devel] Bug#429326: Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-10-15 Thread Takashi Iwai 
At Sun, 14 Oct 2007 23:37:58 +0200,
Elimar Riesebieter wrote:
> 
> forwarded 429326 [EMAIL PROTECTED]
> 
> thanks
> 
> On Sun, 14 Oct 2007 the mental interface of
> Loïc Minier told:
> 
> > On Sun, Oct 14, 2007, Elimar Riesebieter wrote:
> > > From my point of view it is not reasonable to patch Debian packages,
> > > which of course will break 32bit only sound cards. So we've to find
> > > a way to force a solution directly on alsa-devel. Who will forward
> > > the _Debian_bug to [EMAIL PROTECTED] ?
> > 
> >  I don't understand your remark about patching the Debian package?  Why
> >  would reverting the bit shift macros break anything?
> > 
> >  The attached patch for alsa-lib should be enough to fix this, upstream
> >  should revert the broken change in the bit shifts.
> 
> Yeah, we can provide it. But shouldn't it be a solution for all ALSA
> users?
> 
> This is Debian #429326. As well reported as
> https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3360.
> 
> We have to look for a general solution as the distributors don't
> need to patch sources.

Hm, basically the possible overflow is only the case for 32 -> any bit
down conversion, right?  Then let's keep shift_down() for other
conversions, at least.  It's good for sound quality.

We can add a check in shift_down() for 32bit version to not to add if
it's greater or equal (1<<31) - (1<<(bits-1)) - 1 although I'm not
sure whether it's worth to try...


Takashi

> > Alsa #3360; GNOME #436192; LP #116990; Mandriva #33908; Revert HG r2264
> > (changeset 23c4c0f5de40) as the bit shifts overflow when the volume is high
> > for 32 bits to 16 bits resampling
> > 
> > diff -r 23c4c0f5de40 -r 883ae69471d2 src/pcm/plugin_ops.h
> > --- a/src/pcm/plugin_ops.h  Fri Mar 09 17:06:23 2007 +0100
> > +++ b/src/pcm/plugin_ops.h  Fri Mar 09 15:30:38 2007 +0100
> > @@ -85,8 +85,6 @@ static inline u_int32_t sx24s(u_int32_t 
> >  #define _put_triple(ptr,val) _put_triple_be(ptr,val)
> >  #define _put_triple_s(ptr,val) _put_triple_le(ptr,val)
> >  #endif
> > -
> > -#define shift_down(val, bits) (((val) + (1 << ((bits) - 1))) >> (bits))
> >  
> >  #ifdef COPY_LABELS
> >  static void *copy_labels[5] = {
> > @@ -258,14 +256,14 @@ conv_xxx1_009x: as_u32(dst) = sx24s((u_i
> >  conv_xxx1_009x: as_u32(dst) = sx24s((u_int32_t)(as_u8c(src) ^ 0x80) << 8); 
> > goto CONV_END;
> >  conv_xxx1_9000: as_u32(dst) = (u_int32_t)(as_u8c(src) ^ 0x80) << 24; goto 
> > CONV_END;
> >  conv_xxx1_0009: as_u32(dst) = (u_int32_t)(as_u8c(src) ^ 0x80); goto 
> > CONV_END;
> > -conv_xx12_xxx1: as_u8(dst) = shift_down(as_u16c(src), 8); goto CONV_END;
> > +conv_xx12_xxx1: as_u8(dst) = as_u16c(src) >> 8; goto CONV_END;
> >  conv_xx12_xx12: as_u16(dst) = as_u16c(src); goto CONV_END;
> >  conv_xx12_xx21: as_u16(dst) = bswap_16(as_u16c(src)); goto CONV_END;
> >  conv_xx12_x120: as_u32(dst) = sx24((u_int32_t)as_u16c(src) << 8); goto 
> > CONV_END;
> >  conv_xx12_021x: as_u32(dst) = sx24s((u_int32_t)bswap_16(as_u16c(src)) << 
> > 8); goto CONV_END;
> >  conv_xx12_1200: as_u32(dst) = (u_int32_t)as_u16c(src) << 16; goto CONV_END;
> >  conv_xx12_0021: as_u32(dst) = (u_int32_t)bswap_16(as_u16c(src)); goto 
> > CONV_END;
> > -conv_xx12_xxx9: as_u8(dst) = shift_down(as_u16c(src), 8) ^ 0x80; goto 
> > CONV_END;
> > +conv_xx12_xxx9: as_u8(dst) = (as_u16c(src) >> 8) ^ 0x80; goto CONV_END;
> >  conv_xx12_xx92: as_u16(dst) = as_u16c(src) ^ 0x8000; goto CONV_END;
> >  conv_xx12_xx29: as_u16(dst) = bswap_16(as_u16c(src)) ^ 0x80; goto CONV_END;
> >  conv_xx12_x920: as_u32(dst) = sx24((u_int32_t)(as_u16c(src) ^ 0x8000) << 
> > 8); goto CONV_END;
> > @@ -284,57 +282,57 @@ conv_xx12_01Ax: as_u32(dst) = sx24s((u_i
> >  conv_xx12_01Ax: as_u32(dst) = sx24s((u_int32_t)(as_u16c(src) ^ 0x80) << 
> > 8); goto CONV_END;
> >  conv_xx12_A100: as_u32(dst) = (u_int32_t)bswap_16(as_u16c(src) ^ 0x80) << 
> > 16; goto CONV_END;
> >  conv_xx12_001A: as_u32(dst) = (u_int32_t)(as_u16c(src) ^ 0x80); goto 
> > CONV_END;
> > -conv_x123_xxx1: as_u8(dst) = shift_down(as_u32c(src), 16); goto CONV_END;
> > -conv_x123_xx12: as_u16(dst) = shift_down(as_u32c(src), 8); goto CONV_END;
> > -conv_x123_xx21: as_u16(dst) = bswap_16(shift_down(as_u32c(src), 8)); goto 
> > CONV_END;
> > +conv_x123_xxx1: as_u8(dst) = as_u32c(src) >> 16; goto CONV_END;
> > +conv_x123_xx12: as_u16(dst) = as_u32c(src) >> 8; goto CONV_END;
> > +conv_x123_xx21: as_u16(dst) = bswap_16(as_u32c(src) >> 8); goto CONV_END;
> >  conv_x123_x123: as_u32(dst) = sx24(as_u32c(src)); goto CONV_END;
> >  conv_x123_321x: as_u32(dst) = sx24s(bswap_32(as_u32c(src))); goto CONV_END;
> >  conv_x123_1230: as_u32(dst) = as_u32c(src) << 8; goto CONV_END;
> > -conv_x123_0321: as_u32(dst) = shift_down(bswap_32(as_u32c(src)), 8); goto 
> > CONV_END;
> > - conv_x123_xxx9: as_u8(dst) = shift_down(as_u32c(src), 16) ^ 0x80; goto 
> > CONV_END;
> > -conv_x123_xx92: as_u16(dst) = shift_down(as_u32c(src), 8) ^ 0x8000; goto 
> > CONV_END;
> > -conv_x123_xx29: as_u16(dst) = bswap_16(shift_down(as_u32c(src), 8)) ^ 
>

Bug#429326: [Pkg-alsa-devel] Bug#429326: Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-10-14 Thread Elimar Riesebieter 
forwarded 429326 [EMAIL PROTECTED]

thanks

On Sun, 14 Oct 2007 the mental interface of
Loïc Minier told:

> On Sun, Oct 14, 2007, Elimar Riesebieter wrote:
> > From my point of view it is not reasonable to patch Debian packages,
> > which of course will break 32bit only sound cards. So we've to find
> > a way to force a solution directly on alsa-devel. Who will forward
> > the _Debian_bug to [EMAIL PROTECTED] ?
> 
>  I don't understand your remark about patching the Debian package?  Why
>  would reverting the bit shift macros break anything?
> 
>  The attached patch for alsa-lib should be enough to fix this, upstream
>  should revert the broken change in the bit shifts.

Yeah, we can provide it. But shouldn't it be a solution for all ALSA
users?

This is Debian #429326. As well reported as
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3360.

We have to look for a general solution as the distributors don't
need to patch sources.

> Alsa #3360; GNOME #436192; LP #116990; Mandriva #33908; Revert HG r2264
> (changeset 23c4c0f5de40) as the bit shifts overflow when the volume is high
> for 32 bits to 16 bits resampling
> 
> diff -r 23c4c0f5de40 -r 883ae69471d2 src/pcm/plugin_ops.h
> --- a/src/pcm/plugin_ops.hFri Mar 09 17:06:23 2007 +0100
> +++ b/src/pcm/plugin_ops.hFri Mar 09 15:30:38 2007 +0100
> @@ -85,8 +85,6 @@ static inline u_int32_t sx24s(u_int32_t 
>  #define _put_triple(ptr,val) _put_triple_be(ptr,val)
>  #define _put_triple_s(ptr,val) _put_triple_le(ptr,val)
>  #endif
> -
> -#define shift_down(val, bits) (((val) + (1 << ((bits) - 1))) >> (bits))
>  
>  #ifdef COPY_LABELS
>  static void *copy_labels[5] = {
> @@ -258,14 +256,14 @@ conv_xxx1_009x: as_u32(dst) = sx24s((u_i
>  conv_xxx1_009x: as_u32(dst) = sx24s((u_int32_t)(as_u8c(src) ^ 0x80) << 8); 
> goto CONV_END;
>  conv_xxx1_9000: as_u32(dst) = (u_int32_t)(as_u8c(src) ^ 0x80) << 24; goto 
> CONV_END;
>  conv_xxx1_0009: as_u32(dst) = (u_int32_t)(as_u8c(src) ^ 0x80); goto CONV_END;
> -conv_xx12_xxx1: as_u8(dst) = shift_down(as_u16c(src), 8); goto CONV_END;
> +conv_xx12_xxx1: as_u8(dst) = as_u16c(src) >> 8; goto CONV_END;
>  conv_xx12_xx12: as_u16(dst) = as_u16c(src); goto CONV_END;
>  conv_xx12_xx21: as_u16(dst) = bswap_16(as_u16c(src)); goto CONV_END;
>  conv_xx12_x120: as_u32(dst) = sx24((u_int32_t)as_u16c(src) << 8); goto 
> CONV_END;
>  conv_xx12_021x: as_u32(dst) = sx24s((u_int32_t)bswap_16(as_u16c(src)) << 8); 
> goto CONV_END;
>  conv_xx12_1200: as_u32(dst) = (u_int32_t)as_u16c(src) << 16; goto CONV_END;
>  conv_xx12_0021: as_u32(dst) = (u_int32_t)bswap_16(as_u16c(src)); goto 
> CONV_END;
> -conv_xx12_xxx9: as_u8(dst) = shift_down(as_u16c(src), 8) ^ 0x80; goto 
> CONV_END;
> +conv_xx12_xxx9: as_u8(dst) = (as_u16c(src) >> 8) ^ 0x80; goto CONV_END;
>  conv_xx12_xx92: as_u16(dst) = as_u16c(src) ^ 0x8000; goto CONV_END;
>  conv_xx12_xx29: as_u16(dst) = bswap_16(as_u16c(src)) ^ 0x80; goto CONV_END;
>  conv_xx12_x920: as_u32(dst) = sx24((u_int32_t)(as_u16c(src) ^ 0x8000) << 8); 
> goto CONV_END;
> @@ -284,57 +282,57 @@ conv_xx12_01Ax: as_u32(dst) = sx24s((u_i
>  conv_xx12_01Ax: as_u32(dst) = sx24s((u_int32_t)(as_u16c(src) ^ 0x80) << 8); 
> goto CONV_END;
>  conv_xx12_A100: as_u32(dst) = (u_int32_t)bswap_16(as_u16c(src) ^ 0x80) << 
> 16; goto CONV_END;
>  conv_xx12_001A: as_u32(dst) = (u_int32_t)(as_u16c(src) ^ 0x80); goto 
> CONV_END;
> -conv_x123_xxx1: as_u8(dst) = shift_down(as_u32c(src), 16); goto CONV_END;
> -conv_x123_xx12: as_u16(dst) = shift_down(as_u32c(src), 8); goto CONV_END;
> -conv_x123_xx21: as_u16(dst) = bswap_16(shift_down(as_u32c(src), 8)); goto 
> CONV_END;
> +conv_x123_xxx1: as_u8(dst) = as_u32c(src) >> 16; goto CONV_END;
> +conv_x123_xx12: as_u16(dst) = as_u32c(src) >> 8; goto CONV_END;
> +conv_x123_xx21: as_u16(dst) = bswap_16(as_u32c(src) >> 8); goto CONV_END;
>  conv_x123_x123: as_u32(dst) = sx24(as_u32c(src)); goto CONV_END;
>  conv_x123_321x: as_u32(dst) = sx24s(bswap_32(as_u32c(src))); goto CONV_END;
>  conv_x123_1230: as_u32(dst) = as_u32c(src) << 8; goto CONV_END;
> -conv_x123_0321: as_u32(dst) = shift_down(bswap_32(as_u32c(src)), 8); goto 
> CONV_END;
> - conv_x123_xxx9: as_u8(dst) = shift_down(as_u32c(src), 16) ^ 0x80; goto 
> CONV_END;
> -conv_x123_xx92: as_u16(dst) = shift_down(as_u32c(src), 8) ^ 0x8000; goto 
> CONV_END;
> -conv_x123_xx29: as_u16(dst) = bswap_16(shift_down(as_u32c(src), 8)) ^ 0x80; 
> goto CONV_END;
> +conv_x123_0321: as_u32(dst) = bswap_32(as_u32c(src)) >> 8; goto CONV_END;
> +conv_x123_xxx9: as_u8(dst) = (as_u32c(src) >> 16) ^ 0x80; goto CONV_END;
> +conv_x123_xx92: as_u16(dst) = (as_u32c(src) >> 8) ^ 0x8000; goto CONV_END;
> +conv_x123_xx29: as_u16(dst) = bswap_16(as_u32c(src) >> 8) ^ 0x80; goto 
> CONV_END;
>  conv_x123_x923: as_u32(dst) = sx24(as_u32c(src) ^ 0x80); goto CONV_END;
>  conv_x123_329x: as_u32(dst) = sx24s(bswap_32(as_u32c(src)) ^ 0x8000); goto 
> CONV_END;
>  conv_x123_9230: as_u32(dst) = (as_u32c(src) ^ 0x80) << 8; goto CONV_END;
> -conv_x123_0329: as_u32(dst)

Bug#429326: [Pkg-alsa-devel] Bug#429326: Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-10-14 Thread Elimar Riesebieter 
On Sun, 14 Oct 2007 the mental interface of
Loïc Minier told:

> On Sun, Oct 14, 2007, Elimar Riesebieter wrote:
> > From my point of view it is not reasonable to patch Debian packages,
> > which of course will break 32bit only sound cards. So we've to find
> > a way to force a solution directly on alsa-devel. Who will forward
> > the _Debian_bug to [EMAIL PROTECTED] ?
> 
>  I don't understand your remark about patching the Debian package?  Why
>  would reverting the bit shift macros break anything?
> 
>  The attached patch for alsa-lib should be enough to fix this, upstream
>  should revert the broken change in the bit shifts.

It should be a solution for every one! So upstream has to fix it :)



-- 
  On the keyboard of life you have always
  to keep a finger at the escape key;-)


signature.asc
Description: Digital signature

Bug#437827: [Pkg-alsa-devel] Bug#429326: Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-10-14 Thread Loïc Minier 
On Sun, Oct 14, 2007, Elimar Riesebieter wrote:
> From my point of view it is not reasonable to patch Debian packages,
> which of course will break 32bit only sound cards. So we've to find
> a way to force a solution directly on alsa-devel. Who will forward
> the _Debian_bug to [EMAIL PROTECTED] ?

 I don't understand your remark about patching the Debian package?  Why
 would reverting the bit shift macros break anything?

 The attached patch for alsa-lib should be enough to fix this, upstream
 should revert the broken change in the bit shifts.

-- 
Loïc Minier
Alsa #3360; GNOME #436192; LP #116990; Mandriva #33908; Revert HG r2264
(changeset 23c4c0f5de40) as the bit shifts overflow when the volume is high
for 32 bits to 16 bits resampling

diff -r 23c4c0f5de40 -r 883ae69471d2 src/pcm/plugin_ops.h
--- a/src/pcm/plugin_ops.h	Fri Mar 09 17:06:23 2007 +0100
+++ b/src/pcm/plugin_ops.h	Fri Mar 09 15:30:38 2007 +0100
@@ -85,8 +85,6 @@ static inline u_int32_t sx24s(u_int32_t 
 #define _put_triple(ptr,val) _put_triple_be(ptr,val)
 #define _put_triple_s(ptr,val) _put_triple_le(ptr,val)
 #endif
-
-#define shift_down(val, bits) (((val) + (1 << ((bits) - 1))) >> (bits))
 
 #ifdef COPY_LABELS
 static void *copy_labels[5] = {
@@ -258,14 +256,14 @@ conv_xxx1_009x: as_u32(dst) = sx24s((u_i
 conv_xxx1_009x: as_u32(dst) = sx24s((u_int32_t)(as_u8c(src) ^ 0x80) << 8); goto CONV_END;
 conv_xxx1_9000: as_u32(dst) = (u_int32_t)(as_u8c(src) ^ 0x80) << 24; goto CONV_END;
 conv_xxx1_0009: as_u32(dst) = (u_int32_t)(as_u8c(src) ^ 0x80); goto CONV_END;
-conv_xx12_xxx1: as_u8(dst) = shift_down(as_u16c(src), 8); goto CONV_END;
+conv_xx12_xxx1: as_u8(dst) = as_u16c(src) >> 8; goto CONV_END;
 conv_xx12_xx12: as_u16(dst) = as_u16c(src); goto CONV_END;
 conv_xx12_xx21: as_u16(dst) = bswap_16(as_u16c(src)); goto CONV_END;
 conv_xx12_x120: as_u32(dst) = sx24((u_int32_t)as_u16c(src) << 8); goto CONV_END;
 conv_xx12_021x: as_u32(dst) = sx24s((u_int32_t)bswap_16(as_u16c(src)) << 8); goto CONV_END;
 conv_xx12_1200: as_u32(dst) = (u_int32_t)as_u16c(src) << 16; goto CONV_END;
 conv_xx12_0021: as_u32(dst) = (u_int32_t)bswap_16(as_u16c(src)); goto CONV_END;
-conv_xx12_xxx9: as_u8(dst) = shift_down(as_u16c(src), 8) ^ 0x80; goto CONV_END;
+conv_xx12_xxx9: as_u8(dst) = (as_u16c(src) >> 8) ^ 0x80; goto CONV_END;
 conv_xx12_xx92: as_u16(dst) = as_u16c(src) ^ 0x8000; goto CONV_END;
 conv_xx12_xx29: as_u16(dst) = bswap_16(as_u16c(src)) ^ 0x80; goto CONV_END;
 conv_xx12_x920: as_u32(dst) = sx24((u_int32_t)(as_u16c(src) ^ 0x8000) << 8); goto CONV_END;
@@ -284,57 +282,57 @@ conv_xx12_01Ax: as_u32(dst) = sx24s((u_i
 conv_xx12_01Ax: as_u32(dst) = sx24s((u_int32_t)(as_u16c(src) ^ 0x80) << 8); goto CONV_END;
 conv_xx12_A100: as_u32(dst) = (u_int32_t)bswap_16(as_u16c(src) ^ 0x80) << 16; goto CONV_END;
 conv_xx12_001A: as_u32(dst) = (u_int32_t)(as_u16c(src) ^ 0x80); goto CONV_END;
-conv_x123_xxx1: as_u8(dst) = shift_down(as_u32c(src), 16); goto CONV_END;
-conv_x123_xx12: as_u16(dst) = shift_down(as_u32c(src), 8); goto CONV_END;
-conv_x123_xx21: as_u16(dst) = bswap_16(shift_down(as_u32c(src), 8)); goto CONV_END;
+conv_x123_xxx1: as_u8(dst) = as_u32c(src) >> 16; goto CONV_END;
+conv_x123_xx12: as_u16(dst) = as_u32c(src) >> 8; goto CONV_END;
+conv_x123_xx21: as_u16(dst) = bswap_16(as_u32c(src) >> 8); goto CONV_END;
 conv_x123_x123: as_u32(dst) = sx24(as_u32c(src)); goto CONV_END;
 conv_x123_321x: as_u32(dst) = sx24s(bswap_32(as_u32c(src))); goto CONV_END;
 conv_x123_1230: as_u32(dst) = as_u32c(src) << 8; goto CONV_END;
-conv_x123_0321: as_u32(dst) = shift_down(bswap_32(as_u32c(src)), 8); goto CONV_END;
- conv_x123_xxx9: as_u8(dst) = shift_down(as_u32c(src), 16) ^ 0x80; goto CONV_END;
-conv_x123_xx92: as_u16(dst) = shift_down(as_u32c(src), 8) ^ 0x8000; goto CONV_END;
-conv_x123_xx29: as_u16(dst) = bswap_16(shift_down(as_u32c(src), 8)) ^ 0x80; goto CONV_END;
+conv_x123_0321: as_u32(dst) = bswap_32(as_u32c(src)) >> 8; goto CONV_END;
+conv_x123_xxx9: as_u8(dst) = (as_u32c(src) >> 16) ^ 0x80; goto CONV_END;
+conv_x123_xx92: as_u16(dst) = (as_u32c(src) >> 8) ^ 0x8000; goto CONV_END;
+conv_x123_xx29: as_u16(dst) = bswap_16(as_u32c(src) >> 8) ^ 0x80; goto CONV_END;
 conv_x123_x923: as_u32(dst) = sx24(as_u32c(src) ^ 0x80); goto CONV_END;
 conv_x123_329x: as_u32(dst) = sx24s(bswap_32(as_u32c(src)) ^ 0x8000); goto CONV_END;
 conv_x123_9230: as_u32(dst) = (as_u32c(src) ^ 0x80) << 8; goto CONV_END;
-conv_x123_0329: as_u32(dst) = shift_down(bswap_32(as_u32c(src)), 8) ^ 0x80; goto CONV_END;
-conv_123x_xxx3: as_u8(dst) = shift_down(as_u32c(src), 8) & 0xff; goto CONV_END;
-conv_123x_xx32: as_u16(dst) = bswap_16(shift_down(as_u32c(src), 8)); goto CONV_END;
-conv_123x_xx23: as_u16(dst) = shift_down(as_u32c(src), 8) & 0x; goto CONV_END;
+conv_x123_0329: as_u32(dst) = (bswap_32(as_u32c(src)) >> 8) ^ 0x80; goto CONV_END;
+conv_123x_xxx3: as_u8(dst) = (as_u32c(src) >> 8) & 0xff; goto CONV_END;
+conv_123x_xx32: as_u16(dst) = bswap_16(as_u32c(src) >> 8); goto CONV_END;
+conv_123x_x

Bug#437827: [Pkg-alsa-devel] Bug#429326: Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-10-14 Thread Elimar Riesebieter 
On Sun, 14 Oct 2007 the mental interface of
Loïc Minier told:

> On Tue, Aug 14, 2007, Frans Pop wrote:
> > Recently the sound when playing DVDs with vlc started creaking very bad,
> > especially with some voices and loud sound in general.
> 
>  This GNOME bug might have a sensible explanation, even if your issue
>  might have been resolved another way:
> 

Thanks for the info :) It seems that the alsa people don't force a
progress on
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3360
?

From my point of view it is not reasonable to patch Debian packages,
which of course will break 32bit only sound cards. So we've to find
a way to force a solution directly on alsa-devel. Who will forward
the _Debian_bug to [EMAIL PROTECTED] ?

Elimar

-- 
  On the keyboard of life you have always
  to keep a finger at the escape key;-)

Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-10-14 Thread Frans Pop 
On Sunday 14 October 2007, Loïc Minier wrote:
> On Tue, Aug 14, 2007, Frans Pop wrote:
> > Recently the sound when playing DVDs with vlc started creaking very
> > bad, especially with some voices and loud sound in general.
>
>  This GNOME bug might have a sensible explanation, even if your issue
>  might have been resolved another way:
> 

Thanks for the pointer Loïc (although at least my BR had nothing to do with 
Gnome).

Jonathan Matthew actually found my upstream BR and added a comment there, so 
I'd already seen his analysis there:
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3360

I added a comment myself that I discovered the issue has not completely 
disappeared for me, although its a lot less then when I originally reported 
it. I'm not sure if #437827 should be reopened or not; I'll leave that to 
the alsa-lib maintainers.

Cheers,
FJP

Bug#429326: Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-10-14 Thread Loïc Minier 
On Tue, Aug 14, 2007, Frans Pop wrote:
> Recently the sound when playing DVDs with vlc started creaking very bad,
> especially with some voices and loud sound in general.

 This GNOME bug might have a sensible explanation, even if your issue
 might have been resolved another way:


-- 
Loïc Minier

Bug#437827: [Pkg-alsa-devel] Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-09-15 Thread Frans Pop 
On Tuesday 11 September 2007, Elimar Riesebieter wrote:
> > Recently the sound when playing DVDs with vlc started creaking very
> > bad, especially with some voices and loud sound in general. Today I
> > found that the same happens when I play OGG files using vlc.
>
> Which sound driver do you use? I noticed a similar bug from someone
> using APPLE hardware. A digital soundcard is used there.

I have a fairly standard 64-bits Intel-based box:
00:1b.0 Audio device [0403]: Intel Corporation 82801G (ICH7 Family) High 
Definition Audio Controller [8086:27d8] (rev 01)

The alsa module used is snd_hda_intel.

Cheers,
FJP

P.S. I also filed a BR upstream, but no response so far:
https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3360



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#437827: [Pkg-alsa-devel] Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-09-11 Thread Elimar Riesebieter 
On Tue, 14 Aug 2007 the mental interface of
Frans Pop told:

> Package: libasound2
> Version: 1.0.14a-2
> Severity: important
> 
> Recently the sound when playing DVDs with vlc started creaking very bad,
> especially with some voices and loud sound in general. Today I found
> that the same happens when I play OGG files using vlc.

Which sound driver do you use? I noticed a similar bug from someone
using APPLE hardware. A digital soundcard is used there.

Elimar


-- 
  "Talking much about oneself can also 
   be a means to conceal oneself."
 -Friedrich Nietzsche



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#437827: libasound2: Sound creaks badly when playing DVDs or OGG files

2007-08-14 Thread Frans Pop 
Package: libasound2
Version: 1.0.14a-2
Severity: important

Recently the sound when playing DVDs with vlc started creaking very bad,
especially with some voices and loud sound in general. Today I found
that the same happens when I play OGG files using vlc.

If I play the same movies or OGG files in kaffeine, the sound is fine.

My first impulse was to blame vlc, but today I find that I can 100% resolve
the problem by downgrading libasound2 to 1.0.13-2. The problem is
completely reproducible after upgrading libasound2 again.

I guess this is probably related to #429326, although my symptoms are
different (I have no problem with pauses).

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.23-rc3 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages libasound2 depends on:
ii  libc6 2.6.1-1GNU C Library: Shared libraries

libasound2 recommends no packages.

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]