All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] Audio patch
@ 2004-10-09 17:23 Fabrice Bellard
  2004-10-09 18:02 ` David E. Still
  2004-10-10  1:54 ` malc
  0 siblings, 2 replies; 12+ messages in thread
From: Fabrice Bellard @ 2004-10-09 17:23 UTC (permalink / raw)
  To: qemu-devel

I commited the version 6 of malc's audio patch. Tell me if you see 
problems. Maybe SDL audio support could be enabled on Mac OS X ?

The version 7 of the patch is not acceptable in its current state as C++ 
is forbidden in QEMU.

Fabrice.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Audio patch
  2004-10-09 17:23 [Qemu-devel] Audio patch Fabrice Bellard
@ 2004-10-09 18:02 ` David E. Still
  2004-10-10  1:54 ` malc
  1 sibling, 0 replies; 12+ messages in thread
From: David E. Still @ 2004-10-09 18:02 UTC (permalink / raw)
  To: qemu-devel

I would *love* to have SDL audio support for the Mac.  Perhaps even a 
-sdlaudio switch from the command-line?

Another possibility would be using the liboss library 
(http://liboss.sourceforge.net/), although the project doesn't seem to 
have had much activity of late.  I don't know how viable it is, what it 
would take to implement, or even if it's robust enough for Qemu, but it 
might be worth looking at given the limitations of SDL audio that 
you've mentioned in the past.

- Dave
--
David E. Still
Writer/Producer
TripleCat Productions
http://www.triplecatproductions.com

On Oct 9, 2004, at 10:23 AM, Fabrice Bellard wrote:

> I commited the version 6 of malc's audio patch. Tell me if you see 
> problems. Maybe SDL audio support could be enabled on Mac OS X ?
>
> The version 7 of the patch is not acceptable in its current state as 
> C++ is forbidden in QEMU.
>
> Fabrice.
>
>
>
> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel
>
>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Audio patch
  2004-10-09 17:23 [Qemu-devel] Audio patch Fabrice Bellard
  2004-10-09 18:02 ` David E. Still
@ 2004-10-10  1:54 ` malc
  2004-10-10 16:02   ` John R. Hogerhuis
  2004-10-11 14:52   ` Johannes Schindelin
  1 sibling, 2 replies; 12+ messages in thread
From: malc @ 2004-10-10  1:54 UTC (permalink / raw)
  To: qemu-devel

On Sat, 9 Oct 2004, Fabrice Bellard wrote:

> I commited the version 6 of malc's audio patch. Tell me if you see
> problems. Maybe SDL audio support could be enabled on Mac OS X ?
>
> The version 7 of the patch is not acceptable in its current state as C++
> is forbidden in QEMU.

For the time being (until C++ magically disapears) 7c_aqemu which should
in apply cleanly to the current CVS is available at:
http://www.boblycat.org/~malc/code/patches/qemu/

-- 
mailto:malc@pulsesoft.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Audio patch
  2004-10-10 16:02   ` John R. Hogerhuis
@ 2004-10-10 13:35     ` Johannes Martin
  2004-10-10 16:51       ` Fabrice Bellard
  0 siblings, 1 reply; 12+ messages in thread
From: Johannes Martin @ 2004-10-10 13:35 UTC (permalink / raw)
  To: qemu-devel

On Sun, 10 Oct 2004, John R. Hogerhuis wrote:
> On Sat, 2004-10-09 at 18:54, malc wrote:
> > For the time being (until C++ magically disapears)
> A beautiful dream, my friend.
How about somebody uses AT&T's cfront or SUN's GCC2C to convert those
templates in mixeng.cc to plain C? The code won't be pretty, but it will
make more sense than handcoding the stuff.

I guess one could also use some smart macros to replace the templates.

	Johannes

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Audio patch
  2004-10-10  1:54 ` malc
@ 2004-10-10 16:02   ` John R. Hogerhuis
  2004-10-10 13:35     ` Johannes Martin
  2004-10-11 14:52   ` Johannes Schindelin
  1 sibling, 1 reply; 12+ messages in thread
From: John R. Hogerhuis @ 2004-10-10 16:02 UTC (permalink / raw)
  To: qemu-devel

On Sat, 2004-10-09 at 18:54, malc wrote:
> For the time being (until C++ magically disapears)

A beautiful dream, my friend.

-- John.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Audio patch
  2004-10-10 13:35     ` Johannes Martin
@ 2004-10-10 16:51       ` Fabrice Bellard
  2004-10-10 17:21         ` malc
  0 siblings, 1 reply; 12+ messages in thread
From: Fabrice Bellard @ 2004-10-10 16:51 UTC (permalink / raw)
  To: qemu-devel

Moreover mixeng.cc uses floats, which are also forbidden in QEMU, except 
for CPU emulation :-)

Fabrice.

Johannes Martin wrote:
> On Sun, 10 Oct 2004, John R. Hogerhuis wrote:
> 
>>On Sat, 2004-10-09 at 18:54, malc wrote:
>>
>>>For the time being (until C++ magically disapears)
>>
>>A beautiful dream, my friend.
> 
> How about somebody uses AT&T's cfront or SUN's GCC2C to convert those
> templates in mixeng.cc to plain C? The code won't be pretty, but it will
> make more sense than handcoding the stuff.
> 
> I guess one could also use some smart macros to replace the templates.
> 
> 	Johannes
> 
> 
> 
> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel
> 
> 

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Audio patch
  2004-10-10 16:51       ` Fabrice Bellard
@ 2004-10-10 17:21         ` malc
  0 siblings, 0 replies; 12+ messages in thread
From: malc @ 2004-10-10 17:21 UTC (permalink / raw)
  To: qemu-devel

On Sun, 10 Oct 2004, Fabrice Bellard wrote:

> Moreover mixeng.cc uses floats, which are also forbidden in QEMU, except
> for CPU emulation :-)

I'm no big C++ fan myself, but credit where it's due, changing these to
int32_t or whatnot is as simple as changing template argument and clamping
code in ONE place - now beat that stupid C99 ;)

>
> Johannes Martin wrote:
> > On Sun, 10 Oct 2004, John R. Hogerhuis wrote:
> >
> >>On Sat, 2004-10-09 at 18:54, malc wrote:
> >>
> >>>For the time being (until C++ magically disapears)
> >>
> >>A beautiful dream, my friend.
> >
> > How about somebody uses AT&T's cfront or SUN's GCC2C to convert those
> > templates in mixeng.cc to plain C? The code won't be pretty, but it will
> > make more sense than handcoding the stuff.
> >
> > I guess one could also use some smart macros to replace the templates.

-- 
mailto:malc@pulsesoft.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Audio patch
  2004-10-10  1:54 ` malc
  2004-10-10 16:02   ` John R. Hogerhuis
@ 2004-10-11 14:52   ` Johannes Schindelin
  2004-10-11 17:59     ` [Qemu-devel] " Ronald
  1 sibling, 1 reply; 12+ messages in thread
From: Johannes Schindelin @ 2004-10-11 14:52 UTC (permalink / raw)
  To: qemu-devel

[-- Attachment #1: Type: TEXT/PLAIN, Size: 559 bytes --]

Hi,

On Sun, 10 Oct 2004, malc wrote:

> On Sat, 9 Oct 2004, Fabrice Bellard wrote:
> > The version 7 of the patch is not acceptable in its current state as C++
> > is forbidden in QEMU.
>
> For the time being (until C++ magically disapears) 7c_aqemu which should
> in apply cleanly to the current CVS is available at:
> http://www.boblycat.org/~malc/code/patches/qemu/

Please try the attached patch *after* applying 7c_aqemu. It removes the
need for template Fabrice's Way(TM). I could not test it yet (sitting on a
box without sound card...).

Ciao,
Dscho

[-- Attachment #2: Type: APPLICATION/x-gunzip, Size: 4279 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Qemu-devel] Re: Audio patch
  2004-10-11 14:52   ` Johannes Schindelin
@ 2004-10-11 17:59     ` Ronald
  2004-10-11 22:06       ` Johannes Schindelin
  0 siblings, 1 reply; 12+ messages in thread
From: Ronald @ 2004-10-11 17:59 UTC (permalink / raw)
  To: qemu-devel

Le Mon, 11 Oct 2004 16:52:52 +0200, Johannes Schindelin a écrit :

> Hi,
> 
> On Sun, 10 Oct 2004, malc wrote:
> 
>> On Sat, 9 Oct 2004, Fabrice Bellard wrote:
>> > The version 7 of the patch is not acceptable in its current state as
>> > C++ is forbidden in QEMU.
>>
>> For the time being (until C++ magically disapears) 7c_aqemu which should
>> in apply cleanly to the current CVS is available at:
>> http://www.boblycat.org/~malc/code/patches/qemu/
> 
> Please try the attached patch *after* applying 7c_aqemu. It removes the
> need for template Fabrice's Way(TM). I could not test it yet (sitting on a
> box without sound card...).

Doesn't work for me, windows report the device as working but no sound
output. 

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Re: Audio patch
  2004-10-11 17:59     ` [Qemu-devel] " Ronald
@ 2004-10-11 22:06       ` Johannes Schindelin
  2004-10-11 22:20         ` [Qemu-devel] " Ronald
  0 siblings, 1 reply; 12+ messages in thread
From: Johannes Schindelin @ 2004-10-11 22:06 UTC (permalink / raw)
  To: daimon55, qemu-devel

Hi,

On Mon, 11 Oct 2004, Ronald wrote:

> Doesn't work for me, windows report the device as working but no sound
> output.

Sorry, still could not test, but one thing is for sure: At the beginning
of mixeng_template.h, the four definitions of HALFT and HALF: could you
change them to

#ifdef SIGNED
#define HALFT (float)(IN_MAX)
#define HALF (float)(IN_MAX)
#else
#define HALFT (float)((IN_MAX)>>1)
#define HALF HALFT
#endif

and test again?

Sorry,
Dscho

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Qemu-devel] Re: Re: Audio patch
  2004-10-11 22:06       ` Johannes Schindelin
@ 2004-10-11 22:20         ` Ronald
  2004-10-11 22:31           ` Joe Menola
  0 siblings, 1 reply; 12+ messages in thread
From: Ronald @ 2004-10-11 22:20 UTC (permalink / raw)
  To: qemu-devel

Le Tue, 12 Oct 2004 00:06:50 +0200, Johannes Schindelin a écrit :

> Hi,
> 
> On Mon, 11 Oct 2004, Ronald wrote:
> 
>> Doesn't work for me, windows report the device as working but no sound
>> output.
> 
> Sorry, still could not test, but one thing is for sure: At the beginning
> of mixeng_template.h, the four definitions of HALFT and HALF: could you
> change them to
> 
> #ifdef SIGNED
> #define HALFT (float)(IN_MAX)
> #define HALF (float)(IN_MAX)
> #else
> #define HALFT (float)((IN_MAX)>>1)
> #define HALF HALFT
> #endif
> 
> and test again?
> 

Still the same thing: card reported ok but no output at all.

> Sorry,
> Dscho

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Qemu-devel] Re: Re: Audio patch
  2004-10-11 22:20         ` [Qemu-devel] " Ronald
@ 2004-10-11 22:31           ` Joe Menola
  0 siblings, 0 replies; 12+ messages in thread
From: Joe Menola @ 2004-10-11 22:31 UTC (permalink / raw)
  To: qemu-devel

On Mon October 11 2004 5:20 pm, Ronald wrote:
> Le Tue, 12 Oct 2004 00:06:50 +0200, Johannes Schindelin a écrit :
> > Hi,
> >
> > On Mon, 11 Oct 2004, Ronald wrote:
> >> Doesn't work for me, windows report the device as working but no sound
> >> output.
> >
> > Sorry, still could not test, but one thing is for sure: At the beginning
> > of mixeng_template.h, the four definitions of HALFT and HALF: could you
> > change them to
> >
> > #ifdef SIGNED
> > #define HALFT (float)(IN_MAX)
> > #define HALF (float)(IN_MAX)
> > #else
> > #define HALFT (float)((IN_MAX)>>1)
> > #define HALF HALFT
> > #endif
> >
> > and test again?
>
> Still the same thing: card reported ok but no output at all.
>
> > Sorry,
> > Dscho

My soundblaster did the same thing.

Malc had a fix for this...

"If you are using Win98 guest make sure that 16bit DMA for SoundBlaster is
set to 5"

HTH-jm

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2004-10-11 22:39 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-10-09 17:23 [Qemu-devel] Audio patch Fabrice Bellard
2004-10-09 18:02 ` David E. Still
2004-10-10  1:54 ` malc
2004-10-10 16:02   ` John R. Hogerhuis
2004-10-10 13:35     ` Johannes Martin
2004-10-10 16:51       ` Fabrice Bellard
2004-10-10 17:21         ` malc
2004-10-11 14:52   ` Johannes Schindelin
2004-10-11 17:59     ` [Qemu-devel] " Ronald
2004-10-11 22:06       ` Johannes Schindelin
2004-10-11 22:20         ` [Qemu-devel] " Ronald
2004-10-11 22:31           ` Joe Menola

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.